More complex conditional statements

[Lee Bussy]

I'm sure there's a simple way to do this but so far I seem to be making it too hard I guess. Consider this line:

If ($s_ConsolArc = "Archive" And $b_DoArchive = 1) Or $s_ConsolArc = "Consolidated" Then

What I am trying to do is enter the subsequent code if either of these condtions are true:

$s_ConsolArc = "Consolidated"

or

$s_ConsolArc = "Archive" And $b_DoArchive = 1

I assume I am making a mistake somewhere and that it is indeed possible to accomplish this, so a virtual beer for my foolishness to the person who helps.

[gruntydatsun]

If StringCompare($s_ConsolArc,"Archive") And $b_DoArchive = 1 OR $s_ConsolArc = "Consolidate" Then
    msgbox(1,"SUCCESS","SUCCESS")
EndIf

Edited November 19, 2011 by gruntydatsun

[water]

What do you think is wrong with the first code line in your post? I think the code does exactly what you describe below.

Edited November 19, 2011 by water

[Melba23]

Lee Bussy,

Are you saying that your posted If statement does not work?

It looks fine to me both logically and syntactically - furthermore I have just tested it without problem.

expandcollapse popup

; False
$s_ConsolArc = "Archive"
$b_DoArchive = 0

If ($s_ConsolArc = "Archive" And $b_DoArchive = 1) Or $s_ConsolArc = "Consolidated" Then
    ConsoleWrite("True" & @CRLF)
Else
    ConsoleWrite("False" & @CRLF)
EndIf

; True
$s_ConsolArc = "Archive"
$b_DoArchive = 1

If ($s_ConsolArc = "Archive" And $b_DoArchive = 1) Or $s_ConsolArc = "Consolidated" Then
    ConsoleWrite("True" & @CRLF)
Else
    ConsoleWrite("False" & @CRLF)
EndIf

; false
$s_ConsolArc = ""
$b_DoArchive = 1

If ($s_ConsolArc = "Archive" And $b_DoArchive = 1) Or $s_ConsolArc = "Consolidated" Then
    ConsoleWrite("True" & @CRLF)
Else
    ConsoleWrite("False" & @CRLF)
EndIf

; True
$s_ConsolArc = "Consolidated"
$b_DoArchive = 1

If ($s_ConsolArc = "Archive" And $b_DoArchive = 1) Or $s_ConsolArc = "Consolidated" Then
    ConsoleWrite("True" & @CRLF)
Else
    ConsoleWrite("False" & @CRLF)
EndIf

; True
$s_ConsolArc = "Consolidated"
$b_DoArchive = 0

If ($s_ConsolArc = "Archive" And $b_DoArchive = 1) Or $s_ConsolArc = "Consolidated" Then
    ConsoleWrite("True" & @CRLF)
Else
    ConsoleWrite("False" & @CRLF)
EndIf

Does it work for you too?

M23

[gruntydatsun]

Thats odd, it faulted for me the first time using:

If $s_ConsolArc = "Archive" And $b_DoArchive = 1 OR $s_ConsolArc = "Consolidate" Then
msgbox(1,"SUCCESS","SUCCESS")
Else
msgbox(1,"FAIL","FAIL")
EndIf

And now it works. Maybe I typod.

[water]

  On 11/19/2011 at 11:52 AM, 'gruntydatsun said:

If StringCompare($s_ConsolArc,"Archive") And $b_DoArchive = 1 OR $s_ConsolArc = "Consolidate" Then
    msgbox(1,"SUCCESS","SUCCESS")
EndIf

I think your code won't work because StringCompare only returns > 0 when "string1 is greater than string2".

[water]

In such code the brackets can make the difference. Please check here to understand how a statement is evaluated by AutoIt (called operator precedence)

[gruntydatsun]

So this statement works unbracketed because AND has higher precedence than OR

[water]

No. It's pure coincidence. AND and OR are one the same level and therefore the expression is evaluated from left to right.

  Quote

When more than one operator is used in an expression the order in which things happen is controlled by operator precedence. The precedence used in AutoIt is given below. Where two operators have the same precedence the expression is evaluated left to right.

From highest precedence to lowest:

NOT

^

* /

+ -

&

< > <= >= = <> ==

AND OR

That's how I understand it.

[gruntydatsun]

Hooray for coincidence!

[Lee Bussy]

Wow ... well a long story short but my code works now. Honestly I'm noit sure what I was doing wrong at this point, but it's good to know I wasn't completely off my rocker. I don't know if anyone noticed the time I posted but I'm UTC (-05:00) so I think a distinct lack of sleep was my worst problem.

Thanks to all who replied. Reminding me what I needed to know was called "operator precedence" was what got me to th section I was looking for as well.