Vider un combo

Salut à tous,
je remplir un combobox et je sélectionne l'élément voulu puis après des traitements je vide le combo comme suit:

Code:

---

cboAnnonceur.Items.clear();

---

La liste est vidée mais l'élément selectionné (dans cboAnnonceur.text) demeure! (

Code:

---

cboAnnonceur.text.clear();

---

) n'existe pas
Je souhaite l'effacer aussi comment le faire?

Merci d'avance!

Que veux tu faire exactement

1- Eliminer tous les items de ton combo
2- Mettre la selection a vide
3- Les deux ?

MonCombo.SelectedText = ""

Citation:

---

Envoyé par alexadvance

MonCombo.SelectedText = ""

---

à la limite String.Empty mais surtout pas "" ;)

Citation:

---

Envoyé par Louis-Guillaume Morand

à la limite String.Empty mais surtout pas "" ;)

---

Je m'incline ! (A vouloir faire trop vite, on fait souvent trop mal !)

Citation:

---

Envoyé par olibara

Que veux tu faire exactement

1- Eliminer tous les items de ton combo
2- Mettre la selection a vide
3- Les deux ?

---

Je souhaite faire les deux!

Personnelement je travaille essentiellement avec des combo bindé dans lequel je mets le selectedvalue a -1

Mais dans ton cas il me semble que le clear et le String.Empty devraient faire l'affaire

Citation:

---

Envoyé par olibara

Personnelement je travaille essentiellement avec des combo bindé dans lequel je mets le selectedvalue a -1

Mais dans ton cas il me semble que le clear et le String.Empty devraient faire l'affaire

---

j'ai fait les deux mais rien ne se passe:

Code:

---

1 2	cboAnnonceur.Items.clear(); cboAnnonceur.SelectedText=String.empty

---

Mais rien ne se pas. Je poste ici tout mon code:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

private void cboAnnonceur_SelectedIndexChanged(object sender, EventArgs e) { cboAnnonceurGetCode(cboAnnonceur.Text); EditStatus = "I"; FrmEditionStatus(EditStatus); cboMarque.Items.Clear(); cboMarque.SelectedText=string.Empty; RemplirCboMarque(CboAnnonceurCode.Text);   } private void cboMarque_SelectedIndexChanged(object sender, EventArgs e) { cboMarqueGetCode(cboMarque.Text); EditStatus = "I"; FrmEditionStatus(EditStatus); cboProduit.Items.Clear(); cboProduit.SelectedText=string.Empty; RemplirCboProduit(cboMarqueCode.Text); }

---

Ainsi, si l'on change modifie le contenue de cboAnnonceur, le cboMarque se vide de ses items et du texte (eventuellement sélectioné) et Ainsi de suite...

Citation:

---

Envoyé par Louis-Guillaume Morand

à la limite String.Empty mais surtout pas "" ;)

---

Citation:

---

Envoyé par alexadvance

Je m'incline ! (A vouloir faire trop vite, on fait souvent trop mal !)

---

Pourquoi ? (puisqu'il faut une chaîne)

Code:

---

1 2 3 4 5

static void Main(string[] args) { Console.WriteLine(Object.ReferenceEquals("", String.Empty) ? "C'est la même" : "Ce n'est pas la même"); }

---

testé avec VS2008 et VS2010.

Citation:

---

Envoyé par realpasso

Pourquoi ?

---

Je dirai que String.Empty est tout de meme plus lisible. Ca veut dire "J'instancie à vide car la valeur n'a pas d'importance" alors que 'chaine vide' peut vouloir dire "la valeur de ma chaine est vide" ...

Une histoire d'habitude et de clarté, je pense...

Pour en revenir au probleme de "le gris" :

Citation:

---

Envoyé par Le gris

Mais rien ne se pas. Je poste ici tout mon code..

---

Avec ce code, ton "cboMarque" ne se vide pas quand tu changes la sélection de "cboAnnonceur" ?

Citation:

---

Envoyé par alexadvance

Je dirai que String.Empty est tout de meme plus lisible. Ca veut dire "J'instancie à vide car la valeur n'a pas d'importance" alors que 'chaine vide' peut vouloir dire "la valeur de ma chaine est vide" ...

---

:marteau: Avoue, tu ne serais pas consultant toua ? ;-)

