initialisation d'un tableau dynamique dans une list<T>

Version imprimable

Salut,
J'ai créé la structure suivante

Code:

1
2
3
4
5
6
public struct LibelleMultiLangue
    {
        public int IdMsg;
        public string ErrorCode;
        public string[] libelle;
    }

je ne connais pas a l'avance le nombre de champ "libelle" => je voudrais utiliser un tableau dinamyque
J'essaie de créer un fonction test pour remplir le tableau

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
private void btnTestFindIndex_Click(object sender, EventArgs e)
        {
            List<LibelleMultiLangue> lmsg = new List<LibelleMultiLangue>();
            LibelleMultiLangue msg = new LibelleMultiLangue();
            for (int i = 0; i < 1000; i++)
            {
                msg.ErrorCode = (i+1).ToString();
                msg.IdMsg = i+1;
 
                msg.libelle[0]="LIB" + (i + 1).ToString() + "_0";
                msg.libelle[1]="LIB" + (i + 1).ToString() + "_1";
                msg.libelle[2]="LIB" + (i + 1).ToString() + "_2";
                msg.libelle[3]="LIB" + (i + 1).ToString() + "_3";
 
                lmsg.Add(msg);
            }
            GetMessageFromCode(lmsg, "100", 3);
        }

dur la ligne de code suivante

Code:

1
2
 
msg.libelle[0]="LIB" + (i + 1).ToString() + "_0";

j'ai une erreur me disant

Citation:

object reference not set to an instance of an object.

Quelqu'un serait-il me dire où se situe mon erreur?
Merci
Christophe
PS: j'ai essayé en changeant

Code:

public string[] libelle;

par

Code:

public <string> libelle;

et en utilisant ensuite

Code:

1
2
 
msg.libelle.add("LIB" + (i + 1).ToString() + "_3");

Ca me donne le même résultat...

L'erreur est due au fait que tu essayes de stocker des valeurs dans msg.libelle alors qu'il n'est pas initialisé

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
private void btnTestFindIndex_Click(object sender, EventArgs e)
        {
            List<LibelleMultiLangue> lmsg = new List<LibelleMultiLangue>();
            for (int i = 0; i < 1000; i++)
            {
                LibelleMultiLangue msg = new LibelleMultiLangue(); //sinon tu ajoutes toujours la meme variable dans lmsg
 
                msg.libelle=new string[4]; // sinon ton array n'est pas initialisé (c'est mieux d'utiliser une une list mais il faudrait aussi faire un new List<string>())
                msg.ErrorCode = (i+1).ToString();
                msg.IdMsg = i+1;
 
                msg.libelle[0]="LIB" + (i + 1).ToString() + "_0";
                msg.libelle[1]="LIB" + (i + 1).ToString() + "_1";
                msg.libelle[2]="LIB" + (i + 1).ToString() + "_2";
                msg.libelle[3]="LIB" + (i + 1).ToString() + "_3";
 
                lmsg.Add(msg);
            }
            GetMessageFromCode(lmsg, "100", 3);
        }

24/11/2011, 09h36
BenoitM

Citation:

object reference not set to an instance of an object

est du à l'acces à une NullValue donc un petit breakpoint et vérifier les différentes variables aide souvent à trouver l'erreur

Yop, j'etais tombe la dessus aussi, j'ai donc modifié mon code comme ceci

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
 
 
        private void btnTestFindIndex_Click(object sender, EventArgs e)
        {
            List<LibelleMultiLangue> lmsg = new List<LibelleMultiLangue>();
            LibelleMultiLangue msg = new LibelleMultiLangue();
            msg.libelle = new List<string>();
            for (int i = 0; i < 1000; i++)
            {
                msg.ErrorCode = (i+1).ToString();
                msg.IdMsg = i+1;
                msg.libelle.Clear();
                msg.libelle.Add("LIB" + (i + 1).ToString() + "_1");
                msg.libelle.Add("LIB" + (i + 1).ToString() + "_2");
                msg.libelle.Add("LIB" + (i + 1).ToString() + "_3");
                msg.libelle.Add("LIB" + (i + 1).ToString() + "_4");
                msg.libelle.Add("LIB" + (i + 1).ToString() + "_5");
 
                lmsg.Add(msg);
            }
 
            MessageBox.Show(GetMessageFromCode(lmsg, "100", 3));
        }
    }

Mais dans ce cas, mes errorcode, idmsg sont bien différent mais je me retrouve avec les meme valeur pour chaque tableau de libelle (la valeur calculée pour le dernier élément du tableau)

24/11/2011, 09h53
Pukram
en mettant le
Code:

1 2 msg.libelle = new List<string>();
dans ma boucle "for", je n'ai plus ce problème mais cette solution me semble peu élégante.
Qqun aurait-il une autre solution?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
private void btnTestFindIndex_Click(object sender, EventArgs e)
        {
            List<LibelleMultiLangue> lmsg = new List<LibelleMultiLangue>();
            for (int i = 0; i < 1000; i++)
            {
                LibelleMultiLangue msg = new LibelleMultiLangue();
                msg.libelle = new List<string>();
 
                msg.ErrorCode = (i+1).ToString();
                msg.IdMsg = i+1;
                msg.libelle.Add("LIB" + (i + 1).ToString() + "_1");
                msg.libelle.Add("LIB" + (i + 1).ToString() + "_2");
                msg.libelle.Add("LIB" + (i + 1).ToString() + "_3");
                msg.libelle.Add("LIB" + (i + 1).ToString() + "_4");
                msg.libelle.Add("LIB" + (i + 1).ToString() + "_5");
 
                lmsg.Add(msg);
            }
 
            MessageBox.Show(GetMessageFromCode(lmsg, "100", 3));
        }
    }

L'explication était dans la premiere réponse :roll:

24/11/2011, 10h17
Bluedeep

Accessoirement les concaténations de chaine dont on abuse inutilement et à plaisir ici sont contre-performantes au possible.
24/11/2011, 10h41
Pukram

oui, je sais mais c'était juste pour un test, au final, les données viendrons d'une DB et j'aurais juste a reprendre les valeur des champs LIB1,LIB2,... pour populer mes libellés