HellBound Hackers | Computer General | Programming

{
public partial class Form1 : Form
{ //Global Declarations: Accumulated Charges, Total Amount of Groups, and a constant for the Rental Rate
double totalAccumulatedCharges = 0;
double totalAmountOfGroups = 0;
double rentalRate = 200 / 60;



public Form1()
{
InitializeComponent();
}

private void Form1_Load(object sender, EventArgs e)
{

}

private void btnCalculate_Click(object sender, EventArgs e)
{
// Declarations: Input - Name, Minutes used.
// Output - Current charge, Group name, Accumulated Charges, Average Charge, Number of Groups
double dblMinutesUsed;
double dblAverageCharge, dblCurrentCharge, dblAccumulatedCharges, dblNumberOfGroups;

//Input - Gather the data
try
{
dblMinutesUsed = double.Parse(txtMinutesUsed.Text);

}
catch
{
MessageBox.Show("Input was either non-numeric or blank","Invalid Input!");
txtMinutesUsed.Focus();
return;
}

//Process - Calculate Current Charge, Accumulated Charges, Number of Groups, and Average charge
dblAccumulatedCharges += totalAccumulatedCharges;
dblNumberOfGroups += totalAmountOfGroups;
dblAverageCharge = dblAccumulatedCharges/dblNumberOfGroups;
dblCurrentCharge = dblMinutesUsed * rentalRate;



//Output - Display the information

//Clear all the output listboxes
lstAccumulatedCharges.Items.Clear();
lstAverageCharge.Items.Clear();
lstCurrentCharge.Items.Clear();
lstGroupName.Items.Clear();
lstNumberOfGroups.Items.Clear();

//Display the information
lstAccumulatedCharges.Items.Add(dblAccumulatedCharges.ToString("n1"));
lstAverageCharge.Items.Add(dblAverageCharge.ToString("n1"));
lstCurrentCharge.Items.Add(dblCurrentCharge.ToString("n1"));
lstGroupName.Items.Add(txtGroupName.Text);
lstNumberOfGroups.Items.Add(dblNumberOfGroups.ToString("n1"));




}

private void btnClear_Click(object sender, EventArgs e)
{
// Clear all input text boxes and output list boxes
txtGroupName.Clear();
txtMinutesUsed.Clear();
lstAccumulatedCharges.Items.Clear();
lstAverageCharge.Items.Clear();
lstCurrentCharge.Items.Clear();
lstGroupName.Items.Clear();
lstNumberOfGroups.Items.Clear();
txtGroupName.Focus();
}

private void btnExit_Click(object sender, EventArgs e)
{
this.Close();

}
}
}

dblAccumulatedCharges += totalAccumulatedCharges;
dblNumberOfGroups += totalAmountOfGroups;

OP, let me guess, school project?
The thing you're trying to do can be done with Perl/Python, as a CLI scblockedript. 10x faster. 10x more useful. 10x readable.

Is this the first program you've written?
I'll give you a hint:
This line:
dblAccumulatedCharges += totalAccumulatedCharges;
is the same as:
dblAccumulatedCharges = dblAccumulatedCharges + totalAccumulatedCharges;

Dystopia wrote:


No, it's not the first program that I've written. I am aware that those two lines are interchangeable, which is why I didn't bother changing it. I was going to when I finished the program.

But thank you for your advice, I'll fiddle around with the calculations and see what I can come up with.

// Declarations: Input - Name, Minutes used.
// Output - Current charge, Group name, Accumulated Charges, Average Charge, Number of Groups
double dblMinutesUsed;
double dblAverageCharge, dblCurrentCharge, dblAccumulatedCharges, dblNumberOfGroups;
dblAccumulatedCharges = 0;
dblNumberOfGroups = 1;

//Process - Calculate Current Charge, Accumulated Charges, Number of Groups, and Average charge
dblCurrentCharge = dblMinutesUsed * rentalRate;
dblAverageCharge = totalAccumulatedCharges / dblNumberOfGroups;
dblAccumulatedCharges = dblCurrentCharge + totalAccumulatedCharges;
dblNumberOfGroups += totalNumberOfGroups;
totalNumberOfGroups++;

Dystopia wrote:
Alright. I've changed the declarations a bit and now when I run the program, it counts the number of times I have hit the calculate button aka total number of groups. The current charge also works correctly.





The accumulated charges is still not working. Neither is the average charge. But instead of coming up with 'NaN', I get a zero. Which means it must be calculating "0 divided by 1". But I'm not sure how to go about coding it properly so that it doesn't occur.

Here is my new process: