V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member Sign In
推荐学习书目
› Learn Python the Hard Way
Python Sites
› PyPI - Python Package Index
› http://diveintopython.org/toc/index.html
› Pocoo
值得关注的项目
› PyPy
› Celery
› Jinja2
› Read the Docs
› gevent
› pyenv
› virtualenv
› Stackless Python
› Beautiful Soup
› 结巴中文分词
› Green Unicorn
› Sentry
› Shovel
› Pyflakes
› pytest
Python 编程
› pep8 Checker
Styles
› PEP 8
› Google Python Style Guide
› Code Style from The Hitchhiker's Guide
This topic created in 4036 days ago, the information mentioned may be changed or developed.
自己用装饰器写了一个，有什么需要改进的地方吗？
def retry(attempt):
def decorator(func):
def wrapper(*args, **kw):
att = 0
while att < attempt:
try:
return func(*args, **kw)
except Exception as e:
att += 1
return wrapper
return decorator
@rety(attempt=3)
def get_response(url):
import requests
r = requests.get('http://xxx')
return r.content
 |
1
dalang Apr 21, 2015
其实我觉得没什么问题。只是直接捕获 Exception 对后期排障有隐患，而且我更倾向于能明确告知调用者 retry 失败的原因。
def retry(attempt, raise_on_fail=False): def decorator(func): def wrapper(*args, **kw): att = 0 last_except = None while att < attempt: try: return func(*args, **kw) except Exception as e: att += 1 last_except = e else: if raise_on_fail: raise Exception('Hit retry threshold, failed for {0}' % str(last_except)) return None return wrapper return decorator |
|
2
dalang Apr 21, 2015
|
Select Language