Schleife für PDF-Dateien

Question

Mein Programm geht in ein Verzeichnis und sucht nach PDF-Dateien, um sie zu analysieren. Dieses Programm läuft immer, deshalb muss ich sicherstellen, dass die gleiche Datei nicht erneut analysiert wird.

Ich habe eine Liste verwendet, um die Dateinamen zu speichern und dann zu überprüfen, ob sie da sind.

Mein Code funktioniert nicht in Bezug auf das, wenn jemand einen Blick darauf werfen kann, was falsch ist, würde es sehr geschätzt werden.

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

Versuchen Sie folgendes:

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); 
    } 
} 

Wie ich auf Ihre Frage kommentierte der Code, den Sie geschrieben funktioniert nicht, weil Sie eine break-Anweisung einfügen müssen, wie folgt aus:

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); 
    } 
} 

Wie auch immer, ich empfehle Ihnen, eine der in dieser Frage gezeigten Antworten zu verwenden.

Answer 2

A LINQ Enthält Betrieb so viel prägnanter machen würde (usedFileNames Annahme, daß ein 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

prägnant noch:

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); 
} 

Diese abstrahiert schön die Logik weg, die aus Zahlen, die Dateinamen müssen Sie (außerhalb der Schleife) von der Logik verarbeiten, die sie verarbeitet (innerhalb der Schleife).

Answer 4

Während die anderen Antworten bessere Lösungen für das Problem sind, erklären sie nicht, warum der ursprüngliche Code nicht funktioniert hat. Das Problem besteht darin, dass der Algorithmus den Wert der Variablen isInList überschreibt, der daher nur für die letzte Datei in der Liste gilt. Dies würde das Problem beheben:

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); 
    } 
} 

Ich möchte hinzufügen, dass es besser ist, eine HashSet statt einer Liste für Ihre usedFileNames Sammlung zu verwenden. Der Hash-Satz soll effizient bestimmen, ob er ein bestimmtes Element enthält. Die Liste, wenn ich mich richtig erinnere, führt eine lineare Suche durch, die (für große Stückzahlen) ineffizient ist.