Rorka Posted September 22, 2009 Share Posted September 22, 2009 Okay here is a reconsturction of my code... $i = 0 Do MsgBox(0, "Value of $i is:", $i) $i = $i + 1 Until $i = 10 MsgBox(0, "Done", "") The problem is it doesnt do the "MsgBox(0, "Done", "")" it just ends... how come? Link to comment Share on other sites More sharing options...
Richard Robertson Posted September 22, 2009 Share Posted September 22, 2009 The code sample you provided does not "miss" the last message box. Link to comment Share on other sites More sharing options...
Rorka Posted September 22, 2009 Author Share Posted September 22, 2009 It doesnt show the "Done" message box though... Link to comment Share on other sites More sharing options...
JohnOne Posted September 22, 2009 Share Posted September 22, 2009 It does for me AutoIt Absolute Beginners Require a serial Pause Script Video Tutorials by Morthawt ipify Monkey's are, like, natures humans. Link to comment Share on other sites More sharing options...
Rorka Posted September 22, 2009 Author Share Posted September 22, 2009 It does for meso a blank message box shows up with the title Done? Link to comment Share on other sites More sharing options...
JohnOne Posted September 22, 2009 Share Posted September 22, 2009 so a blank message box shows up with the title Done?Yup AutoIt Absolute Beginners Require a serial Pause Script Video Tutorials by Morthawt ipify Monkey's are, like, natures humans. Link to comment Share on other sites More sharing options...
Rorka Posted September 22, 2009 Author Share Posted September 22, 2009 (edited) Yup Weird... it dont for me... are u using BETA? urm... nowit does :/ But my real code still refuses to work Do _log("Searching") $FirstObject = _BMReadUInt($Handle, ($ObjectManager + $ObjManagerFirstObject)) $curobject = _BMReadUInt($Handle, ($curobject + $ObjectNextOffset)) $Name = _BMReadASCIIString($Handle, _BMReadUint($Handle, (_BMReadUInt($Handle, ($curobject + $GAMEOBJECT_NAMEOFFSET_1)) + $GAMEOBJECT_NAMEOFFSET_2)), 30) $curGUID = _BMReadMemory($handle, ($curobject + 0x30), 'uint64') If $name="Rich Saronite Deposit" Then _log("Found: Saronite Deposit") _Move($handle, _getObjectX($handle, $curobject), _getObjectY($handle, $curobject), _getObjectZ($handle, $curobject)) Sleep(4000) _interact($handle, $curGUID) EndIf If $curobject = 0 Then $curobject = $Firstobject EndIf Until _GetX($handle, $pBase) = Round($XCoord) and _GetY($handle, $pBase) = Round($YCoord) and _GetZ($handle, $pBase) = Round($ZCoord) ;Fixhere _Move($handle, $XCoord, $YCoord, $ZCoord) Edited September 22, 2009 by Rorka Link to comment Share on other sites More sharing options...
JohnOne Posted September 22, 2009 Share Posted September 22, 2009 (edited) I'm not qualified to figure that code out mate But I'm unsure if you can put functions inside another functions parameters Edited September 22, 2009 by JohnOne AutoIt Absolute Beginners Require a serial Pause Script Video Tutorials by Morthawt ipify Monkey's are, like, natures humans. Link to comment Share on other sites More sharing options...
Rorka Posted September 22, 2009 Author Share Posted September 22, 2009 I'm not qualified to figure that code out mateBut I'm unsure if you can put functions inside another functions parametersWell the search works. Link to comment Share on other sites More sharing options...
JohnOne Posted September 22, 2009 Share Posted September 22, 2009 So what isnt it doing ? Is it kicking up an error, hanging, stuck in loop etc..... AutoIt Absolute Beginners Require a serial Pause Script Video Tutorials by Morthawt ipify Monkey's are, like, natures humans. Link to comment Share on other sites More sharing options...
SAG Posted September 22, 2009 Share Posted September 22, 2009 So what isnt it doing ?Is it kicking up an error, hanging, stuck in loop etc.....The code shown will show "Done" as the title of the msgbox and will show nothing in text of the box. Link to comment Share on other sites More sharing options...
TurionAltec Posted September 22, 2009 Share Posted September 22, 2009 It goes: 0 1 2 3 4 5 6 7 8 9 DONE as expected... Link to comment Share on other sites More sharing options...
Rorka Posted September 22, 2009 Author Share Posted September 22, 2009 So what isnt it doing ?Is it kicking up an error, hanging, stuck in loop etc.....Its not sending the _move function...Its difficlt to explain without a World of Warcraft account. Link to comment Share on other sites More sharing options...
JohnOne Posted September 23, 2009 Share Posted September 23, 2009 Like I say I'm no expert I would try ending your do loop with until, before starting your if loop Do stuff Until this event If this event = certain value Then Continue wíth other stuff Endif AutoIt Absolute Beginners Require a serial Pause Script Video Tutorials by Morthawt ipify Monkey's are, like, natures humans. Link to comment Share on other sites More sharing options...
GEOSoft Posted September 23, 2009 Share Posted September 23, 2009 (edited) Until (_GetX($handle, $pBase) = Round($XCoord)) and (_GetY($handle, $pBase) = Round($YCoord)) and (_GetZ($handle, $pBase) = Round($ZCoord)) ;Fixhere Edited September 23, 2009 by GEOSoft George Question about decompiling code? Read the decompiling FAQ and don't bother posting the question in the forums.Be sure to read and follow the forum rules. -AKA the AutoIt Reading and Comprehension Skills test.*** The PCRE (Regular Expression) ToolKit for AutoIT - (Updated Oct 20, 2011 ver:3.0.1.13) - Please update your current version before filing any bug reports. The installer now includes both 32 and 64 bit versions. No change in version number. Visit my Blog .. currently not active but it will soon be resplendent with news and views. Also please remove any links you may have to my website. it is soon to be closed and replaced with something else. "Old age and treachery will always overcome youth and skill!" 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