Unnötige Null-Überprüfung in KeyedCollection <TKey, TItem> .Contains (TKey)

Question

Nur eine unnötige Null-Überprüfung in KeyedCollection.Contains (TKey) gefunden.

Schätzen Sie, es ist nur eine sehr kleine Optimierung, aber sollte nicht gedacht, dass diese Art von Ineffizienz von einem automatisierten Code-Analyse-Tool aufgenommen werden?

Hier ist der C# durch den Reflektor erzeugt:

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; 
} 

Auch, was ist der beste Weg, einen Patch zu senden? ;-) Durch die .net forums oder?

Answer 1

Dies wird wahrscheinlich sowieso vom JIT optimiert, so dass Sie sich nicht wirklich darum kümmern müssen. Wie auch immer, die Kosten eines Null-Checks sind nahe Null.

Um einen Fehler zu melden, können Sie die Microsoft Connect-Website verwenden. Aber ich glaube nicht, dass sie es beheben werden ...