Comment modifier correctement un objet?

Version imprimable

Bonjour à tous,

Voilà, je voulais savoir si pour modifier un objet, il valait mieux l'extraire, le modifier puis le remettre dans la liste ou plutot le modifier directement dans la liste car la j'ai fais :

Code:

1
2
3
4
5
6
7
8
for (int i = 0; i < ListeEntree.Count; i++)
                    {
                        if (ListeEntree[i].getNomP().Equals(nomProduit))
                        {
                            ListeEntree[i].setNomP(newNom);
                            ListeEntree[i].setTauxE(newPrix);
                        }
                    }

Le problème est qu'il modifi tous mes objets :s

Merci de votre aide.

30/04/2009, 11h38
StormimOn

Si tous les objets de la collection sont modifiés on pourrait supposer qu'ils ont tous le même nom de produit. La condition serait donc toujours vérifiée.

Tu as essayé de déboguer pour voir ce qu'il se passe ?

en faite je crois que le problème vient plutot lors de l'affichage, en faite j'affiche les données de mon objet dans des labels lorsqu'un objet est sélectionné dans ma comboBox et l'affichage ne change pas. Je n'ai peut être pas sélectionné le bon événement. Voici le code :
Code:

1 2 3 4 5 6 7 8 9 10 private void cmbObjMod_SelectedIndexChanged(object sender, EventArgs e) { panelModEntre.Visible = true; for (int i = 0; i < ListeEntree.Count; i++) { lblNom.Text = ListeEntree[i].getNomP().ToString(); lblPrix.Text = ListeEntree[i].getTauxE().ToString(); } }
En tout cas merci pour ton aide.

30/04/2009, 14h26
StormimOn

C'est un peu normal que tu ais un problème d'affichage. L'événement c'est le bon, par contre tu affiches chaque élément de la collection (boucle for) au lieu d'afficher les informations de l'élément sélectionné dans la ComboBox.

Résultat tu auras beau modifier la sélection dans la ComboBox, tu verras toujours les informations du dernier élément de la collection.
30/04/2009, 14h28
sicilianadev

Ah ok, pourrais tu m'expliquer comment faire pour afficher l'élément sélectionné sil te plait? je pense avec selectIndex non? Mais où le mettre :s
30/04/2009, 14h32
StormimOn

Ca dépend comment tu alimentes la ComboBox. Montre un peu le code :)

Elle est alimenté dans le LOAD de ma page avec ce code :

Code:

1
2
3
4
5
for (int i = 0; i < ListeEntree.Count; i++)
           {
               nomProduit = ListeEntree[i].getNomP();
               cmbObjMod.Items.Add(nomProduit);
           }

30/04/2009, 14h43
StormimOn

A ce moment la propriété SelectedIndex fera l'affaire, pour récupérer l'élément sélectionné.
30/04/2009, 14h50
sicilianadev

d'accord mais je le mets où exatement? Désolé du dérangement :s

Quelque chose comme ça

Code:

1
2
3
4
5
6
7
private void cmbObjMod_SelectedIndexChanged(object sender, EventArgs e)
{
    panelModEntre.Visible = true;
    int i = cmbObjMod.SelectedIndex;
    lblNom.Text = ListeEntree[i].getNomP().ToString();
    lblPrix.Text = ListeEntree[i].getTauxE().ToString();
}

30/04/2009, 14h55
sicilianadev

je le laisse dans le for non?
30/04/2009, 15h06
StormimOn

La boucle for ne sert à rien puisqu'on connait l'index de l'élément. On peut y accéder directement.
30/04/2009, 15h12
sicilianadev

j'ai essayé comme tu me la conseillé, mais il bug sur

Code:

lblNom.Text = ListeEntree[i].getNomP().ToString();

Il me le surligne en jaune :s
30/04/2009, 15h18
StormimOn

Ca ne bug pas. Ce doit être une erreur de compilation, regarde le message d'erreur associé.
30/04/2009, 15h20
sicilianadev

il n'y en a pas justement
30/04/2009, 15h24
StormimOn

Si tu ne peux pas compiler ou exécuter le code, tu dois bien avoir un message d'erreur quelque part. Si tu te places sur le code surligné, tu n'as pas un tooltip qui indique le pourquoi du problème ?
30/04/2009, 15h35
sicilianadev

Même dans le tooltip il ne veux pas l'afiché,
Mais la j contient une chaîne

j'ai resolu mon problème, j'ai fais a peu près comme tu me l'a conseillé, voici le code :
Code:

1 2 3 4 5 6 panelModEntre.Visible = true; int i = cmbObjMod.SelectedIndex; lblNom.Text = ListeEntree[i].getNomP().ToString(); lblPrix.Text = ListeEntree[i].getTauxE().ToString();
Merci pour ton aide :)