PWM Webrowser Pumpenmonitor / Steuerung

Als Anmerkung: Stell den Code als Datei hoch. Die Formatierung zerhaut alles. Und gerade bei Python wo es auf die Formatierung ankommt...

Nun als File, wegen Formatierung

Habe es in github eingestellt, das ist übersichtlicher:

https://github.com/gerontec/pwm_wilo

RTFM!

Wichtige Regeln - erst lesen, dann posten!

1

Groß- und Kleinschreibung verwenden

2

Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang

Schorsch Z. schrieb:

Habe es in github eingestellt, das ist übersichtlicher:

https://github.com/gerontec/pwm_wilo

Danke. Okay, ich habe den Code nur überflogen, aber bereits dabei sind mir einige Dinge aufgefallen. Kleinigkeiten, aber...

Zunächst fällt mir auf, daß dort keine "requirements.txt" vorhanden ist. Die Datei kannst Du in einem Virtual Environment (Modul "venv" oder "virtualenv") mit "pip freeze" erstellen, sie listet dann alle Module und Modulversionen, die Deine Software benutzt und die nicht zum Standardinstallation von Python gehören. Nutzer Deiner Software können diese Module dann ihrerseits in ihr Virtual Environment mit "pip install -r requirements.txt" installieren und so dieselbe Laufzeitumgebung installieren, die Du verwendet hast. Auch in Deinem PHP-Code könntest Du die Redis-Verbindung schön in einer Klasse kapseln.

Mir ist auch nicht ganz klar, warum Du "ujson" benutzt, da Python bereits standardmäßig ein Modul "json" mitbringt.

Du verwendest viele globale Variablen, das könntest Du mit objektorientierter Programmierung deutlich reduzieren, dadurch könntest Du etwa die Verbindungen zum MQTT-Broker, zum WLAN und zum Redis-Server hübsch verpacken -- und Dein Programm zudem "sprechender" und somit lesbarer machen...

Mir fehlt leider auch eine Lizenz zum Projekt, so daß andere es im Prinzip nicht benutzen können, auch wenn der Quellcode öffentlich ist.

Andererseits ist das ein schönes Projekt, viel Spaß und Erfolg dabei -- und Danke, daß Du Deinen Code auch anderen zur Verfügung stellst.