Блог

Итак, у меня ситуация, когда мне нужно выдать исключение, потому что «аргумент не поддерживается». Чтобы объяснить, как я сюда попал, приведу примерную ситуацию:

• Оки бывают многих видов, включая Вкусняшки и Закутки
• Оукс может дружить с другими Оукс, но только с правильными типами
• Юки могут дружить с Зуками, но Зуки не могут дружить с Юками

Пример кода:

 public abstract class Ook
{
    public abstract bool TryBefriendYook(Yook yook);
    public abstract bool TryBefriendZook(Zook zook);

    public bool TryBefriend(Ook o0k)
    {
        Type ookType = ook.GetType;

        if (ookType == typeof(Yook))
        {
            TryBefriendYook((Yook)ook);
            return true;
        }
        else if (ookType == typeof(Zook))
        {
            TryBefriendZook((Zook)ook);
            return true;
        }
        else return false;
    }

    public void Befriend(Ook ook)
    {
        if(!TryBefriend(ook))
            throw new Exception(
                "argument type not supported");
    }
}

public sealed class Yook : Ook
{
    public override bool TryBefriendYook(Yook yook)
    {
        return true;
    }
    public override bool TryBefriendZook(Zook zook)
    {
        return true;
    }
}

public partial sealed class Zook : Ook
{
    public override bool TryBefriendYook(Yook yook)
    {
        return false;
    }
    public override bool TryBefriendZook(Zook zook)
    {
        return true;
    }
}
  

Итак, этот тип подпадает под оба ArgumentException (аргумент не подходит для подкласса) и NotSupportedException (подкласс не принимает аргумент), не так ли?

Итак, какой из них мне следует выбрать — или вместо этого мне следует написать пользовательское исключение для этой ситуации?

Вы не должны выдавать ни одно исключение. Вы объявили этот метод:

 public bool TryBefriend(Ook o0k)
  

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

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

Думайте об этом как о попытке вывести деньги из банка. Если вы попытаетесь вывести 1000 долларов, вы должны получить сообщение о том, что это удалось, или о том, что это не удалось, потому что у вас нет 1000 долларов. Он не должен возвращать успех, если у вас достаточно денег, или выдавать исключение, если у вас этого нет. Отсутствие достаточного количества денег — нормальная, предсказуемая возможность. Если, с другой стороны, приложение не может продолжить работу, потому что оно потеряло соединение с базой данных и нет способа восстановить, то создание исключения имело бы смысл.

Поэтому я бы просто вернул true or false , и тогда вызывающий может решить, что делать, на основе ответа. Вы увидите это, что имеет смысл:

 if(someOok.TryBefriend(someOtherOook))
{
    // friends!
}
else
{
    // not friends!
}
  

…и не это, что неудобно:

 try
{
    someOok.TryBefriend(someOtherOook);
    // friends!
}
catch(NotSupportedException ex)
{
    // not friends!
}
  

Согласно MSDN:

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

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

Итак, согласно MSDN, исключение ArgumentException больше подходит для вашего случая редактировать: если абсолютно необходимо больше аргументов или пользовательский возврат, вы можете написать свое собственное пользовательское исключение, однако нет проблем в использовании исключения ArgumentException, если оно соответствует вашим потребностям.