Блог

У меня есть папка с 198 файлами, среди них 25 файлов с датой последней записи 22.06.2014. Я пытаюсь получить этот набор последних измененных файлов, используя следующий код:

 public static IEnumerable<FileInfo> GetLatestFiles(string path)
{
    return new DirectoryInfo(path)
            .GetFiles("*.log")
            .GroupBy(f => f.Extension)
            .Select(g => g.OrderByDescending(f => f.LastWriteTime
            .Date
            .ToShortDateString())
            .First());
}
  

Однако он возвращает мне только один файл, потому что он также проверяет последнее изменение времени, несмотря на то, что я обрезал дату. Есть ли какой-либо подход для перечисления этого списка файлов без учета времени?

Легче определить, что вы делаете, если вы не просто выравниваете по точкам, а по вызовам верхнего уровня:

 public static IEnumerable<FileInfo> GetLatestFiles(string path)
{
    return new DirectoryInfo(path)
            .GetFiles("*.log")
            .GroupBy(f => f.Extension)
            .Select(g => g.OrderByDescending(f => f.LastWriteTime
                                                   .Date
                                                   .ToShortDateString())
                          .First());
}
  

Теперь более очевидно, что вы выполняете упорядочение на основе строк. Если вы не находитесь в культуре, которая сначала ставит год, затем месяц, затем день, это почти наверняка не то, что вы хотите. Вероятно, вас вообще не волнует строковое представление. Всегда избегайте преобразования в строку, если вы действительно не заинтересованы в текстовом представлении.

Далее вы делаете пару странных вещей:

• Вы извлекаете только файлы «* .log», но группируете по расширению
• Вы получаете только самый первый результат каждой группы после упорядочивания

Я подозреваю, что вы действительно хотите сгруппировать по дате и вернуть все в самую последнюю группу. Например:

 public static IEnumerable<FileInfo> GetLatestFiles(string path)
{
    return new DirectoryInfo(path)
            .GetFiles("*.log")
            .GroupBy(f => f.LastWriteTime.Date)
            .OrderByDescending(g => g.Key) // Order the groups by date (desc)
            .First(); // Just return the most recent group
}
  

(Обратите внимание, что это приведет к сбою, если в каталоге нет файлов… что вы хотите, чтобы это делало в этом случае?)

Вы также должны учитывать, какой часовой пояс вас интересует — два файла могут иметь одинаковую дату последнего изменения в одном часовом поясе, но не в другом…

Использование First() в конце разумно, чтобы вы получили только один файл. Если вы удалите его, все файлы будут упорядочены, как вы уже сделали.

Однако, используя уже созданный вами запрос, вы не получите то, что хотите. Вместо этого вы можете попробовать что-то вроде следующего:

  DirectoryInfo(path).GetFiles("*.log")
                    .Where(x=>x.LastWriteTime.ToShortDateString()=="06/22/2014");
  

Попробуйте снять First() :

 public static IEnumerable<FileInfo> GetLatestFiles(string path)
{
    return new DirectoryInfo(path)
            .GetFiles("*.log")
            .GroupBy(f => f.Extension)
            .Select(g => g.OrderByDescending(f => f.LastWriteTime
            .Date
            .ToShortDateString()));
}