Сколько попыток/исключений слишком много в функции python

Question

У меня есть функция, которая геокодирует адрес. Я не хочу, чтобы эта функция умирала, поэтому я пытаюсь поймать ошибку и вместо нее вернусь. Однако я также хочу различать ошибки, для этого я использую try/except в нескольких местах.

Есть ли такая вещь, как слишком много попыток/кроме? Как бы вы оптимизировали эту функцию?

Вот мой код:

def geocode(address): 
    js = '' 
    try: 
     urlq = urllib.urlencode({'address':address, 'sensor':'false'}) 
    except Exception, e: 
     return (False, "Error url-encoding address. Error:%s" % e, js, 'failed') 
    try: 
     f = urllib2.urlopen(GEO_URL + urlq) 
     d = f.read() 
    except Exception, e: 
     return (False, "Error making connection. Error:%s" % e, js, 'failed') 
    # 
    try: 
     js = json.loads(d) 
    except Exception, e: 
     return (False, "Error converting JSON. Error:%s" % e, js, 'failed') 
    return (True, '', js, 'ok') 

Answer 1

Ловля Exception всегда плохая идея. Вы хотите указать, какую ошибку вы хотите поймать.

try: 
    ... 
except URLError, e: 
    return (False, "Error making connection. Error:%s" % e, js, 'failed') 
except ValueError, e: 
    return (False, "Error converting JSON. Error:%s" % e, js, 'failed') 
except UnicodeEncodeError, e: 
    return (False, "Error unicode formatting. Error:%s" % e, js, 'failed') 

Также возвращающий кортеж для указания ошибки, как правило, не является предпочтительным. Подумайте о том, чтобы положить попытку, кроме как в вызывающую функцию, и позволить распространению ошибки.