Function in itself

[Shadows]

Func Check()
    If "f4ac08" = Hex(PixelGetColor(446, 219), 6) Then
        If "f4a408" = Hex(PixelGetColor(490, 130), 6) Then
            MouseClick ( "left", 800, 500 , 1, 0 )
            Sleep (3000)
            mail()
        Else
            sleep (1000)
            Check()
        EndIf
    Else
        Sleep (1000)
        Check()
    EndIf
EndFunc

In this script i wish to check if two pixels. If they are both the correct colour, then it will click the button.

And im having the problem where its Calling for a function in the function its self, this is causing some stacking problems.

How do i get around it?

[Melba23]

Shadows,

Welcome to the AutoIt forum.

What you are running into is known as recursion - take a look at the Recursion tutorial in the Wiki if you want to know more about it.

The trick is to keep the check loop alive rather then calling the check function over and over again. Here is one way you might do it - although there any many others:

Func Check()

    Local $fDone = False
    Do
        If "f4ac08" = Hex(PixelGetColor(446, 219), 6) And "f4a408" = Hex(PixelGetColor(490, 130), 6) Then
            MouseClick("left", 800, 500, 1, 0)
            Sleep(3000)
            mail()
            $fDone = True ; Now you can escape from the loop
        Else
            Sleep(1000)
        EndIf
    Until $fDone = True ; You keep looping until the mail function has been run

EndFunc   ;==>Check

Note also how you can get 2 If's on the same line.

Please ask if you have any questions.

M23

[Shadows]

Shadows,

Welcome to the AutoIt forum.

What you are running into is known as recursion - take a look at the Recursion tutorial in the Wiki if you want to know more about it.

The trick is to keep the check loop alive rather then calling the check function over and over again. Here is one way you might do it - although there any many others:

Func Check()

    Local $fDone = False
    Do
        If "f4ac08" = Hex(PixelGetColor(446, 219), 6) And "f4a408" = Hex(PixelGetColor(490, 130), 6) Then
            MouseClick("left", 800, 500, 1, 0)
            Sleep(3000)
            mail()
            $fDone = True ; Now you can escape from the loop
        Else
            Sleep(1000)
        EndIf
    Until $fDone = True ; You keep looping until the mail function has been run

EndFunc   ;==>Check

Note also how you can get 2 If's on the same line.

Please ask if you have any questions.

M23

AWESOME! i got it :D and thanks