Есть ли питонский способ попробовать что-то до максимального количества раз? - PullRequest
73 голосов
/ 20 февраля 2009

У меня есть скрипт Python, который запрашивает сервер MySQL на общем хосте Linux. По некоторым причинам, запросы к MySQL часто возвращают ошибку «сервер ушел»:

_mysql_exceptions.OperationalError: (2006, 'MySQL server has gone away')

Если вы попытаетесь выполнить запрос сразу же после этого, он обычно будет успешным. Итак, я хотел бы знать, есть ли в python разумный способ попытаться выполнить запрос, и, если он потерпит неудачу, повторить попытку с фиксированным числом попыток. Вероятно, я бы хотел, чтобы он попробовал 5 раз, прежде чем полностью отказаться.

Вот код, который у меня есть:

conn = MySQLdb.connect(host, user, password, database)
cursor = conn.cursor()

try:
    cursor.execute(query)
    rows = cursor.fetchall()
    for row in rows:
        # do something with the data
except MySQLdb.Error, e:
    print "MySQL Error %d: %s" % (e.args[0], e.args[1])

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

Ответы [ 9 ]

80 голосов
/ 20 февраля 2009

Как насчет:

conn = MySQLdb.connect(host, user, password, database)
cursor = conn.cursor()
attempts = 0

while attempts < 3:
    try:
        cursor.execute(query)
        rows = cursor.fetchall()
        for row in rows:
            # do something with the data
        break
    except MySQLdb.Error, e:
        attempts += 1
        print "MySQL Error %d: %s" % (e.args[0], e.args[1])
74 голосов
/ 20 февраля 2009

Опираясь на ответ Даны, вы можете сделать это в качестве декоратора:

def retry(howmany):
    def tryIt(func):
        def f():
            attempts = 0
            while attempts < howmany:
                try:
                    return func()
                except:
                    attempts += 1
        return f
    return tryIt

Тогда ...

@retry(5)
def the_db_func():
    # [...]

Усовершенствованная версия, использующая модуль decorator

import decorator, time

def retry(howmany, *exception_types, **kwargs):
    timeout = kwargs.get('timeout', 0.0) # seconds
    @decorator.decorator
    def tryIt(func, *fargs, **fkwargs):
        for _ in xrange(howmany):
            try: return func(*fargs, **fkwargs)
            except exception_types or Exception:
                if timeout is not None: time.sleep(timeout)
    return tryIt

Тогда ...

@retry(5, MySQLdb.Error, timeout=0.5)
def the_db_func():
    # [...]

Для установки модуля decorator :

$ easy_install decorator
11 голосов
/ 23 июня 2015

ОБНОВЛЕНИЕ: есть улучшенная ветвь библиотеки повторных попыток под названием tenacity , которая поддерживает больше функций и в целом более гибкая.


Да, есть библиотека повторных попыток , которая имеет декоратор, который реализует несколько видов логики повторных попыток, которые вы можете комбинировать:

Некоторые примеры:

@retry(stop_max_attempt_number=7)
def stop_after_7_attempts():
    print "Stopping after 7 attempts"

@retry(wait_fixed=2000)
def wait_2_s():
    print "Wait 2 second between retries"

@retry(wait_exponential_multiplier=1000, wait_exponential_max=10000)
def wait_exponential_1000():
    print "Wait 2^x * 1000 milliseconds between each retry,"
    print "up to 10 seconds, then 10 seconds afterwards"
7 голосов
/ 20 февраля 2009
conn = MySQLdb.connect(host, user, password, database)
cursor = conn.cursor()

for i in range(3):
    try:
        cursor.execute(query)
        rows = cursor.fetchall()
        for row in rows:
            # do something with the data
        break
    except MySQLdb.Error, e:
        print "MySQL Error %d: %s" % (e.args[0], e.args[1])
6 голосов
/ 22 февраля 2009

Как и S.Lott, мне нравится флаг, чтобы проверить, закончили ли мы:

conn = MySQLdb.connect(host, user, password, database)
cursor = conn.cursor()

success = False
attempts = 0

while attempts < 3 and not success:
    try:
        cursor.execute(query)
        rows = cursor.fetchall()
        for row in rows:
            # do something with the data
        success = True 
    except MySQLdb.Error, e:
        print "MySQL Error %d: %s" % (e.args[0], e.args[1])
        attempts += 1
6 голосов
/ 20 февраля 2009

Я бы рефакторинг это так:

def callee(cursor):
    cursor.execute(query)
    rows = cursor.fetchall()
    for row in rows:
        # do something with the data

def caller(attempt_count=3, wait_interval=20):
    """:param wait_interval: In seconds."""
    conn = MySQLdb.connect(host, user, password, database)
    cursor = conn.cursor()
    for attempt_number in range(attempt_count):
        try:
            callee(cursor)
        except MySQLdb.Error, e:
            logging.warn("MySQL Error %d: %s", e.args[0], e.args[1])
            time.sleep(wait_interval)
        else:
            break

Выделение функции callee, похоже, нарушает функциональность, так что легко увидеть бизнес-логику, не увязнув в коде повторения.

1 голос
/ 04 декабря 2015

1. Определение:

def try_three_times(express):
    att = 0
    while att < 3:
        try: return express()
        except: att += 1
    else: return u"FAILED"

2.Usage:

try_three_times(lambda: do_some_function_or_express())

Я использую его для разбора HTML-контекста.

1 голос
/ 27 марта 2015
def successful_transaction(transaction):
    try:
        transaction()
        return True
    except SQL...:
        return False

succeeded = any(successful_transaction(transaction)
                for transaction in repeat(transaction, 3))
0 голосов
/ 11 января 2013

Это мое общее решение:

class TryTimes(object):
    ''' A context-managed coroutine that returns True until a number of tries have been reached. '''

    def __init__(self, times):
        ''' times: Number of retries before failing. '''
        self.times = times
        self.count = 0

    def __next__(self):
        ''' A generator expression that counts up to times. '''
        while self.count < self.times:
            self.count += 1
        yield False

    def __call__(self, *args, **kwargs):
        ''' This allows "o() calls for "o = TryTimes(3)". '''
        return self.__next__().next()

    def __enter__(self):
        ''' Context manager entry, bound to t in "with TryTimes(3) as t" '''
        return self

    def __exit__(self, exc_type, exc_val, exc_tb):
        ''' Context manager exit. '''
        return False # don't suppress exception

Это позволяет код, подобный следующему:

with TryTimes(3) as t:
    while t():
        print "Your code to try several times"

Также возможно:

t = TryTimes(3)
while t():
    print "Your code to try several times"

Надеюсь, это можно улучшить, обрабатывая исключения более интуитивно понятным способом. Открыт для предложений.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...