Sign in to follow this  
Followers 0
Lee Bussy

More complex conditional statements

11 posts in this topic

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. :D

Share this post


Link to post
Share on other sites



#2 ·  Posted (edited)

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

Edited by gruntydatsun

Share this post


Link to post
Share on other sites

#3 ·  Posted (edited)

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 by water

My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (NEW 2017-04-18 - Version 1.4.8.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2017-02-27 - Version 1.3.1.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2015-04-01 - Version 0.4.0.0) - Download - General Help & Support - Example Scripts
Excel - Example Scripts - Wiki
Word - Wiki
PowerPoint (2015-06-06 - Version 0.0.5.0) - Download - General Help & Support

Tutorials:
ADO - Wiki

 

Share this post


Link to post
Share on other sites

Lee Bussy,

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

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

; 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? :oops:

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

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.

Share this post


Link to post
Share on other sites

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".

My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (NEW 2017-04-18 - Version 1.4.8.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2017-02-27 - Version 1.3.1.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2015-04-01 - Version 0.4.0.0) - Download - General Help & Support - Example Scripts
Excel - Example Scripts - Wiki
Word - Wiki
PowerPoint (2015-06-06 - Version 0.0.5.0) - Download - General Help & Support

Tutorials:
ADO - Wiki

 

Share this post


Link to post
Share on other sites

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


My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (NEW 2017-04-18 - Version 1.4.8.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2017-02-27 - Version 1.3.1.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2015-04-01 - Version 0.4.0.0) - Download - General Help & Support - Example Scripts
Excel - Example Scripts - Wiki
Word - Wiki
PowerPoint (2015-06-06 - Version 0.0.5.0) - Download - General Help & Support

Tutorials:
ADO - Wiki

 

Share this post


Link to post
Share on other sites

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

Share this post


Link to post
Share on other sites

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

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.

My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (NEW 2017-04-18 - Version 1.4.8.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2017-02-27 - Version 1.3.1.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2015-04-01 - Version 0.4.0.0) - Download - General Help & Support - Example Scripts
Excel - Example Scripts - Wiki
Word - Wiki
PowerPoint (2015-06-06 - Version 0.0.5.0) - Download - General Help & Support

Tutorials:
ADO - Wiki

 

Share this post


Link to post
Share on other sites

Hooray for coincidence!

Share this post


Link to post
Share on other sites

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.

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