Python скрипт висит в петле

Question

В одном из моих сценариев, я использую следующий бесконечный цикл для проверки активного подключения к Интернету:

def online_check(): 
    try: 
     con = urllib2.urlopen("http://www.google.com/") 
     data = con.read() 
     logging.debug('{0} Reached the host. Exiting online_check'.format(time.strftime('[ %H:%M:%S ]'))) 
    except: 
     logging.debug('{0} Could not reach host trying again in 3 seconds'.format(time.strftime('[ %H:%M:%S ]'))) 
     time.sleep(3) 
     online_check() 

Я знаю, что это не очень элегантное решение, но проблема заключается в том, что когда я запускаю свой скрипт, он иногда вызывает метод online_check и застревает посередине (один раз примерно в 200 попытках). Скрипт все еще запущен и не генерируется исключение; сценарий просто застрял. Я могу нажать CTRL + C (даже после того, как часы скрипта застряли), и он просто выбросит исключение и продолжит работу со следующим online_check. Я также переписал сценарий, чтобы проверить IP-адрес в «ifconfig», а не на pinging google, к сожалению, с аналогичными результатами.

Что я делаю неправильно? Могу ли я переписать сценарий, чтобы этого не произошло? Есть ли что-то, что я могу сделать, чтобы узнать, что здесь происходит?

справка приветствуется. Btw. Я использую Python2.7.1, и я пробовал этот скрипт как на Linux, так и на Mac.

P.S: Если у вас есть рекомендации по разработке метода, который проверяет подключение, не используя пропускную способность и минимальные накладные расходы, я был бы более чем счастлив услышать это.

Answer 1

В дополнение к бесконечной рекурсии (CPython, насколько я знаю, не поддерживает оптимизированную рекурсию хвоста), вы не закрываете свое соединение.

Возможно, вы столкнетесь с каким-либо ограничением соединения либо с ОС, либо с Google при повторном запуске. Вероятно, это происходит до достижения максимальной глубины рекурсии, поэтому вы не получаете исключения.

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

Также как упоминалось в @senderle, вы должны попытаться поймать более конкретную ошибку.

Try:

def online_check(): 
    while True: 
    try: 
     con = urllib2.urlopen("http://www.google.com/") 
     data = con.read() 
     logging.debug('{0} Reached the host. Exiting online_check'.format(time.strftime('[ %H:%M:%S ]'))) 
    except urllib2.URLError: 
     logging.debug('{0} Could not reach host trying again in 3 seconds'.format(time.strftime('[ %H:%M:%S ]'))) 
     time.sleep(3) 
    finally: 
     con.close() 

(Предупреждение непроверенный код)

Answer 2

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

Такие, как:

def online_check(max_checks=10, current_check=0): 
    try: 
     con = urllib2.urlopen("http://www.google.com/") 
     data = con.read() 
     logging.debug('{0} Reached the host. Exiting online_check'.format(time.strftime('[ %H:%M:%S ]'))) 
    except: 
     logging.debug('{0} Could not reach host trying again in 3 seconds'.format(time.strftime('[ %H:%M:%S ]'))) 
     if max_checks > current_check: 
      time.sleep(3) 
      online_check(max_checks, current_check+1) 

Так вы могли бы сделать:

online_check(5) # for 5 maximum checks 
online_check() # use the default value, which is 10 

Я также предлагаю вам поймать более конкретные исключения, для улучшения навыков кодирования, но и потому, что, когда вы делаете CTRL-C, Python выдает исключение KeyboardInterrupt, которое ваш код действительно ловит, потому что вы улавливаете все исключения.

Answer 3

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

def online_check(): 
    while True: 
    try: 
     con = urllib2.urlopen("http://www.google.com/") 
     data = con.read() 
     logging.debug('{0} Reached the host. Exiting online_check'.format(time.strftime('[ %H:%M:%S ]'))) 
     break 
    except: 
     logging.debug('{0} Could not reach host trying again in 3 seconds'.format(time.strftime('[ %H:%M:%S ]'))) 
     time.sleep(3) 

Это должно дать вам лучшие результаты. Я предполагаю, что у вас уже есть отдельный поток, который запускает эту функцию (иначе код не может выполняться - эта функция никогда не возвращается).

Answer 4

В дополнение к предложениям других, вы действительно должны указать, какие исключения (ы) вы ловли. Control-C не работает, потому что это просто вызывает исключение, и ваш оператор except интерпретирует его как то же, что и все остальные.

Похоже на исключение, которое вы хотите, это urllib2.URLError.