my first script issue - (Moved)

[hc4fun]

Hi all, I'm beginner with autoit scripting so please be gentle

 

Func funkcja1() ;M na przycisk prawy

$sleep = random (450,700,15)

if PixelGetColor ( Random($z1,$z2,1) , 385) = 0xFFAAAA then ; jeżeli w zakresie od z1 do z2 wystąpi kolor wykonuj
      MouseMove(Random($x3, $x4, 1), Random($y1, $y2, 2)) ; przesuń mysz na współżędne z zakresu x3 - x4 i y1 do y2
      While True
         sleep($sleep) ;klikaj
         MouseClick("left")
      WEnd
      sleep(500)
      Else
         if PixelGetColor ( Random($z3,$z4,1) , 385) = 0xCF2424 then ; jeżeli w rpzedziale od z3 do z4 występuje kolor czekaj jeżeli n ie zakończ pętle i zwolnij lpm
         MouseMove(Random($x1, $x2, 1), Random($y1, $y2, 2)) ; przesuń mysz na współżędne z zakresu x3 - x4 i y1 do y2 i trzymaj lpm
        While True
         sleep($sleep)
         MouseClick("left")
      WEnd
      sleep(500)
      Else
        sleep(200)
      EndIf
   EndIf

EndFunc

The idea is that if the first range contains the color 0xF ... do this until the Else condition is met.

And I have a problem that it loops in the first condition and ignore the second, I did something wrong but I don't know what so I would be grateful for your help.

As for the coordinates, I verified it 100x, so that's not the problem here.

Edited May 21, 2022 by hc4fun

[Melba23]

Moved to the appropriate forum.

Moderation Team

[argumentum]

..I have no idea of what you're trying to get done BUT, Random(min,max,flag) , the flag is 0 or 1, not 15.
And, welcome to the forum 

[chelvas]

Declare the global variable for x, y, z, values.