Problème de foreach sur getFiles()

Bonjour,
actuellement j'ai ceci qui fonctionne correctement :

Code:

---

1 2 3 4 5 6 7 8

foreach (FileInfo nextImage in dirRoot.GetFiles()) { if (BoiteAOutils.isImage(nextImage.Extension)) { addImage(path + @"\" + nextImage.Name, newnode); } }

---

Et lorsque je rajoute ce code a la fonction addImage

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

if(System.IO.File.Exists(pathImage)) { Bitmap img = new Bitmap(pathImage); int ratio; if (img.Width < img.Height) ratio = img.Height / 60; else if (img.Height < img.Width) ratio = img.Width / 60; else ratio = 1; Bitmap image = new Bitmap(img, new Size(60, 60)); image.Save(pathDossDest+@"\"+Path.GetFileName(pathImage)); img.Dispose(); image.Dispose(); }

---

Dans mon nextImage, s'ajoute les noms des fichiers en allant c est a dire :
premier passage : nextFile.name=fichier1.jpg
second : nextFile.name=fichier1.jpg.fichier2.jpg
troisisieme : nextFile.name=fichier1.jpg.fichier2.jpg.fichier3.jpg
.... ainsi de suite.
Je ne vois vraiment pas en quoi un code appelant peut poser soucis dans le code appelant.

A mon avis le problème ne se trouve pas dans le code que tu as posté... en tous cas je ne trouve rien qui puisse expliquer ça. Montre un extrait de code plus complet.

Code:

---

foreach (FileInfo nextImage in dirRoot.GetFiles())

---

A ma connaissance, ça va évaluer GetFiles() à chaque itération, non ?

Donc vu qu'on ajoute des fichiers à chaque itération, on arrive dans une boucle infinie, et on rempli le disque...

Moi je fais toujours :

Code:

---

1 2	FileInfo [] files = dirRoot.GetFiles(); foreach (FileInfo nextImage in files)

---

Citation:

---

Envoyé par StringBuilder

A ma connaissance, ça va évaluer GetFiles() à chaque itération, non ?

---

Non, le foreach est transformé par le compilateur en quelque chose comme ça :

Code:

---

1 2 3 4 5 6 7 8 9 10

using (IEnumerator<FileInfo> enumerator = GetFiles().GetEnumerator()) { while(enumerator.MoveNext()) { FileInfo nextImage = enumerator.Current;   // Corps de la boucle foreach ... } }

---

GetFiles n'est donc appelé qu'une seule fois.

ah ok, bon, ben je m'amuse tout le temps à passer par un array inutile alors :aie:

Code:

---

1 2 3 4 5 6 7 8 9 10

DirectoryInfo dirRoot = new DirectoryInfo(path); try { foreach (FileInfo nextImage in dirRoot.GetFiles()) { if (BoiteAOutils.isImage(nextImage.Extension)) { addImage(path + @"\" + nextImage.Name, newnode); } }

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

public void addImage(string path,XmlNode node) { XmlNode Nimage = docXml.CreateElement("image"); XmlNode pathNode = docXml.CreateElement("pathImage"); XmlText text = docXml.CreateTextNode(path); XmlNode pathMiniatureNode = docXml.CreateElement("pathMiniature"); XmlText textmini = docXml.CreateTextNode(pathMiniatures+"\\"+Path.GetFileName(path)); pathMiniatureNode.AppendChild(textmini); pathNode.AppendChild(text); Nimage.AppendChild(pathNode); Nimage.AppendChild(pathMiniatureNode); node.AppendChild(Nimage); BoiteAOutils.addMiniature(path, pathMiniatures); }

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

public static void addMiniature(string pathImage,string pathDossDest) { if(System.IO.File.Exists(pathImage)) { Bitmap img = new Bitmap(pathImage); int ratio; if (img.Width < img.Height) ratio = img.Height / 60; else if (img.Height < img.Width) ratio = img.Width / 60; else ratio = 1; Bitmap image = new Bitmap(img, new Size(60, 60)); image.Save(pathDossDest+@"\"+Path.GetFileName(pathImage)); img.Dispose(); image.Dispose(); } }

---

Si je met cette ligne en commentaire : BoiteAOutils.addMiniature(path, pathMiniatures); plus de problème.

Bah j'ai beau regarder le code sous toutes les coutures, je vois pas... tu as essayé de déboguer en pas à pas ? C'est sans doute la meilleure façon de comprendre ce qui se passe

J'ai essayé et dès le foreach (FileInfo nextImage in dirRoot.GetFiles()) en pas a pas,dans mon nextImage, s'ajoute les noms des fichiers en allant c est a dire :
premier passage : nextFile.name=fichier1.jpg
second : nextFile.name=fichier1.jpg.fichier2.jpg
troisisieme : nextFile.name=fichier1.jpg.fichier2.jpg.fichier3.jpg
.... ainsi de suite.

alors là c'est surnaturel :koi:
Essaie de faire juste ça :

Code:

---

1 2 3 4

foreach (FileInfo nextImage in dirRoot.GetFiles()) { Debug.Print(nextImage.Name); }

---

Si tu obtiens le même résultat, c'est qu'il y a vraiment un truc qui cloche...

Ah, et puis une question bête, mais qu'il faut quand même se poser... tu es sûr que tu n'as pas réellement des fichiers avec ces noms dans le répertoire en question ?

J'ai essayé en pas a pas en mettant en commentaire

Code:

---

BoiteAOutils.addMiniature(path, pathMiniatures);

---

et tout est bon. Donc là je ne vois vraiment pas comment c est possible !

Mon nextImage dans mon foreach est bon du moment que je met en commentaire : BoiteAOutils.addMiniature(path, pathMiniatures);

Citation:

---

Envoyé par gobelet18

Mon nextImage dans mon foreach est bon du moment que je met en commentaire : BoiteAOutils.addMiniature(path, pathMiniatures);

---

Bah je sais bien, ça fait 3 fois que tu le dis :aie:
Mais là vraiment je vois pas du tout comment c'est possible... Si tu peux poster un exemple court (pas ton projet entier) mais complet (assez pour reproduire le problème), je jetterai un coup d'oeil

J'étais pas sûr que ce soit clair ^^.
Sinon voilà :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31

FolderBrowserDialog fbd = new FolderBrowserDialog(); fbd.ShowDialog(); string path = fbd.SelectedPath; DirectoryInfo dirRoot = new DirectoryInfo(path); string pathDossDest = path + "\\miniatures"; Directory.CreateDirectory(pathDossDest); try { foreach (FileInfo nextImage in dirRoot.GetFiles()) { if (nextImage.Extension == ".jpg" || nextImage.Extension == ".jpeg" || nextImage.Extension == ".png") { Bitmap img = new Bitmap(nextImage.FullName); int ratio; if (img.Width < img.Height) ratio = img.Height / 60; else if (img.Height < img.Width) ratio = img.Width / 60; else ratio = 1; Bitmap image = new Bitmap(img, new Size(60, 60)); image.Save(pathDossDest + @"\" + Path.GetFileName(path)); img.Dispose(); image.Dispose(); } } } catch (UnauthorizedAccessException f) { string t = f.ToString(); }

---

Bah il marche très bien ce code :koi:

Mis à part cette ligne :

Code:

---

image.Save(pathDossDest + @"\" + Path.GetFileName(path));

---

qu'il faut remplacer par :

Code:

---

image.Save(pathDossDest + @"\" + Path.GetFileName(nextImage.Name));

---

Aucun problème, ça crée bien une miniature de chaque image...

Bonsoir,

Juste quelques petites remarques "optionnelles" :aie:

• Tu n'utilises jamais ratio
• Tu pourrais remplacer ta concaténation pour le chemin de la miniature par un appel à Path.Combine
• Il serait peut-être judicieux (point de vue extensibilité) de placer les extensions dans un conteneur (tableau, liste, etc.) et de faire un if (allowedExtensions.Contains(nextImage.Extension))(voire appliquer cela directement sur l'énumération du foreach foreach (FileInfo nextImage in dirRoot.EnumerateFiles().Where(fi => allowedExtensions.Contains(di.Extension)))

Cordialement !

J'ai une fonction isImage(string path) pour gerer, c'est juste que c'était plus simple pour le morceau de code demandé. Le ratio n'était pas utilisé, c'est vrai ^^, c'est corrigé et merci pour la remarque j'avais pas vus. Et efféctivement si je remplace dans mon code par Path.GetFileName(nextImage.Name)); ca marche mieux ^^.
Et le problème que j'avais était dans mon dossier de test, des images nommé ".image.jpg" et du coup le foreach concaténait plusieurs fichiers a cause de cela. Merci a tous de vos conseils.