KeyedCollection<TKey, TItem>.Contains(TKey) 中不必要的空值检查

Question

刚刚在 KeyedCollection.Contains(TKey) 中发现了一个不必要的空检查。

感谢这只是一个很小的优化，但不应该认为这种低效率会被自动代码分析工具发现吗？

这是反射器生成的C#：

public bool Contains(TKey key)
{
    if (key == null)
    {
        ThrowHelper.ThrowArgumentNullException(ExceptionArgument.key);
    }
    if (this.dict != null)
    {
        return this.dict.ContainsKey(key);
    }
    if (key != null) // Not needed as key cannot be null
    {
        foreach (TItem local in base.Items)
        {
            if (this.comparer.Equals(this.GetKeyForItem(local), key))
            {
                return true;
            }
        }
    }
    return false;
}

另外，发送补丁的最佳方式是什么？ ;-) 通过.net forums 或？

Answer 1

无论如何，这可能会被 JIT 优化，所以你真的不需要担心它。无论如何，空检查的成本接近于零。

要报告错误，您可以使用 Microsoft Connect 网站。但我不认为他们会解决它...