Discussion :

[sicilianadev]

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 :

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

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.

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

[sicilianadev]

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 :

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

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.

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

[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

[StormimOn]

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

[sicilianadev]

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

Code :

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

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

[StormimOn]

A ce moment la propriété SelectedIndex fera l'affaire, pour récupérer l'élément sélectionné.

[sicilianadev]

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

[StormimOn]

Quelque chose comme ça

Code :

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

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

[sicilianadev]

je le laisse dans le for non?

[StormimOn]

La boucle for ne sert à rien puisqu'on connait l'index de l'élément. On peut y accéder directement.

[sicilianadev]

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

Code :

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

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

Il me le surligne en jaune :s

[StormimOn]

Ca ne bug pas. Ce doit être une erreur de compilation, regarde le message d'erreur associé.

[sicilianadev]

il n'y en a pas justement

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

[sicilianadev]

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

[sicilianadev]

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

Code :

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

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