Beginner need help

Josip Pavic

Can you tell me what is wrong with this IF statement??
I think I mix something with operators.

namespace exercise1
{
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
}

private void label1_Click(object sender, EventArgs e)
{

}

private void button1_Click(object sender, EventArgs e)
{
int age;

age = int.Parse(textBox1.Text);

if (age < 17)
{
MessageBox.Show("You're still a youngster.");
}
if (age <= 25)
{
MessageBox.Show("Fame beckons!");
}
if (age <= 40)
{
MessageBox.Show("There's still time.");
}
if (age > 40)
{
MessageBox.Show("Oh dear, you've probably missed it!");
}

}
}
}

Thanks for help!!!

7 Replies

Know the answer? Post it — somebody with the same question will find it here.

Sam Hobbs15y ago
Accepted answer
Also try to use a more meaningful subject for the threads you create instead of something like "Beginner need help". The best reason for doing that is so that you get the best answer possible since the subject of the thread can influence who looks at the question.
0
Sam Hobbs15y ago
Thank you for marking a post as the answer. Actually Satish's post should be marked as the answer.
0
Josip Pavic15y ago
Thank you for help. I will do so.
0
Mayur Gujrathi15y ago
Dear josip
your code seems correct but due to your code logic
if suppose a no is less than 25 then it executes respective code
then control comes down and will check less than 40 which also becomes true and again executes code in that block

either do as suggested satish or get out of block after displaying massagebox
0
Josip Pavic15y ago
thank you. it¨s working ok. I forget else. thanks!!!
0
Satish Bhat15y ago
Try

if (age < 17)
{
MessageBox.Show("You're still a youngster.");
}
else if (age <= 25)
{
MessageBox.Show("Fame beckons!");
}
else if (age <= 40)
{
MessageBox.Show("There's still time.");
}
else if (age > 40)
{
MessageBox.Show("Oh dear, you've probably missed it!");
}
0
Josip Pavic15y ago
yes I did that. But my code is not working. Only one messageBox shoud appear. instead there are 2 or 3 boxes.
0