0

I have a working code that exports SQL Server data to Excel.

public static void ExportToExcel(DataTable dt)
{
        // Create sql connection string
        string conString = "Data Source=DELL\\SQLSERVER1;Trusted_Connection=True;DATABASE=Test;CONNECTION RESET=FALSE";
        SqlConnection sqlCon = new SqlConnection(conString);
        sqlCon.Open();

        SqlDataAdapter da = new SqlDataAdapter("select * from tStudent", sqlCon);
        System.Data.DataTable dtMainSQLData = new System.Data.DataTable();
        da.Fill(dtMainSQLData);
        DataColumnCollection dcCollection = dtMainSQLData.Columns;

        // Export Data into EXCEL Sheet
        Microsoft.Office.Interop.Excel.ApplicationClass ExcelApp = new Microsoft.Office.Interop.Excel.ApplicationClass();
        ExcelApp.Application.Workbooks.Add(Type.Missing);

        //ExcelApp.Cells.CopyFromRecordset(objRS);
        for (int i = 1; i < dtMainSQLData.Rows.Count + 1; i++)
        {
            for (int j = 1; j < dtMainSQLData.Columns.Count + 1; j++)
            {
                if (i == 1)
                    ExcelApp.Cells[i, j] = dcCollection[j - 1].ToString();
                else
                    ExcelApp.Cells[i, j] = dtMainSQLData.Rows[i - 1][j - 1].ToString();
            }
        }
        ExcelApp.ActiveWorkbook.SaveCopyAs("C:/Users/Administrator.CAMO/Downloads/FtpFilesStorage/Sheet1.xlsx");
        ExcelApp.ActiveWorkbook.Saved = true;
        ExcelApp.Quit();

        Console.WriteLine(".xlsx file Exported succssessfully.");
}

And I am using this method in Main() like this: 

System.Data.DataTable dtMainSQLData = new System.Data.DataTable();
ExportToExcel(dtMainSQLData);

This works fine but the first row is not copied or exported from the original table in SQL Server.

marc_s
  • 732,580
  • 175
  • 1,330
  • 1,459
user2525244
  • 91
  • 1
  • 3
  • 12
  • 1
    Your for is wrong it should be `for (int i = 0; i < dtMainSQLData.Rows.Count; i++` and you'll have to off set it when you write e.g. `ExcelApp.Cells[i+1, j] ` – Conrad Frix Apr 01 '14 at 18:18
  • thanks Conrad Frix... It did work. But now I am not able to export the last row. I changed my code to following after your suggestion to this: for (int i = 0; i < dtMainSQLData.Rows.Count; i++) { for (int j = 1; j < dtMainSQLData.Columns.Count + 1; j++) { if (i == 0) ExcelApp.Cells[i + 1, j] = dcCollection[j - 1].ToString(); else ExcelApp.Cells[i + 1, j] = dtMainSQLData.Rows[i - 1][j - 1].ToString(); } } Plz tell wht mstak i'm makin. – user2525244 Apr 01 '14 at 19:13

2 Answers2

2

You're experiencing one-off errors because of the fact that ExcelApp.Cells isn't zero based and you have a header row.

To combat this you can use foreach and separate out your header export to a separate block. It does take a little bit more code but I find it easier to deal with and less error prone

int i = 1;

int j = 1;   
//header row
foreach(DataColumn col in dtMainSQLData.Columns)
{  
  ExcelApp.Cells[i, j]  = col.ColumnName;
  j++;
}

i++;
//data rows
foreach(DataRow row  in dtMainSQLData.Rows)
{
    for (int k = 1; k < dtMainSQLData.Columns.Count + 1; k++)
    {
         ExcelApp.Cells[i, k]  = row[k-1].ToString();
    }
    i++;
}
Conrad Frix
  • 51,984
  • 12
  • 96
  • 155
  • Using following I get an error saying "object does not contain a definition for column name and no extension for 'ColumnName' " Cannot apply indexing with [] to an expression of type `object' at row[k-1] Regards. – user2525244 Apr 01 '14 at 19:49
  • I changed it to use the actual data types instead of `var` although I don't see why that's needed – Conrad Frix Apr 01 '14 at 19:53
  • Hi Conrad Frix. Can you help me with this one. http://stackoverflow.com/questions/22797917/import-excel-xslx-file-into-sql-server-difficulty-in-updating-table Thanks. – user2525244 Apr 01 '14 at 22:13
0

Your code is fine, except you just need to remove your "else" block and increment the starting row of your Excel spreadsheet:

for (int i = 1; i < dtMainSQLData.Rows.Count + 1; i++)
{
    for (int j = 1; j < dtMainSQLData.Columns.Count + 1; j++)
    {
        if (i == 1)
            ExcelApp.Cells[i, j] = dcCollection[j - 1].ToString();

        ExcelApp.Cells[i + 1, j] = dtMainSQLData.Rows[i - 1][j - 1].ToString();
    }
}
gorf13
  • 1