|
Anzeige
|
PWM Webrowser Pumpenmonitor / SteuerungFalls jemand auch eine oder mehrere PWM Pumpen hat, hier mein neues kleines Projekt: [Mod: Code siehe weiter unten]
Angehängte Dateien:
:
Bearbeitet durch Moderator
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: RTFM! Wichtige Regeln - erst lesen, dann posten!
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. Antwort schreiben |
Anzeige
|
