PWM Webrowser Pumpenmonitor / Steuerung

[Schorsch Z. (schorsch_z)]

Falls jemand auch eine oder mehrere PWM Pumpen hat, hier mein neues 
kleines Projekt:

[Mod: Code siehe weiter unten]

[N. M. (mani)]

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

[Schorsch Z. (schorsch_z)]

Nun als File, wegen Formatierung

[Schorsch Z. (schorsch_z)]

Habe es in github eingestellt, das ist übersichtlicher:

https://github.com/gerontec/pwm_wilo

[Obelix X. (obelix)]

RTFM!

Wichtige Regeln - erst lesen, dann posten!

    Groß- und Kleinschreibung verwenden
    Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang

[Ein T. (ein_typ)]

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.