Why won't this work?

[PartyPooper]

This has to be something I'm doing wrong, but for the life of me I can't see what. Are the operators I'm using to equate the Option 1, Option 2 and Option 3 values incorrect? The Help File has <> Test if two values are not equal. so I assume they are correct. Any help would be appreciated

#include <GUIConstants.au3>

Opt("GUIOnEventMode", 1)

$MainGUI = GUICreate("MyGUI", 200, 100, -1, - 1)

GUISetIcon(@SystemDir & "\pifmgr.dll", 27)

GUISetOnEvent($GUI_EVENT_CLOSE, "OnExit")

$ComboBox = GUICtrlCreateCombo("",10, 25, 100, 30)

GUICtrlSetData($ComboBox,"Option 1|Option 2|Option 3","Option 1")

GUICtrlSetOnEvent(-1,"OnComboChoice")

GUISetState(@SW_SHOW)

; GUI Message Loop

While 1

Sleep(200)

WEnd

Func OnComboChoice()

$Choice = GUICtrlRead($ComboBox)

If $Choice <> "Option 1" OR $Choice <> "Option 2" OR $Choice <> "Option 3" Then

MsgBox(262160,"ERROR", $Choice & " IS NOT equal to any test values")

Else

MsgBox(262160,"ERROR", $Choice & " IS equal to a test value")

EndIF

EndFunc

Func OnExit()

Exit

EndFunc

[SmOke_N]

Try replacing <> with == that way, if they are "Not" equal, it would do your "Else" statement, and if they are equal, they would do the immediate "if" statement.

The way you have it now, if it ever is "Not" equal, it would do your "Else" statement... <> is Less than or Greater than.

Edit:

As an example if you didn't want to change the entire code, you could do:

If Not $Choice <> "Option 1" OR Not $Choice <> "Option 2" OR Not $Choice <> "Option 3" Then

Edit 2:

Gawd!!, I make things so complicated... you could just switch it out like Larry said below!!

Edited November 12, 2005 by ronsrules

[PartyPooper]

Now I just feel stupid. Why didn't I pick that up.

Thanks Larry (and Ron for a good try )