c# – Resharper猛拉自己的尾巴;这是第一次还是最后一次？

我有一些冗长的代码：

private bool AnyUnselectedComBox()
{
    bool anyUnselected = false;
    foreach (Control c in this.Controls)
    {
        if (c is ComboBox)
        {
            if ((c as ComboBox).Selectedindex == -1)
            {
                anyUnselected = true;
                break;
            }
        } 
    }
    return anyUnselected;
}

…… Resharper提供了一个像这样的LINQ表达式的elegantize：

return this.Controls.OfType<ComboBox>().Any(c => (c as ComboBox).Selectedindex == -1);

…但随后的Resharper检查说明了它生成的代码(上图)：“类型转换是多余的”(指“c作为ComboBox”部分),因此它最终为：

return this.Controls.OfType<ComboBox>().Any(c => c.Selectedindex == -1);

Resharper不应该生成Resharper批准的代码吗？或者它有时需要两次通过才能完全“束紧腰部”？

解决方法

是的,有时候ReSharper会自行纠正,需要第二次通过以使其“恰到好处”.我一直认为它使用某些“安全模板”来进行转换,在某些情况下,安全转换的某些部分并不是真正需要的.

所有版本的代码都是正确且等效的,但第一个“pass”转换为Linq,第二个“pass”删除了Linq转换添加的一些冗余代码.

原文地址：https://www.jb51.cc/csharp/92253.html