my loop doesn't work

[sofian89]

expandcollapse popup

HotKeySet("{ESC}", "MyExit")
Global $A=True




While $A=True
      Sleep(2000)
      MouseClick("left",486,706,1)
      Sleep(2000)
      MouseClick("left",602,314,1)
      Sleep(2000)
      MouseClick("left",713,445,1)
      Sleep(2000)
      MouseClick("left",794,584,1)
      Sleep(2000)
      Call("CheckStan")
WEnd






Func CheckStan() 

   While 1
         $CheckStan = PixelGetColor (869,304)
         If $CheckStan = 0x1E872B Then
            MouseClick("left",678,256,1 )
            Sleep(2000)
            MouseClick("left",820,700,1)
         EndIf

   WEnd

EndFunc

Func MyExit() ; Reduce The window and stop the script
    MouseClick("left",1334,16,1)
    Exit
EndFunc

my script supposed to hit some mouse click then check a pixel color then if its right he click again and repeat all the clicks again 

but my script dont 

[sofian89]

Func CheckStan()

i put this fonction on a loop because i want that then script wait until the button become green then do a click 

[BrewManNH]

Try this:

Func CheckStan()

     While 1
          $CheckStan = PixelGetColor(869, 304)
          If $CheckStan = 0x1E872B Then
               MouseClick("left", 678, 256, 1)
               Sleep(2000)
               MouseClick("left", 820, 700, 1)
               Return ; <<<<<<<<<<<<<<<<
          EndIf

     WEnd

EndFunc   ;==>CheckStan

It will check for the color, and when it finds that color, it returns to the main While loop. The way you have it written, once it goes into the ChckStan function, there's no way for it to ever return from it.

[sofian89]

Thank you bro 

i already add exitloop if he find the color so it’s will return to the main loop

[BrewManNH]

One other thing, not a major thing, you don't need to use Call when calling a function, just use the function name instead, for example
 

Call("Checkstan") 
; Can be done this way
Checkstan()

Saves you some typing in the long run.