429

I have a loop starting with for i in range(0, 100). Normally it runs correctly, but sometimes it fails due to network conditions. Currently I have it set so that on failure, it will continue in the except clause (continue on to the next number for i).

Is it possible for me to reassign the same number to i and run through the failed iteration of the loop again?

3
  • 2
    You can use range(100) without the first parameter. If you use Python 2.x you could even use xrange(100), this generates an iterator and uses less memory. (Not that it matters with only 100 objects.) Commented Jan 18, 2010 at 6:50
  • 13
    This question may be helpful: is there a pythonic way to try something up to a maximum number of times? Commented Aug 18, 2010 at 19:58
  • You can use Python retrying package. Retrying It is written in Python to simplify the task of adding retry behavior to just about anything.
    – ManJan
    Commented Oct 18, 2017 at 17:13

29 Answers 29

566

Do a while True inside your for loop, put your try code inside, and break from that while loop only when your code succeeds.

for i in range(0,100):
    while True:
        try:
            # do stuff
        except SomeSpecificException:
            continue
        break
12
  • 44
    @Ignacio, huh? continue retries the while loop, of course, not the for (!), so i is not "the next" anything -- it's exactly the same as it was on a previous (failed) leg of the same while, of course. Commented Jan 18, 2010 at 6:05
  • 21
    As xorsyst notes, it's advisable to put a retry limit on there. Otherwise you could get stuck looping for quite some time.
    – Brad Koch
    Commented Aug 29, 2013 at 0:34
  • 5
    This is an excellent example: medium.com/@echohack/… Commented Dec 5, 2014 at 12:09
  • 10
    I would definitely leave out the while True: line, otherwise the break wil continue the outer loop to exhaustion.
    – Jan
    Commented Sep 17, 2018 at 11:50
  • 6
    @Sankalp, it seems to me that this answer is proper for the question text.
    – zneak
    Commented Jan 17, 2020 at 21:16
345

I prefer to limit the number of retries, so that if there's a problem with that specific item you will eventually continue onto the next one, thus:

for i in range(100):
  for attempt in range(10):
    try:
      # do thing
    except:
      # perhaps reconnect, etc.
    else:
      break
  else:
    # we failed all the attempts - deal with the consequences.
12
  • 1
    What are the consequences of the second else as relates to flow control? Under which circumstances is the "we failed..." part of the code executed?
    – user67416
    Commented Jan 28, 2015 at 5:30
  • 8
    @g33kz0r the for-else construct in Python executes the else clause if the for loop doesn't break. So, in this case, that section executes if we try all 10 attempts and always get an exception.
    – xorsyst
    Commented Jan 28, 2015 at 11:25
  • 3
    Don't you need a break at the end of the try: part? With the additional break in try:, if the process completes successfully the loop will break, if it doesn't complete successfully it will go straight to the exception part. Does that make sense? If I don't put a break at the end of try: it just does the thing 100 times.
    – Tristan
    Commented Aug 11, 2015 at 13:59
  • 3
    @Tristan - the else clause of the try does this "if successful, then break" that you are looking for.
    – PaulMcG
    Commented Nov 10, 2016 at 15:00
  • 2
    I also prefer a for-loop for retrying. A wrinkle in this code is that, if you want to re-raise the exception when you give up trying, you need something like "if attempt=9: raise" inside the except clause, and remember to use 9 and not 10.
    – PaulMcG
    Commented Nov 10, 2016 at 15:02
113

UPDATE 2021-12-01:

As of June 2016, the retrying package is no longer being maintained. Consider using the active fork github.com/jd/tenacity, or alternatively github.com/litl/backoff.


The retrying package is a nice way to retry a block of code on failure.

For example:

@retry(wait_random_min=1000, wait_random_max=2000)
def wait_random_1_to_2_s():
    print("Randomly wait 1 to 2 seconds between retries")
4
  • 5
    More generally, pypi has multiple packages for retry decorators: pypi.python.org/…
    – kert
    Commented Apr 16, 2016 at 21:14
  • is there anyway you can print the number of the retry attempt every time it fails?
    – saul
    Commented Nov 13, 2018 at 23:29
  • 26
    As I understood is not maintained, more active fork is github.com/jd/tenacity and maybe github.com/litl/backoff can be used too. Commented Jan 24, 2019 at 7:45
  • 3
    The retrying package is alive again since 2022-09-03 with a new maintainer.
    – selle
    Commented Feb 14, 2023 at 14:30
59

