[Python-de] Re: Code Style Review

Am 28.11.22 um 16:37 schrieb [email protected]:
> Neben PyLint könntest du auch nochmal "flake8" auf den Code loslassen.

Außerdem könntest du auch noch entweder "black" oder "yapf" oder ein 
ähnliches Tool benutzen, um den Code einheitlich und anhand üblicher 
Gepflogenheiten zu formatieren.

Weiterhin:

- Sortiere die Imports mit "isort"
- Strukturiere den Code in Sektionen. Vorschlag

   - Module docstring
   - Imports
   - Global constants and variables
   - Exceptions
   - Classes
   - Helper functions
   - main function

Momentan sind bei dir Code auf globaler Ebene und Funcktionen wild 
gemischt. Das ist ziemlich unübersichtlich und verleitet m.E. dazu, den 
angemessenen "Separation of concern" aus dem Auge zu verlieren. Auf 
globaler Ebene würde ich idR wirklich nur einige wichtige Konstanten 
definieren und den Rest in eine "main"-Funktion verschieben. Diese wird 
dann am Ende des Scripts aufgerufen mit:

if __name__ == '__main__':
     main()

Das zwingt dazu, sich überlegen zu müssen, welche Daten wirklich global 
sein müssen und wie ansonsten Daten zwischen Funktionen und Klassen 
weitergereicht werden.


Chris
_______________________________________________
python-de Mailingliste -- [email protected]
Zur Abmeldung von dieser Mailingliste senden Sie eine Nachricht an [email protected]
https://mail.python.org/mailman3/lists/python-de.python.org/
Mitgliedsadresse: [email protected]