Delete Files older than x Minutes

[up2late]

Greetings,

I've looked through the forums but haven't had much success.  I am trying to write a script that takes a command line parameter of a path and deletes files older than 5 minutes.  I've hacked a few together but still can't get it working.  Can someone help me figure out where I went wrong?

I've attached the script I am working on and have my basic error checking included.

The array is working to determine which files need to be deleted, but the delete isn't occurring.  I am thinking it may be with my arrary function in the delete command?

Thanks for all of the help already.  Still new to the forums.

#include <File.au3>
#include <Array.au3>
#include <Date.au3>
#include <MsgBoxConstants.au3>

Dim $FilePath
If $CmdLine[0] <> 1 Then
Exit
Else
    $FilePath = $CmdLine[1]
    ; List all the files and folders in the desktop directory using the default parameters and return the full path.
    Local $aFileList = _FileListToArray($FilePath, Default, 1, Default)
    $date = _NowCalc()
    If IsArray($aFileList) Then
        For $i = 1 To $aFileList[0]
            $CreateTime = FileGetTime($FilePath & $aFileList[$i], 1)
            $CreateTimeArray = $CreateTime[0] & "/" & $CreateTime[1] & "/" & $CreateTime[2] & " " & $CreateTime[3] & ":" & $CreateTime[4] & ":" & $CreateTime[5]
            If _DateDiff('n', $CreateTimeArray, $date) > 3 Then
                ;MsgBox ("Test", $i,"")
                FileDelete($FilePath & $aFileList[$i])
            EndIf
        Next
        _ArrayDisplay($aFileList, "$aFileList")
    EndIf
EndIf

Edited December 24, 2014 by up2late

[BrewManNH]

Where is it failing?

Have you checked to see what FileGetTime is returning and what _NowCalc is returning and what $CreateTimeArray contains?

Are you sure that the creation time is what you're looking to get from FileGetTime and not the last modified time? Depending upon where the files are coming from there might be a difference.

is there a trailing backslash at the end of the $FilePath variable? Otherwise the path is going to look like "C:SomePathSomefilename.exe" rather than "C:SomePathSomefilename.exe"

Are any files returned from _FileListToArray?

You need error checking in your script before you can understand where it's not working. As posted you have none.

[guinness]

In my signature there is a function for checking the file date difference.

[Geir1983]

Do you need to add the path to FileDelete($files[$i])?

[TheSaint]

A proper date checking script, no doubt like the one by guinness, will also check the date, not just time, so that a few minutes after midnight (for example), will not be seen as older than a few minutes before midnight.

[Geir1983]

What do you mean TheSaint, he is using _DateDiff, this is already comparing dates?

[TheSaint]

I was just saying, and hadn't really looked at his code ... never do, when AutoIt code tags haven't been used ... unless it is only two or three lines. I just tend to read the responses of others I respect and see whether they have it in hand. My comment was really for those who may require it.

I see things here, to be as much about teaching, as it is about answering a question or request.

Edited December 23, 2014 by TheSaint

[up2late]

Thanks everyone for the help.  I have modified the original post with more information.  Still new to the forums and AutoIt.  I have the script working to determine which files need to be deleted, but the actual delete isn't occurring.  I am wondering if it's something with the FileDelete command and passing array values in?

Any help would be greatly apprciated.

Thanks again.

[Danp2]

I think this has already been mentioned above, but you need to indicate the full filename including path for the FileDelete to work as desired. You can either change the call to _FileListToArray so that it returns the path (see the $bReturnPath parameter) or you can change your call to FileDelete so that it specifies the directory along with the filename.

[up2late]

Ah got it, thanks!  I missed that part.  I added the path and it's working now.  Code has been updated.  Thanks again everyone!