Soll ich ArgumentNullException werfen, wenn eine Zeichenfolge leer ist?

Question

Ich arbeite an einer Methode, die bei einem String-Parameter etwas tut. Ein gültiger Wert für den Zeichenfolgenparameter ist ein anderer Wert als null oder string.Empty. Also sieht mein Code so aus.

 
private void SomeMethod(string someArgument) 
{ 
    if(string.IsNullOrEmpty(someArgument)) 
     throw new ArgumentNullException("someArgument"); 

    // do some work 
} 

Nichts zu aufregend dort. Meine Frage ist, ist es in Ordnung, eine ArgumentNullException zu werfen, auch wenn die Zeichenfolge gleich string.Empty ist? Weil es technisch nicht null ist. Wenn Sie der Meinung sind, dass ArgumentNullException nicht ausgelöst werden soll, welche Ausnahme sollte ausgelöst werden?

Answer 1

ArgumentException sollte für den String.Empty Fall geworfen werden. Dies würde auf ein anderes Problem hinweisen, als dass es null ist. Um eine NullReferenceException zu vermeiden, überprüfe ich zunächst auf Null, dann trimme ich und überprüfe den leeren Fall, um zu verhindern, dass irgendwelche Leerzeichen passieren.

private void SomeMethod(string someArgument) 
{ 
    if(someArgument == null) 
     throw new ArgumentNullException("someArgument"); 

    if (someArgument.Trim() == String.Empty) 
     throw new ArgumentException("Input cannot be empty", "someArgument"); 

    // do some work 
} 

Ab .NET 4.0 können Sie die String.IsNullOrWhiteSpace Methode verwenden, um diese Kontrollen in einem Rutsch durchzuführen. Auf diese Weise verzichten Sie auf die Möglichkeit, einen granularen Ausnahmetyp anzugeben. Daher würde ich mich für die ArgumentException entscheiden und die Nachricht entsprechend aktualisieren.

Answer 2

Sie sollten eine ArgumentException werfen, wenn eine leere Zeichenfolge keine akzeptierte Eingabe für Ihre Methode ist. Es kann für Kunden sehr verwirrend sein, wenn Sie eine ArgumentNullException werfen, während sie kein null Argument zur Verfügung stellten.

Es ist einfach ein anderer Anwendungsfall. Möglicherweise verfügen Sie auch über Methoden, die keine Nulleingabewerte akzeptieren, aber leere Zeichenfolgen akzeptieren. Es ist wichtig, dass Sie in Ihrer gesamten Anwendung konsistent sind.

Answer 3

In .NET Framework wird manchmal die ArgumentNullException für den String.IsNullOrEmpty-Fall verwendet - ein Beispiel ist System.Windows.Forms.Clipboard.SetText.

Also ich denke, es ist sinnvoll, das gleiche in Ihrem Code zu tun, es sei denn, es gibt einen echten Wert bei der Unterscheidung der beiden Fälle.

Beachten Sie, dass diese und andere von ArgumentException abgeleitete Ausnahmen im Allgemeinen einen Programmierfehler anzeigen und daher Informationen bereitstellen müssen, die einem Entwickler bei der Diagnose des Problems helfen. Ich persönlich denke, es ist unwahrscheinlich, dass ein Entwickler es verwirrend finden würde, wenn Sie ArgumentNullException für ein leeres String-Argument verwenden, besonders wenn Sie dieses Verhalten wie im folgenden Beispiel dokumentieren.

/// <summary> 
/// ... description of method ... 
/// </summary> 
/// <param name="someArgument">... description ...</param> 
/// <exception cref="ArgumentNullException">someArgument is a null reference or Empty.</exception> 
public void SomeMethod(string someArgument) 
{ 
    ... 
} 

Answer 4

Wenn man all die Dinge zu berücksichtigen, dass (Joe/Ahmad Mageed) hat gesagt, würde ich dann eine Ausnahme für diesen Fall erstellen.

class ArgumentNullOrEmptyException : ArgumentNullException 

Answer 5

Dies hängt von den Umständen wirklich ab.

Die Frage kommt auf, ist es wirklich ein Fehler? Damit meine ich, Sie immer einen Wert erwarten? Wenn Sie das tun, dann wahrscheinlich die hier beste Wette Ihre eigenen Exception schafft, vielleicht etwa so:

class StringEmptyOrNullException : Exception 
{ 
} 

Wo Sie können auch Ihre eigenen Konstrukteuren hinzufügen und hinzugefügt Informationen etc.

Wenn es jedoch nicht ist "Ausnahme" passiert in Ihrem Programm, wenn es wahrscheinlich eine bessere Idee wäre, null von der Methode zurückzugeben und von dort aus zu behandeln.Denken Sie daran, Exception sind für außergewöhnliche Bedingungen.

hoffe, das hilft,

Kyle

Answer 6

Warum diesen Code nicht verwenden?

private void SomeMethod(string someArgument) 
{ 
//chek only NULL 
if(ReferenceEquals(someArgument,null)) 
    throw new ArgumentNullException("someArgument"); 

// and after trim and check 
if (someArgument.Trim() == String.Empty) 
    throw new ArgumentException("Input cannot be empty", "someArgument"); 

// do some work 
}