Как правильно работать с ошибками в питоне?

Смотри на status_code ответа прежде чем пытаться его парсить.

А если он с нормальным статусом кинет мусор?

А если метеорит упадет на датацентр?

По теме:

Easier to ask for forgiveness than permission. This common Python coding style assumes the existence of valid keys or attributes and catches exceptions if the assumption proves false. This clean and fast style is characterized by the presence of many try and except statements. The technique contrasts with the LBYL style common to many other languages such as C.

Ладно я понял смысл. Туплю что-то

пользуясь случаем, спрошу: как ещё можно работать с ошибками, кроме ветвления и монад?

Можешь в try/except засунуть это.

падать как в ерланге.

Придумали же удобный .format

Зачем так ужасно урл клеить?🧐

А ещё если не ошибаюсь у response есть r.json и тебе ненужно будет жейсонть ответ дальше 🙂

AttributeError: 'HTTPResponse' object has no attribute 'json'

Но json таки в return засунул.

А если метеорит упадет на датацентр?

В нормальной архитектуре программы работающей 24/7 падение датацентра не должно никак влиять на работу программы. Единственное - информация получаемая с датацентра будет отсутствовать, придется довольствоваться тем что есть и ждать когда метеорит уберут, а датацентр починят.

Ага, осталось разобраться как заставить его дрочить функцию до посинения.

стоп стоп,

response = r.json()

http://docs.python-requests.org/en/master/

Да я из urllib пытался вызвать.

https://github.com/pnpnpn/retry-decorator/blob/master/retry_decorator/retry_d... как-то так.

За такой код нужно к стенке. Могли бы и вкласс засунуть вместо то чтобы лесенкой писать

Если совсем просто, то:

shot_number = 5
while shot_number:
    try:
        resp = request.get(url)
        assert resp.code == 200
        return resp.json()
    except:
        pass

    shot_number -= 1

а просто засунуть типа такого?

status_code = 0
while status_code == 0:
    try:    
        ....
        status_code = r.status_code
        ....
    except:
        pass

Не проще?

Да я из urllib пытался вызвать.

Я requests использую, оч удобный.

Если в одном месте — может, и проще, но неограниченное повторение — первый шаг к фейлу.

assert resp.code == 200

assert — для отладки. Они, например, при `python -o` пропускаются.

За такой код нужно к стенке. Могли бы и вкласс засунуть вместо то чтобы лесенкой писать

Stop writing classes. Не вижу, зачем там класс.

Хотя лесенку можно чуть рефакторить, но оно слишком маленькое и без того.

А так вот?

         i = 5        
         while i:
             try:
                 html = requests.get( url ) 
                 html.raise_for_status()
                 i = 0
             except requests.exceptions.HTTPError as err:
                 print(err)
                 i -= 1
         response = html.json()

Правда не понятно что делать, если на датацентр упал ядрён батон, а программа работает как демон. Она же просто упадёт. Поллинг нужен же? Или в питоне ожидание по другому делается?

мы через celery такие задачи крутим. есть падучий сервис, там мы по 5 раз пытаемся связаться с ними если падает то уже пишем лог

Но опять же может тебе celery будет избыточен >_^

Оказалось всё хуже, чем ожидалось, вендопони выкидывают ошибку с 200 кодом. обработку ошибок json decoder придется копать.

Как это дело красиво оформить?

             try:
                 html = requests.get( url, timeout=2.0)
                 html.raise_for_status()
             except requests.exceptions.HTTPError as err:
                 print(err)
                 i -= 1
                 continue
             except requests.exceptions.ConnectTimeout as err:
                 print(err)
                 i -= 1
                 continue
             except requests.exceptions.ConnectionError as err:
                 print(err)
                 i -= 1
                 continue
             except requests.exceptions.ReadTimeout as err:
                 print(err)
                 i -= 1
                 continue

pawnhearts ggrn x3al vvn_black

Ты можешь несколько типов исключений перечистить в одном except блоке.

через or?

https://stackoverflow.com/questions/6470428/catch-multiple-exceptions-in-one-...

https://docs.python.org/3/tutorial/errors.html#handling-exceptions

Заодно можешь почитать в доках как работает or.

In [3]: exceptions.NameError or exceptions.MemoryError
Out[3]: NameError

Человеческие ретраи по статусам/таймаутам/коннект еррорам делаются через Request сессии

https://www.peterbe.com/plog/best-practice-with-retries-with-requests

Ошибки парсинга и долговременную недоступность сервера - это уже сам хендли и продумывай поведение.

Судя по твоему коду, будет достаточно:

try:
    resp = request.get(url)
    assert resp.code == 200
    return resp.json()
except Exception as e:
    print('{} {}'.format(type(e), e)

Так ты поймаешь все ошибки, в том числе и если при 200 коде прилетит не json.

И вопрос, ты хочешь заспамить ресурс? У тебя цикл выполняется непрерывно до тех пор, пока не будет 5 ошибок. Оно разве так надо?

А из-за assert проблем не будет?

И вопрос, ты хочешь заспамить ресурс? У тебя цикл выполняется непрерывно до тех пор, пока не будет 5 ошибок. Оно разве так надо?

Оно со второго или третьего раза всегда прорывается. Там просто хроническая проблема в виде 503, либо ConnectionError.

А с 200 ответом с ошибкой я таки разобрался, сервер возвращал ошибку запроса, они советуют:

nonce=str(int(time.time()))

А на самом деле нужен:

nonce=str(float(time.time()))

А из-за assert проблем не будет?

Каких? Если отладку специально отключить? Не нравится assert, используй то что нравится, я на такой проверке кода ответа не настаиваю.

А сервер реально мудиловский оказывается, меняет структуру json на ответах.