这种字符串操作可以吗？

Question

public string Format { get { return string.Format("{0}{1}v{1}", LastManStanding ? "FFA " : string.Empty, m_Teams.PlayersPerTeam); } }

我应该使用 StringBuilder 吗？

我不太确定这样有条件地格式化字符串有多么错误，而不是这样做

public string Format 
{
    get
    {
        StringBuilder sb = new StringBuilder();

        if(LastManStanding)
            sb.Append("FFA ");

        sb.Append(string.Format("{0}v{0}", m_Teams.PlayersPerTeam);
        return sb.ToString();
    }
}

Answer 1

同意大家的意见——对于这么小的字符串来说是可以的。请注意，使用 StringBuilder 重写会更糟糕，因为您使用 String.Format 而不是 StringBuilder.AppendFormat。

在您的情况下，如果您决定开始用其他语言本地化您的程序，那么使用单一格式字符串创建结果字符串是有益的 - 可以从资源中提取格式化字符串，但是很难使用 StringBuilder 编写代码来尊重所有可能的情况语言。

实际上，故事的另一面可能与使用 StringBuilder 或 String.Format 无关。有些人/团队不想使用 ？ : 运算符。了解为什么建议您重写一段代码将有助于确定重写目标。

Answer 2

我会将 FFA 和普通游戏的格式化逻辑分开：

public string Format 
{
    get
    {
        if(LastManStanding)
            return string.Format("FFA {0}v{0}", m_Teams.PlayersPerTeam);
        else
            return string.Format("{0}v{0}", m_Teams.PlayersPerTeam);
    }
}

这样更容易阅读。从逻辑上讲，格式字符串也是分开的。您很可能会单独更改它们，但我很惊讶您没有这样做。 “FFA 2v2”对我来说没有多大意义。

Answer 3

错了，没有。可读吗？嗯，

一点点格式化就能大有帮助

public string Format 
{ 
  get 
  { 
    return string.Format("{0}{1}v{1}", 
      LastManStanding ? "FFA " : string.Empty,
      m_Teams.PlayersPerTeam); 
  } 
}

Answer 4

我认为第一个版本是首选，它清楚地表达了您的意图，并且比第二个冗长的版本更加简洁和易于阅读。我会这样格式化它：

public string Format 
{ 
    get 
    { 
        return string.Format("{0}{1}v{1}", LastManStanding ? "FFA " : string.Empty,
                                           m_Teams.PlayersPerTeam); 
    } 
}

Answer 5

如果您要对长字符串进行编辑，则首选 StringBuilder，在这种情况下您不需要它。

Answer 6

对于这么小的字符串，你所做的一切都很好。

当您处理数千个串联和/或字符串格式时，请使用 StringBuilder。

---

旁注：

代替：

sb.Append(string.Format("{0}v{0}", m_Teams.PlayersPerTeam));

您可以这样做：

sb.AppendFormat("{0}v{0}", m_Teams.PlayersPerTeam);