I have a recordset that contains 17 tables I want to loop through the recordset and dynamically populate the colums in the gridview control.
Here is my code snippet

 for (int i = 0; i < dsRpt.Tables[i].Rows.Count; i++)
        {
            //Populate the first Question Data
            tempProjectNameData = dsRpt.Tables[i].Rows[i].ItemArray[2].ToString();
            tempNumberData = dsRpt.Tables[i].Rows[i].ItemArray[3].ToString();

            row = table.NewRow();

            if (tempProjectNameData == dsRpt.Tables[i].Rows[i].ItemArray[2].ToString())
            {

                row["No"] = dsRpt.Tables[i].Rows[i].ItemArray[3].ToString();
                row["Question"] = dsRpt.Tables[i].Rows[i].ItemArray[4].ToString();
                row[dsRpt.Tables[i].Rows[i].ItemArray[1].ToString() + " " +
               dsRpt.Tables[i].Rows[i].ItemArray[2].ToString()] = dsRpt.Tables[i].Rows[i].ItemArray[5].ToString();

                table.Rows.Add(row);

            }

            tempProjectNameData = dsRpt.Tables[i].Rows[i].ItemArray[2].ToString();
            tempNumberData = dsRpt.Tables[i].Rows[i].ItemArray[3].ToString();
        }
I then assign the table object to view object and bind it to the gridview control. It works partially hence I need an assistance.

I have 17 Rows in each table and exactly same data structure.

The problem with the code above is that I get only three rows populated and the columns are populate with missing columns for each row.

A very nifty clean code to loop through this dataset object will do me good.

Thanks in advance.

You are using i twice, once for the table, and then again for each row.

Much better to do this:

foreach(DataTable innerTable in dsRpt.Tables)
{
   foreach(DataRow row in innerTable.Rows)
   {
      DataRow newRow = table.NewRow();
      newRow["No"] = row[3].ToString();
      newRow["Question"] = row[4].ToString();
      newRow[row[1].ToString() + " " + row[2].ToString()] = row[5].ToString();
      table.Rows.Add(newRow);
   }
}

Also, you assign tempProjectNameData to the second item in the row, then test to see if it equals that same value, which will always be true. You never use tempNumberData.

Then at the end you assign the variables again, for no reason.

/- Sam Judson : Wrox Technical Editor -/