Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > de.comp.lang.python > #5864

[Python-de] Re: Code Style Review

Path csiph.com!fu-berlin.de!uni-berlin.de!not-for-mail
From c.buhtz@posteo.jp
Newsgroups de.comp.lang.python
Subject [Python-de] Re: Code Style Review
Date Mon, 28 Nov 2022 15:37:34 +0000
Lines 52
Message-ID <480b80d21e58d280c437b4854bfc7cf1@posteo.de> (permalink)
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
X-Trace news.uni-berlin.de WXoS+T65esWLNvalHOIOYwyoM5JBAcOjBTyW0JMqH5Jg==
Authentication-Results mail.python.org; dkim=pass reason="2048-bit key; unprotected key" header.d=posteo.jp header.i=@posteo.jp header.b=B9goC/cj; dkim-adsp=pass; dkim-atps=neutral
DKIM-Signature v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.jp; s=2017; t=1669649855; bh=BVu/DPOokGyZIUia+7VZ+27i1CDdezHj+PhkrgnG7Ek=; h=Date:From:To:Subject:From; b=B9goC/cji9Vw5oWDitwBh3TvI2m3ZQgXzKy/tcxEtsAOrIL48Xtgfp6xTJCllnm0+ /AaRMXyLXfVcYIl9nzcE81/U3BJNJj3xov3C5jM6Cdt9x+HKjPXbzAbuP+0nIb29n7 DVdWE+EIxD55rwXWf1xXuXONb5Jg48/kbFqclafH8CxYQPkc0JUAco8wfgAISrVhtl BtcbcQR3O2JuCE6VrmljCXFrA1bDyABzZNj/v7BMn91VRg7W5pStZeaddS6YnC4Gpw dHS26t2OxVCkoR26e5aSKTul23yEb5kxo1zVTgdYYow01ULPzOKloxwMCcnFTSr0H/ SUpim+rGjuAng==
In-Reply-To <tm2jhn$1k5h8$1@news1.tnib.de>
Message-ID-Hash OM7HM6ZEUDLENJD7XL266QHP67RTRLG5
X-Message-ID-Hash OM7HM6ZEUDLENJD7XL266QHP67RTRLG5
X-MailFrom c.buhtz@posteo.jp
X-Mailman-Rule-Misses dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-python-de.python.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header
X-Mailman-Version 3.3.7
Precedence list
List-Id Die Deutsche Python Mailingliste <python-de.python.org>
Archived-At <https://mail.python.org/archives/list/python-de@python.org/message/OM7HM6ZEUDLENJD7XL266QHP67RTRLG5/>
List-Archive <https://mail.python.org/archives/list/python-de@python.org/>
List-Help <mailto:python-de-request@python.org?subject=help>
List-Owner <mailto:python-de-owner@python.org>
List-Post <mailto:python-de@python.org>
List-Subscribe <mailto:python-de-join@python.org>
List-Unsubscribe <mailto:python-de-leave@python.org>
Xref csiph.com de.comp.lang.python:5864

Show key headers only | View raw

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()

Back to de.comp.lang.python | Previous | Next — Previous in thread | Next in thread | Find similar

Thread

Code Style Review Marc Haber <mh+usenetspam1118@zugschl.us> - 2022-11-28 16:19 +0100
  [Python-de] Re: Code Style Review c.buhtz@posteo.jp - 2022-11-28 15:37 +0000
    [Python-de] Re: Code Style Review Christopher Arndt <chris@chrisarndt.de> - 2022-11-28 16:48 +0100
      [Python-de] Re: Code Style Review c.buhtz@posteo.jp - 2022-11-28 19:53 +0000
      Re: [Python-de] Re: Code Style Review Marc Haber <mh+usenetspam1118@zugschl.us> - 2022-11-28 22:07 +0100
        [Python-de] Re: Code Style Review Stefan Schwarzer <sschwarzer@sschwarzer.net> - 2022-11-29 18:51 +0100
          [Python-de] Re: Code Style Review Marc Haber <mh+python-de@zugschlus.de> - 2022-11-29 21:44 +0100
            [Python-de] Re: Code Style Review Stefan Schwarzer <sschwarzer@sschwarzer.net> - 2022-11-29 23:16 +0100
    Re: [Python-de] Re: Code Style Review Marc Haber <mh+usenetspam1118@zugschl.us> - 2022-11-28 21:52 +0100
      [Python-de] Re: Code Style Review c.buhtz@posteo.jp - 2022-11-28 22:38 +0000
        Re: [Python-de] Re: Code Style Review Enrik Berkhan <Enrik.Berkhan@inka.de> - 2022-11-29 06:03 +0000
        Re: [Python-de] Re: Code Style Review Marc Haber <mh+usenetspam1118@zugschl.us> - 2022-11-29 15:22 +0100
          [Python-de] Re: Code Style Review c.buhtz@posteo.jp - 2022-11-29 14:39 +0000
      [Python-de] Re: Code Style Review Matthias Urlichs <matthias.urlichs@noris.de> - 2022-11-30 10:47 +0000
        [Python-de] Re: Code Style Review Marc Haber <mh+python-de@zugschlus.de> - 2022-12-01 21:30 +0100
          [Python-de] Re: Code Style Review Stefan Schwarzer <sschwarzer@sschwarzer.net> - 2022-12-02 00:07 +0100
            Re: [Python-de] Re: Code Style Review "Peter J. Holzer" <hjp-usenet3@hjp.at> - 2022-12-03 14:32 +0100
          [Python-de] Re: Code Style Review Matthias Urlichs <matthias.urlichs@noris.de> - 2022-12-02 09:29 +0000
            [Python-de] Re: Code Style Review Marc Haber <mh+python-de@zugschlus.de> - 2022-12-02 18:02 +0100
    [Python-de] Re: Code Style Review Stefan Schwarzer <sschwarzer@sschwarzer.net> - 2022-11-29 18:33 +0100
  [Python-de] Re: Code Style Review Stefan Schwarzer <sschwarzer@sschwarzer.net> - 2022-11-29 18:18 +0100
    [Python-de] Re: Code Style Review Hans-Peter Jansen <hpj@urpla.net> - 2022-11-30 15:52 +0100
    [Python-de] Re: Code Style Review Marc Haber <mh+python-de@zugschlus.de> - 2022-12-01 21:16 +0100
      [Python-de] Re: Code Style Review Stefan Schwarzer <sschwarzer@sschwarzer.net> - 2022-12-01 23:58 +0100
        [Python-de] Re: Code Style Review Marc Haber <mh+python-de@zugschlus.de> - 2022-12-02 18:10 +0100
          Re: [Python-de] Re: Code Style Review "Peter J. Holzer" <hjp-usenet3@hjp.at> - 2022-12-03 14:36 +0100

csiph-web