Try: raise except:?

Question

[Mikkkch]

Здравствуйте, есть ли что-то плохое в том, чтобы использовать следующую конструкцию?

try:
    if <condition>:
        raise SomethingError
except SomethingError:
    ...

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

Comments

[Рональд Макдональд]

Но зачем?

[Mikkkch]

Рональд Макдональд, например если я хочу в одном месте выловить ошибку и вернуть ее пользователю.

try:

        if await consumersCRUD.get(form_data.username):
            raise ValueError('Consumer with same username already exist.')

        await security.validate_username(form_data.username)
        await security.validate_password(form_data.password, form_data.confirm_password)

    except ValueError as details:

        context = {
            'request': request,
            'fields':  ConsumerCreation.__fields__.values(),
            'error':   details
        }

        return auth_templates.TemplateResponse('sign-up.html', context=context)

[Влад Григорьев]

Mikkkch, так что мешает вам сразу вернуть ошибку?
зачем эта дополнительная обвязка с except?

[Mikkkch]

Влад Григорьев, а что Вам мешает не обрабатывать ошибки в своем коде вообще? Ну пусть падает там программа, тем более пользователь сам увидит причину того, что его бэкенд регистрировать не хочет.

Answer 1

[Art005]

А не лучше ли
if condition:
*Код который находится под except*

Зачем зря рейзить ошибку и обрабатывать?

Answer 2

[Сергей Горностаев]

Использование исключения для управления ходом выполнения - это общеизвестный антипаттерн. Не надо так делать.

Comments

[Сергей Горностаев]

Dr. Bacon, вполне можно обойтись условными выражениями. Судя по коду в комментариях под вопросом, выброс исключения ради выброса исключения.

[Mikkkch]

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

[Mikkkch]

и если в этой вьюшке я try except'ы не проставлю, а функции, которые валидируют введенные пользователем данные возвращают именно ValueError, то программа вместо того, чтобы вернуть пользователю на фронтенд его оплошности, закроется и хана потоку выполнения

[Сергей Горностаев]

Mikkkch, во-первых, лучше подобные вопросы решать не через исключения. Исключение - это признак исключительной ситуации, а попытка регистрации с занятым именем - это ситуация штатная. Во-вторых, обработчик исключения для этой ситуации должен перехватывать IntegrityError, а не ValueError.

[Mikkkch]

Сергей Горностаев, если не использовать исключения, то пойдет дублирование кода, в котором мне каждый раз придется пересобирать контекст. Зачем ожидать IntegrityError, когда можно достать пользователя из базы, если такой уже есть?Все так делают, даже создатель фреймворка, на котором я пишу. Тем более, если обработать IntegrityError, то придется два разных обработчика заводить.

[Сергей Горностаев]

Dr. Bacon, сразу скажу, бывают ситуации в которых это допустимо, паттеры и антипаттерны - это не абсолютные истинны, ориентироваться надо на конкретную ситуацию. Но я бы сделал отдельно проверку на то, что введено число, и отдельно проверку на вхождение числа в диапазон.

[Mikkkch]

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

[Сергей Горностаев]

Mikkkch, рекомендую прочитать этот ответ и комментарии под ним.

[Mikkkch]

Сергей Горностаев, почитал, спасибо. Попробую отрефакторить код таким образом, чтобы можно было пройтись ифами и сообщить пользователю о местах, в которых он ошибся.