Sign in to follow this  
Followers 0
DevilEvilGod

Help on this *not sure how to name it*

10 posts in this topic

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 advance

GUI.au3

Main.au3

Share this post


Link to post
Share on other sites



#2 ·  Posted (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 by MKISH

----------------------------------------

:bye: Hey there, was I helpful?

----------------------------------------

My Current OS: Win8 PRO (64-bit); Current AutoIt Version: v3.3.8.1

Share this post


Link to post
Share on other sites

#3 ·  Posted (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 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

Share this post


Link to post
Share on other sites

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 Gude
How 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

Share this post


Link to post
Share on other sites

@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

Share this post


Link to post
Share on other sites

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 Gude
How 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

Share this post


Link to post
Share on other sites

BrewManNH,

The control IDs will remain consecutive as long as they're created consecutively

Not 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._______My UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites

@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

Share this post


Link to post
Share on other sites

#9 ·  Posted (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.972

Global $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 by DevilEvilGod

Share this post


Link to post
Share on other sites

#10 ·  Posted

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

Share this post


Link to post
Share on other sites

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 account

Sign in

Already have an account? Sign in here.


Sign In Now
Sign in to follow this  
Followers 0