object references not set to an instance of the object

Question

Am running asp.net application with c#.Am using gridview to update and delete the columns.deleting is working fine.if am clicking the update button i got

System.NullReferenceException: Object reference not set to an instance of an object.

My code is;

    protected bool IsRowModified(GridViewRow row)
    {
        int currentID;
        string currentName;
        string currentLocation;
      currentID = Convert.ToInt32(GridView1.DataKeys
                                [row.RowIndex].Value);

        currentName = ((TextBox)row.FindControl
                                      ("txtName")).Text;
        currentLocation = ((TextBox)row.FindControl
                                  ("txtLocation")).Text;




        **System.Data.DataRow newRow = originalTable.Select
                (String.Format("ID = {0}", currentID))[0];** //got error in this line

        if (!currentName.Equals(newRow["Name"].ToString()))
        { return true; }
        if (!currentLocation.Equals(newRow["Location"].ToString()))
        { return true; }

        return false;

    }

possible duplicate of [What is a NullReferenceException in .NET?](http://stackoverflow.com/questions/4660142/what-is-a-nullreferenceexception-in-net) — R. Martinho Fernandes, Mar 09 '11 at 12:38
and anyway Jeni, edit your question showing us the full exception stacktrace — Davide Piras, Mar 09 '11 at 12:38
Either `originalTable` is null or `originalTable.Select` result is null — Stecya, Mar 09 '11 at 12:38

score 1 · Answer 1 · answered Mar 09 '11 at 13:39

Either originalTable is null, or originalTable.Select(...) is returning null.

Could it be that you've deleted the underlying data from originalTable and not updated the UI?

An alternative method might be to use the DataItem property of the GridViewRow parameter:

protected bool IsRowModified(GridViewRow row)     
{                     
    string currentName = ((TextBox)row.FindControl("txtName")).Text;
    string currentLocation = ((TextBox)row.FindControl("txtLocation")).Text;                            
    DataRow newRow = (DataRow)row.DataItem;

    if (!string.Equals(currentName, newRow["Name"].ToString()))         
    { 
        return true; 
    }         
    if (!string.Equals(currentLocation, newRow["Location"].ToString()))         
    { 
        return true; 
    }          

    return false;      
}

score -2 · Answer 2 · answered Mar 09 '11 at 12:40

-2

My guess is that your currentID variable is null or empty. You should check your values for null or empty before you try to use them.

if(!String.IsNullOrEmpty(currentID))
{
   // Continue processing
}

answered Mar 09 '11 at 12:40

Robert Williams

1,310
9
12

I doubt currentID is the culprit. If there's no dot, there's no `NullReferenceException`. – R. Martinho Fernandes Mar 09 '11 at 12:42
3

`currentID` is an integer - it can't be `null`. – Grant Thomas Mar 09 '11 at 12:43
Ah, Mr. Disappointment, you're right....I missed currentID being assigned as an integer. – Robert Williams Mar 09 '11 at 12:47

object references not set to an instance of the object

2 Answers2