Here is a solution similar to others, but it will raise the exception if it doesn't succeed in the prescribed number or retries.

tries = 3
for i in range(tries):
    try:
        do_the_thing()
    except KeyError as e:
        if i < tries - 1: # i is zero indexed
            continue
        else:
            raise
    break
2
  • 3
    Nice answer, but the variable name retries is misleading. It should much rather be tries.
    – Lukas
    Commented Aug 17, 2016 at 16:48
  • 2
    Very good solution thank you. It could be improved by adding a delay between each try. Very useful when dealing with APIs.
    – Sam
    Commented Dec 31, 2017 at 14:59
35

Alternatives to retrying: tenacity and backoff (2020 update)

The retrying library was previously the way to go, but sadly it has some bugs and it hasn't got any updates since 2016. Other alternatives seem to be backoff and tenacity. During the time of writing this, the tenacity had more GItHub stars (2.3k vs 1.2k) and was updated more recently, hence I chose to use it. Here is an example:

from functools import partial
import random # producing random errors for this example

from tenacity import retry, stop_after_delay, wait_fixed, retry_if_exception_type

# Custom error type for this example
class CommunicationError(Exception):
    pass

# Define shorthand decorator for the used settings.
retry_on_communication_error = partial(
    retry,
    stop=stop_after_delay(10),  # max. 10 seconds wait.
    wait=wait_fixed(0.4),  # wait 400ms 
    retry=retry_if_exception_type(CommunicationError),
)()


@retry_on_communication_error
def do_something_unreliable(i):
    if random.randint(1, 5) == 3:
        print('Run#', i, 'Error occured. Retrying.')
        raise CommunicationError()

for i in range(100):
    do_something_unreliable(i)

The above code outputs something like:

Run# 3 Error occured. Retrying.
Run# 5 Error occured. Retrying.
Run# 6 Error occured. Retrying.
Run# 6 Error occured. Retrying.
Run# 10 Error occured. Retrying.
.
.
.

More settings for the tenacity.retry are listed on the tenacity GitHub page.

4
  • Great answer! I spent hours trying to figure out the decorator, partial stuff.... Could you make your retry_on_communication_error a function that can accept parameters? something like def retry_on_communication_error(para1, para2)?
    – Jerry T
    Commented Jan 2, 2021 at 23:56
  • I'm not entirely sure what you mean, but (1) If you want to pass more parameters to the function that is to be retried, just add more parameters to the function definition. In this example, there is do_something_unreliable(i), but you could have do_something_unreliable(i, j, k) if you wish. (2) If you want to pass some other parameters for the tenacity.retry, just add them to the partial.
    – Niko Fohr
    Commented Jan 3, 2021 at 1:00
  • This answer was super helpful. @np8 what do I do differently if I want to use retry_on_communication_error in a few different files?
    – user613
    Commented Apr 28, 2022 at 15:40
  • You can define it in one module, like above, and then import it as any other function
    – Niko Fohr
    Commented Apr 28, 2022 at 19:28
15

The more "functional" approach without using those ugly while loops:

def tryAgain(retries=0):
    if retries > 10: return
    try:
        # Do stuff
    except:
        tryAgain(retries+1)

tryAgain()
6
  • 27
    I'm sorry, but it seems much uglier than the "ugly while loops" variants; and I am fond of functional programming...
    – lvella
    Commented Sep 9, 2011 at 17:39
  • 11
    You need to make sure you don't recurse deeply though - the default stack size in Python is 1000 Commented Aug 21, 2014 at 10:40
  • 9
    If this is going to be 'functional', the recursion should be: except: tryAgain(retries+1)
    – quamrana
    Commented Aug 1, 2017 at 13:18
  • The problem with this is that we need to pass error around as variables.
    – lowzhao
    Commented Apr 28, 2020 at 1:56
  • Also there is a 1000 limit for recursion Commented Nov 4, 2020 at 15:30
11

The clearest way would be to explicitly set i. For example:

i = 0
while i < 100:
    i += 1
    try:
        # do stuff

    except MyException:
        continue
6
  • 48
    Is that C or C++? I can't tell. Commented Jan 18, 2010 at 6:52
  • 5
    @Georg That's Python, as stated in the question. Or where you being sarcastic for some reason?
    – Jakob Borg
    Commented Jan 18, 2010 at 15:29
  • 4
    This doesn't do what the OP asked for. It might if you put i += 1 just after # do stuff.
    – fmalina
    Commented Oct 20, 2013 at 19:20
  • 9
    Not pythonic. Should use range for this kind of stuff.
    – Mystic
    Commented Dec 1, 2014 at 19:04
  • 2
    I agree, this should definitely use range. Commented May 17, 2016 at 6:00
