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.