Elseif giving me grief

[BiggJohn]

Hi,

I am a casual user of AutoIT, and can usually puzzle it out when I've made a mistake, but I have to admit I'm stumped as to why this is a code violation:

Func UninstallClient()
    If FileExists(@WindowsDir & "\myexec\myexecexec.exe") or (@SystemDir & "\myexec\myexecexec.exe") Then
        If FileExists(@WindowsDir & "\myexecsetup\myexecsetup.exe") Then ShellExecuteWait("myexecsetup.exe","/uninstall",@WindowsDir & "\myexecsetup","",@SW_HIDE)
        Elseif FileExists(@SystemDir & "\myexec\myexecsetup.exe") Then ShellExecuteWait("myexecsetup.exe","/uninstall",@SystemDir & "\myexec","",@SW_HIDE)
        EndIf
    EndIf
 
EndFunc ;==UninstallClient

The error I get indicates I have a syntax error after adding in the nested ElseIf's.  Prior to that it looked like this:

Func UninstallClient()
    If FileExists(@WindowsDir & "\myexec\myexecexec.exe") or (@SystemDir & "\myexec\myexecexec.exe") Then
        If FileExists(@WindowsDir & "\myexecsetup\myexecsetup.exe") Then ShellExecuteWait("myexecsetup.exe","/uninstall",@WindowsDir & "\myexecsetup","",@SW_HIDE)
        if FileExists(@SystemDir & "\myexec\myexecsetup.exe") Then ShellExecuteWait("myexecsetup.exe","/uninstall",@SystemDir & "\myexec","",@SW_HIDE)
    EndIf
    
EndFunc ;==UninstallClient

Which just doesn't look right to me, however the script checker seems to think it's fine.  

Just to clarify what I'm after, I'm looking for a certain file which can be in one of two places, if it exists in either one, then launch an uninstaller which can also be on one of two (other) places.  I know this is not clean and could have been written better, so helpful suggestions as to how to better accomplish this are welcome, however, foremost I'm looking to find out why the Elseif causes a syntax error.

Thoughts?

Thanks!

John

Edited August 21, 2017 by BiggJohn

[Bowmore]

 

In this case you need to put the code after Then on a new line

Func UninstallClient()

    If FileExists(@WindowsDir & "\myexec\myexecexec.exe") Or (@SystemDir & "\CCM\ccmexec.exe") Then
        If FileExists(@WindowsDir & "\myexecsetup\myexecsetup.exe") Then
            ShellExecuteWait("myexecsetup.exe", "/uninstall", @WindowsDir & "\myexecsetup", "", @SW_HIDE)
        ElseIf FileExists(@SystemDir & "\myexec\myexecsetup.exe") Then
            ShellExecuteWait("myexecsetup.exe", "/uninstall", @SystemDir & "\myexec", "", @SW_HIDE)
        EndIf
    EndIf

EndFunc   ;==>UninstallClient

 

[BiggJohn]

I guess I never realized how much formatting counted.  

Thank you for such a speedy and helpful reply.

Cheers!

John