Cornhole Calculator

Shane Stapleton

Hey guys so I'm pretty new at C#. Right now I am trying to create a cornhole calculator (you know, the game where you throw bean bags into a hole). I have the GUI created and some of the coding done, but I can't figure out my current problem which is that my variable that adds up the Total Points for each team resets back to 0 every time. I'm not sure how to update it to where it keeps adding to the total score every time I add a new number in and click my "Add" button.

Anyways, here's my code that I have so far.

public MainPage()

{

this.InitializeComponent();

}

private void addTeamPoints(object sender, RoutedEventArgs e)

{

int oneAdd = int.Parse(teamOneAdd.Text);

int twoAdd = int.Parse(teamTwoAdd.Text);

int oneTotal = 0;

int twoTotal = 0;

if (oneAdd > 0)

{

oneTotal += oneAdd;

teamOnePoints.Text = oneTotal.ToString();

}

else if (twoAdd > 0)

{

twoTotal += twoAdd;

teamTwoPoints.Text = twoTotal.ToString();

}

else

{

string ex = "Enter a number higher than 0";

exception.Text = ex;

}

teamOneAdd.Text = "0";

teamTwoAdd.Text = "0";

}

So there's a Textbox that is automatically set to 0 and then you input a number in to change whichever team's total. There can't be a number higher than 0 in both boxes and both boxes can't be equal to zero. Then, after it updates the total score the box should change back to 0 so you can enter in a new number.

I am also trying to figure out how to not let the total equal over 21 and if it does then it resets the total back to 15 (because the team busted).

3 Replies

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

theLizard10y ago
Then you are not checking the value when you should, eg
int oneAdd = int.Parse(teamOneAdd.Text);
you should be checking what oneAdd or twoAdd is befor any other test
if(oneAdd > 0 && oneAdd < 21)
oneTotal += oneAdd ; }
else if(oneAdd >= 21)
oneTotal = 15;
there are other ways that you can check for lower and upper limits of your total var, this really depends on what your aim is and what other processing you want to do in your method.
If for arguments sake a team busts at totals over 21 no other processing needs to take place then a simple test would be
if((oneAdd=int.Parse(teamOneAdd.Text) >=21)
{
oneTotal = 15;
return;
}
Same for twoAdd...
Shane Stapleton10y ago
Thanks for your reply. All of it worked out except for the last part.
When you type a number over 21 the program adds it to 15 instead of equaling 15. I've tried various things like trying to subtract oneAdd from the total, but it still doesn't equal 15.
theLizard10y ago
Your first problem is declaring int oneTotal = 0; in private void addTeamPoints(object sender, RoutedEventArgs e)

Clicking add resets oneTotal and twoTotal back to 0 which is what you have told it to do.

Put oneTotal and twoTotal here

public partial class Form1 : Form
{
int oneTotal = 0;
int twoTotal = 0;

public Form1()

{

these vars can now be accessed anywhere in the form so you can do oneTotal += oneAdd; without it resetting to zero each time.
the other if(oneTotal >= 21) { oneTotal = 15; }