Python et PEP8

vv222 · 09-02-2024 16:15:43

Ce message était à l’origine posté en réponse à celui-ci : Aide Python : conversion de python2 vers python3 pour un script DFiso.

---

Elzen a écrit :

le code n'a pas trop l'air de respecter PEP8, perso ça ne me dérange pas outre mesure, mais de vrais devs Python pourraient râler

La PEP8 est naze, et les vrais devs Python aussi

Elzen · 09-02-2024 16:53:17

Y a quand même quelques trucs intéressants dedans. Du style, espacer un peu les paramètres lors de l'appel à une fonction ou de sa définition pour aérer un peu, ou essayer d'éviter les lignes de 10km de long !

vv222 · 09-02-2024 17:27:51

Oh, pour ça j’ai tendance à séparer les paramètres par des sauts de ligne plutôt que simplement des espaces. Pas besoin d’une norme soi-disant-optionnelle-mais-on-t’insulte-si-tu-ne-la-suis-pas pour aérer le code intelligemment.

Philou92 · 09-02-2024 21:13:17

Pour la mise en forme j’utilise « black » : https://packages.debian.org/stable/python/black

black monscript.py

et hop ! Plus nécessaire de se fatiguer tout mon code devient « devs naze PEP 8 compliant »

vv222 · 09-02-2024 21:34:51

black est probablement un des outils qui représente le mieux cette hypocrisie des normes soit-disant optionnelles mais qui sont imposées en pratique. Jamais on ne me verra l’utiliser de ma propre initiative (si on me paie suffisamment cher, je veux bien transiger).

arpinux · 09-02-2024 22:59:09

salut et merci @Elzen !

ça fonctionne en mettant à jour les appels aux différents thèmes installés
j'ai supprimé l'appel au wall aussi.
bref, je teste les différents thèmes possibles en modifiant le script au fur et à mesure maintenant que j'ai capté la structure.

je reviens avec le script à tester sur une DFiso dans la soirée

... sur le bon fil de discussion

Dernière modification par arpinux (09-02-2024 23:39:50)

MPython Alaplancha · 10-02-2024 09:16:07

Bonjour,

                        #La PEP8 est naze

TEXTduLabelDuLAYOUTconcernantLaMiseENPage_1         ="Hello"

def CHANGELENOMDUTEXTEDULABELDULAYOUTCONCERNANTLAMISEENPAGE_1(Variabledutextdulabelconcernantlamiseenpage_1):

                                    import     time

                                    for compte_un_deux_trois in range(3)         :

                                                                        TEXTduLabelDuLAYOUTconcernantLaMiseENPage_1=   Variabledutextdulabelconcernantlamiseenpage_1

                                                                        print(              TEXTduLabelDuLAYOUTconcernantLaMiseENPage_1        )

                                                                        time.sleep(                  1

                                                                                                     )

CHANGELENOMDUTEXTEDULABELDULAYOUTCONCERNANTLAMISEENPAGE_1(                  'MDR'

                                                                            )

"""cette fonct fé ceci mé

pas cela.voir le d

étail dans la doc"""

Philou92 · 10-02-2024 17:30:41

J’ai suivi la recommandation de vv222, j’ai ajouté un alias dans mon fichier .bash_aliases :

alias hypocrisie='black'

Un petit coup d’hypocrisie et hop!

hypocrisie LA_PEP8_est_naze.py

# La PEP8 est naze

TEXTduLabelDuLAYOUTconcernantLaMiseENPage_1 = "Hello"

def CHANGELENOMDUTEXTEDULABELDULAYOUTCONCERNANTLAMISEENPAGE_1(

    Variabledutextdulabelconcernantlamiseenpage_1,

):

    import time

    for compte_un_deux_trois in range(3):

        TEXTduLabelDuLAYOUTconcernantLaMiseENPage_1 = (

            Variabledutextdulabelconcernantlamiseenpage_1

        )

        print(TEXTduLabelDuLAYOUTconcernantLaMiseENPage_1)

        time.sleep(1)

CHANGELENOMDUTEXTEDULABELDULAYOUTCONCERNANTLAMISEENPAGE_1("MDR")

"""cette fonct fé ceci mé

pas cela.voir le d

étail dans la doc"""

On y voit plus clair même si ce n’est pas miraculeux. Exeunt le style de nommage des variables et fonction, la position du docstring, etc.

Dernière modification par Philou92 (10-02-2024 17:32:28)

Anonyme · 10-02-2024 18:09:15

Vous me faite rire avec le formatage des sources !
Il y a bien longtemps j'avais développé du code en pascal, surtout chez moi par plaisir et un peu au bureau pour déverminage.
Comme il y avait des heures payées, ma boite a exigé les sources. Très fâché, grrr...
Donc je me suis écrit un soft pourrisseur de sources, le texte était compacté au maxi, structure quasi invisible, commentaires supprimés, variables et fonctions renommées mais bien sûr tout restait compilable sans la moindre erreur. Cette version de code source n'a certainement jamais resservi

