Checkbox if-statement problem

Question

0.00/5 (No votes)

See more:

if (sexch.Checked == true)
            {

                ch.Text = "male";
            }
           else  if (sexch.Checked == false)
            {
                ch.Text = "Female";

            }

I have this code when the checkbox ==true I want a label.text to change to male. The problem is that the program only read the else.

I am getting the sexch checkbox from a database and depending on the output(true=male, false=female) of the database I want to change the label.text.

What I have tried:

if (sexch.Checked == true)
            {

                ch.Text = "male";
            }
           else  if (sexch.Checked == false)
            {
                ch.Text = "Female";

            }

if (sexch.Checked == true)
            {

                ch.Text = "male";
            }
             if (sexch.Checked == false)
            {
                ch.Text = "Female";

            }

Posted 5-Jun-18 10:56am

Ahmed Taha

Updated 7-Jun-18 11:16am

Add a Solution

2 solutions

Solution 2

Use CheckBox.CheckedChanged Event (System.Windows.Forms)[^]

Inside of it, use:

C#

ch.Text = sexch.Checked ? "Male" : "Female";

In case you want to change checkbox state based on database value:

C#

sexch.Checked = database_value;

For further details, please see: CheckBox.Checked Property (System.Windows.Forms)[^]

Posted 7-Jun-18 11:17am

Maciej Los

Updated 7-Jun-18 11:19am

v2

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Dave Kreskowiak · Accepted Answer · 2018-06-05T13:26:00

Solution 1

Your problem is that you don't need the second if statement at all. If the checkbox is not true, it must be false.

C#

if (sexch.Checked == true)
{
    ch.Text = "Male";
}
else
{
    ch.Text = "Female";
}

Posted 5-Jun-18 13:26pm

Dave Kreskowiak

Comments

Ahmed Taha 6-Jun-18 11:42am

Still, I have the same problem.
The program does not go through the (if)part, it only sees the else part whether the checkbox is true or false.

Dave Kreskowiak 6-Jun-18 11:44am

You're going to have to go through the code in the debugger to see what's happening. The code I posted will work and will only pick one path, either ch.Text is going to be "Male" or it's going to be "Female".

BUT! This will only work depending on the code around this code and the context in which it is executing. Since we can only work with exactly what you type into the question, we can't go any farther than this.

Learn to use the debugger and step through the code line-by-line and examine the variable contents. This is the only way you're going to figure this problem out.