c# - 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 表达式，如下所示：

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

...但是随后的 Resharper 检查说明了它生成的代码（上图）：“类型转换是多余的”（指的是“c as ComboBox”部分），因此它最终成为：

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

Resharper 不应该生成 Resharper 认可的代码吗？或者它只是有时需要两次传球才能完全“束腰”？

Answer 1

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

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