ArgumentException или ArgumentNullException для строковых параметров?

Question

Что касается лучших практик, то лучше:

public void SomeMethod(string str) 
{
    if(string.IsNullOrEmpty(str))
    {
        throw new ArgumentException("str cannot be null or empty.");
    }

    // do other stuff
}

или

public void SomeMethod(string str) 
{
    if(str == null) 
    {
        throw new ArgumentNullException("str");
    }

    if(str == string.Empty)
    {
        throw new ArgumentException("str cannot be empty.");
    }

    // do other stuff
}

Вторая версия кажется более точной, но и более громоздкой, чем первая. Я обычно иду с # 1, но решил проверить, есть ли аргумент для # 2.

Answer 1

Я бы сказал, что второй способ действительно более точный - да, он более громоздкий, но вы всегда можете обернуть его в метод, чтобы избежать необходимости делать это все время. Это может быть даже метод расширения:

str.ThrowIfNullOrEmpty("str");


public static void ThrowIfNullOrEmpty(this string value, string name)
{
    if (value == null)
    {
        throw new ArgumentNullException(name);
    }
    if (value == "")
    {
        throw new ArgumentException("Argument must not be the empty string.",
                                    name);
    }
}

Другая потенциально полезная форма - это форма, которая возвращает исходную строку, если все в порядке. Вы могли бы написать что-то вроде этого:

public Person(string name)
{
    this.name = name.CheckNotEmpty();
}

Другой вариант, который следует рассмотреть, - это использование Кодовых контрактов в качестве альтернативы выбрасыванию собственных исключений.

Answer 2

Я бы предложил использовать первый. Если ваш метод не ожидает нулевую или пустую строку, на самом деле не имеет значения, была ли передана нулевая или пустая строка - важно сообщать и сообщать об ошибке, и именно это делает 1-й вариант.

Answer 3

Бросок пустой ссылки на пустую строку может сильно сбить с толку - значение по умолчанию для строки равно нулю и будет указывать на неинициализированную строку, тогда как пустая строка может представлять другие проблемы.

Мне нравится идея Джона Скита, однако мне нравится явно использовать throw, а не иметь отдельную функцию.

Вместо этого вы можете бросить следующий класс:

 public class ArgumentNullOrEmptyException : ArgumentNullException
{
    #region Properties And Fields

    private static string DefaultMessage => $"A value cannot be null or empty{Environment.NewLine}";

    #endregion

    #region Construction and Destruction

    public ArgumentNullOrEmptyException()
        : base(DefaultMessage)
    {
    }

    public ArgumentNullOrEmptyException(string paramName)
        : base(paramName, 
                  $"{DefaultMessage}" +
                  $"Parameter name: {paramName}")
    {
    }

    public ArgumentNullOrEmptyException(string message, Exception innerException)
        : base($"{DefaultMessage}{message}", innerException)
    {
    }

    public ArgumentNullOrEmptyException(string paramName, string message)
        : base(paramName, 
              $"{DefaultMessage}" +
              $"Parameter name: {paramName}{Environment.NewLine}" +
              $"{message}")
    {
    }

    #endregion

    public static void ThrowOnNullOrEmpty(string paramName, string paramValue)
    {
        if(string.IsNullOrEmpty(paramValue))
            throw new ArgumentNullOrEmptyException(paramName);
    }
    public static void ThrowOnNullOrEmpty(string paramValue)
    {
        if (string.IsNullOrEmpty(paramValue))
            throw new ArgumentNullOrEmptyException();
    }


    public static void ThrowOnNullOrEmpty(string paramName, object paramValue)
    {
        //ThrowOnNullOrEmpty another object that could be 'empty' 

        throw new NotImplementedException();
    }
}



        private static void AFunctionWithAstringParameter(string inputString)
    {
        if(string.IsNullOrEmpty(inputString)) throw new ArgumentNullOrEmptyException(nameof(inputString));

        //Or ..
        ArgumentNullOrEmptyException.ThrowOnNullOrEmpty(nameof(inputString), inputString);
    }

Обратите внимание, что мой производный тип вызывает только базовый класс исключений вместо переопределения свойства сообщения. Это связано с тем, что в «дополнительной информации» отладчика Visual Studio указано внутреннее сообщение, а не переопределенный тип. Это означает, что это единственный способ красиво отобразить сообщение во время отладки.

Answer 4

Другая возможность ArgumentOutOfRange исключение:

Исключение, которое выдается, когда значение аргумента выходит за допустимый диапазон значений, как определено вызываемым методом.