Why won't my script work?

[BillLuvsU]

Why won't this script work in a function:

$startup = 1

if $startup = 1 then

$temp = 1

$pass=inputbox("Password", "Password")

if $pass <> "autoit" then

msgbox(0, "Password","Incorrect Password")

$pass2=inputbox("Password", "Password")

if $pass2 <> "uhcakip" then

shutdown(4) else

sleep (10) elseif $error=1 then

shutdown(4)

endif

elseif $error=1 then

shutdown(4)

else

sleep (10)

endif

endif

[killaz219]

im guessing it's the

Shutdown(4) Else
         Sleep(10) ElseIf $error = 1 Then

[Blue_Drache]

Why won't this script work in a function:

CODE

If $startup = 1 Then

  $temp = 1

  $pass = InputBox("Password", "Password")

  If $pass <> "autoit" Then

      MsgBox(0, "Password", "Incorrect Password")

      $pass2 = InputBox("Password", "Password")

      If $pass2 <> "uhcakip" Then

        Shutdown(4)

      Else

        Sleep(10)

      ElseIf $error = 1 Then

        Shutdown(4)

      EndIf

  ElseIf $error = 1 Then

      Shutdown(4)

  Else

      Sleep(10)

  EndIf

EndIf

You put the cart before the horse. You have an "else" in front of an "elseif"

Edit:Killa, you fast...............

Edited January 6, 2005 by Blue_Drache

[BillLuvsU]

I canged it but now it keeps saying there is no if statement for my endif

[BillLuvsU]

heres the full thing

func password()
$startup = 1
if $startup = 1 then
$temp = 1
$pass=inputbox("Password", "Password")
if $pass <> "autoit" then 
msgbox(0, "Password","Incorrect Password")
$pass2=inputbox("Password", "Password")
if $pass2 <> "uhcakip" then
shutdown(4) 
else
sleep (10) 
elseif $error=1 then
shutdown(4)
endif
elseif $error=1 then
shutdown(4)
else 
sleep (10)
endif
endif
end func
winwaitactive("Startup")
password()

Edited January 6, 2005 by fear1313

[Blue_Drache]

I canged it but now it keeps saying there is no if statement for my endif

<{POST_SNAPBACK}>

CODE

$Startup = 1

If $startup = 1 Then

$temp = 1

$pass = InputBox("Password", "Password")

If $pass <> "autoit" Then

MsgBox(0, "Password", "Incorrect Password")

$pass2 = InputBox("Password", "Password")

If $pass2 <> "uhcakip" Then

msgbox(0,"","test")

ElseIf $error = 1 Then

msgbox(0,"","test")

Else

Sleep(10)

EndIf

ElseIf $error = 1 Then

msgbox(0,"","test")

Else

Sleep(10)

EndIf

EndIf

Works fine for me. Just crashes when it's looking for $error because the rest of the code's not here or you're forgetting to set it.

BTW, change the msgbox(0,"","test") back to your rebooting command.

[Jos]

Here is the tidy output.... your have a "end func" which needs to be EndFunc.

Also review your logic now its indented..

Func password()
    $startup = 1
    If $startup = 1 Then
        $temp = 1
        $pass = InputBox("Password", "Password")
        If $pass <> "autoit" Then
            MsgBox(0, "Password", "Incorrect Password")
            $pass2 = InputBox("Password", "Password")
            If $pass2 <> "uhcakip" Then
                Shutdown(4)
            Else
                Sleep(10)
            ElseIf $error = 1 Then
                Shutdown(4)
            EndIf
        ElseIf $error = 1 Then
            Shutdown(4)
        Else
            Sleep(10)
        EndIf
    EndIf
    end Func
    WinWaitActive("Startup")
    password()

Edited January 6, 2005 by JdeB

[BillLuvsU]

Did you try it in a function? Cause thats when it fs up

[BillLuvsU]

gtg

[Blue_Drache]

Did you try it in a function? Cause thats when it fs up

<{POST_SNAPBACK}>

CODE

bootit()

Func Bootit()

$Startup = 1

If $Startup = 1 Then

$temp = 1

$pass = InputBox("Password", "Password")

If $pass <> "autoit" Then

MsgBox(0, "Password", "Incorrect Password")

$pass2 = InputBox("Password", "Password")

If $pass2 <> "uhcakip" Then

MsgBox(0, "", "test")

ElseIf $error = 1 Then

MsgBox(0, "", "test")

Else

Sleep(10)

EndIf

ElseIf $error = 1 Then

MsgBox(0, "", "test")

Else

Sleep(10)

EndIf

EndIf

EndFunc ;==>Bootit

Yup. Works fine.

[BillLuvsU]

mail me

[Jos]

Did you try it in a function? Cause thats when it fs up

<{POST_SNAPBACK}>

no need too... did you change the "end func" to "endfunc" ?

[Blue_Drache]

JdeB: Maybe he needs SciTE.

[Jos]

JdeB: Maybe he needs SciTE.   

<{POST_SNAPBACK}>

yeap and start using Tidy... it really helps a lot with these kind of problems...

[phillip123adams]

   ElseIf $error = 1 Then

   ElseIf $error = 1 Then

<{POST_SNAPBACK}>

$error or @error?