_ArrayMinIndex and Void entries

[any0day@…]

Thought I would just point out that for void entries, when calling _ArrayMinIndex, it always returns those entries. I was expecting it to throw an error if void entries were detected. As a result, I added a bit of code to the function so I could catch this:

; Search
     If $iCompNumeric Then
         For $i = $iStart To $iEnd
             If $avArray[$i] = Chr(0) Then Return SetError(4, 0, -1)
             If Number($avArray[$iMinIndex]) > Number($avArray[$i]) Then $iMinIndex = $i
         Next
     Else
         For $i = $iStart To $iEnd
             If $avArray[$i] = Chr(0) Then Return SetError(4, 0, -1)
             If $avArray[$iMinIndex] > $avArray[$i] Then $iMinIndex = $i
         Next
     EndIf

A minor addition, but one I found to be helpful. It is, however, up to you on how you want to handle it.

[J-Paul Mesnage]

for me it is not a bug as Autoit do implicit conversion.
I move it to "feature request"

[TicketCleanup]

Automatic ticket cleanup.

[any0day@…]

Replying to Jpm:

for me it is not a bug as Autoit do implicit conversion.
I move it to "feature request"

The issue is that sometimes if there are void entries, this can cause a program crash because _ArrayMin uses the min index as the subscript. Error output will catch this, stopping the crash. This is why I labeled bug, and not feature.

[J-Paul Mesnage]

Why a program can crash just the return an "void" entry?
give me a good example of such crash

[any0day@…]

Replying to Jpm:

Here's a simplified example:

While $i <= 10
	$array[$i] = $i
	ToolTip("Lowest Value = " & _ArrayMin($array), 0,0)
WEnd

Related Functions _ArrayMinIndex, _ArrayMaxIndex, _ArrayMin, _ArrayMax.

Reason: Subscript Out Of Bounds

Related Example:

Func _ArrayMin(Const ByRef $avArray, $iCompNumeric = 0, $iStart = 0, $iEnd = 0)
	Local $iResult = _ArrayMinIndex($avArray, $iCompNumeric, $iStart, $iEnd)
	If @error Then Return SetError(@error, 0, "")
	'''Return $avArray[$iResult]'''
EndFunc   ;==>_ArrayMin

Bolded reasons.

Fix: add an error return on _ArrayMinIndex and _ArrayMaxIndex stated above.

[J-Paul Mesnage]

It is over simplified as $array is not declared and $i not set too
can you put a real crashing example
Thanks

[J-Paul Mesnage]

I cannot imagine a real crash but just the script not working as you want.
If you really need to get error on such entries just check them before calling _ArrayMinIndex()