4 replies to this topic

[jbnonn]

I'm trying to get this if statement to work. All the other ones work fine except this if statement, which has two jobs. Can some one please help or direct me in the right direction. Should I use a different method or what. Thanks

Sample Code

private void cbSearch_CheckedChanged(object sender, EventArgs e)
{
if (cbSearch.Checked == true)
MainForm.hdSearchdd.Visible = false;
MainForm.hdSearchtxt.Visible = false;
else
MainForm.hdSearchdd.Visible = true;
MainForm.hdSearchtxt.Visible = true;

[CommittedC0der]

Try adding braces to your code:

private void cbSearch_CheckedChanged(object sender, EventArgs e)
        {
            if (cbSearch.Checked == true)
            {
                MainForm.hdSearchdd.Visible = false;
                MainForm.hdSearchtxt.Visible = false;
            }
            else 
            {
                MainForm.hdSearchdd.Visible = true; 
                MainForm.hdSearchtxt.Visible = true;
            }
        }

Also please encase your code like this: [CODE*]code here[/CODE*] just remove the '*' symbol.

~ Committed.

[jbnonn]

Thank you for you help, been working on this all day.

[sfhdweb]

CommittedC0der said:

Try adding braces to your code:

private void cbSearch_CheckedChanged(object sender, EventArgs e)

        {

            if (cbSearch.Checked == true)

            {

                MainForm.hdSearchdd.Visible = false;

                MainForm.hdSearchtxt.Visible = false;

            }

            else 

            {

                MainForm.hdSearchdd.Visible = true; 

                MainForm.hdSearchtxt.Visible = true;

            }

        }

Also please encase your code like this: [CODE*]code here[/CODE*] just remove the '*' symbol.

~ Committed.

thanks for sharing this great info

[sam_coder]

also, in terms of elegant code.. this is redundant

if (cbSearch.Checked == true) {

instead, just do this

if (cbSearch.Checked) {