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

Question

我有一些冗长的代码:

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批准的代码吗？或者它有时需要两次通过才能完全"束紧腰部"？

Answer 1

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

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