Small error please help... error at bottom

[Mazdryk]

#include <Misc.au3>

HotKeySet("{F9}", "_exit")

Hotkeyset ("{1}", "_fight" )

AutoItSetOption( "PixelCoordMode", 1)

$wgh = WinGetHandle("filegrabber")

$color1 = "ox400000"

While 1

Sleep (1000) ;Keeps script running

WEnd

Func _Fight()

$Black3 = PixelSearch(883, 920, 883, 920, $color1,100, 1, $wgh)

$Black4 = PixelSearch(883, 924, 883, 924, $color1,100, 1, $wgh)

If (IsArray($Black3) = 0 or IsArray($Black4) = 0) Then

Send ("{F4 DOWN}")

Sleep (100)

Send ("{F4 UP}")

Else

$Black = PixelSearch(828, 920, 828, 920, $color1,100, 1, $wgh)

$Black2 = PixelSearch(828, 924, 828, 924, $color1,100, 1, $wgh)

If (IsArray($Black) = 0 or IsArray($Black2) = 0) Then

Send ("{F3 DOWN}")

Sleep (100)

Send ("{F3 UP}")

else

Send ("{2 DOWN}")

Sleep (100)

Send ("{2 UP}")

endif

EndFunc

Func _exit()

Sleep(3000)

Exit 0

EndFunc

GIVES me ERROR

Line 39

EndFunc

Error: "if" statement has no matching "Endif" statement.

[ahmed9100]

here it is

expandcollapse popup

#include <Misc.au3>
HotKeySet("{F9}", "_exit")
Hotkeyset ("{1}", "_fight" )
AutoItSetOption( "PixelCoordMode", 1)

$wgh = WinGetHandle("filegrabber")
$color1 = "ox400000"

While 1
Sleep (1000) ;Keeps script running
WEnd


Func _Fight()
$Black3 = PixelSearch(883, 920, 883, 920, $color1,100, 1, $wgh)
$Black4 = PixelSearch(883, 924, 883, 924, $color1,100, 1, $wgh)
If (IsArray($Black3) = 0 or IsArray($Black4) = 0) Then
    Send ("{F4 DOWN}")
    Sleep (100)
    Send ("{F4 UP}")
Else
    $Black = PixelSearch(828, 920, 828, 920, $color1,100, 1, $wgh)
    $Black2 = PixelSearch(828, 924, 828, 924, $color1,100, 1, $wgh)
    If (IsArray($Black) = 0 or IsArray($Black2) = 0) Then
        Send ("{F3 DOWN}")
        Sleep (100)
        Send ("{F3 UP}")
    else
        Send ("{2 DOWN}")
        Sleep (100)
        Send ("{2 UP}")
    endif
EndIf
EndFunc


Func _exit()
Sleep(3000)
Exit 0
EndFunc

[TheSaint]

@ahmed9100 - We have policies here about Game Automation, so at the very least, when you see a function called 'Fight' and it's pixel related, you should ask questions, and await an appropriate response before giving help.

@Mazdryk - You obviously know enough of the rules here, to use the window name of FileGrabber, but seemingly neglected to rename your function so it didn't look like game related. Of course, this could all just be a coincidence with the word Fight, but I doubt it.

I'll leave it up to a MOD to decide if you transgressed. I recall welcoming you in another post, which I should perhaps check.

EDIT

Yep, as I suspected, a >locked topic and warning.

Edited September 15, 2013 by TheSaint

[Melba23]

Mazdryk,

This is the last time you get asked to read the Forum rules - next time you get a holiday to force the issue.

ahmed9100,

Sensible advice there - please note it for the future.

M23