When I try to edit the data it will prompt an error which is ''input string was not in a correct format.'' at the line "totalpc=totalpc+convert.toint32(row.cells[2].text)"

Question

1.00/5 (1 vote)

See more:

private void calculateSum()
{
    Int32 TotalPC = 0;
    foreach (GridViewRow row in GridView1.Rows)
    {
        TotalPC = TotalPC + Convert.ToInt32(row.Cells[2].Text);
    }
    GridView1.FooterRow.Cells[1].Text = "Total PC";
    GridView1.FooterRow.Cells[2].Text = TotalPC.ToString();

}

protected void GridView1_DataBound(object sender, EventArgs e)
{
    calculateSum();
}

What I have tried:

<form id="form1" runat="server">
        <h2><a href="Mainpage.aspx" style="color:darkorange;text-decoration:none">Total User PCs in Active</a></h2>
        <div>

            <asp:SqlDataSource ID="SqlDataSource1" runat="server" ConnectionString="<%$ ConnectionStrings:ConnectionString %>" DeleteCommand="DELETE FROM [TotalAll] WHERE [No] = @No" InsertCommand="INSERT INTO [TotalAll] ([No], [Department], [TotalPC]) VALUES (@No, @Department, @TotalPC)" SelectCommand="SELECT * FROM [TotalAll]" UpdateCommand="UPDATE [TotalAll] SET [Department] = @Department, [TotalPC] = @TotalPC WHERE [No] = @No">
                <DeleteParameters>
                    <asp:Parameter Name="No" Type="Int32" />
                </DeleteParameters>
                <InsertParameters>
                    <asp:Parameter Name="No" Type="Int32" />
                    <asp:Parameter Name="Department" Type="String" />
                    <asp:Parameter Name="TotalPC" Type="Int32" />
                </InsertParameters>
                <UpdateParameters>
                    <asp:Parameter Name="Department" Type="String" />
                    <asp:Parameter Name="TotalPC" Type="Int32" />
                    <asp:Parameter Name="No" Type="Int32" />
                </UpdateParameters>
            </asp:SqlDataSource>
            <br />
            <asp:GridView ID="GridView1" runat="server" AutoGenerateColumns="False" BackColor="White" BorderColor="#CC9966" BorderStyle="None" BorderWidth="1px" CellPadding="4" DataKeyNames="No" DataSourceID="SqlDataSource1" OnDataBound="GridView1_DataBound" ShowFooter="True" >
                <Columns>
                    <asp:BoundField DataField="No" HeaderText="No" ReadOnly="True" SortExpression="No" />
                    <asp:BoundField DataField="Department" HeaderText="Department" SortExpression="Department" />
                    <asp:BoundField DataField="TotalPC" HeaderText="TotalPC" SortExpression="TotalPC" />
                    <asp:CommandField HeaderText="Operation" ShowEditButton="True" />
                </Columns>
                <FooterStyle BackColor="#FFFFCC" ForeColor="#330099" />
                <HeaderStyle BackColor="#990000" Font-Bold="True" ForeColor="#FFFFCC" />
                <PagerStyle BackColor="#FFFFCC" ForeColor="#330099" HorizontalAlign="Center" />
                <RowStyle BackColor="White" ForeColor="#330099" />
                <SelectedRowStyle BackColor="#FFCC66" Font-Bold="True" ForeColor="#663399" />
                <SortedAscendingCellStyle BackColor="#FEFCEB" />
                <SortedAscendingHeaderStyle BackColor="#AF0101" />
                <SortedDescendingCellStyle BackColor="#F6F0C0" />
                <SortedDescendingHeaderStyle BackColor="#7E0000" />
            </asp:GridView>

        </div>
    </form>

Posted 23-Feb-20 18:51pm

WeiJun420

Updated 23-Feb-20 18:54pm

v2

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

phil.o · Accepted Answer · 2020-02-23T18:55:00

Solution 1

Please do not use Convert class to parse numeric inputs. Use TryParse method instead.

C#

foreach (GridViewRow row in GridView1.Rows)
{
   if (int.TryParse(row.Cells[2].Text, out int value))
   {
      TotalPC += value;
   }
   // Optional: deal with an invalid integer value or an empty string
   else
   {
      // ...
   }
}

Posted 23-Feb-20 18:55pm

phil.o

Updated 23-Feb-20 18:57pm

v2

Comments

WeiJun420 24-Feb-20 1:11am

Ok,it works, Thanks

phil.o 24-Feb-20 1:15am

You're welcome.