在 ASP.Net / C# 中优化循环

Question

我写了一个循环来逐行显示字符串列表中的每一行。 问题是该列表包含超过 45,000 行，并且需要花费大量时间来创建用于显示的页面。

有人可以帮忙优化代码吗！

        List<string> OverrrideLog = lc.getOverrideLog();
        List<string> AccessLog = lc.getAccessLog();

        foreach (string s in OverrrideLog)
            lblOverrideLog.Text += s + "<br/>";

        foreach (string s in AccessLog)
            lblAccessLog.Text += s + "<br/>";

这里 lblOverrideLog 和 lblAccessLog 是文字，每个列表有超过 22,000 行。

Answer 1

您可以使用String.Join 方法（字符串，IEnumerable）：

List<string> OverrrideLog = lc.getOverrideLog();
List<string> AccessLog = lc.getAccessLog();

lblOverrideLog.Text = String.Join("<br />", OverrrideLog);
lblAccessLog.Text = String.Join("<br />", AccessLog);

（另见String.Join vs. StringBuilder: which is faster?）

Answer 2

未经测试，但试试这个：

List<string> OverrrideLog = lc.getOverrideLog();
List<string> AccessLog = lc.getAccessLog();

StringBuilder sb = new StringBuilder();
StringBuilder sb2 = new StringBuilder();

foreach(var el in OverrrideLog)
{
  sb.Append(el);
  sb.Append(" <br />");
}

foreach(var el in AccessLog)
{
  sb2.Append(el);
  sb2.Append(" <br />");
}

lblOverrideLog.Text = sb.ToString();
lblAccessLog.Text = sb2.ToString();

编辑：

哎呀，用val代替var

Answer 3

举个例子

            OverrrideLog.ForEach(x=>{
                if (sbOverrideLog.Length > 0)
                    sbOverrideLog.Append("<br />");

                sbOverrideLog.Append(x);
            });

            AccessLog.ForEach(x =>
            {
                if (sbAccessLog.Length > 0)
                    sbAccessLog.Append("<br />");

                sbAccessLog.Append(x);
            });

Answer 4

您应该使用StringBuilder。

像您这样连接字符串会在内存中创建数千个临时字符串。