int.TryParse() always returns true [closed]

Question

Closed. This question is not reproducible or was caused by typos. It is not currently accepting answers.

---

This question was caused by a typo or a problem that can no longer be reproduced. While similar questions may be on-topic here, this one was resolved in a way less likely to help future readers.

Closed 8 years ago.

Improve this question

My program contains a textbox. I need to check if it gets only numbers, then print.

        int num;
        if (this.Tree.GetType() == Main.TestInt.GetType())
        {
            if (int.TryParse(this.label.Text,out num) == true) // i tried without the == before
            {
                this.Tree.SetInfo(int.Parse(this.TextBox.Text));
                base.label.Text = base.TextBox.Text;
            }
            else
            {
                base.TextBox.Text = "";
                MessageBox.Show("Only Numbers Allowed", "Error");
            }
        }

The problem is, for some reason it always returns true, and goes to the

    this.Tree.SetInfo(int.Parse(this.TextBox.Text));

Why is it happening?

Answer 1

2 changes:

    int num;
    if (this.Tree.GetType() == Main.TestInt.GetType())
    {
        if (int.TryParse(this.TextBox.Text,out num)) //1,  you were parsing label.Text
        {
            this.Tree.SetInfo(num); //2, don't bother parsing it twice!
            base.label.Text = base.TextBox.Text;
        }
        else
        {
            base.TextBox.Text = "";
            MessageBox.Show("Only Numbers Allowed", "Error");
        }
    }

Answer 2

Probably you want to check the value of TextBox not the Label. So it would be this.TextBox.Text instead of this.Label.Text

if (int.TryParse(this.TextBox.Text,out num))
{
  this.Tree.SetInfo(this.TextBox.Text);
  base.label.Text = base.TextBox.Text;
}
else
{
  base.TextBox.Text = string.Empty;
  MessageBox.Show("Only Numbers Allowed", "Error");
}