Forum EduPython

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