DevilEvilGod Posted May 7, 2012 Share Posted May 7, 2012 hi all im trying to do a gui that loads values from an ini file ill attach both files so u can see my problem is loading the keys on the ini file to the respectible label :/ For $a = $label_day1 To $label_day42 Step 1 for $b = 1 to 42 step 1 Next GUICtrlSetData ($a, IniRead ($path, "label_day", "day" & $b, "Error")) Next i tried like this but it only reads key 1 on the ini PS i search and search (maybe not hardly i admit ) but i dont know what to do next... thanks in advanceGUI.au3Main.au3 Link to comment Share on other sites More sharing options...
stormbreaker Posted May 7, 2012 Share Posted May 7, 2012 (edited) You could declare an array like $label_day[42]. New code:Global $label_day[42] For $a = 1 To 42 Step 1 GUICtrlSetData ($label_day[$a], IniRead ($path, "label_day", "day" & $a, "Error")) Next Edited May 7, 2012 by MKISH ---------------------------------------- :bye: Hey there, was I helpful? ---------------------------------------- My Current OS: Win8 PRO (64-bit); Current AutoIt Version: v3.3.8.1 Link to comment Share on other sites More sharing options...
kylomas Posted May 7, 2012 Share Posted May 7, 2012 (edited) DevilEvilGod, You are spinning all the way to the end of the loop controlled by $b before you run the guictrlsetdata. kylomas Edit: comment - If I may, you are also assumming that the control id's will remain consecutive. I think that this is bad programming technique. Edited May 7, 2012 by kylomas Forum Rules Procedure for posting code "I like pigs. Dogs look up to us. Cats look down on us. Pigs treat us as equals." - Sir Winston Churchill Link to comment Share on other sites More sharing options...
BrewManNH Posted May 7, 2012 Share Posted May 7, 2012 The control IDs will remain consecutive as long as they're created consecutively. 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...
kylomas Posted May 7, 2012 Share Posted May 7, 2012 @BrewmanNH, Yes, a bad practice in my opinion. Perhaps there is a trade off that makes it acceptible. kylomas Forum Rules Procedure for posting code "I like pigs. Dogs look up to us. Cats look down on us. Pigs treat us as equals." - Sir Winston Churchill Link to comment Share on other sites More sharing options...
BrewManNH Posted May 7, 2012 Share Posted May 7, 2012 For those not too familiar with Arrays and how to use them, a simple for/next loop with the control id variables is, IMHO, a perfectly acceptable way of accessing those control IDs. If you know what order you created them in, and you don't change it later in a rewrite, the order won't change. I wouldn't do it that way most of the time, but I can't see how it would be a bad coding practice to do it. 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...
Moderators Melba23 Posted May 7, 2012 Moderators Share Posted May 7, 2012 BrewManNH,The control IDs will remain consecutive as long as they're created consecutivelyNot altogether true - you must also not have deleted any controls previously as this would leave holes in the ControlID array which would be filled first. It is an undocumented trick and one which could no longer work at any time if the default behaviour were changed in the future. I use it a lot but fully realise that those sections might have to be changed at some time. I suppose the "proper" way to do it would be to store the returned ControlIDs in an array and then run through 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...
kylomas Posted May 7, 2012 Share Posted May 7, 2012 @BrewmanNH, M23, ...Bowing to the masters... Forum Rules Procedure for posting code "I like pigs. Dogs look up to us. Cats look down on us. Pigs treat us as equals." - Sir Winston Churchill Link to comment Share on other sites More sharing options...
DevilEvilGod Posted May 7, 2012 Author Share Posted May 7, 2012 (edited) thx for all your anxwers (my head spins)now gonna apply all that to my script and ill tell you guys how it went PS - i think i understood the control ids part and im not gonna change them in any way just their values so im guessing that is ok? (for now at least , im hunger for knoledge )hmmmm 2nd edit>"C:Program FilesAutoIt3SciTE..autoit3.exe" /ErrorStdOut "D:Documentsau3Main.au3"D:Documentsau3Main.au3 (15) : ==> Error in expression.:GUICtrlSetData ($label_day & [$a], IniRead ($path, "label_day", "day" & $a, "Error"))GUICtrlSetData (^ ERROR>Exit code: 1 Time: 1.972Global $label_day[42] Func R_Label_day() For $a = 1 To 42 Step 1 GUICtrlSetData ($label_day & [$a], IniRead ($path, "label_day", "day" & $a, "Error")) <--- error on this line :/ Next EndFunc Edited May 7, 2012 by DevilEvilGod Link to comment Share on other sites More sharing options...
DevilEvilGod Posted May 7, 2012 Author Share Posted May 7, 2012 now it works :/ Func R_Label_day() $a = IniRead ($path, "label_day", "day1" , "Error") For $c = $label_day1 To $label_day42 Step 1 $a = $a + 1 Select Case $a < 1 GUICtrlSetData ($c, "") Case $a > IniRead ($path, "label_day", "day_end" , "Error") GUICtrlSetData ($c, "") Case $a > 0 GUICtrlSetData ($c, $a) EndSelect Next EndFunc maybe not perfect but its what i came up with hehehe sugestions are apreciated @kylomas,BrewManSH, Melba23 thx for explanation but couldnt make it work with arrays 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