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 22:38:02 +0000
Lines: 47
Message-ID: <7255f3485a4745df69dc13cedf68e8b0@posteo.de>
References: <tm2jhn$1k5h8$1@news1.tnib.de> <480b80d21e58d280c437b4854bfc7cf1@posteo.de> <tm372k$1lddf$1@news1.tnib.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: quoted-printable
Mail-Followup-To: python-de@python.org
In-Reply-To: <tm372k$1lddf$1@news1.tnib.de>
Message-ID-Hash: NPC3AW7OJ2ZJCGRCFP7QON4MQWJNUE3B
Precedence: list
Archived-At: <https://mail.python.org/archives/list/python-de@python.org/message/NPC3AW7OJ2ZJCGRCFP7QON4MQWJNUE3B/>
Xref: csiph.com de.comp.lang.python:5869

Am 28.11.2022 21:52 schrieb Marc Haber:
> 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=C3=BCsselwort
> benutzt wurde ergibt das f=C3=BCr mich keinen Sinn

Auch wenn es radikal klingen mag; Es hat keinen Sinn "global" zu=20
verwenden, auch wenn Python das im Sprachumfang hat. Ein "global" hat=20
sehr viele Nebeneffekte und birgt potentielle Risiken, die den Gewinn=20
nicht aufwiegen. Es gibt andere L=C3=B6sungen.

> Gerade f=C3=BCr die Konfiguration halte ich die globale Variable auch f=
=C3=BCr
> den vern=C3=BCnftigen Weg, das wirklich =C3=BCberall bekannt zu machen.

Nein, ist es nicht. Das ist nur eine "Abk=C3=BCrzung". Besser w=C3=A4re hie=
r eine=20
Art Singleton-Pattern anzuwenden.
Sagen wir deine Konfiguration ist in der Klasse "Config".

class Config:
     @classmethod
     def instance(cls):
         # Provide the instance if it exists
         if cls._instance:
             return cls._instance

         # But don't created implicit when needed.
         raise Exception(f'No instance of class "{cls}" exists. '
                         'Create an instance first.')

     def __init__(self):
         # Exception when an instance exists
         if __class__._instance:
             raise Exception(
                 f'Instance of class "{self.__class__.__name__}" still=20
exists!'
                 f' Use "{self.__class__.__name__}.instance()" to access=20
it.')

         # Remember the instance as the one and only singleton
         __class__._instance =3D self

Quelle:=20
https://github.com/bit-team/backintime/blob/buhtz_config-singleton/common/c=
onfig.py