Changed value in textbox returning wrong values

Question

This is my problem

 private void P1STRtextbox_TextChanged(object sender, EventArgs e)
    {

        if (Convert.ToInt32(P1STRtextbox.Text) > Convert.ToInt32(P1RemainingStatPointsLabel.Text))
        {
            MessageBox.Show("You have enter a number larger than your remaining stat points!");
        }           
        else
        {
            P1RemainingStatPointsLabel.Text = (PlayerOne[P1ClassNumber].TotalStatPoints - Convert.ToInt32(P1STRtextbox.Text)).ToString();
        }
    }
    private void P1DEXtextbox_TextChanged(object sender, EventArgs e)
    {
        if (Convert.ToInt32(P1DEXtextbox.Text) > Convert.ToInt32(P1RemainingStatPointsLabel.Text))
        {
            MessageBox.Show("You have enter a number larger than your remaining stat points!");
        }
        else
        {
            P1RemainingStatPointsLabel.Text = (PlayerOne[P1ClassNumber].TotalStatPoints - Convert.ToInt32(P1DEXtextbox.Text)).ToString();
        }
    }

So basicly the event is triggered when the text in the box is changed. When I input a number into the strength text box at first, it does the deductions properly, but when I go to the Dexterity box and try to edit the value, it does not do the deduction properly. As you can see in the picture(remaining stat points) that 3000-(250+750) != 2250; Can someone please give me some logical advice? Thank you for your time and effort in advance!

spinalfrontier · Accepted Answer · 2013-12-29 07:34:40Z

2

How about changing you P1DEXtextbox_TextChanged else statement to use the below?

P1RemainingStatPointsLabel.Text = (PlayerOne[P1ClassNumber].TotalStatPoints - (Convert.ToInt32(P1STRtextbox.Text) + Convert.ToInt32(P1DEXtextbox.Text))).ToString();

answered Dec 29, 2013 at 7:34

spinalfrontier

8771 gold badge15 silver badges22 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

MAV · Accepted Answer · 2013-12-29 07:34:52Z

2

You are never adding the strength and dexterity values. You need to take both fields into account when changing the remaining stats value:

int strength = Convert.ToInt32(P1STRtextbox.Text);
int dexterity = Convert.ToInt32(P1DEXtextbox.Text);
P1RemainingStatPointsLabel.Text = (PlayerOne[P1ClassNumber].TotalStatPoints - (strength + dexterity)).ToString();

You also have to make sure that you handle possible errors when converting text to an integer.

answered Dec 29, 2013 at 7:34

MAV

7,4674 gold badges33 silver badges47 bronze badges

Collectives™ on Stack Overflow

Changed value in textbox returning wrong values

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related