Comments on: Le PEP8, en résumé http://sametmax.com/le-pep8-en-resume/ Du code, du cul Sat, 07 Nov 2015 11:08:18 +0000 hourly 1 http://wordpress.org/?v=4.1 By: Biganon http://sametmax.com/le-pep8-en-resume/#comment-113496 Tue, 12 Aug 2014 19:25:14 +0000 http://sametmax.com/?p=8514#comment-113496 Ce que tu appelles “camel case”, c’est pas plutôt du Pascal case ? Sauf erreur, le camel case commence par une minuscule, en tout cas par convention c’est ce qu’on désigne par camel case en informatique.

]]> By: Sam http://sametmax.com/le-pep8-en-resume/#comment-20677 Mon, 03 Mar 2014 14:08:58 +0000 http://sametmax.com/?p=8514#comment-20677 Merci Marien. Pour les commentaires, je ne respecte pas cette convention, donc je ne vais pas la recommander. Pour le from, je corrige tout de suite.

]]> By: Marien http://sametmax.com/le-pep8-en-resume/#comment-20670 Mon, 03 Mar 2014 10:40:48 +0000 http://sametmax.com/?p=8514#comment-20670 Comme je reviens sur ce formidable article et que je compare en même temps avec le document “officiel”, je viens corriger deux petits trucs (je sais pas si ce sera pris en compte mais au moins ce sera là) :

– Les commentaires inline doivent être séparés par deux espaces du code et suivi d’un seul espace (c’est le cas dans certains exemples mais pas toujours)
– Le from collections on namedtuple c’est fait exprès ? Parce que c’était bon dans l’article stocké par mon agrégateur de flux RSS ce qui veut dire que ça a été changé après la publication de l’article (en fait je crois que vous n’avez pas corrigé le bon “import”)

J’en avais d’autres mais ça a été corrigé depuis le temps :)

]]> By: kontre http://sametmax.com/le-pep8-en-resume/#comment-17979 Wed, 08 Jan 2014 13:14:39 +0000 http://sametmax.com/?p=8514#comment-17979 @Tony: souvent les éditeurs utilisent déjà pep8/flake8 pour afficher les warnings, et c’est vrai que c’est super pratique.
@herison: pylint est trop lent pour fonctionner en temps réel contrairement à pep8. Il faut aussi plus de trucs (avec parfois un tas de faux positifs quand il ne trouve pas une librairie).

Il y a l’utilitaire autopep8 qui permet de corriger automatiquement une bonne partie des manquements au pep8.

]]> By: herison http://sametmax.com/le-pep8-en-resume/#comment-17843 Thu, 02 Jan 2014 12:46:55 +0000 http://sametmax.com/?p=8514#comment-17843 Il y a pep8 qui est un outil pour voir si le code répond à la PEP8, mais j’utilise pylint qui est moins sensible à la PEP8 et affiche d’autre erreur comme les variables non utilisées ou les partie de code demandant un refactoring.

]]> By: Tony http://sametmax.com/le-pep8-en-resume/#comment-17840 Thu, 02 Jan 2014 11:50:17 +0000 http://sametmax.com/?p=8514#comment-17840 Certains editeurs integrent des controles pendant l’édition. Mais c’est souvent pas super parametrable ou deconnant. L’utilisation d’outil comme flake8/pep8 est donc de toutes façons fortement recommandées.

]]> By: Sam http://sametmax.com/le-pep8-en-resume/#comment-17760 Mon, 30 Dec 2013 12:28:36 +0000 http://sametmax.com/?p=8514#comment-17760 C’était pour voir si vous suiviez bien ^^

]]> By: JeromeJ http://sametmax.com/le-pep8-en-resume/#comment-17756 Mon, 30 Dec 2013 10:18:54 +0000 http://sametmax.com/?p=8514#comment-17756 Dans l’exemple correct “{str(x) : x for x in range(10)}” tu as laissé l’espace avant les deux points :(

]]> By: Sam http://sametmax.com/le-pep8-en-resume/#comment-17597 Fri, 27 Dec 2013 17:34:39 +0000 http://sametmax.com/?p=8514#comment-17597 Tu veux dire pour l’indentation ?

]]> By: roro http://sametmax.com/le-pep8-en-resume/#comment-17595 Fri, 27 Dec 2013 16:46:52 +0000 http://sametmax.com/?p=8514#comment-17595 Les espaces, c’est bien à la lecture (encore que quoi queue…)
Mais ça fait chier à la rédaction.

]]>