c# - 为什么.Net中Exception类的ToString()方法不使用StringBuilder来构建字符串？

Question

关闭。这个问题是 opinion-based 。目前不接受答案。

---

想要改进这个问题吗？更新问题，以便 editing this post 可以用事实和引文来回答它。 .

已关闭 7 年前。

Improve this question

我希望下面的代码(来自 .Net 框架中的 Exception 类)使用 StringBuilder 构建字符串，然后返回 builder.ToString() 而不是在现有字符串上使用“+”运算符，这最终每次都会创建一个新字符串。

我希望编写 .Net 源代码的人尽可能遵循最佳实践，因此我想仔细检查这是否仍然是最佳的

private String ToString(bool needFileLineInfo, bool needMessage) {
    String message = (needMessage ? Message : null);
    String s;
    if (message == null || message.Length <= 0) {
        s = GetClassName();
    }
    else {
        s = GetClassName() + ": " + message;
    }

    if (_innerException!=null) {
        s = s + " ---> " + _innerException.ToString(needFileLineInfo, needMessage) 
              + Environment.NewLine 
              + "   "
              + Environment.GetResourceString("Exception_EndOfInnerExceptionStack");

    }

    string stackTrace = GetStackTrace(needFileLineInfo);
    if (stackTrace != null)
    {
        s += Environment.NewLine + stackTrace;
    }

    return s;
}

Answer 1

如果不询问代码作者，就不可能确切地知道代码作者为什么选择此实现。

也就是说，值得记住的是，优先使用 StringBuilder 而不是简单的 string 连接的一般指南主要适用于具有任意大迭代计数的循环场景。

在上面的示例中，该方法的主要部分应该最终调用 Concat() 方法，该方法应该可以更好地创建单个新字符串从输入中获取对象，而不是使用 StringBuilder 迭代地附加每个部分。为其他连接引入 StringBuilder，尤其是当它们并不总是会发生时(因为它们是有条件的)，在常见场景中很可能不太理想。

参见例如String concatenation vs String Builder. Performance ，其中特别指出了这种差异。