Path: csiph.com!fu-berlin.de!uni-berlin.de!not-for-mail
From: c.buhtz@posteo.jp
Newsgroups: de.comp.lang.python
Subject: =?utf-8?q?=5BPython-de=5D_Re=3A_Code_Style_Review?=
Date: Mon, 28 Nov 2022 15:37:34 +0000
Lines: 52
Message-ID: <480b80d21e58d280c437b4854bfc7cf1@posteo.de>
References: <tm2jhn$1k5h8$1@news1.tnib.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: quoted-printable
In-Reply-To: <tm2jhn$1k5h8$1@news1.tnib.de>
Message-ID-Hash: OM7HM6ZEUDLENJD7XL266QHP67RTRLG5
Precedence: list
Archived-At: <https://mail.python.org/archives/list/python-de@python.org/message/OM7HM6ZEUDLENJD7XL266QHP67RTRLG5/>
Xref: csiph.com de.comp.lang.python:5864

Hallo Marc,

erst einmal finde ich es super, dass du dich so intensiv damit=20
auseinandersetzt.

Die Liste ist f=C3=BCr ein Code Review etwas ungeeignet. Vielleicht kannst =
du=20
den Code auf Codeberg.org oder wo anders hosten, dann l=C3=A4sst er sich=20
leichter kommentieren.

Neben PyLint k=C3=B6nntest du auch nochmal "flake8" auf den Code loslassen=
=2E

Die Frage ist, warum du solche PyLint Ausnahmen ("# pylint:") definiert=20
hast. IMHO ben=C3=B6tigt man daf=C3=BCr einen wirklich triftigen Grund; ein=
e=20
strenge Indikation. Die Hinweise und Meldungen von Lintern haben ihre=20
Berechtigung und sollten nicht unterdr=C3=BCck werden.

Neben den Lintern w=C3=BCrde ich dir noch empfehlen, ein paar mehr Leerzeil=
en=20
zu machen. Das erh=C3=B6ht IMHO die Lesbarkeit, gerade bei verschachtelten=
=20
Strukturen. Nat=C3=BCrlich ist das Geschmacksache bzw. eine Frage des=20
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()