en boucle pour les fichiers PDF

Question

Mon programme va dans un répertoire et recherche des fichiers pdf pour les analyser. Ce programme est toujours en cours d'exécution, donc je dois m'assurer de ne pas analyser à nouveau le même fichier.

J'ai utilisé une liste pour stocker les noms de fichiers et ensuite vérifier s'ils sont là.

Mon code ne fonctionne pas à cet égard, si quelqu'un peut jeter un oeil et voir ce qui est faux, il serait grandement apprécié.

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

Essayez ceci:

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

Comme je l'ai commenté votre question, le code affiché ne fonctionne pas parce que vous devez insérer une instruction break, comme ceci:

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

Quoi qu'il en soit, je vous recommande d'utiliser l'une des techniques présentées dans cette question réponses.

Answer 2

A LINQ Contient opération ferait autant plus concise (en supposant usedFileNames est un 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

Plus concise encore:

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

Ce abstracts bien loin la logique qui figure sur laquelle les noms de fichiers que vous devez traiter (en dehors de la boucle), à ​​partir de la logique qui les traite (à l'intérieur de la boucle).

Answer 4

Alors que les autres réponses sont de meilleures solutions au problème, elles n'expliquent pas pourquoi le code original n'a pas fonctionné. Le problème est que l'algorithme écrase la valeur de la variable isInList, qui ne sera donc vraie que pour le dernier fichier de la liste. Cela fixerait ce problème:

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

Je voudrais ajouter qu'il est préférable d'utiliser un HashSet au lieu d'une liste pour votre collection usedFileNames. L'ensemble de hachage est conçu pour déterminer efficacement s'il contient un élément donné. La liste, si je me souviens bien, fait une recherche linéaire, qui (pour un grand nombre d'éléments) est inefficace.