编译器能否在字符串上优化ToString()？

Question

我相信每个人都会遇到喜欢这种ToString()方法的开发者.我们都可能看到类似以下的代码:

public static bool CompareIfAnyMatchesOrHasEmpty(List<string> list1, List<string> list2)
{
    bool result = false;
    foreach (string item1 in list1)
    {
        foreach (string item2 in list2)
        {
            if (item1.ToString() == item2.ToString())
            {
                result = true;
            }
            if (item1.ToString() == "")
            {
                result = true;
            }
        }
    }
    return result;
}

我想知道的是ToString(),编译器是否可以优化方法(空的,没有格式化的)？我的假设是,它不是,因为它最初定义object.因此,我提出第二个问题,是否有必要清理此类实例？

Answer 1

C#编译器不会优化它.但是,在运行时,我相信这可能会被CLR中的JIT编译器内联,因为string.ToString()它只返回自身.

String.ToString甚至是声明的TargetedPatchingOptOutAttribute,当它从其他组件调用时,允许它由NGEN内联,所以它显然是一个内联目标.

Answer 2

它当然可以被编译器优化掉,但它们可能不会因为它是微不足道的.在决定是否值得进行任何优化之前,请先尝试一些测试.我们来试试吧!

List<string> strings = Enumerable.Range(1, 10000000).Select(x => Guid.NewGuid().ToString()).ToList();
var sw= Stopwatch.StartNew();

foreach (var str in strings) {
    if (!str.ToString().Equals(str.ToString())) {
        throw new ApplicationException("The world is ending");
    }
}

sw.Stop();
Console.WriteLine("Took: " + sw.Elapsed.TotalMilliseconds);

sw = Stopwatch.StartNew();
foreach (var str in strings) {
    if (!str.Equals(str)) {
        throw new ApplicationException("The world is ending");
    }
}
sw.Stop();
Console.WriteLine("Took: " + sw.Elapsed.TotalMilliseconds);

好的,所以我们处于一个包含1000万件物品的循环中.与非tostring版本相比,tostring(称为两次)版本需要多长时间？

这是我在我的机器上得到的:

Took: 261.6189 
Took: 231.2615

是的.我在1000万次迭代中节省了30整毫秒.所以...是的,我会说不,不值得.完全没有.

现在,代码是否应该更改,因为它是愚蠢的？是.我会把这个论点说成:"这是不必要的,让我一眼就认为这不是一个字符串.它需要我的脑循环来处理,并且完全没有目的.不要这样做." 不要从优化的角度来争论.