Citation:

---

Envoyé par alexadvance

Pour en revenir au probleme de "le gris" :

Avec ce code, ton "cboMarque" ne se vide pas quand tu changes la sélection de "cboAnnonceur" ?

---

le cboMarque se vide de ses items mais pas du texte qui est en sélection dans le "cboMarque.text" or d'après le ode, il est mis à Empty!!!! Je suis un peu embêté j'avoue.

Je continue de chercher les problème! Merci pour vos réponses!

Bonjour à tous je viens d'avoir ce que je cherche:

Code:

---

cboMarque.Text=null;

---

Et le tour est Joué!

Merci infiniment pour votre aide à tous!

ca n'a RIEN à voir avec de la clarté, celui qui utilise "" ne connait pas la CLR tout simplement. les bases types sont par valeur mais pas String, or si tu fais "" tu instancies un objet qui contient "", alors que String.Empty non.

si on suit votre logique, avec dix champs à reset, vous faites dix fois ="" et donc vous instanciez dix objets qui ne servent à rien. pour quelle raison? parce que "ca revient au même"? :D

c'est sûr que si le résultat c'est de mettre à vide, on peut faire ca de mille façons maintenant, y a la façon propre et la façon sale. Votre "", c'est sale car contre-performant.

de la meme façon que je vois souvent

Code:

---

1 2 3 4

if( maChaine == null || maChaine == "") // ou String.Empty) {   }

---

là encore c'est sale car il FAUT utiliser String.IsNullOrEmpty(maChaine)
pourquoi? parce que cette méthode teste la longueur de la chaine et pas sa valeur. Il est BEAUCOUP plus rapide de faire

Code:

---

if (maChaine.Length == 0)

---

que

Code:

---

if(maChaine == String.Empty)

---

performance parlant bien entendu


ca peut paraitre trivial ou superficiel sauf que c'est en cumulant plein de petits best-practices que l'on obtient un code plus propre. Si on garde des mauvaises habitudes, le jour où vous ferez du temps réel ou de l'embarqué, votre code sera inutilisable :/

Citation:

---

Envoyé par Louis-Guillaume Morand

ca n'a RIEN à voir avec de la clarté, celui qui utilise "" ne connait pas la CLR tout simplement. les bases types sont par valeur mais pas String, or si tu fais "" tu instancies un objet qui contient "", alors que String.Empty non.

si on suit votre logique, avec dix champs à reset, vous faites dix fois ="" et donc vous instanciez dix objets qui ne servent à rien. pour quelle raison? parce que "ca revient au même"? :D

c'est sûr que si le résultat c'est de mettre à vide, on peut faire ca de mille façons maintenant, y a la façon propre et la façon sale. Votre "", c'est sale car contre-performant.

de la meme façon que je vois souvent

Code:

---

1 2 3 4

if( maChaine == null || maChaine == "") // ou String.Empty) {   }

---

là encore c'est sale car il FAUT utiliser String.IsNullOrEmpty(maChaine)
pourquoi? parce que cette méthode teste la longueur de la chaine et pas sa valeur. Il est BEAUCOUP plus rapide de faire

Code:

---

if (maChaine.Length == 0)

---

que

Code:

---

if(maChaine == String.Empty)

---

performance parlant bien entendu


ca peut paraitre trivial ou superficiel sauf que c'est en cumulant plein de petits best-practices que l'on obtient un code plus propre. Si on garde des mauvaises habitudes, le jour où vous ferez du temps réel ou de l'embarqué, votre code sera inutilisable :/

---

Merci pour toutes ces précisions! j'en prendrai bonne note.

Citation:

---

Envoyé par Louis-Guillaume Morand

ca n'a RIEN à voir avec de la clarté, celui qui utilise "" ne connait pas la CLR tout simplement. les bases types sont par valeur mais pas String, or si tu fais "" tu instancies un objet qui contient "", alors que String.Empty non.

---

