From DataGridView to multiline commaSeparated string

Question

I have a DataGridView with four Columns and need to crate a multiline string from its content, separated by comma.
This code works, but probably - there is a more elegant way:

string multiLine = "";
string singleLine;    
foreach (DataGridViewRow r in dgvSm.Rows)
{
    if (!r.IsNewRow)
    {
        singleLine = r.Cells[0].Value.ToString() + ","
        + r.Cells[1].Value.ToString() + ","
        + r.Cells[2].Value.ToString() + ","
        + r.Cells[3].Value.ToString() + Environment.NewLine;
        multiLine = multiLine + singleLine;
    }
}

peenut · Accepted Answer · 2012-06-24T09:57:50.770

I don't know about elegant, but:

use StringBuilder for string manipulation, type string is immutable!
if you need to do something in between, separate first or last cycle running (e.g. comma separation) So, basically something like this:

StringBuilder multiLine = new StringBuilder();
foreach (DataGridViewRow r in dgvSm.Rows)
{
 if (!r.IsNewRow)
 {
  if (r.Cells.Count > 0)
  {
   multiLine.Append(r.Cells[0].Value.ToString()); //first separated
   for (int i = 1; i < r.Cells.Count; ++i)
   {
    singleLine.Append(','); //between values
    singleLine.Append(r.Cells[i].Value.ToString());
   }
   multiLine.AppendLine();
  }
 }
}

To illustrate speed difference between StringBuilder concatenation (just dynamic array of characters) and string (new object and copy everything each time you use operator + concatenation), have a look at mini-program:

public static void Main()
{
    var sw = new Stopwatch();
    sw.Start();
    StringBuilder s = new StringBuilder();
    //string s = "";
    int i;
    for (i = 0; sw.ElapsedMilliseconds < 1000; ++i)
        //s += i.ToString();
        s.Append(i.ToString());
    sw.Stop();
    Console.WriteLine("using version with type " + s.GetType().Name + " I did " +
        i + " times of string concatenation.");
}

For my computer it is:

using version with type String I did 17682 times of string concatenation.
using version with type StringBuilder I did 366367 times of string concatenation.

peenut, maybe your code takes more processing time than the first one (if a dgv has 1000 rows, for example. What do you think? — Buena, Jun 24 '12 at 09:46
Buena, I added code to compare string and StringBuilder speed — peenut, Jun 24 '12 at 09:58

score 0 · Answer 2 · answered Jun 24 '12 at 09:39

0

Try this :

    string multiLine = "";
string singleLine;    
foreach (DataGridViewRow r in dgvSm.Rows)
{
if (!r.IsNewRow)
{
singleLine = r.Cells[0].Value.ToString() + ","
+ r.Cells[1].Value.ToString() + ","
+ r.Cells[2].Value.ToString() + ","
+ r.Cells[3].Value.ToString() + "\r\n";
multiLine = multiLine + singleLine;
}
}

answered Jun 24 '12 at 09:39

Ali Vojdanian

2,067
2
31
47

aliboy, is there another difference between "Environment.NewLine" and "\r\n" except the number of characters? – Buena Jun 24 '12 at 09:50
http://stackoverflow.com/questions/1015766/difference-between-n-and-environment-newline – Elisabeth Jun 24 '12 at 10:16
ThankYou, Elisa. Useful for me. – Buena Jun 24 '12 at 10:31

From DataGridView to multiline commaSeparated string

2 Answers2