_ArraySort problem

[TrashBoat]

So I've made this script that detects how long i have held down my left mouse button for and stores the information in an array and then sorts its using _ArraySort but the output is half sorted half broken.

Here's my script:

HotKeySet("{F1}","_exit")
#include <Misc.au3>
#include <Timers.au3>
#include <Array.au3>

Local $dll = DllOpen("user32.dll")
$on = False
Global $array[0]

While(1)
    If _IsPressed(01,$dll) Then
        $timer = _Timer_Init()
        While _IsPressed(01,$dll)
            Sleep(1)
        WEnd
        $time = _Timer_Diff($timer)
        _ArrayAdd($array,"Time: " & Floor($time) & " ms")
;~      ConsoleWrite("Time: " & Floor($time) & " ms" & @CRLF)
    EndIf
    Sleep(50)
WEnd

Func _exit()
    _ArraySort($array)
    _ArrayDisplay($array)
    Exit
EndFunc

And the output:

See how its not sorted?  What is the problem here?

[RTFC]

It's sorting alphanumerically, because you stored strings. Instead, store only the time in ms (numerically, not as string!) .

[Melba23]

TrashBoat,

_ArraySort sorts strings alphabetically - which is why you get what you do as a result. Save the values as numbers and it will sort numerically:

_ArrayAdd($array, Floor($time))

And if you open the DLL, you should really use DLLClose as you exit to shut it down.

M23

[TrashBoat]

How did i not think of that. And does me not closing the dll result in memory loss or smth?

[Melba23]

TrashBoat,

Actually AutoIt will clear up after you automatically - but doing it yourself is a good habit to get into and can save problems later on when you really do need to do so (such as with many GDI functions).

M23

[RTFC]

  On 7/26/2018 at 11:32 AM, TrashBoat said:

does me not closing the dll result in memory loss

Expand  

It's the other way around; if you suffer from memory loss you may forget to close the handle.