Discussion :

[Amnael]

Bonsoir à tous,

je suis en train de debug le programme de quelqun, et je suis tombé sur une condition qui m'intrigue.

Ne connaissant pas vraiment python, je voulais avoir votre avis là dessus.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
if int(typo_d) < 3:
                    return self.generate_random_destination(typo_d, dist_factor * 1.2)
                elif typo_d < 12:
 
                    return self.generate_random_destination(typo_d - 1, 1)
                else:
                    return False

Alors, je ne suis pas habitué à ce langage du tout, mais quand je vois une condition avec un cast et l'autre sans cast, je ne peux pas m'empêcher de penser que ce n'est pas normal.

Soit typo_d est un string et il faut un cast, soit c'est déjà un int et dans ce cas le cast du premier est inutile...Si effectivement il s'agit d'un string, je pense que je ne devrais jamais entrer dans la condition du elif...

Bref, j'aimerais bien l'avis de quelqun plus calé que moi dans ce langage^^

[cervo]

Bingo !...
çà saute aux yeux juste après la première instruction que typo_d est d'un type différent d'un int (du moins en enlevant la possibilité que le dev veuille caster un int en int ! ) donc il va pas tester le elif ... (Notons que si la clause [b]elif[/] etait correcte il allait à la fois exécuter la boucle if et elif)
Débogage partiel:
1 - Indentation non respectée,
2 - variable non castée avant verification dans les boucles,
3 - condition trouble ( , probable repetition d'instruction)

[Amnael]

Pour l'indentation c'est à cause du copié collé =)

Sinon, en fait, j'ai fais une trace avec affichage et il entre bien dans les deux conditions,...ce qui veut dire que dans la première il cast un int en int....

Je comprends pas certaines personnes vraiment --'

[Laurent 1973]

En effet, un souci de cast.

@cervo: je ne suis pas d'accord avec toi sur ta supposition de "condition trouble"

Déjà, un "elif" ne se fait que si le premier "if" et faut, donc dans ce cas, il faudrait que typo_d soit dans [3,12[

Et même si on avait:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
if typo_d < 3:
    return self.generate_random_destination(typo_d, dist_factor * 1.2)
if typo_d < 12:
    return self.generate_random_destination(typo_d - 1, 1)
return False

L'algorithme serait toujours équivalent vu que nous avons un "return" qui interrompe la suite de l’exécution.