Parsing gehört in die Main-Funktion

[penguineer]

ble-spaceauth-manager/ble-spaceauth-manager.py

Line 22 in 6f1cfad

parser = argparse.ArgumentParser(description='BLE Space Authentication Helper script.')

Ich würde den Parser mit in die Main-Funktion (siehe #1) schreiben.

[maz3max]

Die geparsten Argumente sind global, deswegen habe ich den Parser jetzt lediglich in den main-Bereich, aber außerhalb der neuen main-Funktion gesetzt.

[penguineer]

Ah, das sehe ich gerade erst. Damit übergibst Du Argumente als globale Variablen → Code Smell.

Eine Funktion sollte nur über ihre Parameter oder die Klasseninstanz konfiguriert werden. Damit ist an der Funktionssignatur ersichtlich, welche Eingangsparameter es gibt und Tests werden erleichtert.

Die Argumente können zum Beispiel in ein Config-Objekt geschrieben werden.