.
Puis j'ai aussi écrit un "enjoliveur" de code, c'était plus facile, moins drôle mais ça faisait joli.

MPython Alaplancha · 11-02-2024 17:18:50

Philou92 a écrit :

On y voit plus clair même si ce n’est pas miraculeux.

En effet pas de miracle en vue
Il est préférable de passer par un vérificateur de code comme flake8, ou pylint (si son IDE n'en dispose pas ).
Si je passe le code LA_PEP8_est_naze.py (l'original) à la moulinette de pylint voici le retour:

fab@fab-Standard-PC-Q35-ICH9-2009:~$ pylint LA_PEP8_est_naze.py 

************* Module LA_PEP8_est_naze

LA_PEP8_est_naze.py:3:0: C0301: Line too long (109/100) (line-too-long)

LA_PEP8_est_naze.py:4:0: W0311: Bad indentation. Found 36 spaces, expected 4 (bad-indentation)

LA_PEP8_est_naze.py:5:0: W0311: Bad indentation. Found 36 spaces, expected 4 (bad-indentation)

LA_PEP8_est_naze.py:6:0: C0301: Line too long (164/100) (line-too-long)

LA_PEP8_est_naze.py:6:0: W0311: Bad indentation. Found 72 spaces, expected 8 (bad-indentation)

LA_PEP8_est_naze.py:7:0: C0301: Line too long (144/100) (line-too-long)

LA_PEP8_est_naze.py:7:0: W0311: Bad indentation. Found 72 spaces, expected 8 (bad-indentation)

LA_PEP8_est_naze.py:8:0: C0301: Line too long (102/100) (line-too-long)

LA_PEP8_est_naze.py:8:0: W0311: Bad indentation. Found 72 spaces, expected 8 (bad-indentation)

LA_PEP8_est_naze.py:9:0: C0301: Line too long (102/100) (line-too-long)

LA_PEP8_est_naze.py:1:0: C0114: Missing module docstring (missing-module-docstring)

LA_PEP8_est_naze.py:1:0: C0103: Module name "LA_PEP8_est_naze" doesn't conform to snake_case naming style (invalid-name)

LA_PEP8_est_naze.py:2:0: C0103: Constant name "TEXTduLabelDuLAYOUTconcernantLaMiseENPage_1" doesn't conform to UPPER_CASE naming style (invalid-name)

LA_PEP8_est_naze.py:3:0: C0116: Missing function or method docstring (missing-function-docstring)

LA_PEP8_est_naze.py:3:0: C0103: Function name "CHANGELENOMDUTEXTEDULABELDULAYOUTCONCERNANTLAMISEENPAGE_1" doesn't conform to snake_case naming style (invalid-name)

LA_PEP8_est_naze.py:3:62: C0103: Argument name "Variabledutextdulabelconcernantlamiseenpage_1" doesn't conform to snake_case naming style (invalid-name)

LA_PEP8_est_naze.py:6:72: W0621: Redefining name 'TEXTduLabelDuLAYOUTconcernantLaMiseENPage_1' from outer scope (line 2) (redefined-outer-name)

LA_PEP8_est_naze.py:4:36: C0415: Import outside toplevel (time) (import-outside-toplevel)

LA_PEP8_est_naze.py:6:72: C0103: Variable name "TEXTduLabelDuLAYOUTconcernantLaMiseENPage_1" doesn't conform to snake_case naming style (invalid-name)

LA_PEP8_est_naze.py:5:40: W0612: Unused variable 'compte_un_deux_trois' (unused-variable)

LA_PEP8_est_naze.py:14:0: W0105: String statement has no effect (pointless-string-statement)

-----------------------------------

Your code has been rated at 0.00/10

fab@fab-Standard-PC-Q35-ICH9-2009:~$

Comme on peut le voir le code est noté 0/10. Le but est d'avoir après correction une note proche du 10/10 (pylint est particulièrement pointilleux, trop peut être)

Le code corrigé par 'black' est aussi noté 0/10...

vv222 · 11-02-2024 17:40:22

MPython Alaplancha a écrit :

Le but est d'avoir après correction une note proche du 10/10

Ça dépend pour quoi, moi par exemple je me fous complètement des signalements du type "Line too long" ou "Bad indentation". Tout simplement parce que je considère (contrairement aux concepteurs de la PEP8 ou de ceux de black) que le développeur est suffisamment intelligent pour faire les meilleurs choix.

MPython Alaplancha · 11-02-2024 18:30:41

Libre à toi de respecter ou non la PEP8, l'interpréteur python ne t'y contraindra pas.
La raison d'établir des conventions de codage, d'établir 'les bonnes pratiques', facilite surtout la collaboration de plusieurs sur personnes (dev) sur un projet. ça facilite aussi la maintenance du code.( Si Y doit maintenir le code de X, mais que X à ses propres conventions de codage, ça peut devenir vite galère...) .
Après tu peux déplorer certains choix de la PEP, n'oublie pas cependant que ces choix ne sont pas le fruit d'une personne, mais du fruit de la collaboration de la communauté de python.

Dernière modification par MPython Alaplancha (11-02-2024 20:07:05)

vv222 · 11-02-2024 21:33:20

MPython Alaplancha a écrit :

Libre à toi de respecter ou non la PEP8, l'interpréteur python ne t'y contraindra pas.

Justement, c’est ce que je trouve hypocrite : l’interpréteur te laisse faire ce que tu veux, mais la totalité de la communauté Python va te tomber dessus si tu oses ne pas suivre la PEP8 pour un logiciel que tu partages publiquement. Quand une recommandation est imposée via ce genre de pression, je ne pense pas que le terme "recommandation" reste approprié.

MPython Alaplancha a écrit :

Après tu peux déplorer certains choix de la PEP, n'oublie pas cependant que ces choix ne sont pas le fruit d'une personne, mais du fruit de la collaboration de la communauté de python.

C’est bien pour ça que je ne suis pas fâché contre une personne en particulier, mais contre l’ensemble de la communauté Python

MPython Alaplancha · 12-02-2024 13:36:14

vv222 a écrit :

mais la totalité de la communauté Python va te tomber dessus si tu oses ne pas suivre la PEP8 pour un logiciel que tu partages publiquement. Quand une recommandation est imposée via ce genre de pression, je ne pense pas que le terme "recommandation" reste approprié.

Nous avons une litanie, un genre de mantra que nous récitons chaque matin lors de nos ablutions:

>>> import this

The Zen of Python, by Tim Peters

Beautiful is better than ugly.

Explicit is better than implicit.

Simple is better than complex.

Complex is better than complicated.

Flat is better than nested.

Sparse is better than dense.

Readability counts.

Special cases aren't special enough to break the rules.

Although practicality beats purity.

Errors should never pass silently.

Unless explicitly silenced.

In the face of ambiguity, refuse the temptation to guess.

There should be one-- and preferably only one --obvious way to do it.

Although that way may not be obvious at first unless you're Dutch.

Now is better than never.

Although never is often better than *right* now.

If the implementation is hard to explain, it's a bad idea.

If the implementation is easy to explain, it may be a good idea.

Namespaces are one honking great idea -- let's do more of those!

>>>

traduction:

>>> import this
Le Zen de Python, par Tim Peters

Le beau est mieux que le laid.
L'explicite vaut mieux que l'implicite.
La simplicité est préférable à la complexité.
Complexe vaut mieux que compliqué.
Plat vaut mieux qu'imbriqué.
Mieux vaut une structure éparse qu'une structure dense.
La lisibilité compte.
Les cas particuliers ne sont pas assez spéciaux pour enfreindre les règles.
Même si l'aspect pratique l'emporte sur la pureté.
Les erreurs ne devraient jamais être passées sous silence.
À moins qu'elles ne soient explicitement réduites au silence.
En cas d'ambiguïté, refusez la tentation de deviner.
Il doit y avoir une - et de préférence une seule - façon évidente de procéder.
Même si cette façon n'est pas évidente au premier abord, à moins d'être hollandais.
Mieux vaut maintenant que jamais.
Bien que jamais soit souvent mieux que *tout de suite*.
Si la mise en œuvre est difficile à expliquer, c'est une mauvaise idée.
Si l'implémentation est facile à expliquer, c'est peut-être une bonne idée.
Les espaces de noms sont une très bonne idée - faisons-en plus !
>>>

Traduit avec DeepL.com (version gratuite)

Ceci explique peut être cela

C’est bien pour ça que je ne suis pas fâché contre une personne en particulier, mais contre l’ensemble de la communauté Python

Tous contre un , un contre tous!

Debian-facile

#1 09-02-2024 16:15:43

Python et PEP8

#2 09-02-2024 16:53:17

Re : Python et PEP8

#3 09-02-2024 17:27:51

Re : Python et PEP8

#4 09-02-2024 21:13:17

Re : Python et PEP8

#5 09-02-2024 21:34:51

Re : Python et PEP8

#6 09-02-2024 22:59:09

Re : Python et PEP8

#7 10-02-2024 09:16:07

Re : Python et PEP8

#8 10-02-2024 17:30:41

Re : Python et PEP8

#9 10-02-2024 18:09:15

Re : Python et PEP8

#10 11-02-2024 17:18:50

Re : Python et PEP8

#11 11-02-2024 17:40:22

Re : Python et PEP8

#12 11-02-2024 18:30:41

Re : Python et PEP8

#13 11-02-2024 21:33:20

Re : Python et PEP8

#14 12-02-2024 13:36:14

Re : Python et PEP8

Pied de page des forums