row is not updating in gridview

Question

0.00/5 (No votes)

See more:

want to update row in gridview.i had written this code.i am updating from ID and id is int type.i think so that is the problem that row is not updating.
please help me with that....

C#

protected void GridView1_RowUpdating(object sender, GridViewUpdateEventArgs e)
{
    GridViewRow row = GridView1.Rows[e.RowIndex];

    TextBox ID = (TextBox)row.FindControl("TextBoxEditID");
    TextBox node = (TextBox)row.FindControl("TextBoxEditNode");
    TextBox title = (TextBox)row.FindControl("TextBoxEdittitle");
    TextBox source = (TextBox)row.FindControl("TextBoxEditsource");
    TextBox scope = (TextBox)row.FindControl("TextBoxEditscope");
    

    //int IDD = Int32.Parse(GridView1.DataKeys[e.RowIndex].Value.ToString());

    int IDD = Convert.ToInt32(ID.ToString());

    string nod = node.Text;
    string tit = title.Text;
    string sou = source.Text;
    string scpe = scope.Text;
    


    Update(IDD, nod, tit, sou, scpe);
    


}
private void Update(int IDD, string nod, string tit, string sou, string scpe)
{
    try
    {
        string query = "UPDATE ['2013 Projects$'] SET Node =@nod, Title =@tit, Source =@sou, Scope =@scpe, WHERE ID=@IDD";
        con = new SqlConnection(connstring);
        cmd = new SqlCommand(query, con);


        cmd.Parameters.Add("@nod", SqlDbType.VarChar).Value = nod;
        cmd.Parameters.Add("@tit", SqlDbType.VarChar).Value = tit;
        cmd.Parameters.Add("@sou", SqlDbType.VarChar).Value = sou;
        cmd.Parameters.Add("@scpe", SqlDbType.VarChar).Value = scpe;
     

        con.Open();
        cmd.ExecuteNonQuery();
        con.Close();

        GridView1.EditIndex = -1;
        data();
    }
    catch (Exception ex)
    {
        throw ex;
    }


}

Posted 26-May-13 18:53pm

fak_farrukh

Updated 26-May-13 19:29pm

_Amy

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)

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) · Accepted Answer · 2013-05-26T20:19:00

Solution 1

There is a problem in your code.

You have forgot to assign the parameter @IDD.

Please do something like below.

C#

cmd.Parameters.Add("@IDD", SqlDbType.Int).Value = IDD;

Posted 26-May-13 20:19pm

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ)

Comments

fak_farrukh 27-May-13 2:23am

i had add this same result

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 2:28am

Debug the code and at the line con.Open();, just hover on cmd and check what is query. See whether all the values are replaced in parameter or not. Post that query here.

One more thing, post the connection string you are using.

fak_farrukh 27-May-13 2:37am

i have debug the code but it does not goes to rowupdating event

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 2:40am

Can you post the GridView Markup in aspx page here ?

fak_farrukh 27-May-13 2:44am

<asp:GridView ID="GridView1" runat="server" AutoGenerateColumns="False"
onrowcancelingedit="GridView1_RowCancelingEdit"
onrowcommand="GridView1_RowCommand" onrowdatabound="GridView1_RowDataBound"
onrowdeleted="GridView1_RowDeleted" onrowdeleting="GridView1_RowDeleting"
onrowediting="GridView1_RowEditing" onrowupdated="GridView1_RowUpdated"
onrowupdating="GridView1_RowUpdating"
onselectedindexchanged="GridView1_SelectedIndexChanged" Width="1009px"
style="margin-left: 0px" BackColor="White" BorderColor="White"
BorderStyle="Ridge" BorderWidth="2px" CellPadding="3" CellSpacing="1"
GridLines="None">
<columns>
<asp:TemplateField HeaderText="ID">
<edititemtemplate>
<asp:TextBox ID="TextBoxEditID" runat="server" Text='<%# Bind("ID") %>' />

<itemtemplate>
<asp:Label ID="LabelID" runat="server" Text='<%# Bind("ID") %>' />

<footertemplate>
<asp:TextBox ID="TextBoxID" runat="server" />

<asp:TemplateField HeaderText="Node">
<edititemtemplate>
<asp:TextBox ID="TextBoxEditNode" runat="server" Text='<%# Bind("Node") %>' />

