Exporting datagridview selectedrows to a datatable won't work

Question

0.00/5 (No votes)

See more:

Hello there. I want to export all selectedRows from a datagridview to a DataTable. When clicking (selecting) on more than 2 rows, the next error appears:

"An exception error of type System.IndexOutOfRangeException occurred in System.Data.dll."

What I have tried:

C#

DataTable table = new DataTable();
            for (int i = 0; i < dataGridView_auswahlen.Rows.Count; i++) {
                if (dataGridView_auswahlen.Rows[i].Selected) {
                    table.Rows.Add( );
                    for (int j = 0; j < dataGridView_auswahlen.Columns.Count; j++) {
                        table.Rows[i][j] = dataGridView_auswahlen[j, i].Value;
                    }
                }
            }

Posted 10-Nov-16 2:43am

TheRealProgrammer

Updated 10-Nov-16 19:49pm

Add a Solution

Comments

[no name] 10-Nov-16 8:51am

The "IndexOutOfRangeException" is pretty much self explanatory. So, what exactly is your question?

#realJSOP 10-Nov-16 8:59am

The veracity of your user ID comes immediately into question.

2 solutions

Solution 1

Why would you assume that i is valid for both dataGridView_auswahlen (row source) and table (row destination)?
Since you are only transferring selected rows, unless your selection always starts with the first row and has no gaps, you will always get an exception.
Instead of using an index, create a new row using DataTable.NewRow[^], and add that to your DataTable. You can then use the DataRow object to add your cell values.

Posted 10-Nov-16 2:54am

OriginalGriff

Comments

TheRealProgrammer 10-Nov-16 8:59am

I understand, but how can I implement that? I can't manage to understand how I can integrate this in my for.

OriginalGriff 10-Nov-16 9:18am

Create your DataTable then add the columns before the "for" loop - that way you get to give them names which helps.
Then inside your "if", create an instance of a DataRow from the DataTable, and use that instance to set the column values in a loop, pretty much as you do now. Then add the row to the table.

What part of that is difficult? Perhaps I'm explaining it badly...

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)

Karthik_Mahalingam · Accepted Answer · 2016-11-10T17:37:00

refer inline comments

C#

DataTable dt = new DataTable(); // create a table for storing selected rows
            var dtTemp = dataGridView1.DataSource as DataTable; // get the source table object
            dt = dtTemp.Clone();  // clone the schema of the source table to new table
            DataTable table = new DataTable();
            for (int i = 0; i < dataGridView1.Rows.Count; i++)
            {
                if (dataGridView1.Rows[i].Selected)
                {
                  var row =   dt.NewRow();  // create a new row with the schema 
                  for (int j = 0; j < dataGridView1.Columns.Count; j++)
                    {
                        row[j] = dataGridView1[j, i].Value;
                    }
                  dt.Rows.Add(row);  // add rows to the new table
                }
            }