SHould this function work?

[SteveO]

Okay, i've got this function checking to see if a box is checked on the GUI, and if the box is checked it's suppose to Enable certain input fields. It just seems to me that my IF Statements might be wrong, because the GUICtrlSetState are correct.

expandcollapse popup

Func UserCheck()
    
    
    If $Checkbox1 = $GUI_CHECKED Then
        GUICtrlSetState($Username2, $GUI_ENABLE)
        GUICtrlSetState($Password2, $GUI_ENABLE)
    EndIF
    
    If $Checkbox2 = $GUI_CHECKED Then
        GUICtrlSetState($Username3, $GUI_ENABLE)
        GUICtrlSetState($Password3, $GUI_ENABLE)
    EndIF
    
    If $Checkbox3 = $GUI_CHECKED Then
        GUICtrlSetState($Username4, $GUI_ENABLE)
        GUICtrlSetState($Password4, $GUI_ENABLE)
    EndIF
    
    If $Checkbox4 = $GUI_CHECKED Then
        GUICtrlSetState($Username5, $GUI_ENABLE)
        GUICtrlSetState($Password5, $GUI_ENABLE)
    EndIF
    
    If $Checkbox5 = $GUI_CHECKED Then
        GUICtrlSetState($Username6, $GUI_ENABLE)
        GUICtrlSetState($Password6, $GUI_ENABLE)
    EndIF
    
    If $Checkbox6 = $GUI_CHECKED Then
        GUICtrlSetState($Username7, $GUI_ENABLE)
        GUICtrlSetState($Password7, $GUI_ENABLE)
    EndIF
    
    If $Checkbox7 = $GUI_CHECKED Then
        GUICtrlSetState($Username8, $GUI_ENABLE)
        GUICtrlSetState($Password8, $GUI_ENABLE)
    EndIF
    
    If $Checkbox8 = $GUI_CHECKED Then
        GUICtrlSetState($Username9, $GUI_ENABLE)
        GUICtrlSetState($Password9, $GUI_ENABLE)
    EndIF
    
    If $Checkbox9 = $GUI_CHECKED Then
        GUICtrlSetState($Username10, $GUI_ENABLE)
        GUICtrlSetState($Password10, $GUI_ENABLE)
    EndIF
    
    If $Checkbox10 = $GUI_CHECKED Then
        GUICtrlSetState($Username11, $GUI_ENABLE)
        GUICtrlSetState($Password11, $GUI_ENABLE)
    EndIF
    
EndFunc

[GEOSoft]

No, but this should

If GUICtrlRead($Checkbox1) = $GUI_CHECKED Then

I would also add an Else Statement in each along with GUICtrlGetState() (to minimize flickering)

If GUICtrlRead($Checkbox1) = $GUI_CHECKED Then
        If GUICtrlGetState($Username2) > 80 Then GUICtrlSetState($Username2, 80)
        If GUICtrlGetState($Password2) > 80 Then GUICtrlSetState($Password2, 80)
    Else
        If GUICtrlGetState($Username2) < 144 Then GUICtrlSetState( $Username2, 144) 
        If GUICtrlGetState($Password2) < 144 Then  GUICtrlSetState( $Password2, 144)
    EndIF

Edit: Syntax error

Edited January 2, 2008 by GEOSoft

[SteveO]

Woot! Thank Alot! Works perfectly now!

[GEOSoft]

Woot! Thank Alot! Works perfectly now!

Glad it worked. Remember that it's always better to check for state before setting state. Otherwise you will be back asking "Why do my controls flicker?".