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表达式来表达：

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

...但随后的Resharper检查说明了它生成的代码（上图）：“Type cast is redundant”（指“c as ComboBox”部分），以便结束up as：

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

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

Answer 1

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

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

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

1 个答案: