if not

[gempie]

what am i doing wrong here? $split is not SKE or 201TL or 67GA or SPEEKS so it should give me the msgbox right?

[/code]
        $split ="t"
        
    If not $split = "SKE" _
        or not $split ="201TL" _
        or not $split ="67GA" _
        or not $split ="SPEEKS"Then
        
        MsgBox(262192, "Warning", $split&" is an unknown code")
;ContinueLoop
    EndIf
[code]

TY

[Monamo]

$split = "t"

If Not $split <> "SKE" _
        Or $split <> "201TL" _
        Or $split <> "67GA" _
        Or $split <> "SPEEKS" Then
    MsgBox(0, "Warning", $split & " is an unknown code")
    ;ContinueLoop
EndIf

[gempie]

$split = "t"

If Not $split <> "SKE" _
        Or $split <> "201TL" _
        Or $split <> "67GA" _
        Or $split <> "SPEEKS" Then
    MsgBox(0, "Warning", $split & " is an unknown code")
    ;ContinueLoop
EndIf

That doesn't seem right. Cause when $split = "SKE" then it still gives the msgbox....

Any idea's anyone?

thnx

[Monamo]

That's what I get for going over the code too quickly - try this:

$split = "t"

If $split <> "SKE" _
        And $split <> "201TL" _
        And $split <> "67GA" _
        And $split <> "SPEEKS" Then
    MsgBox(0, "Warning", $split & " is an unknown code")
    ;ContinueLoop
EndIf

[gempie]

That's what I get for going over the code too quickly - try this:

$split = "t"

If $split <> "SKE" _
        And $split <> "201TL" _
        And $split <> "67GA" _
        And $split <> "SPEEKS" Then
    MsgBox(0, "Warning", $split & " is an unknown code")
    ;ContinueLoop
EndIf

TY, that worked! while waiting i also found this solution:

[/code]
$split = "t"

If $split = "SKELET" _
Or $split = "201TL" _
Or $split = "67GA" _
Or $split = "SPEEKS" Then
    MsgBox(0, "Warning", $split & " is an known code")
    $split = $split
else 
    MsgBox(0, "Warning", $split & " is an unknown code")
   ;ContinueLoop
EndIf
[code]

[weaponx]

TY, that worked! while waiting i also found this solution:

[/code]
$split = "t"

If $split = "SKELET" _
Or $split = "201TL" _
Or $split = "67GA" _
Or $split = "SPEEKS" Then
    MsgBox(0, "Warning", $split & " is an known code")
    $split = $split
else 
    MsgBox(0, "Warning", $split & " is an unknown code")
  ;ContinueLoop
EndIf
[code]

Your code tags are switched, the forward slash goes on the last code tag.

[PsaltyDS]

what am i doing wrong here? $split is not SKE or 201TL or 67GA or SPEEKS so it should give me the msgbox right?

$split ="t"
        
    If not $split = "SKE" _
        or not $split ="201TL" _
        or not $split ="67GA" _
        or not $split ="SPEEKS"Then
        
        MsgBox(262192, "Warning", $split&" is an unknown code")
;ContinueLoop
    EndIf

You've already got your work-around, but it is important to see why the original didn't work:

Look in the help file under "Operators", and the section below the table that give the "Operator Precedence":

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

Notice that NOT has the highest precedence of all, and therefore gets checked FIRST.

So the first part of your If condition becomes:

; If not $split = "SKE"
If (not $split) = ("SKE")oÝ÷ Ù:,yèhÂËkx,mèZ½è§n%y©Ü¢wb¶*'²»§¶¬jëh×6$x = ""
ConsoleWrite("Debug: $x = " & $x & ";  Not $x = " & Not $x & @LF)
$x = "xyz"
ConsoleWrite("Debug: $x = " & $x & ";  Not $x = " & Not $x & @LF)oÝ÷ Ù(¦¦®²)භö[`·h¥êÚ¶àzÛ2¢ëa{M¢oÝ÷ Ù8^½©nzôß«)+b±Ç«µ¨§¶è¶f­r­æ¬·z.¶íî¢wb¶*'²Ê-âÂ)ejÊÆÞ¶»J¨º«ªlʦy©íjëh×6If $split <> "SKE" And $split <> "201TL" And $split <> "67GA" And $split <> "SPEEKS" Then

; Or, another way to put it...

If Not ($split = "SKE" Or $split = "201TL" Or $split = "67GA" Or $split = "SPEEKS") Then

Edited August 10, 2007 by PsaltyDS

[gempie]

thnx for the lesson!! preciate it alot!

Edited August 16, 2007 by gempie