Discussion :

[mekongtso]

Bonjour tout le monde. le bout de code que vous voyez devrait servir à activer et desactiver suite à un clic un bouton radio isolé. Seulement lorsque le bouton est activé le clic parvient à le desactiver mais jamais le contraire. Coup de main SVP.
private void radioButton1_Click(object sender, EventArgs e)
{

if (radioButton1.Checked == true)
radioButton1.Checked = false;
if (radioButton1.Checked != true)
radioButton1.Checked = true;
}

[micka132]

Pourquoi ne pas utiliser une checkbox?
Bon sinon le problème c'est que le radio button va faire son travail avant d'arriver sur l'eventement du click.
Ce qui veut dire que : s'il n'est pas checked il va le devenir, puis tu va rentrer dans ton evenement.
Et donc executer ta premiere instruction

Code :

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

1
2
if (radioButton1.Checked == true)
                radioButton1.Checked = false;

Puis vu qu'il n'y a pas de "else" il fera la suivante dans la foulée (donc remettre à true).

S'il est deja checked, il va rien faire puis rentrer dans ton event. La ton instruction

Code :

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

1
2
if (radioButton1.Checked == true)
                radioButton1.Checked = false;

va se reproduire comme au dessus.

Bref tu peux utiliser plusieurs astuces comme stocker la valeur précedente du checked dans le Tag, utiliser une deuxieme radioBtn invisible, utiliser une variable de classe...
M'enfin ca devient vite un peu tordu si tu n'as finalement besoin que d'une checkbox

[mekongtso]

merci micka132. Je suis passé passé par un checkbox et par là moins compliqué

[sisqo60]

Bonjour,

C'est encore plus simple que ce que vous proposez, ça se fait en une ligne :

Code :

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

radioButton1.Checked = !radioButton1.Checked

Mais l'utilisation d'un checkbox est plus adaptée bien sûr

Bon dév.

[ClaudeBg]

Salut
-----

C'est encore plus simple que ce que vous proposez, ça se fait en une ligne :

Code :

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

radioButton1.Checked = !radioButton1.Checked

C'est plus simple, mais je crains que ça ne fonctionne pas mieux, et exactement pour la même raison que le problème d'origine:

Le fonctionnement même du radiobutton va faire qu'il va d'abord se cocher avant d'appeler l'évènement clic. Du coup, checked sera toujours "true" et "click" invalidera toujours l'action. Le radiobutton ne sera donc jamais coché.

Il n'y a pas moyen d'obtenir le résultat demandé via "click" seul, du moins pas sans le genre d'astuce indiquée par micka132, parce qu'on ne sait pas tester dans "click" l'état du radiobutton avant le dit "click", il sera toujours "true." Il faut utiliser CheckedChanged en combinaison avec click ou tout mécanisme de variable supplémentaire.

Mais je plussoie que pour faire ce genre de chose, ça correspond exactement au fonctionnement d'une checkbox, faite pour ça de façon "naturelle".

Pour le reste, on sent un peu le manque de références sur le C#, parce qu'on n'écrit guère "a = true" on écrit "a" tout simplement. Idem pour "a = false" plutôt que "!a". Ça sent le passage depuis le basic.

idem avec if (a) .... if(!a) plutôt que if(a) else, dans ce cas de figure ou a n'est pas modifié entre le if et le else.
Et, au final, if(a) a = false else a = true, amène la simplification indiquée : a = !a.

A+
Claude