For with array

[Queener]

How do you write this code to an array. I have 3675 to 3682 and another number which isn't in the same range (25834).

Func IsTest($id)
For $i = 3675 To 3682
If $id = Hex($i, 8) Then
Return True
Else
Return False
EndIf
EndFunc

I was thinking about

Func IsTest($id)
For $i = 3675 To 3682 & 25834
If $id = Hex($i, 8) Then
Return True
Else
Return False
EndIf
EndFunc

or

Func IsTest($id)
For $i = 3675 To 3682
If $id = Hex($i, 8) Then
Return True
Else
Return False
EndIf
Next
For $l = 25834
If $id = Hex($l, 8) Then
Return True
Else
Return False
EndIf
EndFunc

But these test doesn't work. Return as array incorrect prefix.

Edited April 16, 2013 by asianqueen

[kylomas]

Close your for...to loop with a "next" for starters...

You want to write the return value to an array outside of the function?

edit: question

Edited April 16, 2013 by kylomas

[SmOke_N]

Be much more specific.

I have no idea what you're asking.

What is id, what does 25834 have to do with anything, are you comparing a string value on purpose, is the "True/False" what you want in the array, how do you want to match it to 3675/3682, etc.

[Queener]

Long story make it short; I have another list that this file will be pulling.

[item]
$id[3675]= '000A'
$id[3676]= '000B'
etc...
$id[25834]='0300'

If IsTest = i then show else don't show. If I don't include 25834 then the code is already working fine, but like to know how to add this number to it. I wouldn't want to do 3675 To 25834 because all other code above 3682 represent something else other than color.

[tlman12]

Be much more specific.

I have no idea what you're asking.

What is id, what does 25834 have to do with anything, are you comparing a string value on purpose, is the "True/False" what you want in the array, how do you want to match it to 3675/3682, etc.

What about

Func IsTest($id)
If $id = 25834 Then
Return True
Else
For $i = 3675 To 3682
If $id = Hex($i, 8) Then
Return True
Else
Return False
EndIf
Next
EndIf
EndFunc   ;==>IsTest

[Queener]

autoit is so picky about how you actually format it... working on the script; if it works; I'll ping you guys. Thank

EDIT:

Error: "Next" statement with no matching "For" statement.

Edited April 16, 2013 by asianqueen

[kylomas]

tlman12,

Your code iterates the loop once and returns. It should probably look more like this (untested)

Func IsTest($id)

    If $id = 25834 Then Return True

    For $i = 3675 To 3682
        If $id = Hex($i, 8) Then Return True
    Next

    return false            ; this needs to be outside of for loop

EndFunc   ;==>IsTest

@asianqueen,

It is difficult to understand what you are asking. The following example will update an array with true or false.

#include <array.au3>

local $aRET[1]

$aRET[ubound($aRET) - 1] = IsTest(hex(3677))        ; set last element of array to value returned from function
redim $aRET[ubound($aRET) + 1]                        ; increase array (presumably you are going to loop on this

_arraydisplay($aRET)                                ; display array

Func IsTest($id)

    For $i = 3675 To 3682
        If $id = Hex($i) Then Return True
    Next

    Return False                                    ; <-- this need to be outside of the for loop

EndFunc

autoit is so picky about how you actually format it...

As opposed to what?

kylomas

Edited April 16, 2013 by kylomas

[tlman12]

tlman12,

Your code iterates the loop once and returns. It should probably look more like this (untested)

ha your right, didn't even notice that. i only copied what he had and added the if statement. didn't try to run it

[Queener]

As opposed to what?

kylomas

You can't type for example where all can't be aligh left:

Call function
If statement
Return True
Else
False

It has to be in the correct format with tabs and spaces.

By the way

Func IsTest($id)

If $id = 25834 Then Return True

For $i = 3675 To 3682
If $id = Hex($i, 8) Then Return True
Next

return false ; this needs to be outside of for loop

EndFunc ;==>IsTest

does not have an endif recall.

Edited April 16, 2013 by asianqueen

[kylomas]

asianqueen,

It has to be in the correct format with tabs and spaces.

Not true...

does not have an endif recall.

Does not need one...

Let me suggest two things:

1 - Spend some time reading the help file

2 - When posting for advice, think through what you are asking and see it from a readers perspective.

Sometime we get too close to the code to see "the forest for the trees", so to speak.

Good Luck,

kylomas

[guinness]

Returning in a loop is back coding practice. It's better to exit the loop and use a return variable.