11
for _ in range(5):
    try:
        # replace this with something that may fail
        raise ValueError("foo")

    # replace Exception with a more specific exception
    except Exception as e:
        err = e
        continue

    # no exception, continue remainder of code
    else:
        break

# did not break the for loop, therefore all attempts
# raised an exception
else:
    raise err

My version is similar to several of the above, but doesn't use a separate while loop, and re-raises the latest exception if all retries fail. Could explicitly set err = None at the top, but not strictly necessary as it should only execute the final else block if there was an error and therefore err is set.

2
  • I think this is the best looking pattern here out of all. However, for someone not coming from python it is really strange that "err" escapes the scope of the for block.
    – niid
    Commented Mar 29, 2021 at 21:20
  • 1
    Yeah, it might look better with an explicit err = None at the top in some ways, but as I noted I don't think it's actually ever used. As a matter of fact, one could then omit the else and use a golang-esque if err or if err is not None in case err could somehow have a falsely value.
    – n8henrie
    Commented Mar 29, 2021 at 21:25
7

A generic solution with a timeout:

import time

def onerror_retry(exception, callback, timeout=2, timedelta=.1):
    end_time = time.time() + timeout
    while True:
        try:
            yield callback()
            break
        except exception:
            if time.time() > end_time:
                raise
            elif timedelta > 0:
                time.sleep(timedelta)

Usage:

for retry in onerror_retry(SomeSpecificException, do_stuff):
    retry()
4
  • Is it possible to specify a separate function for error checking? It would take the output of the callback and pass to the error checking function to decide if it was a failure or success instead of using a simple except exception: Commented Oct 10, 2017 at 21:04
  • Instead of a try … except you can use a if statement. But it is less pythonic. Commented Oct 10, 2017 at 21:07
  • This solution does not work. trinket.io/python/caeead4f6b The exception thrown by do_stuff does not bubble to the generator. Why would it, anyway? do_stuff is called in the body of the for loop, which is on an outer level on its own, not nested in the generator.
    – isarandi
    Commented Apr 17, 2018 at 8:56
  • Your right, but for a different reason: the callback function is never called. I have forgotten the parenthesis, replace by callback(). Commented Apr 17, 2018 at 12:50
5

There is something similar in the Python Decorator Library.

Please bear in mind that it does not test for exceptions, but the return value. It retries until the decorated function returns True.

A slightly modified version should do the trick.

1
5

Using while and a counter:

count = 1
while count <= 3:  # try 3 times
    try:
        # do_the_logic()
        break
    except SomeSpecificException as e:
        # If trying 3rd time and still error?? 
        # Just throw the error- we don't have anything to hide :)
        if count == 3:
            raise
        count += 1
5

Using recursion

for i in range(100):
    def do():
        try:
            ## Network related scripts
        except SpecificException as ex:
            do()
    do() ## invoke do() whenever required inside this loop
3
  • 1
    Exit condition? Or does this run 100 * infinity?
    – ingyhere
    Commented May 17, 2019 at 23:10
  • Nice and simple - no other packages needed - thanks
    – Alex
    Commented May 11, 2022 at 11:45
  • break if no exception are not present Commented Aug 29, 2022 at 6:45
5

attempts = 3
while attempts:
  try:
     ...
     ...
     <status ok>
     break
  except:
    attempts -=1
else: # executed only break was not  raised
   <status failed>

5

Decorator is a good approach.

from functools import wraps
import time

class retry:
    def __init__(self, success=lambda r:True, times=3, delay=1, raiseexception=True, echo=True):
        self.success = success
        self.times = times
        self.raiseexception = raiseexception
        self.echo = echo
        self.delay = delay
    def retry(fun, *args, success=lambda r:True, times=3, delay=1, raiseexception=True, echo=True, **kwargs):
        ex = Exception(f"{fun} failed.")
        r = None
        for i in range(times):
            if i > 0:
                time.sleep(delay*2**(i-1))
            try:
                r = fun(*args, **kwargs)
                s = success(r)
            except Exception as e:
                s = False
                ex = e
                # raise e
            if not s:
                continue
            return r
        else:
            if echo:
                print(f"{fun} failed.", "args:", args, kwargs, "\nresult: %s"%r)
            if raiseexception:
                raise ex
    def __call__(self, fun):
        @wraps(fun)
        def wraper(*args, retry=0, **kwargs):
            retry = retry if retry>0 else self.times
            return self.__class__.retry(fun, *args, 
                                        success=self.success, 
                                        times=retry,
                                        delay=self.delay,
                                        raiseexception = self.raiseexception,
                                        echo = self.echo,
                                        **kwargs)
        return wraper

