Path: csiph.com!fu-berlin.de!uni-berlin.de!not-for-mail
From: Christopher Arndt <chris@chrisarndt.de>
Newsgroups: de.comp.lang.python
Subject: =?utf-8?q?=5BPython-de=5D_Re=3A_Code_Style_Review?=
Date: Mon, 28 Nov 2022 16:48:38 +0100
Lines: 36
Message-ID: <d60e87cd-51b9-084e-3fb6-d15b7588004d@chrisarndt.de>
References: <tm2jhn$1k5h8$1@news1.tnib.de> <480b80d21e58d280c437b4854bfc7cf1@posteo.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-US, de-DE
In-Reply-To: <480b80d21e58d280c437b4854bfc7cf1@posteo.de>
Message-ID-Hash: XSEUEJ6WCAFUATUUKOCY5J4L5VPAT3GG
Precedence: list
Archived-At: <https://mail.python.org/archives/list/python-de@python.org/message/XSEUEJ6WCAFUATUUKOCY5J4L5VPAT3GG/>
Xref: csiph.com de.comp.lang.python:5865

Am 28.11.22 um 16:37 schrieb c.buhtz@posteo.jp:
> 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