johnmcloud Posted July 23, 2012 Posted July 23, 2012 (edited) Hi guys, i don't understand why this not work: Global $Test = @HOUR & ":" & @MIN ConsoleWrite($Test) While 1 If $Test = "14:58" Then MsgBox(0,0,"Test") ExitLoop EndIf Sleep(50) WEnd My object is scheduler some function at specific time if i have the script opened ( play sound or a msgbox, like example ) I have make the If...WEnd in a loop but seems not work, what is the problem? Thanks Edited July 23, 2012 by johnmcloud
BrewManNH Posted July 23, 2012 Posted July 23, 2012 You never update the $Time variable, so the comparison in the If statement will only work if you start it at exactly 14:52. You need to move the line $Test = @HOUR & ":" & @MIN inside the While loop so that it gets updated. If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag GudeHow to ask questions the smart way! I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from. Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays. - ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script. - Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label. - _FileGetProperty - Retrieve the properties of a file - SciTE Toolbar - A toolbar demo for use with the SciTE editor - GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI. - Latin Square password generator
johnmcloud Posted July 23, 2012 Author Posted July 23, 2012 Yeah, you have right, thanks Do you think is a good choise add a Sleep in the While...WEnd? I have see some people using it, someone not. What do you think?
BrewManNH Posted July 23, 2012 Posted July 23, 2012 It's always best to put a sleep in a While loop like this one, because the script is going to eat up you CPU cycles because it's always running. If you're using a GUI and GUIGetMsg it's not needed but for something like this, I would definitely put in at least a Sleep(100), you can even go lower. Play with it to see what works best for your script. If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag GudeHow to ask questions the smart way! I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from. Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays. - ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script. - Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label. - _FileGetProperty - Retrieve the properties of a file - SciTE Toolbar - A toolbar demo for use with the SciTE editor - GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI. - Latin Square password generator
johnmcloud Posted July 23, 2012 Author Posted July 23, 2012 No, i don't have GUI and GUIGetMsg, so i'll use the Sleep Thanks again
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