some usage examples:

@retry(success=lambda x:x>3, times=4, delay=0.1)
def rf1(x=[]):
    x.append(1)
    print(x)
    return len(x)
> rf1()

[1]
[1, 1]
[1, 1, 1]
[1, 1, 1, 1]

4
@retry(success=lambda x:x>3, times=4, delay=0.1)
def rf2(l=[], v=1):
    l.append(v)
    print(l)
    assert len(l)>4
    return len(l)
> rf2(v=2, retry=10) #overwite times=4

[2]
[2, 2]
[2, 2, 2]
[2, 2, 2, 2]
[2, 2, 2, 2, 2]

5
> retry.retry(lambda a,b:a+b, 1, 2, times=2)

3
> retry.retry(lambda a,b:a+b, 1, "2", times=2)

TypeError: unsupported operand type(s) for +: 'int' and 'str'
4

Here is a quick decorator to handle this. 7 lines, no dependencies.

def retry(exception=Exception, retries=3, delay=0):
    def wrap(func):
        for i in range(retries):
            try:
                return func()
            except exception as e:
                print(f'Retrying {func.__name__}: {i}/{retries}')
                time.sleep(delay)
        raise e
    return wrap

@retry()
def do_something():
  ...
@retry(HTTPError, retries=100, delay=3)
def download_something():
  ...

An addition that could be made is extending exception to handle multiple exceptions (splat a list).

2
  • 1
    Nice code, but I think the e is unbound here.
    – scriptboy
    Commented Nov 29, 2022 at 6:50
  • I don't understand how this answer is being voted up. The code doesn't run...
    – gbeaven
    Commented Jun 14, 2023 at 15:56
3

I use following in my codes,

   for i in range(0, 10):
    try:
        #things I need to do
    except ValueError:
        print("Try #{} failed with ValueError: Sleeping for 2 secs before next try:".format(i))
        time.sleep(2)
        continue
    break
3

I use this, which can be used on any function:

def run_with_retry(func: callable, max_retries: int = 3, wait_seconds: int = 2, **func_params):
num_retries = 1
while True:
    try:
        return func(*func_params.values())
    except Exception as e:
        if num_retries > max_retries:
            print('we have reached maximum errors and raising the exception')
            raise e
        else:
            print(f'{num_retries}/{max_retries}')
            print("Retrying error:", e)
            num_retries += 1
            sleep(wait_seconds)

Call like this:

    def add(val1, val2):
        return val1 + val2

    run_with_retry(func=add, param1=10, param2=20)
2

This is my answer, using a decorator approach, a decor function with parameters:

import time
def retry(retries=3, delay=0.5):
    def decor(func):
        def wrap(*args, **kwargs):
            for i in range(retries):
                try:
                    return func(*args, **kwargs)
                except Exception as e:
                    print(f'Retrying {func.__name__}: {i}/{retries}')
                    time.sleep(delay)
                    if(i == retries-1):
                        raise e
        return wrap
    return decor

@retry(retries = 5, delay=0.5)
def divide(a,b):
    print(a/b)

When you call the divide function like divide(6,0), it will retry 5 times, then throw the 'division by zero' exception.

divide(6,0)
Retrying divide: 0/5
Retrying divide: 1/5
Retrying divide: 2/5
Retrying divide: 3/5
Retrying divide: 4/5
Traceback (most recent call last):

  Cell In[48], line 1
    divide(6,0)
    ...
    ...
    print(a/b)

ZeroDivisionError: division by zero
1

If you want a solution without nested loops and invoking break on success you could developer a quick wrap retriable for any iterable. Here's an example of a networking issue that I run into often - saved authentication expires. The use of it would read like this:

client = get_client()
smart_loop = retriable(list_of_values):

for value in smart_loop:
    try:
        client.do_something_with(value)
    except ClientAuthExpired:
        client = get_client()
        smart_loop.retry()
        continue
    except NetworkTimeout:
        smart_loop.retry()
        continue
1

