Ist C# SortedList <TKey, TValue> .Keys.Contains-Methode, die eine Null-Ausnahme wirft, eine gute Sache?

Question

Betrachten Sie dieses Stück Code (auch 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); 
    } 
} 

Problem

Der Ausgang ist

Falsch

Argument

Nach d Igling durch, wie funktioniert die SortedList<TKey, TValue> funktioniert Ich fand, dass dieser Aufruf entspricht Aufruf SortedList<TKey, TValue>.ContainsKey(TKey key) Methode, die eine ArgumentNullException dokumentiert hat.

Auf der anderen Seite, die Dokumentation für die ICollection<T>.Contains Methode heißt es:

Returns: true, wenn das Element in den ICollection<T> gefunden wird; andernfalls false.

Nun, null ist kein Teil der zweiten Sammlung in meinem Beispiel, so erwarte ich false zurückgegeben werden. Eine ArgumentNullException ist eine Verletzung der Schnittstelle und somit ein Fehler.

Was ist eine Problemumgehung für das in Produktionscode? Die Typüberprüfung ist sehr hässlich, da die .Keys -Eigenschaft eine Instanz einer privaten generischen geschachtelten Klasse zurückgibt, und ich habe keine anderen Ideen. Plus, Typprüfung ist ein wirklich böser Code-Geruch.

Answer 1

Die SortedList Klasse implementiert IDictionary, die die Key, value Natur von SortedList angibt. Aufgrund der Art und Weise, wie IDictionary Implementationen funktionieren sollen (unter Verwendung von Hashcodes, um die Schlüssel zu sortieren oder zu indizieren), können Sie kein null Objekt als Schlüssel eingeben, weil null keinen gültigen Hashcode hat. Aus diesem Grund wirft die Contains(...) Methode einen ArguementInvalidException, wenn versucht wird, null in den Parameter zu übergeben, da er weiß, dass null kein gültiger Schlüssel ist.

Wenn Sie dies für eine Produktionsumgebung "beheben" möchten (was ich nicht empfehlen würde, da dies nicht wirklich ein Problem sein sollte), würde ich empfehlen, die SortedList Klasse zu erweitern und die Contains(...) Methode zu umgehen, sicher zurückzukehren wenn die null Argument übergeben.

Answer 2

Ich habe diese Zeile auf MSDN gefunden: Typ: T Das Objekt in der Sammlung zu suchen. Der Wert kann für Referenztypen null sein.

EDIT: Sie müssen diese Anweisung using für System.Linq

System.Linq verwendet wird;

Ich denke durch Verwendung von Object als mein Parameter anstelle von String, der unveränderlich ist Mein Parameter wird wirklich als Referenztyp erkannt. In jedem Fall wurde diese Funktion getestet und funktioniert. Viel Glück.