Discussion :

[gobelet18]

Bonjour,
actuellement j'ai ceci qui fonctionne correctement :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[tomlev]

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.

[StringBuilder]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[tomlev]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[StringBuilder]

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

[gobelet18]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[tomlev]

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

[gobelet18]

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.

[tomlev]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

[gobelet18]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

BoiteAOutils.addMiniature(path, pathMiniatures);

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

[gobelet18]

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

[tomlev]

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
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

[gobelet18]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[tomlev]

Bah il marche très bien ce code

Mis à part cette ligne :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

qu'il faut remplacer par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

[Sehnsucht]

Bonsoir,

Juste quelques petites remarques "optionnelles"

• 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 !

[gobelet18]

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.