Efficient way to join strings with a separator?

Question

foreach (SessionTeacher sessionTeacher in sessionTeachers)
{
   Person person = EMS.PWDAL.DALHelper.GetPersonByID(sessionTeacher.PersonID);
   ltrlSpeakers.Text += person.Name1 + ", ";
}

here ltrlSpeakers.Text will have something like this "abc, def, " that is extra comma and space at the end which I have to remove.

// removing extra comma and space

ltrlSpeakers.Text = ltrlSpeakers.Text.ToString().Remove(ltrlSpeakers.Text.ToString().Length - 2, 2);

it's working fine but is there a better more efficient simple way?

Is this really one of the least efficient parts of your application? I would be more concerned with the fact that you are using [magic numbers](http://stackoverflow.com/q/47882/299327). — Ryan Gates, Mar 01 '13 at 12:46

score 4 · Accepted Answer · answered Mar 01 '13 at 12:38

4

Something like this:

var personNames = sessionTeachers.Select(
      st => EMS.PWDAL.DALHelper.GetPersonByID(st.PersonID).Name1);
ltrlSpeakers.Text = string.Join(", ", personNames);

answered Mar 01 '13 at 12:38

Andrei

55,890
9
87
108

Stefano Altieri · Answer 2 · 2013-03-01T12:45:47.197

3

Try Using LINQ:

string result = string.Join(",",
(from t in sessionTeachers select EMS.PWDAL.DALHelper.GetPersonByID(t.PersonID).Name1))

edited Mar 01 '13 at 12:45

answered Mar 01 '13 at 12:39

Stefano Altieri

4,550
1
24
41

I believe the call to ToArray() is not required. – Trevor Tubbs Mar 01 '13 at 12:42
Call to `ToArray` is required for all .NET versions before 4, so this call might be necessary d(http://stackoverflow.com/questions/10092815/why-is-toarray-used-when-using-string-join-with-a-liststring). – Ivan Golović Mar 01 '13 at 13:04

score 1 · Answer 3 · answered Mar 01 '13 at 12:37

1

Use the built-in String.Join method instead.

ltrlSpeakers.Text = string.Join(
    ", ", 
    sessionTeachers.Select(z => z.EMS.PWDAL.DALHelper.GetPersonByID(z.PersonID).Name1);

answered Mar 01 '13 at 12:37

ken2k

48,145
10
116
176

score 1 · Answer 4 · answered Mar 01 '13 at 12:40

1

Try it like this:

        ltrlSpeakers.Text = 
        String.Join(", ",
            sessionTeachers
                .Select(teacher => EMS.PWDAL.DALHelper.GetPersonByID(teacher.PersonID).Name1)
                .ToArray()
        );

answered Mar 01 '13 at 12:40

Ivan Golović

8,732
3
25
31

score -1 · Answer 5 · edited Jul 11 '23 at 06:37

-1

best way is

ltrlSpeakers.Text = 
        String.Join(", ",
            sessionTeachers
                .Select(teacher => EMS.PWDAL.DALHelper.GetPersonByID(teacher.PersonID).Name1)
                .ToArray()
        );
ltrlSpeakers.Text = ltrlSpeakers.Text.trimend(',')

edited Jul 11 '23 at 06:37

Cœur

37,241
25
195
267

answered Mar 01 '13 at 13:02

kaushik

70
5

Efficient way to join strings with a separator?

5 Answers5