Stupid Basic Question

[sheeple]

Really basic macro, want it to press F1 when the pixel is white, if not, delay 3000 and run agian in a loop.

Its coming up with an error "Else statement with no matching If statement...

First time using autoit and trying to write a simple macro, used ACTool forever but I think its time to move on. -.-

while 1 = 1
$var = PixelGetColor( 628 , 282 )
if Hex($var, 6) = FFFFFF Then Send ("{f1}")
else 
sleep (3000)
endif
Wend

[dorkbrains]

try this.

sleep(5000) ;gives you time to only open what you want the pixel color of

while 1 ;this only needs to be one

If Hex($var) = FFFFFF Then

Send ("{f1}") ;after "Then" start a new line

else

sleep (3000)

endif

Wend

[sheeple]

Added PixelGetColor back in and its giving me the following error

"Error parsing fuction call"

sleep(5000);gives you time to only open what you want the pixel color of

while 1;this only needs to be one
$var = PixelGetColor( 628 , 282 )
If Hex($var) = FFFFFF Then 
Send ("{f1}");after "Then" start a new line
else
sleep (3000)
endif
Wend

[oMBRa]

EDIT: I misread sorry

btw: If Hex($var) = 0xFFFFFF Then

Edited May 21, 2009 by oMBRa

[trancexx]

Hex() returns a string.

[ProgAndy]

You can check the Return value in 2 different ways:

;A)
If Hex($var) = "FFFFFF" Then 
;B)
If $var = 0xFFFFFF Then

Edited May 21, 2009 by ProgAndy

[Bam]

Some one else that used ACtools

Some differences between ACtools and AutoIt,

Thiers not a "Beautify" button but, you can get Tidy witch is just as nice if not nicer sense it backs up your scripts.

When trying to find out the color of a pixel you can't as easly get the RGB.

But once you get to know AutoIt, it becomes real nice.

GL

[Ixidior]

try this.

sleep(5000) ;gives you time to only open what you want the pixel color of

while 1 ;this only needs to be one

If Hex($var) = FFFFFF Then

Send ("{f1}") ;after "Then" start a new line

else

sleep (3000)

endif

Wend

you can use the IF BOOLEAN THEN WHATEVER only when you WON'T use any ELSE or END IF