Delete file only if exists

[OmarPapi]

Hello friends! 

I want to delete the file only if it exists, if not, must not appear msgbox

code:

$MyBox = MsgBox(1, "Cancel file", "the file records.dat exists .. want to delete it!")
If $MyBox == 1 Then
if FileExist  then 
FileRecycle("records.dat") 
return false
EndIf

 

[Melba23]

OmarPapi,

Place the FileExists condition outside the MsgBox/FileRecycle code, which then only runs if the file exists..

M23

[OmarPapi]

11 minutes ago, Melba23 said:

M23

$MyBox = MsgBox(1, "Cancel file", "the file records.dat exists .. want to delete it!")
If $MyBox == 1 Then
EndIf
if FileExist  then 
FileRecycle("records.dat") 
return false
EndIf

no working

Edited February 4, 2016 by OmarPapi

[Melba23]

OmarPapi,

As you have no done what I suggested:

20 minutes ago, Melba23 said:

Place the FileExists condition outside the MsgBox/FileRecycle code, which then only runs if the file exists

I am not surprised. This is what I meant:

#include <MsgBoxConstants.au3>

$sFile = "records.dat"

; Check if file exists
If FileExists($sFile) Then
    ; If it does then ask if it should be deleted
    If MsgBox($MB_OKCANCEL, "Cancel file", "the file records.dat exists .. want to delete it!") = $IDOK Then
        ; And delete it if required
        FileRecycle($sFile)
        ;Return False
    EndIf
EndIf

Please ask if you have any questions.

M23

[OmarPapi]

10 hours ago, Melba23 said:

M23

this code is correct ?

:

$sFile  = " records1.dat"
$sFile2 = " records2.dat"
$sFile3 = " records3.dat"
$sFile4 = " records4.dat"

If FileExists($sFile) Then
    If MsgBox($MB_YESNO, "Cancel file", "the file records1.dat exists .. want to delete it!") = $IDYES Then
        FileRecycle($sFile)
    EndIf
EndIf
If FileExists($sFile2) Then
    If MsgBox($MB_YESNO, "Cancel file", "the file records2.dat exists .. want to delete it!") = $IDYES Then
        FileRecycle($sFile2)
    EndIf
EndIf
If FileExists($sFile3) Then
    If MsgBox($MB_YESNO, "Cancel file", "the file records3.dat exists .. want to delete it!") = $IDYES Then
        FileRecycle($sFile3)
    EndIf
EndIf
If FileExists($sFile4) Then
    If MsgBox($MB_YESNO, "Cancel file", "the file records4.dat exists .. want to delete it!") = $IDYES Then
        FileRecycle($sFile4)
    EndIf
EndIf

 

Edited February 4, 2016 by OmarPapi

[InunoTaishou]

Did you try running the code and see if it works?

Also, if you're going to be checking for a lot of files I'd suggest storing the file names in an array and using a for loop instead (or if your files are all named the same with an additional number you can forego the array and just use a string)

; Use an array to hold the names
Local $aFiles[] = ["my file.txt", "my other file.txt", "some random file.bat", "broken script.au3", "random code.au3"]

For $i = 0 to UBound($aFiles) - 1
    ; Good practice to include the full path to the file, even if it's in the same directory as the script
    If (FileExists(@ScriptDir & "\" & $aFiles[$i])) Then
        If MsgBox($MB_YESNO, "Cancel file", "the file " & $aFiles[$i] & " exists .. want to delete it!") = $IDYES Then
            FileRecycle(@ScriptDir & "\" & $aFiles[$i])
        EndIf
    EndIf
Next

; Same concept just not using an array, works well if the files are all the same name but have sequential numbers
For $i = 1 To 4
    If (FileExists(@ScriptDir & "\records" & $i & ".dat")) Then
        If MsgBox($MB_YESNO, "Cancel file", "the file records" & $i & ".dat exists .. want to delete it!") = $IDYES Then
            FileRecycle(@ScriptDir & "\records" & $i & ".dat")
        EndIf
    EndIf
Next