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表并将表作为字符串返回。

使用多线程或async-await模式可以改进此代码吗？有问题的代码是我将行添加到StringBuilder对象的位置。

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

Answer 1

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

现在需要多长时间？
这需要花费时间吗？

如果有显着的好处（证明有大量的努力）那么肯定！然而，我强烈怀疑答案是“不”，在这种情况下，请保持孤立。在单个操作上处理多个线程很复杂。

您可以将单独的文档部分视为并行任务，但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); }
);

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

3 个答案: