Badly formatted "Func" statement

[Wolfeinstein]

Why i get this error? "Badly formatted "Func" statement"

Sorry for my bad english

#include <GUIConstants.au3>

#Region ### START Koda GUI section ### Form=

$Form1 = GUICreate("Defcon0815 LogApp by epvp", 172, 247, 499, 116)

$Button1 = GUICtrlCreateButton("Metin2", 8, 208, 153, 33, 0)

GUICtrlSetFont(-1, 14, 400, 0, "MS Sans Serif")

GUISetState(@SW_SHOW)

#EndRegion ### END Koda GUI section ###

While 1

$nMsg = GUIGetMsg()

Switch $nMsg

Case $GUI_EVENT_CLOSE

Exit

Case $F3

run ("d:\Jocuri\Metin2 RO\bypass.bat","d:\Jocuri\Metin2 RO")

EndSwitch

HotKeySet ( "{F2}", "log" )

WEnd

func log()

send ("accountname")

Send ("{Enter}")

sleep (1000)

send ("password")

sleep (1000)

Send ("{Enter}")

sleep (5000)

Send ("{Enter}")

EndFunc

[UEZ]

Use [ autoit] your code [ /autoit] to format your code example properly when posting an example!

Then you defined Log() as your function but Log() is already a build-in function (Calculates the natural logarithm of a number.)!

Use a different name, e.g. _Log()

#include <GUIConstants.au3>

#Region ### START Koda GUI section ### Form=
$Form1 = GUICreate("Defcon0815 LogApp by epvp", 172, 247, 499, 116)
$Button1 = GUICtrlCreateButton("Metin2", 8, 208, 153, 33, 0)
GUICtrlSetFont(-1, 14, 400, 0, "MS Sans Serif")
GUISetState(@SW_SHOW)
#EndRegion ### END Koda GUI section ###

While 1
    $nMsg = GUIGetMsg()
    Switch $nMsg
        Case $GUI_EVENT_CLOSE
            Exit
        Case $F3
            run ("d:\Jocuri\Metin2 RO\bypass.bat","d:\Jocuri\Metin2 RO")
    EndSwitch
    HotKeySet ( "{F2}", "_log" )
WEnd

func _log()
    send ("accountname")
    Send ("{Enter}")
    sleep (1000)
    send ("password")
    sleep (1000)
    Send ("{Enter}")
    sleep (5000)
    Send ("{Enter}")
EndFunc

Br,

UEZ

Edited January 15, 2011 by UEZ

[shanet]

Looks suspicious... What exactly is this for?

I have fixed your code and commented it so you can tell what is wrong:

#include <GUIConstants.au3>

HotKeySet ( "{F2}", "_log" )        ;Hotkey decleration is not put in the while loop.
#Region ### START Koda GUI section ### Form=
$Form1 = GUICreate("Defcon0815 LogApp by epvp", 172, 247, 499, 116)
$Button1 = GUICtrlCreateButton("Metin2", 8, 208, 153, 33, 0)
GUICtrlSetFont(-1, 14, 400, 0, "MS Sans Serif")
GUISetState()                   ;Does not need to contain @SW_SHOW
#EndRegion ### END Koda GUI section ###

While 1                         ;properly indented to make code easier to read
    Switch GUIGetMsg()          ;No need for $nMsg variable
        Case $GUI_EVENT_CLOSE
            Exit
        Case $Button1           ;Should be $Button1, not $F3 - That's the impression I get anyway
            run ("d:\Jocuri\Metin2 RO\bypass.bat","d:\Jocuri\Metin2 RO")
    EndSwitch
WEnd

func _log()                     ;Functions always start with an underscore
    send ("accountname")
    Send ("{Enter}")
    sleep (1000)
    send ("password")
    sleep (1000)
    Send ("{Enter}")
    sleep (5000)
    Send ("{Enter}")
EndFunc

shanet

[guinness]

UEZ makes a good point I always add "_" to the start of any Function e.g. _Custom() because this is what I read somewhere on the Forum. I was told it was good programming practice and now I understand why!

[Wolfeinstein]

thank you all!what an epic fail=)

[GEOSoft]

You will always get that error when you name your function the same as one of the AutoIt built in functions.

Func Run() would fail as would Func StringInStr(). By preceding the name with an underscore (or any legal character for that matter) will solve the problem. It's just an AutoIt convention that user defined functions start with an underscore but not absolutely required.