|
|
|
|
|
by Throwadev
4445 days ago
|
|
Tantalor, your post would be much better if you simply gave the solutions you think are better rather than being all smug. For example, what tool is he using that's wrong for the job, and what are examples of the right tool(s)? Yes, radius varies, but the OP's solution addresses that, so what's your point? 10/10 would be drenched in smugness again. |
|
Michael complained that the query repeated the same computation each time, which is an odd thing to say, because that is its purpose. He expected memoization, which the library function did not offer, so he added that. But there's no reason to be dissatisfied with the code for not doing something its not supposed to do, or its developers for not adding an unnecessary feature.
His memoization does support varying radius, but only in a narrow subset of {5, 10, 20, 30, 40, 50}. Of course this is a huge performance optimization for the app, but in general it is impractical for provide memoization like this for an arbitrary radius.