Here is my take on this issue. The following retry function supports the following features:

  • Returns the value of the invoked function when it succeeds
  • Raises the exception of the invoked function if attempts exhausted
  • Limit for the number of attempts (0 for unlimited)
  • Wait (linear or exponential) between attempts
  • Retry only if the exception is an instance of a specific exception type.
  • Optional logging of attempts
import time

def retry(func, ex_type=Exception, limit=0, wait_ms=100, wait_increase_ratio=2, logger=None):
    attempt = 1
    while True:
        try:
            return func()
        except Exception as ex:
            if not isinstance(ex, ex_type):
                raise ex
            if 0 < limit <= attempt:
                if logger:
                    logger.warning("no more attempts")
                raise ex

            if logger:
                logger.error("failed execution attempt #%d", attempt, exc_info=ex)

            attempt += 1
            if logger:
                logger.info("waiting %d ms before attempt #%d", wait_ms, attempt)
            time.sleep(wait_ms / 1000)
            wait_ms *= wait_increase_ratio

Usage:

def fail_randomly():
    y = random.randint(0, 10)
    if y < 10:
        y = 0
    return x / y


logger = logging.getLogger()
logger.setLevel(logging.INFO)
logger.addHandler(logging.StreamHandler(stream=sys.stdout))

logger.info("starting")
result = retry.retry(fail_randomly, ex_type=ZeroDivisionError, limit=20, logger=logger)
logger.info("result is: %s", result)

See my post for more info.

1

I like to use bool values for this, like so:

success = False
num_try = 0
while success is False:
    if num_try >= 10: # or any number
        # handle error how  you please
    try:
        # code
        success = True
    except Exception as e:
        # record or do something with exception if needed
        num_try += 1
1

with this decorator, you can easily control errors

class catch:
    def __init__(self, max=1, callback=None):
        self.max = max 
        self.callback = callback 
    
    def set_max(self, max):
        self.max = max
    
    def handler(self, *args, **kwargs):
        self.index = 0
        while self.index < self.max: 
            self.index += 1
            try:
                self.func(self, *args, **kwargs)
        
            except Exception as error:
                if callable(self.callback):
                    self.callback(self, error, args, kwargs)
                
    def __call__(self, func):
        self.func = func
        return self.handler

import time
def callback(cls, error, args, kwargs):
    print('func args', args, 'func kwargs', kwargs)
    print('error', repr(error), 'trying', cls.index)
    if cls.index == 2:
        cls.set_max(4)
    
    else:
        time.sleep(1)
    
    
@catch(max=2, callback=callback)  
def test(cls, ok, **kwargs):
    raise ValueError('ok')

test(1, message='hello')
1

If retrying a failed attempt x number of times is what you are looking for, a single for else loop is probably what you want. Consider this example with 3 attempts:

attempts = 3

for attempt in range(1, attempts+1):
    try:
        if attempt < 4:
            raise TypeError(f"Error raised on attempt: {attempt}")
        else:
            print(f'Attempt {attempt} finally worked.')
    except (TypeError) as error:
        print(f'Attempt {attempt} hit the exception.')
        continue
    else:
        break
else:
    print(f'Exit after final attempt: {attempt}')

print(f'\nGo on to execute other code ...')

Gives the output:

Attempt 1 hit the exception.
Attempt 2 hit the exception.
Attempt 3 hit the exception.
Exit after final attempt: 3

Go on to execute other code ...

And with one more attempt it succeeds:

attempts = 4

Gives the output:

Attempt 1 hit the exception.
Attempt 2 hit the exception.
Attempt 3 hit the exception.
Attempt 4 finally worked.

Go on to execute other code ...
1

You could have a dedicated function using return to short circuit the result. For example like this:

def my_function_with_retries(..., max_retries=100):
    for attempt in range(max_retries):
        try:
            return my_function(...)
        except SomeSpecificException as error:
            logging.warning(f"Retrying after failed execution: {error}")

    raise SomeOtherException()
1
  • clear and concise solution! Commented Aug 21, 2022 at 16:12
1

I liked laurent-laporte's answer. Here's my version of it wrapped in a class with static methods and some examples. I implemented a retry count as another way to retry. Also added kwargs.

from typing import List
import time