<itemtemplate>
<asp:Label ID="LabelNode" runat="server" Text='<%# Bind("Node") %>' />

<footertemplate>
<asp:TextBox ID="TextBoxNode" runat="server" />

<asp:TemplateField HeaderText="Title">
<edititemtemplate>
<asp:TextBox ID="TextBoxEdittitle" runat="server" Text='<%# Bind("title") %>' />

<itemtemplate>
<asp:Label ID="Labeltitle" runat="server" Text='<%# Bind("title") %>' />

<footertemplate>
<asp:TextBox ID="TextBoxrtitle" runat="server" />

<asp:TemplateField HeaderText="Source">
<edititemtemplate>
<asp:TextBox ID="TextBoxEditsource" runat="server" Text='<%# Bind("source") %>' />

<itemtemplate>
<asp:Label ID="Labelsource" runat="server" Text='<%# Bind("source") %>' />

<footertemplate>
<asp:TextBox ID="TextBoxsource" runat="server" />

<asp:TemplateField HeaderText="Scope">
<edititemtemplate>
<asp:TextBox ID="TextBoxEditscope" runat="server" Text='<%# Bind("scope") %>' />

<itemtemplate>
<asp:Label ID="Labelscope" runat="server" Text='<%# Bind("scope") %>' />

<footertemplate>
<asp:TextBox ID="TextBoxscope" runat="server" />

<asp:CommandField ButtonType="Button" ShowEditButton="True" />

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 2:51am

You have mentioned onrowupdating="GridView1_RowUpdating".
So, why it is not going to that event ?

How you exactly bind the GridView ?

fak_farrukh 27-May-13 2:58am

using this code on pageload

public void data()
{
try
{
//con = new SqlConnection(connstring);
string query = "SELECT ['2013 Projects$'].* FROM ['2013 Projects$']";
da = new SqlDataAdapter(query, connstring);
dt = new DataTable();
da.Fill(dt);
GridView1.DataSource = dt;
GridView1.DataBind();
}
catch (Exception)
{

}

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 3:02am

You have forgot to check the IsPostBack property. So, it is creating the problem.

Do like below.

try
{
if (!IsPostBack)
{
//con = new SqlConnection(connstring);
string query = "SELECT ['2013 Projects$'].* FROM ['2013 Projects$']";
da = new SqlDataAdapter(query, connstring);
dt = new DataTable();
da.Fill(dt);
GridView1.DataSource = dt;
GridView1.DataBind();
}
}

fak_farrukh 27-May-13 3:01am

when i press f 11 its says 'F' is already defined 'aa' is already defined and many more

fak_farrukh 27-May-13 3:13am

int IDD = Convert.ToInt32(ID.ToString());
input string was not in correct format

error on this line

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 3:16am

Assuming the IDD is a Integer field in your database, please use the query I suggested initially in my answer like below.

Don't convert to string and again to int. Because it is already in INT.

cmd.Parameters.Add("@IDD", SqlDbType.Int).Value = IDD;

fak_farrukh 27-May-13 3:23am

int IDD = Convert.ToInt32(ID.ToString());
but what is about to do with this error

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 3:28am

Oh ok. It should be like below.

int IDD = Convert.ToInt32(ID.Text);

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 3:44am

Did you try ?

fak_farrukh 27-May-13 3:55am

yesssss and it is done
thank yu sir

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 3:57am

Most Welcome. My Pleasure... :)

fak_farrukh 27-May-13 3:59am

ok i have another question but not this post related can i ask yu here

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 4:02am

Yes, of course.

fak_farrukh 27-May-13 4:08am

i have an index page in which i have my task tab in menu.now when a person name john login in then in my task tab it should show only john task if another person login its should show his task but the index page must be same for all of them
how can i do this...

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 4:15am

Use session to store the info of the logged in user.

Session["UserName"] = CurrentUserName;

In the index page, where you fetch the data for the logged in user, just query the database with this name like below.

String currentUser = Session["UserName"].ToString();

Query will be like...

Select Column1, Column2
FROM
Table
WHERE User = currentUser

Note:- This is just an idea. You need to implement as per your logic and requirements.

fak_farrukh 27-May-13 4:16am

thanks again

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 27-May-13 4:33am

Most welcome.