i am getting an error "FormatException was unhandled by user code"while executing update command on gridview?

Question

0.00/5 (No votes)

See more:

this is the ccode i m using under gridview rowupdating event:

C#

protected void GridView1_RowUpdating(object sender, GridViewUpdateEventArgs e)
    {
        if (con.State == ConnectionState.Closed)
        {
            con.Open();
        }

        int autoid = Convert.ToInt32(((Label)GridView1.Rows[e.RowIndex].FindControl("Label12")).Text);// error is in this line
        string pcode = ((Label)GridView1.Rows[e.RowIndex].FindControl("lblp")).Text;
        string fyyear = ((Label)GridView1.Rows[e.RowIndex].FindControl("lblf")).Text;
        string date = ((TextBox)GridView1.Rows[e.RowIndex].FindControl("TextBox3")).Text;
        string salary = ((TextBox)GridView1.Rows[e.RowIndex].FindControl("TextBox4")).Text;
        string ta = ((TextBox)GridView1.Rows[e.RowIndex].FindControl("TextBox5")).Text;
        string contigency = ((TextBox)GridView1.Rows[e.RowIndex].FindControl("TextBox6")).Text;
        string  nrc= ((TextBox)GridView1.Rows[e.RowIndex].FindControl("TextBox7")).Text;
        string institcharges = ((TextBox)GridView1.Rows[e.RowIndex].FindControl("TextBox8")).Text;
        string others = ((TextBox)GridView1.Rows[e.RowIndex].FindControl("TextBox9")).Text;
        string str = "update monthly set date='" + date + "',salary='" + salary + "',ta='" + ta + "',contigency='" + contigency+ "',nrc='" + nrc+ "',institcharges='" + institcharges+"' where autoid=" + autoid;
        SqlCommand cmd = new SqlCommand(str, con);
        cmd.Connection = con;
        // con.Open();
        cmd.ExecuteNonQuery();
        cmd.Dispose();

        GridView1.EditIndex = -1;

        grid_show();
    }

please help me with this?

Posted 6-Feb-13 20:07pm

a2ulthakur

Add a Solution

Comments

a2ulthakur 7-Feb-13 2:24am

help !!!!

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)

Ankur^\m/ · Accepted Answer · 2013-02-06T20:27:00

Solution 1

int autoid = Convert.ToInt32(((Label)GridView1.Rows[e.RowIndex].FindControl("Label12")).Text);// error is in this line

The text of your Label Label12 isn't an interger value. FormatException[^] is thrown when the format of an argument does not meet the parameter specifications of the invoked method.

You could use Int32.TryParse Method[^] instead. This will try to parse your string and if it's a correct integer, it will return true and the value will be returned in an out variable which then can be used as a parameter.

Put a debugger on that line, check the label text and things will be very clear to you.

Hope this helps!

Posted 6-Feb-13 20:27pm

Ankur^\m/

Comments

a2ulthakur 7-Feb-13 2:31am

ankur my autoid field is an int field an autoincrement field

Ankur^\m/ 7-Feb-13 3:02am

And you are assigning value to that field from a Label in your HTML page. The text that Label contains is not a valid int. IT could be blank or a number with space. Add a watch for ((Label)GridView1.Rows[e.RowIndex].FindControl("Label12")).Text
and see what it returns.

a2ulthakur 7-Feb-13 2:34am

and how to use Int32.TryParse method in my above mentioned line??

Ankur^\m/ 7-Feb-13 3:03am

Click on the first overload link which is http://msdn.microsoft.com/en-us/library/f02979c7.aspx
You will find a usage example there.

a2ulthakur 7-Feb-13 3:29am

when i am doing [ ((Label)GridView1.Rows[e.RowIndex].FindControl("Label12")).Text] its saying cannot implicitly convert type 'string'to 'int'

Ankur^\m/ 7-Feb-13 3:46am

Just add watch for the text I gave you above. Do not include Convert.ToInt32

a2ulthakur 7-Feb-13 3:51am

ankur: i have a question can i use autoincrement id for the purpose of updating my records as somebody told me that i can not work with auto increment field. plus one more thing do u knw how can i custom generate autoid field any other method except the sql auto increment field

Ankur^\m/ 7-Feb-13 3:58am

Yes you can. The best way would be to create an identity column in the database. If you do not want that, create a static global variable in your page. Set it to the max id value when the gridview first fills. Everytime an update is made, increment that and use that as id field. But again, I would suggest let the database handle it. You just read the value and use in in updates/delete.
And please use 'reply' link to reply to a comment.

a2ulthakur 7-Feb-13 4:40am

at System.Number.StringToNumber(String str, NumberStyles options, NumberBuffer& number, NumberFormatInfo info, Boolean parseDecimal)
at System.Number.ParseInt32(String s, NumberStyles style, NumberFormatInfo info)
at System.Convert.ToInt32(String value)
at gridshow.GridView1_RowUpdating(Object sender, GridViewUpdateEventArgs e) in c:\Users\a2ul\Desktop\test123\gridshow.aspx.cs:line 167
at System.Web.UI.WebControls.GridView.OnRowUpdating(GridViewUpdateEventArgs e)
at System.Web.UI.WebControls.GridView.HandleUpdate(GridViewRow row, Int32 rowIndex, Boolean causesValidation)
at System.Web.UI.WebControls.GridView.HandleEvent(EventArgs e, Boolean causesValidation, String validationGroup)
at System.Web.UI.WebControls.GridView.OnBubbleEvent(Object source, EventArgs e)
at System.Web.UI.Control.RaiseBubbleEvent(Object source, EventArgs args)
at System.Web.UI.WebControls.GridViewRow.OnBubbleEvent(Object source, EventArgs e)
at System.Web.UI.Control.RaiseBubbleEvent(Object source, EventArgs args)
at System.Web.UI.WebControls.LinkButton.OnCommand(CommandEventArgs e)
at System.Web.UI.WebControls.LinkButton.RaisePostBackEvent(String eventArgument)
at System.Web.UI.WebControls.LinkButton.System.Web.UI.IPostBackEventHandler.RaisePostBackEvent(String eventArgument)
at System.Web.UI.Page.RaisePostBackEvent(IPostBackEventHandler sourceControl, String eventArgument)
at System.Web.UI.Page.RaisePostBackEvent(NameValueCollection postData)
at System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint)

this is the stack trace