Ggtcf

I'm a student and I'm experimenting with c# (I'm noob)
My code:

private void button1_Click(object sender, EventArgs e)
 
 if (textBox1.Text == "test")
 
 label1.Show();
 label2.Hide();
 
 else
 label1.Hide();
 label2.Show();
 

My problem is this:
I have two labels, one button and one textbox.

When the TextBox equals "test" if I click on it, I want to show label1 and hide label2.

If the TextBox doesn't contain "test" I want to hide label1 and show label2.

Now when I press the button without the text "test":
label2 shows and label1 is hidden

But if the text is "test" label1 shows and label2 still shows, why?

You forgot braces in the else statement:

private void button1_Click(object sender, EventArgs e)
 
 if (textBox1.Text == "test")
 
 label1.Show();
 label2.Hide();
 
 else
 
 label1.Hide();
 label2.Show();
 
 

As stated by JayV you are missing curly brackets. The reason that label1 will still show though (if you have the text "test") is because without brackets the next statement will (and only will) be executed as part of the Else (see https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/if-else) and anything after that immediate next statement is not part of the if-else i.e. what you have written is:

• If the text in textbox1 is "test" then show label1, hide label2.


• If the text in textbox1 is not "test" then hide label1.


• Regardless of what the text is the code will run the final label2.show.

Effectively you've written

 if (textBox1.Text == "test")
 
 label1.Show();
 label2.Hide();
 
 else
 
 label1.Hide();
 

 label2.Show();

When you want:

 if (textBox1.Text == "test")
 
 label1.Show();
 label2.Hide();
 
 else
 
 label1.Hide();
 label2.Show();
 

I highly recommend you look at debugging, as stepping through your code would quickly show you this (see https://docs.microsoft.com/en-us/visualstudio/debugger/debugger-feature-tour?view=vs-2017)