Danstiv / cm-assistant

1 stars 1 forks source link

except Exception #22

Open TheZavitaev opened 2 years ago

TheZavitaev commented 2 years ago

https://github.com/Danstiv/cm-assistant/blob/master/tgbot/core.py#L95

Лучше создать свое исключение, чтобы иметь возможность отловить ошибку конкретно в этом месте.

except Exceptions as error:
    raise SomeError(  
        (  
            'Во время выполнения произошла'  
            ' непредвиденная ошибка: {error}'        
                ' params = {params};'    
        ).format(  
            error=error,  
            **params  
        )  
    ) from error
Danstiv commented 2 years ago

Не вижу в этом смысла. Сейчас если при выполнении таска происходит exception, при вызове result он рейзится, перехватывается и логируется как есть. Т.е. первой строкой лога идёт заголовок, содержащий название таска, а далее чистый exception из него. Не совсем понимаю, зачем его ещё во что-то оборачивать.