hampkade Posted June 6, 2015 Share Posted June 6, 2015 (edited) I am writing a script that backs up my Teamspeak.So far everything else works, except this:While 1=1 $Date = @MDAY & '_' & @MON & '_' & @YEAR Backup() FileChangeDir("C:/") $Yday = @YDAY Do $Tday = @YDAY Sleep(6000) Until $Tday <> $Yday WEndIt runs through the loop once then exits. I can't seem to fix this.Any suggestions? Edited June 6, 2015 by hampkade Link to comment Share on other sites More sharing options...
Bowmore Posted June 6, 2015 Share Posted June 6, 2015 I see no reason why this should ever exit, unless you have an EXIT statement in your Backup() function or some other code that causes your script to terminate. "Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the universe trying to build bigger and better idiots. So far, the universe is winning."- Rick Cook Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted June 6, 2015 Moderators Share Posted June 6, 2015 hampkade,There appears to be nothing wrong with the basic logic as you can see in this rather faster reproducer:While 1 = 1 $Yday = @SEC ConsoleWrite($Yday & @CRLF) Do $Tday = @SEC ConsoleWrite($Tday & @CRLF) Sleep(100) Until $Tday <> $Yday ConsoleWrite("Here" & @CRLF) WEndCould the Backup function be causing the exit - there is nothing in the snippet itself which would cause it.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...
hampkade Posted June 6, 2015 Author Share Posted June 6, 2015 (edited) This is the full code to the program.#include <MsgBoxConstants.au3> #include <FileConstants.au3> #include <Process.au3> #RequireAdmin Global $TSDIR = "C:/Teamspeak" Global $Date = "Default" Local $Yday = "" Local $Tday = "" While 1=1 $Date = @MDAY & '_' & @MON & '_' & @YEAR Backup() FileChangeDir("C:/") $Yday = @YDAY Do $Tday = @YDAY Sleep(6000) Until $Tday <> $Yday WEnd Func Backup() If ProcessExists("ts3server_win64.exe") Then ProcessClose ("ts3server_win64.exe") DirCopy($TSDIR, "C:/Teamspeakbackup/" & $Date) FileChangeDir($TSDIR) ShellExecute("ts3server_win64.exe") Else MsgBox($MB_SYSTEMMODAL,"TS", "Teamspeak server isn't running") EndIfI don't see anything that would cause the loop to exit Edited June 6, 2015 by hampkade Link to comment Share on other sites More sharing options...
Developers Jos Posted June 6, 2015 Developers Share Posted June 6, 2015 I hope there is an EndFunc at the end as well. Jos 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...
hampkade Posted June 6, 2015 Author Share Posted June 6, 2015 I hope there is an EndFunc at the end as well. Jos Oh god I can't believe I missed that >.< Link to comment Share on other sites More sharing options...
hampkade Posted June 6, 2015 Author Share Posted June 6, 2015 Upon adding the EndFunc, the script works perfectly Thanks for pointing out my idiocracy Link to comment Share on other sites More sharing options...
Developers Jos Posted June 6, 2015 Developers Share Posted June 6, 2015 (edited) Just install the separate Full SciTE4AutoIt3 installer which would have avoided this issue since it runs au3check before running it.Also tidying your source would have helped here. Jos Edited June 6, 2015 by Jos 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
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