Closed cspaier closed 5 years ago
Je vois pas mal de fonction du type
C'est franchement inutile si on fait un import from math import *
non?
non si on met les noms en français Square Root qui devient sqrt avec les futurs STMG ? attention je vais forker !
je ferme car il me semble qu'on a fait le tour au moins sur la partie dév
ouais... Cf. programme 2019 : page 12
Il faut réécrire la librairie stats/proba
ouaip. Globalement il y a encore du travail. On y trouves des mauvaise pratiques comme:
fonctions acceptant un argument de type variable : https://github.com/cspaier/pydiderot/blob/52d60062e8b26c3efde62848e74f827a731430c9/Thonny/Lib/site-packages/stats_proba.py#L180-L191
fonctions retournant une valeur non typée: https://github.com/cspaier/pydiderot/blob/52d60062e8b26c3efde62848e74f827a731430c9/Thonny/Lib/site-packages/stats_proba.py#L469-L475
fonctions dont les paramètres influencent beaucoup trop la sortie (KISS): https://github.com/cspaier/pydiderot/blob/52d60062e8b26c3efde62848e74f827a731430c9/Thonny/Lib/site-packages/stats_proba.py#L248-L262
C'est moins grave mais ca me gêne et je pense que c'est considéré comme mauvaise pratique d'initialiser une variable de type int
avec le str
"optionel"
. Il est courant d'initialiser les variables à None
ce qui permet ensuite de tester simplement if var is None
.
on peut virer 'histop' et 'barre' :
On est bon la non?
on est pas mal.
On ferme alors!
La librairie lycee contient de nombreuses fonctions dont l'utilité n'est pas toujours claire.
L’intérêt de cette fonction par rapport à la fonction
range
de python ne m’apparait pas clairement par exemple: https://github.com/cspaier/thonny/blob/4de3ed26fd274d69d54d2a1d93f6fadc06a4896c/Thonny/Lib/site-packages/lycee.py#L298-L314Je pense qu'il faudra faire cela en même temps que la documentation. Fonction par fonction, évaluer l’intérêt. Si on la garde, on documente.