Examples to explain issues that should be paid attention to when using Python function closures

from functools import wraps
from time import sleep

def retry(attempts=3, wait=2):
  if attempts < 0 or attempts > 5:
    retry_times = 3
  else:
    retry_times = attempts
  if wait < 0 or wait > 5:
    retry_wait = 2
  else:
    retry_wait = after
  def retry_decorator(func):
    @wraps(func)
    def wrapped_function(*args, **kwargs):
      while retry_times > 0:
        try:
          return func(*args, **kwargs)
        except :
          sleep(retry_wait)
          retry_times -= 1
    return wrapped_function
  return retry_decorator

The simple version of the retry decorator, the required variables are perfectly captured by the closure, and the logic is quite simple and clear. The person who asked said that the logic seemed to be quite normal, but the error message that the variable retry_times could not be found (unresolved reference) kept getting reported.

Yes, check it carefully, this is a scoring question: the variables captured by the closure (retry_times, retry_wait) are equivalent to the local variables of the retry function that are referenced at the time. When the local effect of wrapped_function is used to operate the immutable type, When data is generated, a new local variable will be generated, but the newly generated local variable retry_times has not had time to be initialized before use, so it will prompt that the variable cannot be found; retry_wait can be used properly.

Python is a duck-typing programming language. Even if there is a warning, it will still run. Write a function as simple as possible, use a decorator, and put a breakpoint in the wrapped_function logic to see the value of each variable. It can be done very quickly. Found the problem (you can also see the error by running directly: UnboundLocalError: local variable 'retry_attempts' referenced before assignment, at least more useful than warning msg):

@retry(7, 8)
def test():
  print 23333
  raise Exception('Call me exception 2333.')

if __name__ == '__main__':
  test()

output: UnboundLocalError: local variable 'retry_times' referenced before assignment

def retry(attempts=3, wait=2):
  temp_dict = {
    'retry_times': 3 if attempts < 0 or attempts > 5 else attempts,
    'retry_wait': 2 if wait < 0 or wait > 5 else wait
  }

  def retry_decorate(fn):
    @wraps(fn)
    def wrapped_function(*args, **kwargs):
      print id(temp_dict), temp_dict
      while temp_dict.get('retry_times') > 0:
        try:
          return fn(*args, **kwargs)
        except :
          sleep(temp_dict.get('retry_wait'))
          temp_dict['retry_times'] = temp_dict.get('retry_times') - 1
        print id(temp_dict), temp_dict

    print id(temp_dict), temp_dict

    return wrapped_function

  return retry_decorate

@retry(7, 8)
def test():
  print 23333
  raise Exception('Call me exception 2333.')

if __name__ == '__main__':
  test()

4405472064 {'retry_wait': 2, 'retry_times': 3}
4405472064 {'retry_wait': 2, 'retry_times': 3}
23333
4405472064 {'retry_wait': 2, 'retry_times': 2}
23333
4405472064 {'retry_wait': 2, 'retry_times': 1}
23333
4405472064 {'retry_wait': 2, 'retry_times': 0}

>>> test.func_closure[1].cell_contents
{'retry_wait': 2, 'retry_times': 2}