Dois-je lancer l'exception ArgumentNullException si une chaîne de caractères est vide ?

Question

Je travaille sur une méthode qui fait quelque chose avec un paramètre de type chaîne. Une valeur valide pour le paramètre string est tout autre que null ou string.Empty. Mon code ressemble donc à ceci.

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

    // do some work
}

Rien de bien excitant. Ma question est la suivante : est-il possible de lancer une exception ArgumentNullException même si la chaîne est égale à string.Empty ? Parce que techniquement, elle n'est pas nulle. Si vous pensez que l'exception ArgumentNullException ne devrait pas être levée, quelle exception devrait être levée ?

Answer 1

ArgumentException devrait être lancée pour le String.Empty cas. Cela indiquerait un problème autre que le fait qu'il soit nul. Pour éviter un NullReferenceException Je vérifie d'abord s'il y a des nullités, puis je coupe et vérifie s'il y a des vides pour éviter que des espaces blancs ne passent.

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
}

À partir de la version 4.0 de .NET, vous pouvez utiliser la fonction String.IsNullOrWhiteSpace pour effectuer ces vérifications en une seule fois. En procédant ainsi, vous renoncez à la possibilité de spécifier un type d'exception granulaire, et j'opterais donc pour la méthode ArgumentException et mettre à jour le message en conséquence.

Answer 2

Vous devriez lancer un ArgumentException si une chaîne vide n'est pas une entrée acceptée pour votre méthode. Cela peut être très déroutant pour les clients si vous lancez un message de type ArgumentNullException alors qu'ils n'ont pas fourni null argument.

Il s'agit simplement d'un autre cas d'utilisation. Vous pouvez également avoir des méthodes qui n'acceptent pas les valeurs d'entrée nulles mais qui acceptent les chaînes vides. Il est important d'être cohérent dans l'ensemble de votre application.

Answer 3

En tenant compte de tout ce qui a été dit (Joe / Ahmad Mageed), je créerais alors une exception pour ce cas.

class ArgumentNullOrEmptyException : ArgumentNullException

Answer 4

ArgumentNullException est parfois utilisé dans le .NET Framework pour le cas String.IsNullOrEmpty - un exemple est le suivant System.Windows.Forms.Clipboard.SetText .

Je pense donc qu'il est raisonnable de faire la même chose dans votre code, à moins qu'il n'y ait un réel intérêt à distinguer les deux cas.

Notez que cette exception et les autres exceptions dérivées de ArgumentException indiquent généralement une erreur de programmation, et doivent donc fournir les informations nécessaires pour aider un développeur à diagnostiquer le problème. Personnellement, je pense qu'il est peu probable qu'un développeur trouve déroutant que vous utilisiez ArgumentNullException pour un argument de type chaîne vide, surtout si vous documentez ce comportement comme dans l'exemple ci-dessous.

/// <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 5

Cela dépend des circonstances.

La question qui se pose est la suivante : s'agit-il vraiment d'une erreur ? Je veux dire par là que vous siempre attendre une valeur ? Si c'est le cas, votre meilleure option est de créer votre propre code d'accès. Exception peut-être comme ça :

class StringEmptyOrNullException : Exception
{
}

Vous pouvez également y ajouter vos propres constructeurs, des informations supplémentaires, etc.

Toutefois, s'il ne s'agit pas d'un événement "exceptionnel" dans votre programme, il serait probablement plus judicieux de renvoyer la méthode null et de la gérer à partir de là. N'oubliez pas, Exception sont destinés à des conditions exceptionnelles.

J'espère que cela vous aidera,

Kyle