P.S. Couldn't status be just 0 or 1 instead of "Act"? You are doing 38k * 3
letters comparisons, instead of 38k. Also, shouldn't you check the status
of j too, or it doesn't matter?

On Thu, Aug 4, 2016 at 1:43 AM, Javier Guerrero García <javi...@gmail.com>
wrote:

> Another quick one: since you're comparing points and distances and
> sin/cos/tans, does it make sense to compare points i=25 and j=50, and then
> again points i=50 and j=25? Would the calculations yield the same results?
> In that case, you could just halve your computation to (i<j) instead of
> (i!=j), and instead of getting rid of the diagonal of the matrix, get rid
> of the whole triangle below it (or just do something checking propTwelve if
> applies)
>
> Besides Justin recommendation, also a precalculated look-up table for
> sqrt, sin,cos and atan2 might come in handy if you can bear with the error
> tradeoff (they are REALLY expensive functions). Also try a*a directly
> instead of Math.Pow(a,2).
>
> Also, isn't "count" exactly the length of tempCompArray? Why keep track
> of it?
>
> Also, if you write the first three ifs as (if a && b && c), b && c are
> NEVER calculated if a is false, so you can skip 3 lines (and make your
> branch predictor happier :)
>
> Also, since you're referencing array[i] and array[j] a vast number of
> times, maybe is a good idea at the top of the loop:
>
> itemi=tempArray[i]
> itemj=temArray[j]
>
> hence having a direct memory reference instead of a double reference
> (address of tempArray + address of the ith element)
>
> Also, you use a few times "Math.abs(a-b)<=1". Try a oneliner function
> near1(a,b) { c=a+b; return c>=-1&&c<=1}, I think this might be faster (and
> again you get rid of the IF involved in the ABS function).
>
> Well, 1:41AM here, have to sleep :)
>
> On Thu, Aug 4, 2016 at 12:21 AM, bilbosax <waspenc...@comcast.net> wrote:
>
>> Distances are selected by the user using a dropdownlist in the range of
>> .25
>> to 4 miles.  Distance is the key to the whole thing here and has to be
>> exact, I can't be off by 10ths of a mile, it needs to be within say, 20
>> feet.
>>
>>
>>
>> --
>> View this message in context:
>> http://apache-flex-users.2333346.n4.nabble.com/Workers-and-Speed-tp13098p13196.html
>> Sent from the Apache Flex Users mailing list archive at Nabble.com.
>>
>
>

Reply via email to