Help with looping a search for a pixel color (with hotkey)

[East_Arctica]

I was trying to make an AutoIt script that any pixel with the color 0xB200 in the coordinate range 0,0 to 100, 200 but I also wanted it to repeatedly search for that pixel until I use my hotkey {esc} to stop the search, and {esc} to start it again. Heres my current code, the issue is that it won't do anything when I click {esc}
 

#include <AutoItConstants.au3>
 HotKeySet("{esc}","Loop")

$looping = 0

Func Loop()
   If $looping == 1 Then ; if it is searching stop searching
      $looping = 0
   Else
      Do
         $coord = PixelSearch(0, 0, 100, 200, 0xB200)
         If Not @error Then
            MouseClick($MOUSE_CLICK_LEFT, $coord[0], $coord[1], 2, 1)
            MsgBox(0, " Clicked at X:", $coord[0] & "and Y:" & $coord[1])
         EndIf

      Until $looping <> 0
      $looping = 1
   EndIf
EndFunc

Thanks in advance!

[badcoder123]

#include <AutoItConstants.au3>
HotKeySet("{esc}", "Loop")

$looping = 0
Loop()

While 1
    ConsoleWrite("Searching" & @LF)
    $coord = PixelSearch(0, 0, 100, 200, 0xB200)
    If Not @error Then
        MouseClick($MOUSE_CLICK_LEFT, $coord[0], $coord[1], 2, 1)
        MsgBox(0, " Clicked at X:", $coord[0] & "and Y:" & $coord[1])
    EndIf
WEnd

Func Loop()
    $looping = Not $looping
    ConsoleWrite("Paused" & @LF)
    While $looping
        Sleep(200)
    WEnd
EndFunc   ;==>Loop

 

[FrancescoDiMuro]

@East_Arctica
Which application are you trying to automate?
PixelSearch() is not the most reliable function to automate applications

[East_Arctica]

Im using this to automate chrome. (Should i just go down the javascript path instead?

[East_Arctica]

9 hours ago, badcoder123 said:

#include <AutoItConstants.au3>
HotKeySet("{esc}", "Loop")

$looping = 0
Loop()

While 1
    ConsoleWrite("Searching" & @LF)
    $coord = PixelSearch(0, 0, 100, 200, 0xB200)
    If Not @error Then
        MouseClick($MOUSE_CLICK_LEFT, $coord[0], $coord[1], 2, 1)
        MsgBox(0, " Clicked at X:", $coord[0] & "and Y:" & $coord[1])
    EndIf
WEnd

Func Loop()
    $looping = Not $looping
    ConsoleWrite("Paused" & @LF)
    While $looping
        Sleep(200)
    WEnd
EndFunc   ;==>Loop

 

Hmmm, First off you cant just use a function like this "Loop()" you need to call it like this Call("Loop") in order for it to run. Second, wouldn't the Loop needs to be run after it was defined? It is only sending "Paused" to console and never sends "Searching" when I try to use this. After some more testing, it seems that it wasn't detecting the ESC key being pressed, I edited it to +1 (Shift+1) and it showed the Paused when it started and the Paused when I clicked it. It then stopped the script with exit code 0 

[BrewManNH]

You don't need to use the Call function to call a function, it's redundant.

Here's a modified script that might work as intended.

#include <AutoItConstants.au3>
HotKeySet("{esc}", "Loop")
HotKeySet("{tab}", "_Exit")

$Paused = 0
;~ Loop()

While 1
     If Not $Paused Then
          ConsoleWrite("Searching" & @LF)
          $coord = PixelSearch(0, 0, 100, 200, 0xB200)
          If Not @error Then
               MouseClick($MOUSE_CLICK_LEFT, $coord[0], $coord[1], 2, 1)
               MsgBox(0, " Clicked at X:", $coord[0] & "and Y:" & $coord[1])
          EndIf
          Sleep(1000) ; <<<< Sleeps for 1 second before searching again
     Else
          ConsoleWrite("Paused" & @CRLF)
          Sleep(1000) ; << Used so it's not spamming the console all the time
     EndIf
WEnd

Func Loop()
     $Paused = Not $Paused
EndFunc   ;==>Loop
Func _Exit()
    Exit
EndFunc

 

[East_Arctica]

9 hours ago, BrewManNH said:

You don't need to use the Call function to call a function, it's redundant.

Here's a modified script that might work as intended.

#include <AutoItConstants.au3>
HotKeySet("{esc}", "Loop")
HotKeySet("{tab}", "_Exit")

$Paused = 0
;~ Loop()

While 1
     If Not $Paused Then
          ConsoleWrite("Searching" & @LF)
          $coord = PixelSearch(0, 0, 100, 200, 0xB200)
          If Not @error Then
               MouseClick($MOUSE_CLICK_LEFT, $coord[0], $coord[1], 2, 1)
               MsgBox(0, " Clicked at X:", $coord[0] & "and Y:" & $coord[1])
          EndIf
          Sleep(1000) ; <<<< Sleeps for 1 second before searching again
     Else
          ConsoleWrite("Paused" & @CRLF)
          Sleep(1000) ; << Used so it's not spamming the console all the time
     EndIf
WEnd

Func Loop()
     $Paused = Not $Paused
EndFunc   ;==>Loop
Func _Exit()
    Exit
EndFunc

 

Thanks Brew, that worked perfectly, didn't know about the Call not being needed. when i tried to use mine it was erroring so i was confused.