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

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

Re: [Python-de] Re: Code Style Review

Show all headers | View raw

c.buhtz@posteo.jp wrote:
>Neben PyLint könntest du auch nochmal "flake8" auf den Code loslassen.

Das war eine gute Anregung, vim-ale hat das direkt aufgegriffen und
noch ein paar Anregungen im Code hinterlassen.

>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.

Das sehe ich mindestens teilweise anders. Wenn ich irgendwo explizit
hinschreibe, dass eine Funktion eine globale Variable verwendet (wie
z.B. das dictionary, in dem die Konfiguration drin steht) und der
Linter das unkonditional anmeckert nur allein weil das Schlüsselwort
benutzt wurde ergibt das für mich keinen Sinn: Wenn ich das nicht
gewollt hätte hätte ich es nicht hingeschrieben.

Gerade für die Konfiguration halte ich die globale Variable auch für
den vernünftigen Weg, das wirklich überall bekannt zu machen. Die
Alternative, jeder Funktion das Konfig-Dictionary als Parameter
einzufüllen, finde ich noch schlechter lesbar. Wie ist in Python das
normale Idiom für sowas?

Und das hier:
|debug = 0
|if args.debug:
|    debug = 1
zu flaggen mit "ey, debug ist eine Konstante, schreib die gefälligst
groß" finde ich einen echten Bug. Dass das keine Konstante ist, ergibt
sich doch direkt aus dem Code.

>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.

Ja, das ist eine Geschmackssache, ich habe lieber mehr Informationen
auf dem immer zu kleinen Bildschirm. Aber das mag sich ändern wenn ich
mich in der Sprache ein bissche mehr daheim fühle.

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

Dafür habe ich noch nicht genug python-Auge um zu sehen dass das alles
zusammen gehört (meinen Augen fehlt die schließende Klammer, aber das
ist halt so). Vermutlich stimme ich Dir da in einem halben Jahr zu.

Grüße
Marc
-- 
-------------------------------------- !! No courtesy copies, please !! -----
Marc Haber         |   " Questions are the         | Mailadresse im Header
Mannheim, Germany  |     Beginning of Wisdom "     | 
Nordisch by Nature | Lt. Worf, TNG "Rightful Heir" | Fon: *49 621 72739834

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