kor Posted July 10, 2012 Posted July 10, 2012 (edited) I'm not sure why, but the While loop seems to process EVERYTHING all the time. It doesn't actually wait for a button to be pressed. Thus, I've had to do some ugly hacks to prevent parts of the loop from processing until very specific conditions are met. I am having a problem now that the program is done though that if a user clicks outside of the GUI or the gui loses focus then the whole thing crashes. I have posted the entire code and hopefully someone can help me see what I did wrong. Warning, this script relies a lot on our internal active directory so you might have to change a bit around to get it to run on your machines. snip Edited July 10, 2012 by kor
BrewManNH Posted July 10, 2012 Posted July 10, 2012 You have 2 controls in your script that don't exist, $cUnlock and $cReset, so the GUIGetMsg return will be 0 if nothing is being done on the GUI, and because these controls don't exist the value of the variables is going to be 0, so the Select Case for Case $msg = $cReset is going to be true because $msg = 0 and $cReset = 0. Either create those controls before entering the While loop, or change the values of the variables to 9999 so that they won't be acted on before they're created. 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
kor Posted July 10, 2012 Author Posted July 10, 2012 (edited) The controls do exist, but aren't created until after $hLookup is pressed. Does your suggestion still apply? If so, where would I define the values of these not yet existing controls? Edited July 10, 2012 by kor
BrewManNH Posted July 10, 2012 Posted July 10, 2012 You declare the 2 variables at the top of the script, I would give them a value there. 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
kor Posted July 10, 2012 Author Posted July 10, 2012 You declare the 2 variables at the top of the script, I would give them a value there.that did it. thank you!
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