Gridview.SelectedRow problem

toshesh · November 27th, 2006, 03:45 PM

Hi everyone,

I am trying to use the update function on gridview.
The SelectedRow function should return the current row to be
updated, however, I get "Object reference not set to an instance of an object" error. I have checked the variable names and they are correct. could please someone help me!!!

Code:

protected void gviewAccountSub_RowUpdating(object sender, GridViewUpdateEventArgs e) {
        try {
            GridViewRow row = gviewAccountSub.SelectedRow;
            row.RowState = DataControlRowState.Edit;

            DropDownList ddlCurrency = (DropDownList)row.FindControl("ddlvAccountSub_gviewCurrencyEdit");
            DropDownList ddlAccount = (DropDownList)row.FindControl("ddlvAccountSub_gviewAccountEdit");

            e.NewValues["Currency"] = ddlCurrency.SelectedValue;
            e.NewValues["Account"] = ddlAccount.SelectedValue;
        }

thenoseknows · November 28th, 2006, 11:54 PM

You'd think that would work...

Instead of assuming the selected row is the same as the row generating the event, I think it would be a better idea to cast the sender object (the first argument to this event handler) as GridViewRow and make that your row to work with.

I'm also not sure you can assume those are the names of those two drop downs you are trying to find in the row. That, if I remember correctly doesn't work. ASP.NET names the ddl in each row something different because each row has to have a set of controls with unique ID's right?

Step through the code and find out exactly which statement throws the exception!

Neil Timmerman
Programmer
Veris Consulting

SriRamaKrishna · November 29th, 2006, 12:23 PM

Hi ,
use e.SelectedRow;
instead of
GridViewRow row = gviewAccountSub.SelectedRow;

toshesh · November 30th, 2006, 01:01 PM

Thank you for your reply!! I managed to get it working by
using the sender and works fine.
GridViewRow row = ((GridView)sender).Rows[e.RowIndex];
I can see now that e.SelectedRow works as well...