C#
C#

"System.NullReferenceException was unhandled" error after last valid row.

Hi, I'm using Visual Studio 2010 C#. I have a DataGridView on one of my Forms that is connected to SQL database table. That part works. I'm also using the dataGridView1_RowEnter Event to populate the selected information into different Textboxes as I scroll down the grid. That works until I get past the last row.

As soon as I go beyond the length of the table (past the last row) and enter the first NULL row, I get the following error.

ERROR: >> System.NullReferenceException was unhandled

Why is the "if (e.RowIndex >= 0)" logic statement not preventing the error and how do I solve this problem?

private void dataGridView1_RowEnter(object sender, DataGridViewCellEventArgs e)
        {
            DataGridViewRow row = this.dataGridView1.Rows[e.RowIndex];
            if (e.RowIndex >= 0)
            {                
                textBox1.Text = row.Cells[1].Value.ToString();
                textBox2.Text = row.Cells[2].Value.ToString();
                textBox3.Text = row.Cells[3].Value.ToString();
                textBox4.Text = row.Cells[4].Value.ToString();
            }

Open in new window

Mr_FulanoAsked: 2016-08-19

Who is Participating?

Solutions

LVL 45

AndyAinscowConnect With a Mentor Freelance programmer / ConsultantCommented: 2016-08-19

I suspect you want something like:

if (!row.IsNewRow) //if (e.RowIndex >= 0)

https://msdn.microsoft.com/en-us/library/system.windows.forms.datagridviewrow.isnewrow(v=vs.110).aspx

LVL 45

AndyAinscowFreelance programmer / ConsultantCommented: 2016-08-19

>>Why is the "if (e.RowIndex >= 0)" logic statement not preventing the error and how do I solve this problem?

What is the value of e.RowIndex when it fails. You are apparently assuming it is less than zero.

LVL 64

Fernando SotoRetiredCommented: 2016-08-19

This statement needs to be in the if statement, first statement inside if statement.

DataGridViewRow row = this.dataGridView1.Rows[e.RowIndex];

LVL 45

AndyAinscowFreelance programmer / ConsultantCommented: 2016-08-19

@Fernando
Why does that assignment need to be inside the if block?

Mr_FulanoAuthor Commented: 2016-08-19

Hi Fernando, I had it that way and retried your suggestion just to make sure, but that still fails.

Mr_FulanoAuthor Commented: 2016-08-19

Hi Andy, yes...your suggestion works. The last row is negative and keeps increasing each time I enter it (i.e. -1. -2. -3...etc).

Is there a way to not show that last row at all?

LVL 64

Fernando SotoRetiredCommented: 2016-08-19

Hi Andy, I was thinking that if this was happening when he passed the last row it had a -1 as the index and therefore would be an out of bounds index or no row. Was not near a computer to test and wanted him to have something to test for the moment.

Mr_FulanoAuthor Commented: 2016-08-19

Thank you both...!!!

LVL 45

AndyAinscowFreelance programmer / ConsultantCommented: 2016-08-19

Hi Fernando, I could understand why you suggested that for some of the other possible events but was puzzled why for the RowEnter event. Thanks for clarifying.