Как выкинуть кастомное исключение?

Question

[embiid]

к примеру есть такой кастомный эксепшн:

public class CustomerExistsException
    {
        public void DisplayException()
        {
            throw new AuthException("The email has already been taken.");
        }
    }

    public class AuthException : Exception
    {
        public AuthException(string message) : base(message) { }
        public AuthException(string message, Exception exception) 
            : base(message, exception) { }
    }

И как потом его закинуть в сервисе?

public async Task<CustomerDto> Register(CustomerDto customerModel, string password)
        {
            if (await EmailExists(customerModel.Email))
            {
                //Cannot implicitly convert type CustomerExistsException to System.Exception
                //throw new CustomerExistsException();
            }
        }

Answer 1

[cicatrix]

Проброс исключения - самое "дорогое" из всех возможных решений проблемы в плане производительности. К тому же, это и не исключение вовсе, а вполне штатная ситуация. Простая проверка в виде вызова bool CustomerExists будет выглядеть здесь гораздо уместнее. К тому же, эта проверка у вас уже производится.
А так, да, класс любого исключения должен быть наследником System.Exception

Answer 2

[Василий Банников]

1. Любое исключание, которое вы хотите кинуть должно наследоваться от System.Exception
2. CustomerException выглядит странно. Зачем там вообще метод "DisplayException"?

Comments

[embiid]

Я хочу сделать вот как в этой статье, но у меня не получается выкинуть этот кастомный эксепшн в сервисе.
https://www.exceptionnotfound.net/the-asp-net-web-...

[Василий Банников]

embiid, посмотри внимательно, как испольщуется CustomExceptionService

[embiid]

Василий Банников,
То есть, 1) у меня есть CustomerExistsException, CustomerNotFoundException, CustomerDataException - они должны быть все унаследованы от класса Exception?
Типу такого:

public class CustomerDataException : Exception
    {
        public CustomerDataException(string message) : base(message) { }
        public CustomerDataException(string message, Exception exception)
            : base(message, exception) { }
    }

2) И там в коде где этот эксепшн должен выкинуться -> throw new CustomerDataException?
3) И потом уже в фильтре я бросаю на соответствующий экспшн статус код?

Спасибо, вроде как разобрался)

[Василий Банников]

embiid, 1. кажется вы хотите логику на исклбчениях строить - не надо так.
2. да
2. да

Answer 3

[edward_freedom]

Обычная функция
new CustomerExistsException().DisplayException();