komar98 Posted March 14, 2021 Share Posted March 14, 2021 Hi all, experiencing some difficulties with trying to brake my main loop from a function inside of a function file. This is the 2 files I use - first is the code in the "main": ............................................. $criticalError = 0 $criticalError = runStage4() Sleep(1000) If $criticalError == 1 Then ExitLoop EndIf ; Leave team to exit leaveTeam() Sleep(3000) ..................................................... Both below are part of the functional file: This is the function "runStage4()": ................................................ blablabla1(blablabla) passStage4Door(blablabla) Sleep(100) blablabla2(blablabla) ........................................................... This is the function "passStage4Door" checking part of what to do when it fails X many times: ............................................................. If $failCounter == 1 Then ;;;;;;;;; ElseIF $failCounter == 2 Then $CriticalError = 1 $failCounter = 0 EndIf $failCounter = $failCounter + 1 ToolTip($failCounter,0,0,"",0,"") WEnd Return $CriticalError ............................................................................................ So far, ExitLoop literally did nothing even when adding it own "While 1" cycle for runStage4(). Replacing ExitLoop with ContinueLoop resulted into skipping the passDoor and went straight for the next "blablabla2(blablabla)" instead of braking the cycle in the main file and going for "leaveTeam()" Any suggestions? Link to comment Share on other sites More sharing options...
Musashi Posted March 14, 2021 Share Posted March 14, 2021 20 minutes ago, komar98 said: Any suggestions? Yes, provide a simplified but complete script , not just a fragmentary description like : blablabla1(blablabla) passStage4Door(blablabla) Sleep(100) blablabla2(blablabla) "In the beginning the Universe was created. This has made a lot of people very angry and been widely regarded as a bad move." Link to comment Share on other sites More sharing options...
komar98 Posted March 14, 2021 Author Share Posted March 14, 2021 @Musashi I admit, I forgot a very significant detail which is Func runStage4() $CriticalError = 0 $done = 0 While $CriticalError == 0 And $done == 0 blablabla1(blablabla) passStage4Door(blablabla) Sleep(100) blablabla2(blablabla) $done = 1 WEnd Return $CriticalError EndFunc Sorry but runStage4() is 400 lines long and its just repeating mostly of this - I really cant make it any simpler than this and since only the passStage4Door is relevant and should be the one, causing the function to exit, this is why I renamed the rest to blabla... Link to comment Share on other sites More sharing options...
mLipok Posted March 14, 2021 Share Posted March 14, 2021 51 minutes ago, komar98 said: Any suggestions? My first sugestion is, to read this: How to post scriptsource and, I want to you to the forum. Signature beginning:* Please remember: "AutoIt"..... * Wondering who uses AutoIt and what it can be used for ? * Forum Rules ** ADO.au3 UDF * POP3.au3 UDF * XML.au3 UDF * IE on Windows 11 * How to ask ChatGPT for AutoIt Code * for other useful stuff click the following button: Spoiler Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind. My contribution (my own projects): * Debenu Quick PDF Library - UDF * Debenu PDF Viewer SDK - UDF * Acrobat Reader - ActiveX Viewer * UDF for PDFCreator v1.x.x * XZip - UDF * AppCompatFlags UDF * CrowdinAPI UDF * _WinMergeCompare2Files() * _JavaExceptionAdd() * _IsBeta() * Writing DPI Awareness App - workaround * _AutoIt_RequiredVersion() * Chilkatsoft.au3 UDF * TeamViewer.au3 UDF * JavaManagement UDF * VIES over SOAP * WinSCP UDF * GHAPI UDF - modest begining - comunication with GitHub REST API * ErrorLog.au3 UDF - A logging Library * Include Dependency Tree (Tool for analyzing script relations) * Show_Macro_Values.au3 * My contribution to others projects or UDF based on others projects: * _sql.au3 UDF * POP3.au3 UDF * RTF Printer - UDF * XML.au3 UDF * ADO.au3 UDF * SMTP Mailer UDF * Dual Monitor resolution detection * * 2GUI on Dual Monitor System * _SciLexer.au3 UDF * SciTE - Lexer for console pane * Useful links: * Forum Rules * Forum etiquette * Forum Information and FAQs * How to post code on the forum * AutoIt Online Documentation * AutoIt Online Beta Documentation * SciTE4AutoIt3 getting started * Convert text blocks to AutoIt code * Games made in Autoit * Programming related sites * Polish AutoIt Tutorial * DllCall Code Generator * Wiki: * Expand your knowledge - AutoIt Wiki * Collection of User Defined Functions * How to use HelpFile * Good coding practices in AutoIt * OpenOffice/LibreOffice/XLS Related: WriterDemo.au3 * XLS/MDB from scratch with ADOX IE Related: * How to use IE.au3 UDF with AutoIt v3.3.14.x * Why isn't Autoit able to click a Javascript Dialog? * Clicking javascript button with no ID * IE document >> save as MHT file * IETab Switcher (by LarsJ ) * HTML Entities * _IEquerySelectorAll() (by uncommon) * IE in TaskScheduler * IE Embedded Control Versioning (use IE9+ and HTML5 in a GUI) * PDF Related: * How to get reference to PDF object embeded in IE * IE on Windows 11 * I encourage you to read: * Global Vars * Best Coding Practices * Please explain code used in Help file for several File functions * OOP-like approach in AutoIt * UDF-Spec Questions * EXAMPLE: How To Catch ConsoleWrite() output to a file or to CMD *I also encourage you to check awesome @trancexx code: * Create COM objects from modules without any demand on user to register anything. * Another COM object registering stuff * OnHungApp handler * Avoid "AutoIt Error" message box in unknown errors * HTML editor * winhttp.au3 related : * https://www.autoitscript.com/forum/topic/206771-winhttpau3-download-problem-youre-speaking-plain-http-to-an-ssl-enabled-server-port/ "Homo sum; humani nil a me alienum puto" - Publius Terentius Afer"Program are meant to be read by humans and only incidentally for computers and execute" - Donald Knuth, "The Art of Computer Programming" , be and \\//_. Anticipating Errors : "Any program that accepts data from a user must include code to validate that data before sending it to the data store. You cannot rely on the data store, ...., or even your programming language to notify you of problems. You must check every byte entered by your users, making sure that data is the correct type for its field and that required fields are not empty." Signature last update: 2023-04-24 Link to comment Share on other sites More sharing options...
komar98 Posted March 14, 2021 Author Share Posted March 14, 2021 expandcollapse popupWhile 1 (main) ; Clearing stage 4 ; PassiveCheck() While 1 $criticalError = 0 $criticalError = runStage4() Sleep(1000) If $criticalError == 1 Then ExitLoop EndIf WEnd ; Leave team to exit HI leaveTeam() Sleep(3000) WEnd Func runStage4() $CriticalError = 0 autoTrackWithoutCheck() $CriticalError = passStage4Door() Sleep(100) Return $CriticalError autoTrackWithoutCheck() Return $CriticalError EndFunc Func passStage4Door($) $failCounter = 0 While $error == 1 If $failCounter == 1 Then Sleep(150) passiveAttack(1) Sleep(15000) passiveAttack(0) Sleep(250) $CriticalError = 1 Return $CriticalError Sleep(250) ElseIF $failCounter == 2 Then Sleep(150) $CriticalError = 1 Sleep(500) ;$failCounter = 0 EndIf $failCounter = $failCounter + 1 ToolTip($failCounter,0,0,"",0,"") WEnd Return $CriticalError EndFunc This is the code (like 1/10 of it). By trial and error with placing $CriticalError = 1 and Return $CriticalError in different places, I now for sure know that Func passStage4Door doesnt return the value to the main while . Now my question is, how to make it return it so the main while can see it. Link to comment Share on other sites More sharing options...
komar98 Posted March 15, 2021 Author Share Posted March 15, 2021 Okay, debugging something you dont really understand while extremely frustrated, is not a good idea.. If $failCounter == 1 Then Sleep(150) passiveAttack(1) Sleep(15000) passiveAttack(0) Sleep(250) $CriticalError = 1 Return $CriticalError Sleep(250) ElseIF $failCounter == 2 Then Sleep(150) $CriticalError = 1 Sleep(500) ContinueLoop EndIf This is the correct IF I was trying to return the value after the IF, while being infinitely stuck in that IF.... Link to comment Share on other sites More sharing options...
Developers Jos Posted March 15, 2021 Developers Share Posted March 15, 2021 Welcome to the AutoIt forum. Unfortunately you appear to have missed the Forum rules on your way in. Please read them now - particularly the bit about not discussing game automation - and then you will understand why you will get no help and this thread will now be locked. See you soon with a legitimate question I hope. The Moderation team SciTE4AutoIt3 Full installer Download page - Beta files Read before posting How to post scriptsource Forum etiquette Forum Rules Live for the present, Dream of the future, Learn from the past. Link to comment Share on other sites More sharing options...
Recommended Posts