Sign in to follow this  
Followers 0
drx

Quick debug request for a small script

6 posts in this topic

Morning folks,

New to the forum and the program, so I'll say Hello! But to the point. Got a simple script here I am putting in my new images, but it keeps giving me the error:

"Line 4 (File""): Error:Illegal text at the end of statement (one statement per line)"

I'm not the best coder around, but I don't have any text at the end of this code that I can see. Can anyone spot the issue? Thanks in advance....

If FileExists("C:\temp\setup\Setup.exe") Then
RunWait("C:\temp\setup\Setup.exe")
Msgbox(0,"First Install","Application Setup Completed Succesfully! Starting Next Instalation...")
else call(_Error)
EndIf
If FileExists("C:\temp\setup2.exe") Then
RunWait("C:\temp\setup2.exe")
MsgBox(0,"Second Install","All Installations completed Succesfully! Exiting Script")
else call(_Error)
EndIf
Func _Error()
    MsgBox(0,"Setup","One or more required files are not found. Installation Aborted!")
    Exit
EndFunc

Share this post


Link to post
Share on other sites



Split the lines "else call(_Error)" into two lines:

If FileExists("C:\temp\setup\Setup.exe") Then
    RunWait("C:\temp\setup\Setup.exe")
    MsgBox(0, "First Install", "Application Setup Completed Succesfully! Starting Next Instalation...")
Else
    Call("_Error")
EndIf
If FileExists("C:\temp\setup2.exe") Then
    RunWait("C:\temp\setup2.exe")
    MsgBox(0, "Second Install", "All Installations completed Succesfully! Exiting Script")
Else
    Call("_Error")
EndIf
Func _Error()
    MsgBox(0, "Setup", "One or more required files are not found. Installation Aborted!")
    Exit
EndFunc   ;==>_Error

My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (NEW 2017-04-18 - Version 1.4.8.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2017-02-27 - Version 1.3.1.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2015-04-01 - Version 0.4.0.0) - Download - General Help & Support - Example Scripts
Excel - Example Scripts - Wiki
Word - Wiki
PowerPoint (2015-06-06 - Version 0.0.5.0) - Download - General Help & Support

Tutorials:
ADO - Wiki

 

Share this post


Link to post
Share on other sites

drx,

Welcome to the AutoIt forum. :x

There is nothing at the end of your code, but the error tells you to look at Line 4: :shifty:

else call(_Error)

You need to have the function call on a separate line to the Else - like this: :P

If FileExists("C:\temp\setup\Setup.exe") Then
    RunWait("C:\temp\setup\Setup.exe")
    MsgBox(0, "First Install", "Application Setup Completed Succesfully! Starting Next Instalation...")
Else
    _Error()
EndIf
If FileExists("C:\temp\setup2.exe") Then
    RunWait("C:\temp\setup2.exe")
    MsgBox(0, "Second Install", "All Installations completed Succesfully! Exiting Script")
Else
    _Error()
EndIf

Func _Error()
    MsgBox(0, "Setup", "One or more required files are not found. Installation Aborted!")
    Exit
EndFunc   ;==>_Error

By the way, there is no need to use Call - that should only used when you are evaluating a variable which holds the function name. You just use the function name directly as shown above. :nuke:

M23


Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind._______My UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites

Is that all?? Just split the lines eh? Man that's to simple to work :]

Thanks guys!

Share this post


Link to post
Share on other sites

Yeah, it's that simple!

The error message is exactly telling what's wrong. A look in the helpfile tells you the syntax of the statement. That's all :x


My UDFs and Tutorials:

Spoiler

UDFs:
Active Directory (NEW 2017-04-18 - Version 1.4.8.0) - Download - General Help & Support - Example Scripts - Wiki
OutlookEX (NEW 2017-02-27 - Version 1.3.1.0) - Download - General Help & Support - Example Scripts - Wiki
ExcelChart (2015-04-01 - Version 0.4.0.0) - Download - General Help & Support - Example Scripts
Excel - Example Scripts - Wiki
Word - Wiki
PowerPoint (2015-06-06 - Version 0.0.5.0) - Download - General Help & Support

Tutorials:
ADO - Wiki

 

Share this post


Link to post
Share on other sites

I was looking at a tutorial file that used the wrong syntax, took for granted they wouldnt' have typos.....lesson learned :]

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now
Sign in to follow this  
Followers 0