Comparing results from GetFileTime()

[vivec45]

MsgBox(0, "File comparison", CmpFileAge("C:\f1.txt", "C:\f2.txt"))
 
Func CmpFileAge($FilePath, $FilePath2)
$t =  FileGetTime($FilePath)
$t1 = FileGetTime($FilePath2)
 
 
 
If Not @error Then
 
If $t[0] < $t1[0] And $t[1] < $t1[1] And $t[2] < $t2[2] And $t[3] < $t2[3] And $t[4] < $t2[4] And $t[5] < $t2[5] Then
     return "f1"
Else
     return "f2"
EndIf
 
EndIf
 
EndFunc

This is my first attempt at checking which file was last modified. I just want to know if theres a function I can use for this before I try to write something custom.

All I need out of this is which file was last modified.

Currently the script returns "f2" regardless of which was actually last modified.

I haven't thought too deeply about the way I'm doing it, like I said I want to know if there's a function for this already.

Thanks.

Edited November 4, 2011 by vivec45

[JohnOne]

I don't know where $t2 Comes from.

Try this, should return true if $FilePath Is newer than $FilePath2

If Not @error Then

Return ($t[0] + $t[1] + $t[2] + $t[3] + $t[4] + $t[5]) > ($t1[0] + $t1[1] + $t1[2] + $t1[3] + $t1[4] + $t1[5])

EndIf

[vivec45]

I don't know where $t2 Comes from.

Try this, should return true if $FilePath Is newer than $FilePath2

If Not @error Then

Return ($t[0] + $t[1] + $t[2] + $t[3] + $t[4] + $t[5]) > ($t1[0] + $t1[1] + $t1[2] + $t1[3] + $t1[4] + $t1[5])

EndIf

That was a typo, It's actually just the $t1.

And your solution works fine, thanks.

Edited November 4, 2011 by vivec45

[BrewManNH]

This might work more reliably:

MsgBox(0, "File comparison", CmpFileAge("C:f1.txt", "C:f2.txt"))

Func CmpFileAge($FilePath, $FilePath2)
    $t = FileGetTime($FilePath, 0, 1)
    $t1 = FileGetTime($FilePath2, 0, 1)
    If Not @error Then
        If $t < $t1 Then
            Return "F1 is older"
        ElseIf $t = $t1 Then
            Return "Files are the same age"
        Else
            Return "F2 is older"
        EndIf
    EndIf
EndFunc   ;==>CmpFileAge

EDIT: I forgot to include a check to see if they're the same age.

Edited November 4, 2011 by BrewManNH

[vivec45]

This might work more reliably:

MsgBox(0, "File comparison", CmpFileAge("C:f1.txt", "C:f2.txt"))
 
Func CmpFileAge($FilePath, $FilePath2)
    $t = FileGetTime($FilePath, 0, 1)
    $t1 = FileGetTime($FilePath2, 0, 1)
    If Not @error Then
        If $t < $t1 Then
            Return "F1 is older"
        ElseIf $t = $t1 Then
            Return "Files are the same age"
        Else
            Return "F2 is older"
        EndIf
    EndIf
EndFunc   ;==>CmpFileAge

EDIT: I forgot to include a check to see if they're the same age.

Thanks.

[JohnOne]

This might work more reliably:

Blimey!

I never knew you could compare arrays like that.

Nice.

[Jos]

Blimey!

I never knew you could compare arrays like that.

Nice.

He isn't comparing arrays

[JohnOne]

Ah! of course not.

Serves me right for not reading fully, duh!

[BrewManNH]

The problem with comparing the dates as individual pieces in an array is that there are far too many checks to be made to make it an easy comparison.

For instance, if the month on one file is 10 and the other is 11, but the days are 13 and 12 (November 12 and October 13, so I don't confuse anyone as to which date format I'm using), you couldn't just add them all together because those 2 numbers would be the same. You would need to check if the years are different, then check to see if the months are different, then days etc., it gets to be annoying fast.

When it comes back as a string "YYYYMMDDHHMMSS" the comparison is very straightforward.

[JohnOne]

But they come in that order in the array, so shirley it would still work.

[BrewManNH]

If you use the array return, to get a comparison that's as easy to code, you'd probably end up putting it into a string anyways.