可以使用多线程改进此代码吗？

Question

我有一个简单的 Windows 服务，它每天只运行一次。它在数据库中执行一些查询，生成适当的 html 内容（表格、div、...）并将其以电子邮件正文的形式发送给多个收件人。

电子邮件的正文是这样创建的：

private static string GenerateBody()
{
    using (var stringWriter = new StringWriter())
    using (var htmlWriter = new HtmlTextWriter(stringWriter))
    {
        htmlWriter.RenderBeginTag("html");
        htmlWriter.RenderBeginTag(HtmlTextWriterTag.Head);
        htmlWriter.WriteLine("<meta http-equiv=\"Content-Type\" content=\"text/html; charset=utf-8\" />");
        htmlWriter.RenderEndTag();
        htmlWriter.RenderBeginTag("body");

        htmlWriter.Write(
            new StringBuilder()
                .Append(OverviewParagraph.GenerateHTMLContent())
                .Append(PackageWeightParagraph.GenerateHTMLContent())
                .Append(BoxWeightParagraph.GenerateHTMLContent())
                .Append(CodeQualityParagraph.GenerateHTMLContent())
                .Append(ChecksParagraph.GenerateHTMLContent())
                .ToString()
        );

        htmlWriter.RenderEndTag();
        htmlWriter.RenderEndTag();

        return stringWriter.ToString();
    }
}

所有GenerateHTMLContent 方法几乎相同——它们在我的数据库中执行查询，在 HTMLTextWriter 的帮助下构建 HTML 表，然后将表作为字符串返回。

能否通过使用多线程或异步等待模式来改进此代码？有问题的代码是我将行附加到 StringBuilder 对象的位置。

编辑：我问这个问题是因为我以前从未使用过多线程，只是想知道它是否可能。此外，程序现在运行速度也够快了。

Answer 1

如果它只生成一件事，则并行化很复杂，因为您需要考虑同步化。当您可以执行 task 并行化（并行完成的单独和隔离操作）时，并行化是更明显的候选者。您也没有提供足够的信息来表明是否需要进行复杂的工作：

• 现在需要多长时间？
• 花费这么多时间是个问题吗？

如果有显着的好处（证明付出巨大努力的理由），那么当然可以！然而，我强烈怀疑答案是否定的，在这种情况下，别管它。在单个操作中处理多个线程很复杂。

您也许可以将单独的文档部分视为并行任务，但 HTML 生成通常很快 - 所以除非您对此进行了分析并且知道它们需要时间，否则不要打扰。更有可能：您的数据查询是障碍。在这种情况下，请花一些时间改进它，而不必担心并行化。

Answer 2

如果GenerateHTMLContent 方法是隔离的（即如果同时运行它们不会相互干扰），您可以将它们全部启动并在它们可用时收集结果：

// start tasks
Task<string> overviewParagraph =
    Task.Factory.StartNew( () => OverviewParagraph.GenerateHTMLContent() );

Task<string> packageWeightParagraph =
    Task.Factory.StartNew( () => PackageWeightParagraph.GenerateHTMLContent() );

....

// collect results
string overviewParagraphHtml = overviewParagraph.Result;
string packageWeightParagraphHtml = packageWeightParagraph.Result;
...

Answer 3

StringBuilder sb = new StringBuilder();
Parallel.Invoke(
    () => { var s = OverviewParagraph.GenerateHTMLContent(); lock (sb) sb.Append(s); },
    () => { var s = PackageWeightParagraph.GenerateHTMLContent(); lock (sb) sb.Append(s); },
    () => { var s = BoxWeightParagraph.GenerateHTMLContent(); lock (sb) sb.Append(s); },
    () => { var s = CodeQualityParagraph.GenerateHTMLContent(); lock (sb) sb.Append(s); },
    () => { var s = CodeQualityParagraph.GenerateHTMLContent(); lock (sb) sb.Append(s); }
);