Logic wrong

[Docfxit]

What am I doing wrong?  I'm trying to skip over some code if a file exists.

Opt("WinWaitDelay", 100)
Opt("WinTitleMatchMode", 4)
Opt("WinDetectHiddenText", 1)
Opt("MouseCoordMode", 0);While 1 <> 1
; Install Servant Salamander

If _OSVersion() = 'Win7' Or _OSVersion() = 'Win7SP1' Or _OSVersion() = 'Win7X64' Or _OSVersion() = 'Win7SP1x64' Then
    $ProgramFiles = "C:\Programs\"
Else
    $ProgramFiles = "C:\Program Files\"
EndIf

;*~*~*~*~*~*~*~*~*~*~ This is where my problem is
msgbox(48, "Salamander = ", $ProgramFiles & "Salamander\salamand.exe")
;This file does exist   C:\Programs\Salamander\salamand.exe
If not FileExists("$ProgramFiles & Salamander\salamand.exe") then
    ;This doesn't recognize that it does exist
Msgbox(48, "This should not display","This should not display")

EndIf
Exit
Func _OSVersion()
    Local $OS_Version, $servicepack_version
    $OS_Version = StringStripWS(StringRegExpReplace(@OSVersion, "(WIN_)|(Microsoft )|(Windows )|(\(TM\))|( Ultimate)", ""), 8)
    $OS_Version = StringRegExpReplace(StringRegExpReplace($OS_Version, '2008', 'Win7'), 'VISTA', 'Vista')
    If RegRead("HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion", 'CurrentBuildNumber') > 7000 Then $OS_Version = 'Win7'
    $servicepack_version = StringReplace(@OSServicePack, "Service Pack ", "SP")
    If @error = -1 Then $servicepack_version = ""

    ConsoleWrite($OS_Version & $servicepack_version & StringRegExpReplace(@OSArch, '(X86)', '') & @CRLF)
    Return $OS_Version & $servicepack_version & StringRegExpReplace(@OSArch, '(X86)', '')
EndFunc   ;==>_OSVersion

 

Thanks,

Docfxit

[water]

Shouldn't it be:

If Not FileExists($ProgramFiles & "Salamander\salamand.exe") Then

 

[Docfxit]

You're great. 

I changed it on my msgbox to test it, but not on the executable code.

Thank you very much.

Docfxit

[water]

 

[Docfxit]

This is a silly one to keep.  I don't think people need to waste their time to read it.

Is there a way of deleting this thread?

Thanks,

Docfxit

[water]

No. You can't delete threads.
But don't bother. There are many similar threads around