What's wrong with this?

[Pyrex]

$var = PixelGetColor(320,67)
$var2 = PixelGetColor(781,178)


If $var = 13552384 And $var2 Not 16718362 Then
Send ("1")
Sleep(1700)
EndIf

It keeps saying:

If $var = 13552384 And $var2 Not 16718362 Then
If ^ ERROR

Error in expression

All I'm trying to do is If PixelGetColor(320,67) is 13552384 AND PixelGetColor(781,178) does not equal 16718362 Then send bla bla bla

[Airwolf]

Change Not to <>.

$var = PixelGetColor(320,67)
$var2 = PixelGetColor(781,178)


If (($var = 13552384) And ($var2 <> 16718362)) Then
Send ("1")
Sleep(1700)
EndIf

Edited June 11, 2008 by Airwolf

[Pyrex]

That works, thank you

[Tomb]

Change Not to <>.

$var = PixelGetColor(320,67)
$var2 = PixelGetColor(781,178)


If (($var = 13552384) And ($var2 <> 16718362)) Then
Send ("1")
Sleep(1700)
EndIf

can really just do this

$var = PixelGetColor(320,67)
$var2 = PixelGetColor(781,178)


If $var = 13552384 And $var2 <> 16718362 Then
Send ("1")
Sleep(1700)
EndIf

[Airwolf]

can really just do this

$var = PixelGetColor(320,67)
$var2 = PixelGetColor(781,178)


If $var = 13552384 And $var2 <> 16718362 Then
Send ("1")
Sleep(1700)
EndIf

The parenthesis are a bit arbitrary, but there are hundreds of ways you could write the statement to produce the same result. I just got into the habit of encapsulating statements, because if you use Not, then the parenthesis are very important. For example:

$var = 1
$var2 = 2

; You will see Yup
If $var = 1 And Not ($var2 = 1) Then
    MsgBox(0,"","Yup")
Else
    MsgBox(0,"","Nope")
EndIf

; You will see Nope
If $var = 1 And Not $var2 = 1 Then
    MsgBox(0,"","Yup")
Else
    MsgBox(0,"","Nope")
EndIf

[Tomb]

don't really care, just telling the thread starter that he/she doesn't need to use them.