Sign in to follow this  
Followers 0
Shadows

Function in itself

3 posts in this topic

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?

Share this post


Link to post
Share on other sites



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. :P

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. :D

Please ask if you have any questions. :)

M23


Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind._______My UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites

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. :D

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:D and thanks :P

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now
Sign in to follow this  
Followers 0