AlmarM Posted March 30, 2009 Share Posted March 30, 2009 (edited) Hello, Did I miss something? It keeps going on, ignoreing the $_Pause and $Num... Why? expandcollapse popupFunc CreateLeftRight() $_Pause = Not $_Pause While $_Pause Local $MousePos = MouseGetPos() Local $Color = 0x88C859 Local $Read_Length = GUICtrlRead($Length) Send("2") Do PixelSearch($MousePos[0], $MousePos[1], $MousePos[0], $MousePos[1], $Color, 35) If @error Then TrayTip("Status", "Searching...", 1) MouseMove($MousePos[0] +1, $MousePos[1], 1) CreateLeftRight() Else TrayTip("Stauts", "Found.", 1) MouseClick("left", $MousePos[0], $MousePos[1], 1, 1) $Num += 1 EndIf Until $Num = $Read_Length WEnd $Num = 0 EndFunc #cs Func CreateLeftRight() $MousePos = MouseGetPos () $Color = 0x88C859 $ReadLength = GUICtrlRead ( $Length ) Send ( "2" ) While $Num <> $ReadLength + 1 PixelSearch( $MousePos[0], $MousePos[1], $MousePos[0], $MousePos[1], $Color, 35 ) If @Error Then TrayTip("", "SEARCHING...", 1) MouseMove ( $MousePos[0]+1, $MousePos[1], 1 ) CreateLeftRight() Else TrayTip("", "FOUND...", 1) MouseClick ( "Left", $MousePos[0], $MousePos[1], 1, 1 ) $Num += 1 EndIf WEnd EndFunc #ce P.S. The commented func was the first one made, I tried making it with "Do Until" AlmarM Edited March 30, 2009 by AlmarM Minesweeper A minesweeper game created in autoit, source available. _Mouse_UDF An UDF for registering functions to mouse events, made in pure autoit. 2D Hitbox Editor A 2D hitbox editor for quick creation of 2D sphere and rectangle hitboxes. Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted March 30, 2009 Moderators Share Posted March 30, 2009 AlmarM,Have you declared $Length as a Global variable? If not then within your function $Read_Length will be undefined and Until $Num = $Read_Length will never action - hence the never-ending Do...Until loop.M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area Link to comment Share on other sites More sharing options...
AlmarM Posted March 30, 2009 Author Share Posted March 30, 2009 AlmarM,Have you declared $Length as a Global variable? If not then within your function $Read_Length will be undefined and Until $Num = $Read_Length will never action - hence the never-ending Do...Until loop.M23Oh my ...Yep, not a Global.Thx AlmarM Minesweeper A minesweeper game created in autoit, source available. _Mouse_UDF An UDF for registering functions to mouse events, made in pure autoit. 2D Hitbox Editor A 2D hitbox editor for quick creation of 2D sphere and rectangle hitboxes. Link to comment Share on other sites More sharing options...
Recommended Posts
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 accountSign in
Already have an account? Sign in here.
Sign In Now