checkbox and if statements

[Queener]

Don't know if this makes any sense, but my goal is to check each checkbox and if it's checked then do the following action. Though when I do the if statement, it doesn't write to file on all of them. I'm sure I might mis-used the if statements. Any help is appreciated

 

    if GUICtrlRead($cb1) = $gui_checked Then
    ;code below
        $sReplcb1 = stringregexpreplace($sFile , "(?s)(?<=//\[offstart\])" , "" & "Not available")
        filewrite(FileOpen($dummies, $FO_READ + $FO_OVERWRITE), $sReplcb1)
Elseif GUICtrlRead($cb1) = $gui_checked Then
    ;code below
        $sReplcb1 = stringregexpreplace($sFile , "(?s)(?<=//\[offstart\])" , "" & "Not available")
        filewrite(FileOpen($dummies, $FO_READ + $FO_OVERWRITE), $sReplcb1)
Else
exit
endif

If I check both boxes, only one codes will be written to file. Shouldn't it check the first if then to the second if because I'm using the elseif?

[BrewManNH]

An If statement only actions one comparison, whichever one it finds that matches the test first is actioned, so you can not expect it to do more comparisons after it has found a match.

You should use an And operator to test for all possibilities if you want it to check more than one checkbox at a time.

[AutoBert]

Try this way:

$bCB1=GUICtrlRead($cb1)
$bCB2=GUICtrlRead($cb2)
If $bCB1 = $gui_checked Then
    ;code below
    $sReplcb1 = StringRegExpReplace($sFile, "(?s)(?<=//\[offstart\])", "" & "Not available")
    FileWrite(FileOpen($dummies, $FO_READ + $FO_OVERWRITE), $sReplcb1)
EndIf
If $bCB2 = $gui_checked Then
    ;code below
    $sReplcb1 = StringRegExpReplace($sFile, "(?s)(?<=//\[offstart\])", "" & "Not available")
    FileWrite(FileOpen($dummies, $FO_READ + $FO_OVERWRITE), $sReplcb1)
EndIf
if Not $bCB1 And Not $bCB2 Then Exit

 

[Queener]

An If statement only actions one comparison, whichever one it finds that matches the test first is actioned, so you can not expect it to do more comparisons after it has found a match.

You should use an And operator to test for all possibilities if you want it to check more than one checkbox at a time.

how would i apply the and with if statement. I don't see how that would operate if i'm trying to compare more than 2 if in the future i want to add more.

 

 

edit: i think i have an idea. Instead, of looking for checked, why don't i look for not checked... 

Edited July 1, 2015 by asianqueen