Help on this *not sure how to name it*

[DevilEvilGod]

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

[stormbreaker]

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

[kylomas]

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

[BrewManNH]

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

[kylomas]

@BrewmanNH,

Yes, a bad practice in my opinion. Perhaps there is a trade off that makes it acceptible.

kylomas

[BrewManNH]

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.

[Melba23]

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

[kylomas]

@BrewmanNH, M23,

...Bowing to the masters...

[DevilEvilGod]

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 May 7, 2012 by DevilEvilGod

[DevilEvilGod]

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