butterfly Posted June 7, 2014 Share Posted June 7, 2014 is there a way to change more variables on a single line when an if statement is met? as example: Global XYOptionWidth = 3 If $XYOptionWidth = 3 Then $SelectedWidth = 1600 and $troll = 89 , $autoitforum = "I Love you all" any possible way to make it look as clean as written above? ps* i know above example does not work Link to comment Share on other sites More sharing options...
BrewManNH Posted June 7, 2014 Share Posted June 7, 2014 Not in a single line probably not, you'd have to enclose it in a multiline If statement. If $XYOptionWidth = 3 Then $SelectedWidth = 1600 $troll = 89 $autoitforum = "I Love you all" EndIf I wouldn't say no because someone may come along with some totally unwieldy code that allows you to do it. But the way I wrote it is much easier to troubleshoot and debug if something was to be wrong in it. butterfly 1 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 Link to comment Share on other sites More sharing options...
jguinch Posted June 7, 2014 Share Posted June 7, 2014 If your variables are not already declared, maybe you can use. If $XYOptionWidth = 3 Then Local $SelectedWidth = 1600, $troll = 89, $autoitforum = "I Love you all" This seems to be clean and understandable ? Spoiler Network configuration UDF, _DirGetSizeByExtension, _UninstallList Firefox ConfigurationArray multi-dimensions, Printer Management UDF Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted June 7, 2014 Moderators Share Posted June 7, 2014 jguinch, This seems to be clean and understandable ?Personally I would still go for an If...EndIF structure:If $XYOptionWidth = 3 Then Local $SelectedWidth = 1600, $troll = 89, $autoitforum = "I Love you all" EndIfI have never understood why anyone would want to cram everything onto one line at the expense of readability. Given the relative slowness of AutoIt, any tiny increase in time required to run a multi-line If...EndIf compared to a single line version is in my opinion well worth the increase in readability. 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...
mikell Posted June 7, 2014 Share Posted June 7, 2014 The one-liner is possible Global $SelectedWidth = 0 ,$troll = 0 , $autoitforum = "nul", $XYOptionWidth = 3 Msgbox(0,"", $SelectedWidth &@crlf& $troll &@crlf& $autoitforum) If $XYOptionWidth = 3 Then Execute( Assign("SelectedWidth" , "1600") and Assign("troll", "89") and Assign("autoitforum", "I Love you all") ) Msgbox(0,"", $SelectedWidth &@crlf& $troll &@crlf& $autoitforum) Though personally I would never use something like this Link to comment Share on other sites More sharing options...
Werty Posted June 7, 2014 Share Posted June 7, 2014 jguinch, Personally I would still go for an If...EndIF structure: If $XYOptionWidth = 3 Then Local $SelectedWidth = 1600, $troll = 89, $autoitforum = "I Love you all" EndIf I have never understood why anyone would want to cram everything onto one line at the expense of readability. Given the relative slowness of AutoIt, any tiny increase in time required to run a multi-line If...EndIf compared to a single line version is in my opinion well worth the increase in readability. M23 I'm one of those it's old habit from the 80's, programming on computers with 16KB, 48KB and 64KB of RAM, everything was crammed into as little as possible. So we are used to it, and having just as hard a time with a gazilion lines, as you have with reading crammed code. Some guy's script + some other guy's script = my script! Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted June 7, 2014 Moderators Share Posted June 7, 2014 Werty,I too started coding in that era (on a 1k RAM ZX-81) - but that did not stop me adopting more reasonable habits when the chance presented itself. As a young child I was also brought up in a house with no bathroom and an outside toilet - but I much preferred indoor plumbing when we moved to a new home. 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...
Werty Posted June 7, 2014 Share Posted June 7, 2014 Ahh, yes, problem is I didnt go to the bathroom for 20 years (well, more like 15 years, as in doing something else than programming) But anyways, you DO understand it anyway now Some guy's script + some other guy's script = my script! Link to comment Share on other sites More sharing options...
czardas Posted June 7, 2014 Share Posted June 7, 2014 I have never understood why anyone would want to cram everything onto one line at the expense of readability. Not at the expense of readability (smaller regions, overall perspective, navigation and sometimes conformity). The field of view is limited by both the width and height of your screen. operator64 ArrayWorkshop Link to comment Share on other sites More sharing options...
czardas Posted June 7, 2014 Share Posted June 7, 2014 (edited) If your variables are not already declared, maybe you can use. If $XYOptionWidth = 3 Then Local $SelectedWidth = 1600, $troll = 89, $autoitforum = "I Love you all" This seems to be clean and understandable ? Unless this has changed recently, variables can be redeclared in AutoIt. On a side note: it's generally not advisable to declare variables based upon a condition. Edited June 7, 2014 by czardas operator64 ArrayWorkshop Link to comment Share on other sites More sharing options...
jguinch Posted June 7, 2014 Share Posted June 7, 2014 I agree with you all, but I just ask to the OP question. Obviously, I do not use this type of variable declaration, except for the Guinness's challenges Czardas, has you said, variables can be redeclared in AutoIt : yes but i think it is a bad habit, no ? Spoiler Network configuration UDF, _DirGetSizeByExtension, _UninstallList Firefox ConfigurationArray multi-dimensions, Printer Management UDF Link to comment Share on other sites More sharing options...
czardas Posted June 7, 2014 Share Posted June 7, 2014 (edited) Czardas, has you said, variables can be redeclared in AutoIt : yes but i think it is a bad habit, no ? It makes little sense to redeclare a variable in most cases I can think of. If the variable already exists, why declare it again when all you have to do is change its value? However redeclaration can be used as a quick method to empty a large array. Edited June 7, 2014 by czardas operator64 ArrayWorkshop Link to comment Share on other sites More sharing options...
DatMCEyeBall Posted June 7, 2014 Share Posted June 7, 2014 @czardas You can just assign the value of the array to 0$avData = 0 "Just be fred, all we gotta do, just be fred." -Vocaliod "That is a Hadouken. A KAMEHAMEHA would have taken him 13 days and 54 episodes to form." - Roden Hoxha @tabhooked Clock made of cursors ♣ Desktop Widgets ♣ Water Simulation Link to comment Share on other sites More sharing options...
czardas Posted June 7, 2014 Share Posted June 7, 2014 (edited) @czardas You can just assign the value of the array to 0 $avData = 0 Not if you just want to delete the array's contents (to free memory) while still leaving the array intact - ready to be used again. Edited June 7, 2014 by czardas operator64 ArrayWorkshop Link to comment Share on other sites More sharing options...
DatMCEyeBall Posted June 8, 2014 Share Posted June 8, 2014 (edited) Re-dim it to 1 element, set the first one to "", and resize it to the original size after? I see where you're going, and it would be easier to just redeclare it - but would it be considered a "good coding practice"? Edited June 8, 2014 by DatMCEyeBall "Just be fred, all we gotta do, just be fred." -Vocaliod "That is a Hadouken. A KAMEHAMEHA would have taken him 13 days and 54 episodes to form." - Roden Hoxha @tabhooked Clock made of cursors ♣ Desktop Widgets ♣ Water Simulation Link to comment Share on other sites More sharing options...
czardas Posted June 8, 2014 Share Posted June 8, 2014 It was documented at one time (variables can be redeclared), I guess it still is. ReDim is best avoided due to its latency. I haven't made a speed comparison, but other than that I agree that redeclaration of an array for this reason may be a dubious thing to do. I mentioned it as being plausible. I imagine the fastest method would be to have an empty array template that you can assign to the old array. operator64 ArrayWorkshop Link to comment Share on other sites More sharing options...
mikell Posted June 8, 2014 Share Posted June 8, 2014 The helpfile says : "Declaring the same variable name again will erase all array values and reset the dimensions to the new definition." As this is not mentioned as "not recommended" let's assume that such a way is correct DatMCEyeBall 1 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