Pixel Search usage help

[Physical]

I am trying to use the pixel search function to search for any 1 of 4 colors. It doesnt matter which one it comes across first. I am not sure what the best way to go about it is. If i use 4 lines of PixelSearch then it will just hop from one to the next before it does an action. I considered doing the folowing.

$Array[0] = xxxxxxxx     ;store the 4 values in an array
$Array[1] = xxxxxxxx
$Array[2] = xxxxxxxx
$Array[3] = xxxxxxxx

While 1
        $TCoord = PixelSearch (75, 150, 950, 600, $Array[$Count])
        If @error Then
            If $Count = 3 Then
                $Count = 0
            EndIf
            $Count + 1
            Return
        ElseIf Not @error Then
            $Count = 0
            Call ("Function")
        EndIf
    WEnd

I am not sure this will even work. I am still learning this so i may even have done some code wrong i am still working on it. Hopefully you can get an idea of what i am trying to do. Any help and suggestions would be greatly appreciated.

[james3mg]

This is a perfect fit for a For...Next loop, instead of a While loop:

$Array[0] = xxxxxxxx;store the 4 values in an array
$Array[1] = xxxxxxxx
$Array[2] = xxxxxxxx
$Array[3] = xxxxxxxx

For $count=0 To 3
    $TCoord = PixelSearch (75, 150, 950, 600, $Array[$Count])
    If NOT @error Then
        Function()
        ExitLoop
    EndIf
Next

The key to getting it to work is the ExitLoop line, which allows it to continue without checking for the other colors, if one is found. And, unless there's more to the code that dictates it, you don't have to use Call() to run a function in this way- just type in the function name and () after it to run it. Call() is usually used when the name of the function is stored in a string or variable or similar.

Other than that, the only real error I see in my cursory glance over your code is that your $Count + 1 line should be $Count+=1

Hope that's what you were looking for

Hope that helps

Edited May 12, 2008 by james3mg

[Physical]

This is a perfect fit for a For...Next loop, instead of a While loop:

$Array[0] = xxxxxxxx;store the 4 values in an array
$Array[1] = xxxxxxxx
$Array[2] = xxxxxxxx
$Array[3] = xxxxxxxx

For $count=0 To 3
    $TCoord = PixelSearch (75, 150, 950, 600, $Array[$Count])
    If NOT @error Then
        Function()
        ExitLoop
    EndIf
Next

Other than that, your $Count + 1 line should be $Count+=1

Hope that helps

Thanks for the help. Right as you were posting i was doing more searching and found a thread with a little info but not much. So i rewrote it and it suggested the For Next thing as well. This is what i have now:

$Colors[4] = [0x15100A, 0x553826, 0x46332B, 0x7D5A45]
    For $X = 0 to Ubound($Colors) - 1
        $TCoord = PixelSearch ( 75, 150, 950, 600, $Colors[$X], 10)
    Next

The more i look at it tho i am wondering, will this pick 1 of the 4 colors and move on or will this wait till it finds all 4 then move on? thx for any help

Edited May 12, 2008 by Physical

[james3mg]

Thanks for the help. Right as you were posting i was doing more searching and found a thread with a little info but not much. So i rewrote it and it suggested the For Next thing as well. This is what i have now:

$Colors[4] = [0x15100A, 0x553826, 0x46332B, 0x7D5A45]
    For $X = 0 to Ubound($Colors) - 1
        $TCoord = PixelSearch ( 75, 150, 950, 600, $Colors[$X], 10)
    Next

The more i look at it tho i am wondering, will this pick 1 of the 4 colors and move on or will this wait till it finds all 4 then move on? thx for any help

Check my edit...look at ExitLoop

[Physical]

Check my edit...look at ExitLoop

Ok yes this is helping a lot, thankyou very much. I have one more question. Would it be better to use the method of $Count = $Count + 1 to incriment the array or is the UBound function a better method? thx very much for your help

[MikeP]

The problem with PixelSearch is it gives the first pixel found.. from left to right, top to bottom.. don't forget this.

[Physical]

The problem with PixelSearch is it gives the first pixel found.. from left to right, top to bottom.. don't forget this.

Thanks i didn't think about that. I will adjust it so it doesnt find the wrong thing.