T-Ratnosaure / DiscordVoteBot

Bot which organizes the HackademINT's passation vote
0 stars 0 forks source link

if / else #3

Open zteeed opened 4 years ago

zteeed commented 4 years ago

Pour la lisibilité du code et éviter d'avoir beaucoup de niveau d'intendation on essaye d'éviter les if/else imbriqué

ce que je ne veux pas:

    if not admin:
        message += "You're not an admin..."
    elif not started.is_started:
        message += "A session is already started"
    else:
        if not post:
            message += "Please enter a valid post"
        else:
            start = session.query(Start).first()

ce qui est mieux:

    if not admin:
        return "You're not an admin..."
    if not started.is_started:
        return "A session is already started"
   if not post:
        return "Please enter a valid post"
   start = session.query(Start).first()

On élimine les possibilités en arretant ta fonction le plus tot possible si un cas ne te convient pas