Discussion :

[Pukram]

Salut,
J'ai créé la structure suivante

Code :

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

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 :

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

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

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

j'ai une erreur me disant

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 :

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

public string[] libelle;

par

Code :

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

public <string> libelle;

et en utilisant ensuite

Code :

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

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

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

[BenoitM]

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

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

[BenoitM]

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

[Pukram]

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

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

[Pukram]

en mettant le

Code :

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

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?

[BenoitM]

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

[Bluedeep]

Accessoirement les concaténations de chaine dont on abuse inutilement et à plaisir ici sont contre-performantes au possible.

[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