Discussion :

[NazOok]

Bonjour,

J'aurais besoin d'un petit coup de main svp

Enfaite j'ai une form principal "AppliMainForm" ou se trouve une Listbox "listVille" et je voudrais a partir d'une classe que j'ai créée "XMLVille" créer une méthode "ChargementListe" qui me charge ma Listbox avec les élements de mon fichier XML..

J'arrive a faire le chargement de ma listebox dans le programme principal mais dès que j'essaye avec ma méthode "ChargementListe" elle ne se charge plus..
J'ai passé ma ListBox en public, je n'ai pas d'erreur de compilation mais elle reste vide...

voici mon code :

Méthode ChargelentListe de ma class XMLVilles :

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
 
public void ChargementListe()
        {
            AppliMainForm form1;
            form1=new AppliMainForm();
 
            tab = new string[50];
            XmlDocument docVille = new XmlDocument();
            docVille.Load(@".\XMLVilles.xml");
            XmlNodeList listeNoeudsVilles = docVille.GetElementsByTagName("nom");
            foreach (XmlNode node in listeNoeudsVilles)
            {
                tab[nbVilles] = node.InnerText;
                nbVilles++;
            }
            form1.listVille.Items.Clear();
            for(int i = 0; i < this.nbVilles; i++)
            {
                form1.listVille.Items.Add(tab[i]);
            }
        }

Utilisation de la méthode :

Code :

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

1
2
3
4
5
6
 
private void AppliMainForm_Load(object sender, EventArgs e)
        {
            XMLVilles ChL = new XMLVilles();
            ChL.ChargementListe();
        }

Et je souhaite me servir de cette méthode dans plusieurs form par la suite..
Merci de votre aide

[The_badger_man]

Dans ta méthode ChargementList tu crées une nouvelles Form

Code :

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

form1=new AppliMainForm();

et tu charges la ListBox de cette nouvelle form, pas la ListBox de la form principale...


Dans ta méthode ChargementListe tu mélanges contrôles graphique et chargement de données. Si tu veux du code réutilisable tu dois séparer tout ça.

Déjà tu dois créer des classes pour représenter tes données. Par exemple une classe Ville.

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
    public class Ville
    {
        private string _name ;
 
        public  Ville(string name)
        {
            this._name = name;
        }
 
        public string Name
        {
            get
            {
                return _name ;
            }
        }
    }

Ensuite tu crées une classe qui s'occupe de charger les données (villes) et des les renvoyer (liste de villes).

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
    public class VilleDataAccess
    {   
        public List<Ville> ChargementListeVilles()
        {
            List<Ville> listVilles = new List<Ville>();
            XmlDocument docVille = new XmlDocument();
            docVille.Load(@".\XMLVilles.xml");
            XmlNodeList listeNoeudsVilles = docVille.GetElementsByTagName("nom");
            foreach (XmlNode node in listeNoeudsVilles)
            {
                Ville ville = new Ville(node.InnerText);
                listVilles.Add(ville);
            }
            return listVilles;
        }        
    }

Pour charger la grille tu utilises la propriété DataSource de la Listebox et tu lui donnes la liste de villes.

Code :

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

1
2
3
4
5
6
7
8
private void AppliMainForm_Load(object sender, EventArgs e)
{
    VilleDataAccess villeDataAccess = new VilleDataAccess();
 
    listVille.DataSource = villeDataAccess.ChargementListeVilles();
    listVille.DisplayMember = "Name";
    listVille.ValueMember = "Name";    
}

[NazOok]

Ah ouiii en effet j'ai pas était doué sur le fait de réinstancié une form.. je comprend!!

Parcontre j'avais pas vu la réponse alors j'ai réussi a avancé en passant en parametre à ma fonction ChargementListe ma ListBox et cela fonctionne et mon code ne change quasiment pas..
Mais c'est surement pas la meilleure solution?

La je vois que tu utilise des accesseurs c'est sa ?
Je ne connais pas encore cette façon de faire, j'ai du mal a voir un peu la chose mais je vais essayer, sa va m'aidé a progresser

Parcontre je ne suis pas obligé de recréer une classe ? je peux utiliser celle que j'ai déjà : XMLVilles ?

Merci beaucoup pour ton aide