C# help

Anyone here wouldn't happen to know a little bit about c#, would they?

 

Do you mean C+ or C++? I dont know anything about it but what are you trying to do? I work on Cisco networks. Maybe i can help

 

Nope C#,

I'm trying to basically calculate a running total, and average of a double

PHP:

private void btnDisp_Click(object sender, EventArgs e)
{
//Calculate Amount
Amount += Price;
 
 
//Calculate Average
Average = (Amount / Count);
 
//Display values
txtCount.Text = Count.ToString();
txtAmt.Text = Amount.ToString();
txtAvg.Text = Average.ToString();
}


basically trying to get something like this to work.

 

So what's the problem?

 

The Amount(sum) and Average aren't calculating/showing up. It still shows the values as zero. All the code seems to be right.

Sorry I'm kind of new to this whole thing.

 

Here's the program file.

PHP:

/*
* Adam Licht
* Project2
* CIS220 Section 2
* 1/27/2010
* 
* This program takes values from textboxes, places them in to varibles and then in to new text boxes. 
* It also keeps track of the count of inputs and the sum and average of the price.
* 
*/
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
namespace Project2
{
public partial class frmMain : Form
{
//Define Data Members
Int32 ID = 0;
String Description = "";
Double Price = 0;
Int32 Quantity = 0;
Int32 Count = 0;
Double Amount = 0;
Double Average = 0;
//end Data Members
 
 
public frmMain()
{
 
InitializeComponent();
}
private void frmMain_Load(object sender, EventArgs e)
{
DateTime dtmCurrent = DateTime.Today;
lblDate.Text = dtmCurrent.ToString("D");
lblTitle.Text = "Project Two";
txtIDIn.Focus();
 
 
 
 
 
}
private void btnExit_Click(object sender, EventArgs e)
{
Application.Exit();
}
private void btnAbout_Click(object sender, EventArgs e)
{
MessageBox.Show("This is project 2. It does a little more than Project 1, yet the headache that came with programing it increased exponentally.","About");
 
}
private void lblTitle_Click(object sender, EventArgs e)
{
}
private void label1_Click(object sender, EventArgs e)
{
}
private void lblDate_Click(object sender, EventArgs e)
{
 
 
 
}
 
 
private void label2_Click(object sender, EventArgs e)
{
}
private void btnGet_Click(object sender, EventArgs e)
{
//txt to data members
Int32 ID = Convert.ToInt32(txtIDIn.Text);
String Description = txtDiscIn.Text;
Double Price = Convert.ToDouble(txtPriceIn.Text);
Int32 Quantity = Convert.ToInt32(txtQtyIn.Text);
//end txt to data members
 
//Counter
Count = Count + 1;
//end counter
 
//Data Member to Output
 
txtIDOut.Text = ID.ToString();
txtDiscOut.Text = Description;
txtPriceOut.Text = Price.ToString();
txtQtyOut.Text = Quantity.ToString();
 
 
 
 
//end Data member to Output
 
 
 
}
private void btnClrIn_Click(object sender, EventArgs e)
{
//Clear Inputs
txtIDIn.Clear();
txtDiscIn.Clear();
txtPriceIn.Clear();
txtQtyIn.Clear();
//Reset Focus
txtIDIn.Focus();
}
private void btnClrOut_Click(object sender, EventArgs e)
{
//Clear Outputs
txtIDOut.Clear();
txtDiscOut.Clear();
txtPriceOut.Clear();
txtQtyOut.Clear();
//Reset Focus
txtIDIn.Focus();
}
private void btnDisp_Click(object sender, EventArgs e)
{
//Calculate Amount
Amount += Price;
 
 
//Calculate Average
Average = (Amount / Count);
 
//Display values
txtCount.Text = Count.ToString();
txtAmt.Text = Amount.ToString();
txtAvg.Text = Average.ToString();
}
private void btnReset_Click(object sender, EventArgs e)
{
//Clear Values
txtCount.Clear();
txtAmt.Clear();
txtAvg.Clear();
 
//Set values to 0
Count = 0;
Amount = 0;
Average = 0;
}
private void txtCount_TextChanged(object sender, EventArgs e)
{
 
}
private void txtAmt_TextChanged(object sender, EventArgs e)
{
}
 
}
}


 

You're redefining Price. Don't redefine it. In btnGet_Click() just do Price = ...

If you do Double Price = .... , price is local to that function.

 

And actually, btw, you're redefining all variables inside of btnGet_Click()

If you define a variable in that function, you won't use the one that is global to your frmMain() class. you'll make new ones with that name that only hold value for the duration of that function call.

So you're defining a new variable Price, setting THAT to the Price, and then in display, you're using the global Price, which never gets adjusted from 0.

So you're also not going to see the correct id or any of the rest of that stuff you manipulate in btnGet_Click()

 

That's it! Thanks a ton! Apreciate it.

 

So basically only use the prefixes (String, int, double) the first time when you define them globaly?

 

Alright cool.


Thanks everyone! Apreciate it!

 

Happy we could help