OK. Pour valider votre propos, expliquez le résultat suivant :

Code:

---

1 2	Console.WriteLine(Object.ReferenceEquals("", String.Empty) ? "Aucune instance créée" : "instance créée");

---

aucune instance créée

Citation:

---

Each string literal does not necessarily result in a new string instance. When two or more string literals that
are equivalent according to the string equality operator (§14.9.7), appear in the same assembly, these string
literals refer to the same string instance.

---

J'ai cru entendre que MS compte en faire une option de compile. Je file réviser la CLR...

edit: la bonne solution c'est olibara qui l'a donnée : selectedindex = -1 avant le clear.

Citation:

---

Envoyé par realpasso

edit: la bonne solution c'est olibara qui l'a donnée : selectedindex = -1 avant le clear.

---

Je teste cette solution! Merci encore une fois.

Citation:

---

Envoyé par realpasso

OK. Pour valider votre propos, expliquez le résultat suivant :

Code:

---

1 2	Console.WriteLine(Object.ReferenceEquals("", String.Empty) ? "Aucune instance créée" : "instance créée");

---

aucune instance créée


Each string literal does not necessarily result in a new string instance. When two or more string literals that
are equivalent according to the string equality operator (§14.9.7), appear in the same assembly, these string
literals refer to the same string instance.
J'ai cru entendre que MS compte en faire une option de compile. Je file réviser la CLR...

edit: la bonne solution c'est olibara qui l'a donnée : selectedindex = -1 avant le clear.

---

parce qu'il te faut aussi prendre en compte, si je dis pas de betise, que String.Empty pointe vers une valeur readonly déjà existante qui te retourne "", et c'est là que tu y gagnes en performance (même si c'est ultra mineur).
Après, il te suffit de taper "String.Empty vs "" sur google, ce sujet a été débattu par moultes personnes et accepté de façon unanime, y compris par les équipes Dev de Microsoft, je ne puis que suivre leur avis tel un mouton :D

Citation:

---

Envoyé par Louis-Guillaume Morand

parce qu'il te faut aussi prendre en compte, si je dis pas de betise, que String.Empty pointe vers une valeur readonly déjà existante qui te retourne "", et c'est là que tu y gagnes en performance (même si c'est ultra mineur).
Après, il te suffit de taper "String.Empty vs "" sur google, ce sujet a été débattu par moultes personnes et accepté de façon unanime, y compris par les équipes Dev de Microsoft, je ne puis que suivre leur avis tel un mouton :D

---

J'ai participé à ces discussions EN 2002, MS a trouvé amusant pendant ces 8 années passées d'ajouter cette optimisation au JIT.

Mais ce n'est pas tout, en utilisant "" le compilateur peut faire certaines optimisations qu'il ne pourra pas faire avec String.Empty
(c'est un exemple stupide je l'admets mais qui sert à démontrer ce que je dis)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

for (int x = 0; x < 20; x++) { System.Threading.Thread.Sleep(1); var timer = new System.Diagnostics.Stopwatch(); timer.Start();   for (int i = 0; i < 10000000; i++) { String test = "" + "hello"; }   timer.Stop(); Console.WriteLine("Plop " + timer.ElapsedMilliseconds + " ms"); }

---

Sur mon pc ça prends environ 20ms avec "" (le compilateur peut virer tout l'interieur de la boucle i car "" n'a pas d'effet de bord) et + de 100 ms avec String.Empty car le compilateur ne connait pas l'effet de bord qu'il y aurait à ne pas appeler String.Empty.

"early optimisation is the root of all evil", laissez le compilateur et le JIT travailler.

Pour ma part, j'arrête là ce thread qui consistait à vider une combo. Désolé d'avoir rebondi sur quelque chose qui n'avait rien à voir.

Et voila olibara qui revient a la surface :)

+1 pour realpasso et JGM

J'apprecie a armes egales la constance academique de JGM qui peut aider beaucoup les moutons égarés
Tout autant que la sagacité investigatrice de realpasso qui permet de mieux comprendre certains nouveaux chemins !
Grace a des membres comme vous, ce forums a toute son âme et sa qualité !