class Retry:
    @staticmethod
    def onerror_retry(exception, callback, retries: int = 0, timeout: float = 0, timedelta: float = 0,
                      errors: List = None, **kwargs):
        """

        @param exception: The exception to trigger retry handling with.
        @param callback: The function that will potentially fail with an exception
        @param retries: Optional total number of retries, regardless of timing if this threshold is met, the call will
                        raise the exception.
        @param timeout: Optional total amount of time to do retries after which the call will raise an exception
        @param timedelta: Optional amount of time to sleep in between calls
        @param errors: A list to receive all the exceptions that were caught.
        @param kwargs: An optional key value parameters to pass to the function to retry.
        """
        for retry in Retry.__onerror_retry(exception, callback, retries, timeout, timedelta, errors, **kwargs):
            if retry: retry(**kwargs)  # retry will be None when all retries fail.

    @staticmethod
    def __onerror_retry(exception, callback, retries: int = 0, timeout: float = 0, timedelta: float = 0,
                        errors: List = None, **kwargs):
        end_time = time.time() + timeout
        continues = 0
        while True:
            try:
                yield callback(**kwargs)
                break
            except exception as ex:
                print(ex)
                if errors:
                    errors.append(ex)

                continues += 1
                if 0 < retries < continues:
                    print('ran out of retries')
                    raise

                if timeout > 0 and time.time() > end_time:
                    print('ran out of time')
                    raise
                elif timedelta > 0:
                    time.sleep(timedelta)


err = 0

#
# sample dumb fail function
def fail_many_times(**kwargs):
    global err
    err += 1
    max_errors = kwargs.pop('max_errors', '') or 1
    if err < max_errors:
        raise ValueError("I made boo boo.")
    print("Successfully did something.")

#
# Example calls
try:
    #
    # retries with a parameter that overrides retries... just because
    Retry.onerror_retry(ValueError, fail_many_times, retries=5, max_errors=3)
    err = 0
    #
    # retries that run out of time, with 1 second sleep between retries.
    Retry.onerror_retry(ValueError, fail_many_times, timeout=5, timedelta=1, max_errors=30)
except Exception as err:
    print(err)

0

I wrote this one. Hopefully it's clear to understand and easy to use.

from functools import wraps
from random import randrange
from time import sleep


def retry(retry_limit=3, delay=None, max_random_delay=60):
    """
    Retrying decorator with simple backoff logic.
    :param retry_limit: amount of retries before throwing exception
    :param delay: delay between attempts. If None, random delay each time
    :param max_random_delay: maximum random delay in seconds
    """
    def decorator(func):
        @wraps(func)
        def wrapper(*args, **kwargs):
            retries = 0
            while retries < retry_limit:
                try:
                    print(f'trying to execute {func.__name__}: {retries + 1}/{retry_limit}')
                    return func(*args, **kwargs)

                except Exception as e:
                    retries += 1
                    if retries == retry_limit:
                        raise e

                    if delay is None:
                        sleep_time = randrange(max_random_delay)
                        print(f'sleep for {sleep_time} seconds')
                        sleep(sleep_time)
                    else:
                        print(f'sleep for {delay} seconds')
                        sleep(delay)
        return wrapper
    return decorator
-3

Here's my idea on how to fix this:

j = 19
def calc(y):
    global j
    try:
        j = j + 8 - y
        x = int(y/j)   # this will eventually raise DIV/0 when j=0
        print("i = ", str(y), " j = ", str(j), " x = ", str(x))
    except:
        j = j + 1   # when the exception happens, increment "j" and retry
        calc(y)
for i in range(50):
    calc(i)
1
  • 9
    This is way off base. Commented Aug 6, 2016 at 22:01
-3

i recently worked with my python on a solution to this problem and i am happy to share it with stackoverflow visitors please give feedback if it is needed.

print("\nmonthly salary per day and year converter".title())
print('==' * 25)


def income_counter(day, salary, month):
    global result2, result, is_ready, result3
    result = salary / month
    result2 = result * day
    result3 = salary * 12
    is_ready = True
    return result, result2, result3, is_ready


i = 0
for i in range(5):
    try:
        month = int(input("\ntotal days of the current month: "))
        salary = int(input("total salary per month: "))
        day = int(input("Total Days to calculate> "))
        income_counter(day=day, salary=salary, month=month)
        if is_ready:
            print(f'Your Salary per one day is: {round(result)}')
            print(f'your income in {day} days will be: {round(result2)}')
            print(f'your total income in one year will be: {round(result3)}')
            break
        else:
            continue
    except ZeroDivisionError:
        is_ready = False
        i += 1
        print("a month does'nt have 0 days, please try again")
        print(f'total chances left: {5 - i}')
    except ValueError:
        is_ready = False
        i += 1
        print("Invalid value, please type a number")
        print(f'total chances left: {5 - i}')
-11

increment your loop variable only when the try clause succeeds

0

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.