冗余对象.ToString

Question

我正在查看以下代码。

private string MakeKey(int SnapshotID, string UserName, string BrandName)
{
    return string.Format("{0}:{1}:{2}", SnapshotID.ToString(), UserName, BrandName);
}

SnapshotID.ToString()冗余调用ToString()方法。
可能发生的潜在问题是什么？ 例如速度。

编辑
如果有人想改进，我已经添加了一个测试：http: //pastebin.com/gynNjwT1

With ToString:      0.010884
Without ToString:   0.001446

With ToString:      0.005506
Without ToString:   0.002852

With ToString:      0.001155
Without ToString:   0.009117

With ToString:      0.003210
Without ToString:   0.001546

Answer 1

在这段代码中......不，它只是多余的，甚至有点慢，因为String.Format()将在string本身上再次调用ToString() 。

如果String.Format()与IFormatProvider一起使用，则显式ToString()将覆盖它。

编辑（来自评论）：关于性能的注释。 与原始类型（ int ）装箱相比，它可能会或不会更慢（无用的ToString()调用）。 我们应该衡量这个，但很难说。 什么更慢？ 虚函数调用还是拳击？ 与String.Format()总时间相比，这真的很重要吗？ 已经在这里回答了 SO 。 IMO，如果它是作为一种智能优化完成的，那它就毫无用处了。

表现

OP为此做了一个小测试（见评论），只是为了好玩，我的测试机器上的结果：

Test               Time [ms]
With ToString      0.002929
Without ToString:  0.003414

我重写了不使用线程的测试（因为它会添加更多变量，内核可能会有所不同，操作系统会动态加载它们）。 使用这个（非常简单！）测试代码（源自 OP 测试）：

Stopwatch sw = new Stopwatch();
sw.Start();
for (int x = 0; x < Count; x++)
{
    _temp = MakeKeyToString(1, "Ashley", "MyBrand");
}
sw.Stop();
TimeSpan test = TimeSpan.FromMilliseconds((double)sw.ElapsedMilliseconds);

我知道ToString()版本更快（对于Count = 1000000 ）平均大约 10/20 纳秒。 IMO 要测量这么小的东西，我们需要更好的测试环境和更专业的方法。 将他的String.Format()代码更改为使用IFormatProvider ：

string MakeKeyToString(int SnapshotID, string UserName, string BrandName)
{
    return string.Format(CultureInfo.InvariantCulture,
        "{0}:{1}:{2}", SnapshotID.ToString(), UserName, BrandName);
}

再次更改所有内容：没有ToString()会更快 200 纳秒。 同样太小而无法以这种方式测量。

结论

你甚至不能开始考虑这是一种优化，太多的因素会围绕它发挥作用，而且与String.Format()的总时间相比，它是如此之小以至于不会被注意到。 更糟糕的是，如果您将String.Format()更改为使用IFormatProvider ，它可能会引入细微的错误，因为它会让您停下来思考“为什么会这样？应该有与文化相关的原因。也许......”现实......（可能）没有。

Answer 2

在此示例中，通过调用Int32.ToString()您可以避免将int装箱到Object中的开销。

然而，调用.ToString()的开销，然后让String.Format调用.ToString()的开销可能超过装箱的开销。 就String.Format而言，所有这些都可以忽略不计。

Answer 3

不，代码是多余的并停止。 SnapshotID也不能为空，因为它是整数而不是可为空的整数 ( int? )。 在这种情况下，我认为ToString方法将被调用两次（一次显式调用，一次调用第一次调用的结果），因此可能会面临一些性能差异。