C# SortedList<TKey, TValue>.Keys.Contains 方法抛出空异常是件好事吗？

Question

考虑这段代码（也可以使用in Ideone）：

using System;
using System.Collections.Generic;

class Program {
    static void Main() {
        Console.WriteLine(HasItem(new List<string>(), null));
        try {
            Console.WriteLine(HasItem(new SortedList<string, int>().Keys, null));
        } catch (ArgumentNullException) {
            Console.WriteLine("ArgumentNullException");
        }
    }

    public static bool HasItem(ICollection<string> collection, string item) {
        return collection.Contains(item);
    }
}

问题

输出是

错误

ArgumentNullException

在深入了解SortedList<TKey, TValue> 的工作原理后，我发现此调用等效于调用SortedList<TKey, TValue>.ContainsKey(TKey key) 方法，该方法记录了ArgumentNullException。

另一方面，ICollection<T>.Contains 方法的文档指出：

如果在ICollection<T> 中找到项目，则返回：true；否则，false。

好吧，null 在我的示例中不是第二个集合的一部分，所以我希望返回 false。 ArgumentNullException 违反了接口，因此是一个错误。

生产代码中的解决方法是什么？鉴于.Keys 属性返回私有通用嵌套类的实例，类型检查将非常难看，而我没有任何其他想法。另外，类型检查是一种非常讨厌的代码气味。

Answer 1

SortedList 类实现了IDictionary，它指定了SortedList 的Key, value 性质。由于IDictionary 实现的预期工作方式（使用哈希码对键进行排序或索引），您不能输入null 对象作为键，因为null 没有有效的哈希码。出于这个原因，Contains(...) 方法将在尝试将null 传递到参数时抛出ArguementInvalidException，因为它知道null 不是有效的键。

如果您想为生产环境“修复”此问题（我不建议这样做，因为这应该不是问题），我建议扩展 SortedList 类并覆盖 Contains(...) 方法通过null 争论后安全返回。

Answer 2

我在 MSDN 上找到了这一行：Type: T 要在集合中定位的对象。对于引用类型，该值可以为 null。

编辑：您需要 System.Linq 的 using 语句

使用 System.Linq；

       SortedList<string, int> temp = new SortedList<string, int>();
        bool test = HasItem(temp.Keys, null);


    public bool HasItem(ICollection<string> collection, object item)
    {
        return collection.Contains(item);
    }//This returns false when fed a null

我认为通过使用 Object 作为我的参数而不是不可变的字符串，我的参数真正被识别为引用类型。无论哪种方式，此功能都已经过测试并且可以正常工作。祝你好运。