Welches Ergebnis sollte ich zurückgeben?

Question

Ich schreibe einfache Bibliothek, die Liste der Namen zurückgibt.

Aber, was ich sollte zurückkehrt, wenn ich nichts finden können?

return new List<String>(); 

oder

return null; 

Beispiel:

var resultColl=FindNames(...); 

Dieser Code kann von anderen Komponenten verwendet werden, und ich will nicht, es beschädigt. Wenn ich null zurückgebe - ich denke, es ist der richtige Weg, es zu überprüfen. Aber, kann ich leere Liste zurückgeben?

Vielen Dank.

Answer 1

Sie sollten immer leere Liste zurückgeben. Siehe Guidelines for Collections.

Geben Sie NULL-Werte NICHT von den Auflistungseigenschaften oder von den Methoden returning collections zurück. Geben Sie stattdessen eine leere Sammlung oder ein leeres Array zurück.

Answer 2

Rückkehr leere Auflistung ist besser aus der Perspektive Design, da Client-Code muss nicht null Prüfung durchzuführen. Siehe Null Object pattern.

Answer 3

Ich würde Enumerable.Empty<string>() zurückgeben, oder wenn Ihre Methode eine Liste zurückgeben soll, tun Sie return new List<string>(). Wenn Sie viele Situationen haben, in denen Sie eine leere Liste zurückgeben müssen, könnten Sie eine statische Liste erstellen, die jedes Mal zurückgegeben wird. Dies würde verhindern, dass jedes Mal neue leere Listen erstellt werden müssen, wie von @YuvalItzchakov gezeigt.

Eine leere Sammlung ist besser als null da wäre es, zu saubereren Code meiner Meinung nach führen.

Answer 4

Eine ausführlichere Antwort wäre ein Tuple<bool, List<string>>. Dies ist eine klare Lösung, die modifiziert werden kann, andere Details über die Suche einbezogen:

var thereAreResults = foundList.Count > 0; 
return new Tuple<bool, List<String>>(thereAreResults, foundList); 

Answer 5

Es ist wahrscheinlich, dass der anrufende Code die Liste zu durchlaufen will, oder mit einer Liste etwas zu tun. Durch das Zurückgeben einer leeren Liste sollte der aufrufende Code gut funktionieren. Wenn Sie null zurückgeben, muss der aufrufende Code sicherstellen, dass sie zuerst eine Liste haben.

Es ist wahrscheinlich nur eine Frage der Präferenz, aber eine leere Liste zurückkehr bekommt meine Stimme ... bist du das Ding der Vertrag es zurückkommen würde, sagte der Rückkehr.

Answer 6

Das hängt von der Semantik Ihres Codes ab.

Wenn keine Ergebnisse ein akzeptables Ergebnis ist, dann sollten Sie eine leere Auflistung zurück.

Wenn keine Ergebnisse eine Fehlerbedingung sind, geben Sie null zurück.

Answer 7

Ich würde vorschlagen, dass Sie in Einklang stehen, was Sie tun, wenn Sie nichts haben, zurückzukehren. Auf diese Weise wissen Sie immer, dass Sie für alles, was Sie anrufen, die gleiche Art von Antwort für nichts erwarten. So wie Sie wissen, dass Sie immer eine leere Menge oder eine leere Zeichenfolge oder 0 usw. zurückgeben werden.

Was tun Sie in Bezug auf die anderen Bibliothekselemente, die Sie haben?

Answer 8

die leere Liste zurückkehrend ist bequemer für die Nutzer Ihrer Funktion:

foreach (string name in FindNames(...)) 
{ 
    Display(name); 
} 

null Kräfte Ihre Anrufer zurückkehrend zusätzlichen Code zu schreiben:

• einen Test für null und

• eine zusätzliche lokale Variable (um zu vermeiden, dass Sie Ihre Funktion zweimal aufrufen müssen)

  List<string> names = FindNames(...); 
  
  if (names != null) 
  { 
      foreach (string name in names) 
      { 
       Display(name); 
      } 
  } 
  

Also eine leere Liste zurückgeben ist besser.