c# - これは効率的ですか？

Question

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

ここltrlSpeakers.Textには、この「abc、def」のようなものがあります。これは、最後に余分なコンマとスペースを削除する必要があります。

// removing extra comma and space

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

それはうまく機能していますが、より効率的な簡単な方法はありますか？

score 3 · Accepted Answer

このようなもの：

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

score 3 · Accepted Answer

LINQを使用してみてください：

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

score 1 · Accepted Answer

代わりに、組み込みのString.Joinメソッドを使用してください。

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

score 1 · Accepted Answer

このようにしてみてください：

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

score 0 · Accepted Answer

最善の方法は、ltrlSpeakers.Text = String.Join（ "、"、sessionTeachers .Select（teacher => EMS.PWDAL.DALHelper.GetPersonByID（teacher.PersonID）.Name1）.ToArray（））;です。ltrlSpeakers.Text = ltrlSpeakers.Text.trimend（'、'）

c# - これは効率的ですか？

5 に答える 5

Related

Reference