drx Posted January 11, 2011 Share Posted January 11, 2011 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 Link to comment Share on other sites More sharing options...
water Posted January 11, 2011 Share Posted January 11, 2011 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 2022-02-19 - Version 1.6.1.0) - Download - General Help & Support - Example Scripts - WikiExcelChart (2017-07-21 - Version 0.4.0.1) - Download - General Help & Support - Example ScriptsOutlookEX (2021-11-16 - Version 1.7.0.0) - Download - General Help & Support - Example Scripts - WikiOutlookEX_GUI (2021-04-13 - Version 1.4.0.0) - DownloadOutlook Tools (2019-07-22 - Version 0.6.0.0) - Download - General Help & Support - WikiPowerPoint (2021-08-31 - Version 1.5.0.0) - Download - General Help & Support - Example Scripts - WikiTask Scheduler (NEW 2022-07-28 - Version 1.6.0.1) - Download - General Help & Support - Wiki Standard UDFs:Excel - Example Scripts - WikiWord - Wiki Tutorials:ADO - WikiWebDriver - Wiki Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted January 11, 2011 Moderators Share Posted January 11, 2011 drx,Welcome to the AutoIt forum. There is nothing at the end of your code, but the error tells you to look at Line 4: else call(_Error)You need to have the function call on a separate line to the Else - like this: 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 ;==>_ErrorBy 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. M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area Link to comment Share on other sites More sharing options...
drx Posted January 11, 2011 Author Share Posted January 11, 2011 Is that all?? Just split the lines eh? Man that's to simple to work :] Thanks guys! Link to comment Share on other sites More sharing options...
water Posted January 11, 2011 Share Posted January 11, 2011 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 My UDFs and Tutorials: Spoiler UDFs:Active Directory (NEW 2022-02-19 - Version 1.6.1.0) - Download - General Help & Support - Example Scripts - WikiExcelChart (2017-07-21 - Version 0.4.0.1) - Download - General Help & Support - Example ScriptsOutlookEX (2021-11-16 - Version 1.7.0.0) - Download - General Help & Support - Example Scripts - WikiOutlookEX_GUI (2021-04-13 - Version 1.4.0.0) - DownloadOutlook Tools (2019-07-22 - Version 0.6.0.0) - Download - General Help & Support - WikiPowerPoint (2021-08-31 - Version 1.5.0.0) - Download - General Help & Support - Example Scripts - WikiTask Scheduler (NEW 2022-07-28 - Version 1.6.0.1) - Download - General Help & Support - Wiki Standard UDFs:Excel - Example Scripts - WikiWord - Wiki Tutorials:ADO - WikiWebDriver - Wiki Link to comment Share on other sites More sharing options...
drx Posted January 11, 2011 Author Share Posted January 11, 2011 I was looking at a tutorial file that used the wrong syntax, took for granted they wouldnt' have typos.....lesson learned :] Link to comment Share on other sites More sharing options...
Recommended Posts
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 accountSign in
Already have an account? Sign in here.
Sign In Now