adding timers in my loops

[EmyJaneAlovera]

Hi there my name is Emy , I'm a newbie autoit user. With the help of the documentation I successfully made few scripts . Now im trying to learn more by optimizing them,  I want to add timers to my loops so that if the pixel is not found within 10 seconds then exit the script. Again with the help of the documentation i came up with this below , however it does not give me any error but the timer doesn't work it will stay in the pixelsearch loop for ever. Im kinda stuck there , i tried several other things but no success either. Any help would be greatly appreciate

1. #include <MsgBoxConstants.au3>
2. $start = TimerInit()
3. Do
4.     PixelSearch(971,636,977,638,0xFFFFFF) 
5. Until Not @error Or TimerDiff($start) = 10000
6.     If TimerDiff($start) = 10000 Then
7.         MsgBox($MB_OK,"ERROR","TIMEOUT!!!",0)
8.         Exit
9.     ElseIf Not @error then
10.     EndIf
11. MouseClick("left", Random(800,1100), Random(580,666),5,0)

 

 

 

[water]

Welcome to AutoIt and the forum!

How high is the probability that TimerDiff returns exactly the value 10000?

[EmyJaneAlovera]

hello, and thank you for your response. i did in fact tried  >= and no success either.

 

[JockoDundee]

7 minutes ago, water said:

How high is the probability that TimerDiff returns exactly the value 10000?

Just a little higher than the probability that it returns exactly 10000 twice, in succeeding statements

Until Not @error Or TimerDiff($start) = 10000
    If TimerDiff($start) = 10000 Then

 

[SkysLastChance]

I am betting you only changed one.

You need to change both of them. 

#include <MsgBoxConstants.au3>
$start = TimerInit()
Do
    PixelSearch(971,636,977,638,0xFFFFFF)
Until Not @error Or TimerDiff($start) >= 10000
    If TimerDiff($start) >= 10000 Then
        MsgBox($MB_OK,"ERROR","TIMEOUT!!!",0)
        Exit
    ElseIf Not @error then
    EndIf

 

[EmyJaneAlovera]

omg so simple thx you guys , good call water!