PDFファイルのルーピング

Question

私のプログラムは、ディレクトリに入り、それらを解析するためにpdfファイルを検索します。このプログラムは常に実行されているので、同じファイルをもう一度解析しないようにする必要があります。

リストを使用してファイル名を保存し、ファイル名が存在するかどうかを確認しました。

誰かが見て、何が間違っているかを確認することができれば、私のコードはそれに関しては機能しません。

FileInfo[] filePaths = di.GetFiles("*.pdf"); 
for (int i = 0; i < filePaths.Length; i++) 
{ 
    foreach (string fileName in usedFileNames) 
    { 
     if (fileName.Equals(filePaths[i].Name)) 
     { 
      isInList = true; 
     } 
     else 
     { 
      isInList = false; 
     } 
    } 
    if (isInList == false) 
    { 
     PDFReaderChooser chooser = new PDFReaderChooser(filePaths[i].Name); 
     usedFileNames.Add(filePaths[i].Name); 
    } 

} 

Answer 1

このお試しください：

for (int i = 0; i < filePaths.Length; i++) 
{ 
    bool isInList = false; 

    foreach (string fileName in usedFileNames) 
    { 
     if (fileName.Equals(filePaths[i].Name)) 
      isInList = true; 
    } 

    if (isInList == false) 
    { 
     Console.WriteLine("Not in list! #{0}", x); 
     usedFileNames.Add(filePaths[i].Name); 
    } 
} 

：私はあなたの質問にコメントとして、あなたは、break文を挿入する必要があるため、あなたが投稿したコードは次のように、動作しません

FileInfo[] filePaths = di.GetFiles("*.pdf"); 
foreach(FileInfo fInfo in filePaths) 
{ 
    if (!usedFileNames.Contains(fInfo.Name)) 
    { 
     PDFReaderChooser chooser = new PDFReaderChooser(fInfo.Name); 
     usedFileNames.Add(fInfo.Name); 
    } 
} 

を

とにかく、この質問回答に示されているテクニックの1つを使用することをお勧めします。

Answer 2

LINQ操作は、このはるかに簡潔な（usedFileNamesと仮定するとList<string>である）になるだろう含まれています：まだ

FileInfo[] filePaths = di.GetFiles("*.pdf"); 
foreach(FileInfo myInfo in filePaths) 
{ 
    if (!usedFileNames.Contains(myInfo.Name)) 
    { 
     PDFReaderChooser chooser = new PDFReaderChooser(myInfo.Name); 
     usedFileNames.Add(myInfo.Name); 
    } 

} 

Answer 3

より簡潔：

var fileNames = di.GetFiles("*.pdf") 
        .Select(f => f.Name) 
        .Where(n => !usedFileNames.Contains(n)); 
usedFileNames.AddRange(fileNames); 

foreach (var fileName in fileNames) 
{ 
    var chooser = new PDFReaderChooser(fileName); 
} 

これはうまく割り出しロジックを抽象化処理する必要があるファイル名（ループ外）、処理するロジック（ループ内）からのファイル名です。

Answer 4

他の回答は問題の解決策としてはより優れていますが、元のコードが機能しなかった理由は説明していません。問題は、アルゴリズムがisInList変数の値を上書きするため、リスト内の最後のファイルについてのみ真となることです。これにより、この問題が解決されます。

FileInfo[] filePaths = di.GetFiles("*.pdf"); 
for (int i = 0; i < filePaths.Length; i++) 
{ 
    isInList = false 
    foreach (string fileName in usedFileNames) 
    { 
     if (fileName.Equals(filePaths[i].Name)) 
     { 
      isInList = true; 
      break; 
     } 
    } 
    if (isInList == false) 
    { 
     PDFReaderChooser chooser = new PDFReaderChooser(filePaths[i].Name); 
     usedFileNames.Add(filePaths[i].Name); 
    } 
} 

usedFileNamesコレクションのListの代わりにHashSetを使用する方がよいと付け加えます。ハッシュセットは、それが所与のアイテムを含むかどうかを効率的に決定するように設計されている。このリストは、正しくリコールすると線形検索を行いますが、これは（多数の項目に対して）非効率的です。