[Python-de] Re: Code Style Review

Hallo Marc,

erst einmal finde ich es super, dass du dich so intensiv damit 
auseinandersetzt.

Die Liste ist für ein Code Review etwas ungeeignet. Vielleicht kannst du 
den Code auf Codeberg.org oder wo anders hosten, dann lässt er sich 
leichter kommentieren.

Neben PyLint könntest du auch nochmal "flake8" auf den Code loslassen.

Die Frage ist, warum du solche PyLint Ausnahmen ("# pylint:") definiert 
hast. IMHO benötigt man dafür einen wirklich triftigen Grund; eine 
strenge Indikation. Die Hinweise und Meldungen von Lintern haben ihre 
Berechtigung und sollten nicht unterdrück werden.

Neben den Lintern würde ich dir noch empfehlen, ein paar mehr Leerzeilen 
zu machen. Das erhöht IMHO die Lesbarkeit, gerade bei verschachtelten 
Strukturen. Natürlich ist das Geschmacksache bzw. eine Frage des 
Team-Konsent. Z.B. hier:

Am 28.11.2022 16:19 schrieb Marc Haber:

> while True:
>     time.sleep(30)
>     if debug > 0:
>         print('run evaluate() and cleanup_table() from main loop')
>     cleanup_table()
>     if debug > 0:
>         print_table()
>     evaluate()

while True:

     time.sleep(30)

     if debug > 0:
         print('run evaluate() and cleanup_table() from main loop')

     cleanup_table()

     if debug > 0:
         print_table()

     evaluate()


_______________________________________________
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]