Блог им. Albus

Обработка таймаута на Питоне

Коллеги, помогите написать красиво кусочек кода. Обработка ситуации, когда ты заказал котировки с Финама, они не пришли, и ты пробуешь ещё раз. Сейчас я кривенько (чтобы описать задачу) написал так:

try: 
        txt=urlopen(url, timeout=20).readlines()
except timeout:
        print ("Exception!\nWait...")
        sleep (20)
        try:
                txt=urlopen(url, timeout=20).readlines()
        except timeout:
                print ("Exception!\nWait...")
                sleep (20)
                try:
                        txt=urlopen(url, timeout=20).readlines()
                except timeout:
                        print ("Котировки с Финама не пришли")
Как это написать красиво внутри цикла?
10 попыток. Если 10-я неудачная, выводим сообщение
print ("Котировки с Финама не пришли")
★4
23 комментария
+3
def get_data(url, timeout, tries=0, max_tries=10):
  try:
    txt = urlopen(url, timeout=timeout).readlines()
  except TimeoutError:
    print(f«Retry {tries + 1} for {url}»)
    if tries + 1 <= max_tries:
       txt = get_data(url, timeout, tries + 1, max_tries)
    else:
      raise TimeoutError
  return txt
avatar
Михаил
Михаил, круто! С рекурсией! Я бы так не смог ))))
avatar
Albus (Игорь Китаев)
+1
Михаил, так вроде тоже можно:



avatar
Albus (Игорь Китаев)
+2
Albus (Игорь Китаев), можно естественно миллионом способов. Вопрос, как в некотором смысле правильнее для долгосрочной поддержки программы. 

Лучше делить программу на очень небольшие функции.

В Питоне цикл по range обычно признак криво написанной программы.

Создание флагов перед циклом тоже обычно антипатерн. 

Захотите вы потом поменять время ожидания или количество попыток — вам прийдется читать код и менять в нескольких местах. У меня вы просто вызов функции с другими параметрами сделаете.
avatar
Михаил
+1
Михаил, все отлично

только вот вызывать рекурсивно функцию да еще много раз в exception не очень здорово

Инженер по контролю качества ПО наверняка придерется и не пропустит в продакшн
avatar
_sg_
_sg_, в питоне цикл с помощью exception реализован — возможно в других языках это и не очень хорошо. 
avatar
Михаил
Михаил, 
вызывать эту функцию так?
get_data(url, timeout=60, tries=0, max_tries=10)
tries = 0 — уточняю из-за неё потому что она тут глубоко участвует в рекурсии. 
avatar
Albus (Игорь Китаев)
+1
Albus (Игорь Китаев), при вызове вы передаете url и timeout — остальные параметры не используете get_data(url, 60). 
avatar
Михаил
Михаил, 
спасибо, работает.
Только с этой строчкой что то не то, я её заменил
    print(f«Retry {tries + 1} for {url}»)
(правильные кавычки "" тоже ставил — безрезультатно)
avatar
Albus (Игорь Китаев)
Albus (Игорь Китаев), кавычки smartlab автоматически заменяет. f-строки могут не работать на старой версии Питона — у вас какая?
avatar
Михаил
Михаил, 3.8.2 — самая свежая
avatar
Albus (Игорь Китаев)
Albus (Игорь Китаев), тогда не знаю — должно работать с 3.6. Дистанционно сложно разобраться. 
avatar
Михаил
Albus (Игорь Китаев), это самый правильный и читабельный способ, очевидно
avatar
_xXx_
+1
И так сойдёт! Ты же трейдер, а не программист! ))
avatar
Пафос Респектыч
+1
Eugene Logunov,
не нужно одновременно и false и NULL использовать.
Можно просто done = NULL
исправить while
и убрать
  if (data != NULL) {
    done = TRUE
avatar
_sg_
+1
рекурсия нужна )
avatar
siesta00
рекурсию поюзать
avatar
tashik

pypi.org/project/retrying/ если хочется прям красиво сделать.

а так в цикле считать попытки и брейкать цикл при удаче.

avatar
day0markets.ru
day0markets, +1
сделать отдельно функцию запроса которая считывает данные и возвращает сами данные и флаг 1\0 — (есть ошибка получения данных/ нет ошибки)
выглядит на питоне так
for ...
 data, err = LoadData(параметры запроса)
 if err == 0:
   break
 else:
   timeout ()


avatar
sis12qw
+1
sis12qw, ну такое. Имхо, вызывающая функция не должна знать о таймаутах и прочем. Ну и когда потребуется ее дернуть в другом придется лезть и смотреть что она там за флаги возвращает и опять городить огород с retry. Флаги вообще далеко не best practice, лучше exception выбрасывать и его обрабатывть
avatar
day0markets.ru
def get_data(url, timeout=10, max_tries=10):<br>    data = None<br>    try:<br>        data = urlopen(url, timeout=timeout).readlines()<br>    except TimeoutError:<br>        max_tries -= 1<br>    if data:<br>        return data<br>    if max_tries == 0:<br>        return data<br>    get_data(url, timeout, max_tries)
avatar
Сергей

полезные записи за 24 часа

★11 84 Ситуация на текущий момент
★7 3 Топ-10 облигаций с высокой доходностью от аналитиков РБК
★5 3 ​​Самолет - цифры говорят сами за себя
★5 26 😞Ключевая ставка: есть версия, что 16% с нами надолго
★4 35 Деприватизации не будет твердо и четко
★3 83 Индекс МБ сегодня
★3 6 Стагфляционный шок: ВВП удивил самым низким показателем за 2 года, ниже самых низких оценок, PCE горяч
★2 9 Падение доходности облигаций показывает, что в Китае наступило время кризиса
★2 1 🎱Топ-8 перспективных акций. Обновление
★2 5 Московская биржа выбрала систему управления базами данных от «Диасофт»
+225 84 Ситуация на текущий момент
+126 83 Индекс МБ сегодня
+114 50 Азия. Пятница.
+91 2 Ставка осталась на месте🔥Акции и инвестиции
+86 29 В инвесторов в очередной раз плюнули 3% аллокацией😁 зато рынок может расти
+80 188 "Цель она есть". Погнали! (GOLD).
+66 35 Деприватизации не будет твердо и четко
+54 5 ЛУКОЙЛ отчет по РСБУ за 1-й квартал 2024 года: следим теперь за базой для дивидендов именно в этих отчетах
+54 6 Ставка осталась 16% и комментарии ЦБ РФ
+48 0 230 бесплатных роботов для HTX API.

самые обсуждаемые сегодня

21к Неужели здесь всего лишь с десяток трейдеров торгующих на Форекс? Это какой-то капец. -Ау! -Вы где подевались?
3к ‼️ПОЧЕМУ НАШ СЕНТИМЕНТ РЫНКА ПРОГНИВШИЙ
2к $1 трлн — военный бюджет США: обнищание американцев и обогащение производителей оружия, — The Hill
2к Запрет оборота и рекламы криптовалют в России с 01.09.24. Законопроект.
4к Индекс МБ сегодня
1к Пятничный поцелуй
1к Азия. Суббота.

теги блога Albus (Игорь Китаев)

....все тэги



UPDONW
Новый дизайн