How do you deal with checkboxes in C#???

[adonisv]

I'm just trying to toggly between two check boxes a draw x values and a draw y values. I found this code but I'm getting a stack overflow error when I click on the check box....:(( private void XvaluescheckBox_CheckedChanged(object sender, System.EventArgs e) { // Toggle boolean switch(this.XvaluescheckBox.CheckState) { case CheckState.Checked: // Code for checked state. this.m_bDrawXValues = false; this.XvaluescheckBox.CheckState = CheckState.Unchecked; break; case CheckState.Unchecked: // Code for unchecked state. this.m_bDrawXValues = true; this.XvaluescheckBox.CheckState = CheckState.Checked; break; case CheckState.Indeterminate: // Code for indeterminate state. break; } } private void YvaluescheckBox_CheckedChanged(object sender, System.EventArgs e) { // Toggle boolean switch(this.YvaluescheckBox.CheckState) { case CheckState.Checked: // Code for checked state. this.m_bDrawXValues = true; this.YvaluescheckBox.CheckState = CheckState.Unchecked; break; case CheckState.Unchecked: // Code for unchecked state. this.m_bDrawXValues = false; this.YvaluescheckBox.CheckState = CheckState.Checked; break; case CheckState.Indeterminate: // Code for indeterminate state. break; } } :doh:

[Judah Gabriel Himango]

You're changin the check state of the checkbox inside the CheckChanged event handler -- i.e. as soon as your check state has changed, your function gets called. Your function then changes the check state, once again calling your function, which goes forever in a loop until the stack overflows. If you need to change the check state of the checkbox inside the check change function, unhook the event handler first. Try something like

private void YvaluescheckBox_CheckedChanged(object sender, System.EventArgs e)
{
// Toggle boolean
switch(this.YvaluescheckBox.CheckState)
{
case CheckState.Checked:
// Code for checked state.
this.m_bDrawXValues = true;

// temporarily unhook the event handler for a moment
this.YvaluescheckBox.CheckChanged -= new EventHandler(YvaluescheckBox_CheckedChanged);

// set the check state
this.YvaluescheckBox.CheckState = CheckState.Unchecked;

// re-setup the event handler
this.YvaluescheckBox.CheckChanged += new EventHandler(YvaluescheckBox_CheckedChanged);
break;

Or, just don't set the check state of the check box inside your check state change event handler like you're doing there. --------------------------- He who knows that enough is enough will always have enough. -Lao Tsu

[Daniel Turini]

If you have two (or more) checked buttons and only one of them should be set at a time, then you should use option buttons. I see dumb people

[Nnamdi Onyeyiri]

Isnt CheckedChanged called after the checkboxes state is changed? Basically, your recieving the event, and changing a property of the checkbox, that causes the event to be fired again, so its a vicious circle. Instead, try:

private void XvaluescheckBox_CheckedChanged(object sender, System.EventArgs e)
{
this.m_bDrawXValues = this.XvaluescheckBox.Checked;
}

private void YvaluescheckBox_CheckedChanged(object sender, System.EventArgs e)
{
this.m_bDrawYValues = this.YvaluescheckBox.Checked;
}

I havent tested it, but that should work. Strictly speaking, you dont even need the m_bDrawXValues and m_bDrawYValues variables, you can use the Checked property of the checkboxes.

---

To those who didn't make it, we will remember you. To those who did :bob: is back. - Megan Forbes in Black Friday
Another Post by NnamdiOnyeyiri

[adonisv]

So do I, all the time! :laugh: I used the radio buttons, those work nicely..Doh! :sigh: