Ist diese if-Anweisung redundant oder nicht?

Question

Wenn ich String.Join Methode Implementierung anschaute, sah ich eine for-Schleife wie folgt aus:

public static string Join(string separator, params object[] values) 
{ 
    ... 
    for (int index = 1; index < values.Length; ++index) 
    { 
    sb.Append(separator); 
    if (values[index] != null) // first if statement 
    { 
     string str2 = values[index].ToString(); 
     if (str2 != null) // second if statement 
     sb.Append(str2); 
    } 
    } 
    ... 
} 

Hier zweite if-Anweisung überflüssig scheint me.I dachte, wenn values[index] != nullwahr dann ist, wie values[index].ToString() == null könnte wahr? Soviel ich weiß ToString muss immer etwas zurückgegeben werden, oder? Selbst wenn der Typ die Methode ToString nicht überschreibt, sollte er vollständig qualifizierten Namen (Namespace + Klassenname) zurückgeben. Wenn ich es in .NET Framework-Quellcode sehe, dachte ich, dass es vielleicht einen Grund gibt und ich vermisse etwas. Wenn es einen Grund gibt, was ist es?

Answer 1

Technisch ist es nicht überflüssig, da es möglich ist, dass eine ToString Implementierung des Objekts null zurückgibt. Natürlich ist das nicht wirklich nützlich und benutzerdefinierte Typen sollten das nicht tun.

Praktisch in Ihrem Fall jedoch ist die Überprüfung redundant, weil StringBuilder in Ordnung ist, wenn das Argument Appendnull ist. Es wird dann hängen Sie einfach nichts:

StringBuilder sb = new StringBuilder("A"); 
sb.Append((string) null); 
sb.Append("B"); 

Console.WriteLine(sb.ToString() == "AB"); // true 

Answer 2

Ich nehme an, Sie könnten ToString überschreiben und Null zurückgeben.

public override string ToString() 
{ 
    return null; 
}