I need help with a loop

[parkswalker]

While 1
    $a = PixelGetColor(835,302) 
    $b = PixelGetColor(840,435)
    If $a = 0x5383E8 Then
    MouseClick("Left",835,302,1)
    Elseif $b = 0x818AA0
    ExitLoop
    EndIf
    WEnd

What I'm trying to do is wait for $a to become a color and then left click (takes about 40 seconds usually). Problem is that $a doesn't always change to the color I want so I want to look for either $a = 0x5383E8 or $b = 0x818AA0 (which happens if $a doesn't show up). What I am getting with the above code is that the loop exits immediatly instead of looping until either $a or $b = the color I need it to. How do I loop this so that it keeps looking until either happens?

[Nessie]

Try this:

While 1
    $a = PixelGetColor(835, 302)
    $b = PixelGetColor(840, 435)
    If $a = 0x5383E8 Or $b = 0x818AA0 Then
        MouseClick("Left", 835, 302, 1)
        ;Exit ;Uncomment this if you want to exit after the left mouse click ;)
    EndIf
WEnd

BTW be carefull of what are you doing

Hi!

[Mechaflash]

PixelGetColor returns a decimal representation of a color. If you wish to compare it to a hex color, specify the output as Hex. Also use quotations for the values.

Also in your code, the loop never ends if $a is found.

While 1
  $a = Hex(PixelGetColor(835,302), 6)
  $b = Hex(PixelGetColor(840,435), 6)
  If $a = "5383E8" Then
    MouseClick("Left",835,302,1)
    ConsoleWrite("$a was found" & @CRLF)
  Elseif $b = "818AA0" Then
    ConsoleWrite("$b was found" & @CRLF)
    ExitLoop
  Else
    ConsoleWrite("Neither value found, continue loop" & @CRLF)
  EndIf
WEnd

Edited March 12, 2013 by Mechaflash

[BrewManNH]

PixelGetColor returns a decimal representation of a color. If you wish to compare it to a hex color, specify the output as Hex. Also use quotations for the values.

You can directly compare a hex value with a decimal value in AutoIt, no need to convert it from one to the other.