optimisation Fermé

Question

Bonjour, Je suis débutant en python en je voudrais savoir comment réduire toutes ces lignes (si possible).. Merci :) price = 0if (engine_displacement <= 50): price =+ 5if (engine_displacement > 50) and (engine_displacement <= 125): price =+ 6if (engine_displacement > 125) and (engine_displacement <= 250): price =+ 7if (engine_displacement > 250) and (engine_displacement <= 300): price =+ 8if (engine_displacement > 300) and (engine_displacement <= 450): price =+ 9if (engine_displacement > 450) and (engine_displacement <= 500): price =+ 10if (engine_displacement > 500) and (engine_displacement <= 690): price =+ 11if (engine_displacement > 690) and (engine_displacement <= 700): price =+ 12if (engine_displacement > 700) and (engine_displacement <= 800): price =+ 13if (engine_displacement > 800) and (engine_displacement <= 900): price =+ 14if (engine_displacement > 900) and (engine_displacement <= 1000): price =+ 16if (engine_displacement > 1000) and (engine_displacement <= 10000): price =+ 18print(price)

vortex · Answer

Bonsoir,

Comme toujours avec ce genre de chose, conteneur comme list ou tuple combiné à une simple boucle.

Dans ton code, tu aurais déjà dû utiliser des elif, et également gérer le cas où ta valeur est hors limite.

Ce qui pourrait donner par exemple :

def get_price(v, min_price=0):
    prices = ((50, 5), (125, 6), (250, 7), (300, 8), (450, 9), (500, 10),
    (690, 11), (700, 12), (800, 13), (900, 14), (1_000, 16), (10_000, 16),)
    # Parcours du tuple de la fin vers le début
    for i in range(len(prices)-1, -1, -1):
        # Si 1er élément du tuple, on prend la valeur de min_price
        p_min = prices[i-1][0] if i > 0 else min_price
        # Oui, on peut faire ça en python =D
        if p_min < v <= prices[i][0]:
            return prices[i][1]
    # On lance une erreur si valeur hors plages
    error = f'Invalid value, got {v}, min is {min_price}, max is {prices[-1][0]}'
    raise ValueError(error)

# On teste un peu cette jolie fonction
for engine_displacement in (100_564, 564, 125, -56):
    print(engine_displacement, end=': ')
    try:
        print(get_price(engine_displacement))
    except ValueError as er:
        print(er)

Bon, c'est sans doute un peu compliqué si tu es totalement débutant, mais pas tant que ça.

Optimisation

1 réponse

Discussions similaires