Supprimer des fichiers plus anciens qu'une date

vous avez été forcé de considérer seulement la partie date de l'horodatage de création puis la condition est satisfaite et le fichier sera supprimé (plus tôt) de toute façon je suggère quelques modifications à ce code:

static class Helpers {
    public static void DeleteOldFiles(string folderPath, uint maximumAgeInDays,
                                      params string[] filesToExclude) {
        DateTime minimumDate = DateTime.Now.AddDays(-maximumAgeInDays);

        var filesToDelete = Directory.EnumerateFiles(folderPath)
            .Where(x => !IsExcluded(x, filesToExclude));

        foreach (var eligibleFileToDelete in filesToDelete)
            DeleteFileIfOlderThan(eligibleFileToDelete, minimumDate);
    }

    private const int RetriesOnError = 3;
    private const int DelayOnRetry = 1000;

    private static bool IsExcluded(string item, string[] exclusions) {
        return exclusions.Contains(item, StringComparer.CurrentCultureIgnoreCase);
    }

    private static void DeleteFileIfOlderThan(string path, DateTime date)
    {
        for (int i = 0; i < RetriesOnError; ++i) {
            try {
                var file = new FileInfo(path);
                if (file.CreationTime < date)
                    file.Delete();
            }
            catch (IOException) {
                System.Threading.Thread.Sleep(DelayOnRetry);
            }
            catch (UnauthorizedAccessException) {
                System.Threading.Thread.Sleep(DelayOnRetry);
            }
        }
    }
}

Notes

• j'utilise toujours DateTime.Now , je suppose que pour ce genre d'opérations vous n'avez pas besoin de mesure de précision (et vous parlez de jours afin que votre fil peut avoir un horaire programmé de heures ).
• Si votre application utilise plusieurs fichiers journaux vous pouvez spécifier tous les paramètres, ils seront ignorés.
• si vous appelez DeleteOldFiles avec 0 pour maximumAgeInDays alors vous retarderez tous les fichiers journaux non utilisés (comme spécifié dans la liste d'exclusion).
• parfois des fichiers peuvent être utilisés (même si cela devrait se produire rarement dans votre cas). La fonction DeleteFileIfOlderThan essaiera de les supprimer après un court délai (il imite le comportement Explorer.exe ).

vous pouvez appeler cette fonction de cette façon:

Helpers.DeleteOldFiles(@"c:\mypath\", logAge, currentLog);

quelques autres notes:

• ce code ne combine pas chemin et nom de fichier mais si vous devez le faire , vous devez utiliser Path.Combine() , je suppose que vous ne voulez pas réinventer la roue à chaque fois pour vérifier si un chemin se termine avec un antislash arrière ou non.
• les opérations d'E / S peuvent échouer! Vérifiez toujours les exceptions.

je suppose qu'un problème supplémentaire doit être dans

File.Delete(string.Format("{0}/{1}", directory, file));

votre fichier est de type FileSystemInfo . Vous vouliez peut-être utiliser le fichier .Nom . Exemple: disons que répertoire est "c:\" et fichier points "c:\myfile.log" , votre code essaiera de supprimer " c:/c:\myfile.log ". J'ai du mal à deviner ce que vous avez dans ces variables.

le remplacement Correct est suggéré par @HenkHolterman:

file.Delete();