Pen^2 (./3) :
privilégier la clause la plus courte avant le else pour la lisibilité.

Oui je suis entièrement d'accord, je me suis fait la même réflexion en collant le code ici ^^

Pen^2 (./3) :
utiliser, peut-être, un seul return, mais ça dépend un peu des cas, je ne suis pas totalement inflexible sur ce point

Du coup c'est presque contradictoire avec ta règle précédente je trouve. Pour la même raison que celle qui me ferait mettre la branche courte en premier, j'aime bien "fail early" et mettre des return assez tôt quand il n'est pas utile d'exécuter le corps de la fonction (en cas d'erreur par exemple).

Pen^2 (./5) :
Sinon, pour résumer je ne vois rien d'horrible mais vu que c'est très court c'est sûr que ça ne peut pas remettre en question l'architecture

Ah bah on verra, si on tourne en rond peut-être qu'il sera temps de poster des choses plus abstraites

Pen^2 (./5) :
C'est le genre de trucs qui pourrait peut-être me chagriner un peu. L'avantage d'utiliser les standards c'est que tout le monde peut s'y retrouver facilement sans temps de formation. Oui, c'est un peu une évidence, mais, j'ai déjà vu des gens qui réimplémentaient leur classe vector sans raison majeure, donc bon

Oui, ça aussi je suis d'accord, c'est pour ça que j'ai précisé pourquoi j'avais fait ça. J'aimerais bien avoir l'avis de quelqu'un de mieux habitué à Python (Flanker ?), mais autant je trouve l'essentiel de la bibliothèque standard facile à utiliser, autant le module "subprocess" me semble vraiment trop complexe même pour des cas d'utilisation simple.