c# checkbox problem

Question

I want to minimize the amount of code i have to write for this small problem. I have 1 textbox that has a relationship with 2 checkboxes as yes and no. The textbox on form load is set to disabled. When the yes checkbox is changed this event occurs -

private void checkYes1_CheckedChanged(object sender, EventArgs e)
{
    textBox14.Enabled = true;
    checkNo1_cbx.Checked = false;
}

and when the no checkbox is changed -

private void checkNo1_cbx_CheckedChanged(object sender, EventArgs e)
{
    textBox14.Enabled = false;
    checkYes1_cbx.Checked = false;
}

Although another problem is that i have to press yes twice to get it to check.

This is for a question on a form and so far it goes up to 11 questions and more will be added in the future. So my 2 problems so far is -

How can I fix the problem when the checkbox is changed I have to press it again to check it.
Is it possible to improve this code to minimize the amount of code i will have to write in the future.

Jalal Said · Accepted Answer · 2011-07-24 01:47:49Z

2

private void checkYes1_CheckedChanged(object sender, EventArgs e)
{
    OnCheck(true);
}
private void checkNo1_cbx_CheckedChanged(object sender, EventArgs e)
{
    OnCheck(false);
}

private void OnCheck(bool yes)
{
    textBox14.Enabled = yes;
    checkNo1_cbx.CheckedChanged -= checkNo1_cbx_CheckedChanged;
    checkNo2_cbx.CheckedChanged -= checkYes1_CheckedChanged;
    checkNo1_cbx.Checked = !yes;
    checkNo2_cbx.Checked = yes;
    checkNo1_cbx.CheckedChanged += checkNo1_cbx_CheckedChanged;
    checkNo2_cbx.CheckedChanged += checkYes1_CheckedChanged;
}

However consider using RadioBox instead of CheckBox because you want to if one being checked uncheck the other..

Edit: In your previous design, you get it wrong changed I have to press it again to check it because of you have two event handlers assigned to each of the check boxes. now at your code when the first one checked, you are disable the text box and make the other unchecked, but when you call the other unchecked Checked = false you are calling the second check box event handler also so it will enable the text and make the first one disable... you should remove the event handler by -= when updating at your code if you don't want the handler handler to be triggered again.. And that what I am doing in the code sample provided.

edited Jul 24, 2011 at 1:47

answered Jul 24, 2011 at 1:40

Jalal Said

16.2k8 gold badges47 silver badges68 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Mrchief Over a year ago

You need to expand your code to accomodate more checkbox/textboxes as per OP's ask.

Yahia · Accepted Answer · 2011-07-24 01:47:50Z

2

Why are you using 2 checkoboxes ? One checkbox (check1) would be enough:

private void check1_CheckedChanged(object sender, EventArgs e)
{
    textBox14.Enabled = check1.Checked;
}

EDIT: Assuming that each question mean 1 textbox, then you need 1 checkbox per textbox... this could be further improved by using a more complex approach

edited Jul 24, 2011 at 1:47

answered Jul 24, 2011 at 1:39

Yahia

70.5k9 gold badges117 silver badges147 bronze badges

Comments

szeliga · Accepted Answer · 2011-07-24 02:16:32Z

0

unless there is a reason that you are making a round trip back and forth to the server to disabled a textbox on a checkbox selection change, why don't you just do that all on the client side via javascript?

answered Jul 24, 2011 at 2:16

szeliga

6216 silver badges12 bronze badges

Comments

Scott Munro · Accepted Answer · 2011-07-24 02:55:53Z

0

I agree with Yahia. If you do need to explicitly provide the two options though, then you should consider using RadioButtons.

edited Jul 24, 2011 at 2:55

answered Jul 24, 2011 at 1:43

Scott Munro

13.7k3 gold badges76 silver badges82 bronze badges

Collectives™ on Stack Overflow

c# checkbox problem

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related