Trying to sum several fields into a total field

Norris Chappell

int lblQtr1 = int.Parse(Q1IntProc.Text) + int.Parse(Q1IntProj.Text) + int.Parse(Q1ExtBSI.Text) + int.Parse(Q1ExtCMMI.Text); This line is giving me the error:

Agent__007

A quick check would be - just put a debug point on the line throwing the error, and hover (move your mouse pointer over) Q1IntProc, Q1IntProj, Q1ExtBSI and Q1ExtCMMI one by one (or with a quick watch (right click -> Quick Watch)) and check which one's null. There must be (atleast) one of them which is null. If so, look at corresponding "FindControl" statement and check if the string param you are specifying for the "FindControl" call ("txtQ1IntProc", for instance) does actually represent a valid TextBox in your (GridView's) row. Hope that helps.

You have just been Sharapova'd.

Norris Chappell

The qtr1 field is 0. However the other 4 fields are showing in debug a Null value.

Norris Chappell

I did that and all of them are null. aspx: I changed to my code to TextBox but still getting the null errors: Qtr1 value is 0 the other 4 are null. :(

Lost User

The ids of the textboxes in your ASP file do not (as far as I can see) match the names you are using in your code. Also coding a line such as

int lblQtr1 = int.Parse(Q1IntProc.Text) + int.Parse(Q1IntProj.Text) + int.Parse(Q1ExtBSI.Text) + int.Parse(Q1ExtCMMI.Text);

is asking for trouble. You have no idea what may be in any of the text fields, so if the user enters garbage it will just fail. You shoud use int.TryParse to capture the numeric values, so you can alert the user if any is invalid. You should also assign each value to an individual variable before summing them. That way, you can debug it much more easily.

Pete OHanlon

Null is not an integer value. This is why it's failing - as a plain int is not a nullable type, you cannot just parse it, you should actually use TryParse instead. You need to break your logic up into something that looks a little bit like this:

//int lblQtr1 = int.Parse(Q1IntProc.Text) + int.Parse(Q1IntProj.Text) + int.Parse(Q1ExtBSI.Text) + int.Parse(Q1ExtCMMI.Text);
private int GetValue(TextBox textBox)
{
int output;
if (int.TryParse(textBox.Text, out output))
{
return output;
}
return 0;
}
int lblQtr1 = GetValue(Q1IntProc) + GetValue(Q1IntProj) + GetValue(Q1ExtBSI) + GetValue(Q1ExtCMMI);

Agent__007

Norris Chappell wrote:

I changed to my code to TextBox but still getting the null errors:

Yes, that should be Label only and not TextBox, my mistake. Not sure what's wrong here. But in your ASP markup, are you missing an <ItemTemplate> wrapped around the controls? You can also try finding the control in a "Cell" (like row.Cells[N].FindControl(), where N-> index of the container cell). Not sure it's related, but you can also check the actual IDs of the controls after they are rendered and there's a property "ClientIDMode"(?) of GridView (not sure but I remember to have faced something similar a few years ago when I worked on ASP.NET WebForms) which you can use if there's a mismatch. Good luck!

You have just been Sharapova'd.

Norris Chappell

Hi Everyone I finally figured it out. I didn't need to do it in C# after all. I modified my SQL to sum the fields I needed. I want to thank everyone that contributed.

Norris Chappell

I getting that error that The name 'Q1IntProc' does not exist in the current context. So I should have an if statement for each field that I am summing? So what goes here textBox.Text?

Norris Chappell

Hi, I not sure how to use this code? Please let me know if I need to make changes to it?

Lost User

I already suggested that you check the variable names that you are using. They appear not to match the names you have set in your ASP page.

Norris Chappell

It is what I have in my asp. Did I not set it properly?

Norris Chappell

Pete, I not for sure how to use this code. Can you let me know what I need to do? I still getting the error for the 4 fields. The fields do not exist in the current context.