Forum EduPython

Echanges autour d'EduPython.

Vous n'êtes pas identifié(e).

#1 2019-10-08 14:49:52

ivkravchin
Membre
Inscription : 2019-10-08
Messages : 5

lycee.py

Chers Messieurs!
J'ai analysé votre bibliothèque lycee.py et j'ai été surpris de trouver plusieurs erreurs dans celle-ci.
De plus, ce programme ne répond pas aux exigences du PEP -8.
Le contrôle a été effectué par pylint
Veuillez respecter vos utilisateurs, même s'ils sont des enfants, et apportez le code du programme conformément aux normes acceptées dans la communauté Python.

error from pylint log:

lycee.py:817: error (E0602, undefined-variable, mediane) Undefined variable 'trier_liste'
lycee.py:858: error (E0602, undefined-variable, quartile) Undefined variable 'trier_liste'
lycee.py:878: error (E0602, undefined-variable, quartile) Undefined variable 'liste'
lycee.py:878: error (E0602, undefined-variable, quartile) Undefined variable 'liste'
lycee.py:879: error (E0602, undefined-variable, quartile) Undefined variable 'liste'
lycee.py:879: error (E0602, undefined-variable, quartile) Undefined variable 'liste'
lycee.py:905: error (E0602, undefined-variable, decile) Undefined variable 'trier_liste'
lycee.py:958: error (E0102, function-redefined, uniform) function already defined line 281

PEP-8 for examples:
lycee.py:242: convention (C0103, invalid-name, binomial) Argument name "n" doesn't conform to snake_case naming style
lycee.py:242: convention (C0103, invalid-name, binomial) Argument name "p" doesn't conform to snake_case naming style
lycee.py:248: refactor (R1705, no-else-return, binomial) Unnecessary "else" after "return"
lycee.py:256: warning (W0622, redefined-builtin, randint) Redefining built-in 'min'
lycee.py:256: warning (W0622, redefined-builtin, randint) Redefining built-in 'max'
lycee.py:265: warning (W0622, redefined-builtin, choice) Redefining built-in 'list'
lycee.py:281: warning (W0622, redefined-builtin, uniform) Redefining built-in 'min'
lycee.py:281: warning (W0622, redefined-builtin, uniform) Redefining built-in 'max'
lycee.py:299: refactor (R1705, no-else-return, intervalle) Unnecessary "else" after "return"
lycee.py:315: refactor (R1705, no-else-return, range) Unnecessary "else" after "return"
lycee.py:347: warning (W0702, bare-except, fich2chaine) No exception type(s) specified
lycee.py:349: convention (C0121, singleton-comparison, fich2chaine) Comparison to None should be 'expr is not None'
lycee.py:349: convention (C0321, multiple-statements, fich2chaine) More than one statement on a single line


Cordialement
Igor

Hors ligne

Pied de page des forums