Теряются файлы при упаковке, где искать причину?

Question

[MonkAlex]

Имеется простой код упаковки файлов в архив, с использованием стандартного System.IO.Compression с 4.5 дотнета:

var directories = new DirectoryInfo(folder);
var files = directories
  .GetFiles("*", SearchOption.TopDirectoryOnly)
  .Select(f => f.FullName);
foreach (var file in files)
{
  var fileName = file
    .Replace(directories.FullName + Path.DirectorySeparatorChar, string.Empty)
    .Replace(directories.FullName, string.Empty);
  var fileInZip = zip.Entries.FirstOrDefault(f => f.FullName == fileName);
  if (fileInZip != null)
    fileInZip.Delete();
  zip.CreateEntryFromFile(file, fileName, CompressionLevel.NoCompression);
}

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

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

Answer 1

[Виталий Пухов]

похоже что проблема тут

var fileInZip = zip.Entries.FirstOrDefault(f => f.FullName == fileName);
 if (fileInZip != null)
 fileInZip.Delete();

тут идет поиск первого элемента, но чисто теоретически элемент может быть не один, если упаковано не совсем правильно. в таком случае будет удаление первого и попытка создать новую запись, которая скорее всего либо не отработает либо отработает создав 2 записи на 1 файл (путь)

Comments

[MonkAlex]

По идее, поиск идёт по FullName, который включает в себя относительный путь. Плюс, учитываем, что удаляются только файлы, которые есть в папке и будут заново добавлены. Единственное что мне в голову приходит, это вместо удаления и добавления - скипать файл. Как минимум не будет явно вызванного удаления. Це сделать не сложно. Больше идей нет?

[Виталий Пухов]

MonkAlex: можно повесить вторую итерацию var fileInZip = zip.Entries.FirstOrDefault(f => f.FullName == fileName);
if (fileInZip != null)
и если вдруг не null то это тот случай, что я описал, если нет нужно проверять дальше