NEWS
[gelöst] Elegantere Programmierung?
-
@skorpil sagte:
function thermometerSetzen(geraetKanal, modus) { sendTo('hm-rpc.0', 'putParamset', {ID: geraetKanal, paramType: 'MASTER', params: {'MODE_TEMPERATUR_REGULATOR': modus}}, res => { log(res); }); };
Du hast auch noch die alten HM-CC-TC?
Dann würde ich das Aufsplitten der ID auch in dieser Funktion machen.function thermometerSetzen(idKanal, modus) { let ID = idKanal.split('.'); ID = ID[2] + ':' + ID[3]; sendTo('hm-rpc.0', 'putParamset', {ID: ID, paramType: 'MASTER', params: {'MODE_TEMPERATUR_REGULATOR': modus}}, res => { if(res.error) log(res.error, 'warn'); }); };
-
@paul53 ja,habe ich. Macht Sinn!
PS: bin auch froh, dass ich die noch habe, denn die lassen sich Heizkörper unabhängig irgendwo im Raum platzieren und damit die Temperatur präzise erfassen
-
Jetzt habe ich mir ein kleines Programm zur Schaltung der Klimaanlage geschrieben:
// ########################################### // Klima Schlafzimmer Automatik // Deklarationen // ########################################### const IDthermSchlaf = 'hm-rpc.0.JEQ0710462.1.TEMPERATURE'/*Thermostat Elternschlaf 0 TEMPERATURE*/; const IDanwesend = 'hm-rega.0.39533'/*Anwesend*/; const IDklimaSchlafManuell = 'hm-rega.0.3116'/*Klima Schlaf manuell*/; const IDklimaSchlaf = 'hm-rega.0.46642'/*Klima Schlaf*/; const IDsommer = 'hm-rega.0.11457'/*SOMMER Hzg Wohnb ALLES (Var )*/; // ########################################### // Programm // Schaltung ein, wenn > 22 Grad oder < 21.5 // u. Anwesend u Manuell ein u. Sommer ein // ########################################### on(IDthermSchlaf, function () { // triggert bei Wertänderung eines DP if(getState(IDthermSchlaf).val > 22 && getState(IDanwesend).val == true && getState(IDklimaSchlafManuell).val == 1 && getState(IDsommer).val == 1) { setState(IDklimaSchlaf, true); }; // Ende IF if(getState(IDthermSchlaf).val < 21.5 && getState(IDanwesend).val == true && getState(IDklimaSchlafManuell).val == 1 && getState(IDsommer).val == 1) { setState(IDklimaSchlaf, false); }; // Ende IF }); // Ende ON
Sie soll ausgeschaltet werden, wenn die Temperatur unter 21,5 fällt und eingeschlalter werden, wenn die Temüeratur über 22 steigt. Das soll aber nur passieren, wenn die Schalter (hier: Systemvariablen) Sommer + anwesend + klimaschlafmanuell eingeschaltet sind.
Meine Frage: ist dass Programm mit der ON und anschließenden IF Anweisung so okay?
-
Ich habe noch einmal weiterüberlegt und programmiert. Führt folgendes Programm zum gleichen Ergebnis wie das vorherige?
// ########################################### // Klima Schlafzimmer Automatik // Deklarationen // ########################################### const IDthermSchlaf = 'hm-rpc.0.JEQ0710462.1.TEMPERATURE'/*Thermostat Elternschlaf 0 TEMPERATURE*/; const IDanwesend = 'hm-rega.0.39533'/*Anwesend*/; const IDklimaSchlafManuell = 'hm-rega.0.3116'/*Klima Schlaf manuell*/; const IDklimaSchlaf = 'hm-rega.0.46642'/*Klima Schlaf*/; const IDsommer = 'hm-rega.0.11457'/*SOMMER Hzg Wohnb ALLES (Var )*/; // ########################################### // Programm // Schaltung ein, wenn > 22 Grad oder < 21.5 // u. Anwesend u Manuell ein u. Sommer ein // ########################################### on(IDthermSchlaf, function () { // triggert bei Wertänderung eines DP const temp = getState(IDthermSchlaf).val; const anwesend = getState(IDanwesend).val; const manuell = getState(IDklimaSchlafManuell).val === 1; const sommer = getState(IDsommer).val === 1; setState(IDklimaSchlaf, temp > 22 && anwesend && manuell && sommer); setState(IDklimaSchlaf, temp < 21.5 && anwesend && manuell && sommer); });
Was sagst Du, @paul53
-
@skorpil sagte: Führt folgendes Programm zum gleichen Ergebnis wie das vorherige?
Nein, die letzte Bedingung siegt und es wird zweimal gesendet.
Besser so (Teil ohne Konstanten-Deklaration):var klima = getState(IDklimaSchlaf).val; // Einlesen bei Skriptstart wegen Hysterese on(IDthermSchlaf, function (dp) { // triggert bei Wertänderung eines DP if(getState(IDanwesend).val && getState(IDklimaSchlafManuell).val && getState(IDsommer).val) { if(dp.state.val > 22) klima = true; else if(dp.state.val < 21.5) klima = false; } else klima = false; // nur bei erforderlicher Wertänderung wird gesendet: if(klima != getState(IDklimaSchlaf).val) setState(IDklimaSchlaf, klima); }); // Ende ON
Die DP "Manuell" und "Sommer" nehmen nur die Werte 0 und 1 an?
-
-
@paul53 ist es eigentlich eine kluge Idee, ständig wiederkehrende Geräte, die man in verschiedenen Programmen nutzt, einmal quasi pauschal unter“global“ in einen Script zu definieren? Geht das überhaupt ?
Also zum Beispiel
const IDxxxxxxxxx = „hm-rpc.0.JEQ0710462.1.TEMPERATURE“ Usw.
-
@skorpil
Unter "global" würde ich nur häufig verwendete eigene Funktionen erstellen.
Für Datenpunkte würde ich Alias-DP verwenden, die schon in der ID selbsterklärend sind. -
@paul53 Ich habe mir da ein Script geschrieben, was ermitteln soll, ob die Terrassentuer länger als 10 minuten geöffnet ist. Und wenn das der Fall ist, soll eine SV entsprechend ausgelöst werden. Ist das so richtig?
// ########################################### // Lueften Terassentuer lang offen? // Deklarationen // ########################################### const IDtfkOptTerrasse = 'hm-rpc.1.0000DD89B24B3E.1.STATE'/*TFK opt Sensor Terrasse:1 STATE*/ const IDTerTuerSV = 'hm-rega.0.50484'/*TerTuer offen fuer lueften*/ const sommer = 'hm-rega.0.11457'/*SOMMER Hzg Wohnb ALLES (Var )*/; // log(getState(IDtfkOptTerrasse).val); // ########################################### // Programm // ########################################### // Prüfe, ob das Gerät "geöffnet" ist, indem der Status abgerufen wird on(IDtfkOptTerrasse, function () { // triggert bei Wertänderung eines DP if (getState(sommer).val == 0) { var status = getState(IDtfkOptTerrasse).val; if(status == 1) {// Wenn die Tür geöffnet ist, starte eine 10-minütige Verzögerung setTimeout(() => { // Überprüfe erneut, ob die Tür immer noch geöffnet ist var status = getState(IDtfkOptTerrasse).val; if (status) { // Wenn die Tür immer noch geöffnet ist // Setze die Zielvariable entsprechend setState(IDTerTuerSV, true, true); } else { // Wenn die Tür geschlossen ist // Setze die Zielvariable auf "falsch" setState(IDTerTuerSV, false, true); } }, 10 * 60 * 1000); // 10 Minuten in Millisekunden } else { // Wenn die Tür geschlossen ist // Setze die Zielvariable auf "falsch" setState(IDTerTuerSV, false, true); } }; });
-
@skorpil sagte: ob die Terrassentuer länger als 10 minuten geöffnet ist. Und wenn das der Fall ist, soll eine SV entsprechend ausgelöst werden. Ist das so richtig?
Wenn die Tür geschlossen wird, stoppt man den Timeout, wozu man eine Timer-Variable benötigt. Außerdem wird an die CCU nur gesendet (SV), wenn Ack nicht auf true gesetzt wird.
var timer = null; on(IDtfkOptTerrasse, function (dp) { // triggert bei Wertänderung des DP if (!getState(sommer).val) { if(dp.state.val) {// Wenn die Tür geöffnet wird, starte eine 10-minütige Verzögerung timer = setTimeout(() => { timer = null; // Wenn die Tür immer noch geöffnet ist, setze die Zielvariable entsprechend setState(IDTerTuerSV, true); }, 10 * 60 * 1000); // 10 Minuten in Millisekunden } else { // Wenn die Tür geschlossen wird if(timer) clearTimeout(timer); // Setze die Zielvariable auf "falsch" else setState(IDTerTuerSV, false); } } });
-
@paul53 Dankeschön
-
@paul53 ich habe weiter an den Programmen zum "Lüften" gearbeitet.
Ziel: nur im Winter, wenn also die Heizung aus ist (i.e. SV Sommer = aus(0)) sollen beim Lüften länger als 10 Minuten (hier: optischer TFK der Terrassentür), die Thermostate in der Kueche, dem Wohn- und Esszimmer ausgeschaltet (also auf CEN und Themperatur auf 0) werden. Wird die Tür geschlossen, sollen die vorher "gemerkten" Themperaturen wieder eingestellt und das Thermostat auf "auto" geschaltet werden.
Ich habe jetzt dazu mittels "createState" die entsprechenden Datenpunkte von der CCU in den IObroker verschoben. Dann habe ich 2 Scripte erstellt (das erste mit Deiner Hilfe):
// ########################################### // Lueften Terassentuer lang offen? // Deklarationen // ########################################### // Datenpunkt erstellen createState('BenutzerVariablen.TerrTuerLueften', '', { name: 'TerTuer', type: 'boolean' }); const IDtfkOptTerrasse = 'hm-rpc.1.0000DD89B24B3E.1.STATE'/*TFK opt Sensor Terrasse:1 STATE*/; const IDTerTuer = 'javascript.0.BenutzerVariablen.TerrTuerLueften'/*TerTuer*/; const sommer = 'hm-rega.0.11457'/*SOMMER Hzg Wohnb ALLES (Var )*/; // ########################################### // Programm // ########################################### /* Wenn die Tür geschlossen wird, stoppt man den Timeout, wozu man eine Timer-Variable benötigt. Außerdem wird an die CCU nur gesendet (SV), wenn Ack nicht auf true gesetzt wird. */ setState(IDTerTuer,false); var timer = null; on(IDtfkOptTerrasse, function (dp) { // triggert bei Wertänderung des DP if (!getState(sommer).val) { if(dp.state.val) {// Wenn die Tür geöffnet wird, starte eine 10-minütige Verzögerung timer = setTimeout(() => { timer = null; // Wenn die Tür immer noch geöffnet ist, setze die Zielvariable entsprechend setState(IDTerTuer, true); }, 10 * 60 * 1000); // 10 Minuten in Millisekunden } else { // Wenn die Tür geschlossen wird if(timer) clearTimeout(timer); // Setze die Zielvariable auf "falsch" else setState(IDTerTuer, false); } } });
Hier wird zunächst nur der Zustand der Terassentüre (länger als 10 Minuten offen?) ermittelt. In dem nun folgenden Script sollen die o.g. Operationen ausgefüht werden:
// ########################################### // Lueften Terassentuer Werte merken // und schreiben // Deklarationen // ########################################### // Datenpunkte erstellen; createState('BenutzerVariablen.ThempMerkerKueche', '', { name: 'ThempKueche', type: 'number' }); createState('BenutzerVariablen.ThempMerkerWohnzi', '', { name: 'ThempWohnzi', type: 'number' }); createState('BenutzerVariablen.ThempMerkerEsszi', '', { name: 'ThempEsszi', type: 'number' }); const IDthermKueche = 'hm-rpc.0.JEQ0553018.2.SETPOINT'/*Thermostat Kueche 1 SETPOINT*/; const IDthermWohnzi = 'hm-rpc.0.JEQ0552318.2.SETPOINT'/*Thermostat Wohnzimmer 1 SETPOINT*/; const IDthermEsszi = 'hm-rpc.0.JEQ0553158.2.SETPOINT'/*Thermostat Esszimmer 1 SETPOINT*/ var thempKuecheMerk = 'javascript.0.BenutzerVariablen.ThempMerkerKueche'/*ThempKueche*/; var thempWohnziMerk = 'javascript.0.BenutzerVariablen.ThempMerkerWohnzi'/*ThempWohnzi*/; var thempEssziMerk = 'javascript.0.BenutzerVariablen.ThempMerkerEsszi'/*ThempEsszi*/; const terTuerZustand = 'javascript.0.BenutzerVariablen.TerrTuerLueften'/*TerTuer*/; // ########################################### // Programm // ########################################### on(terTuerZustand, function (dp) { if(!dp.state.val) { // Terassentuer ist "false = geschlossen", dann setze die vorher // gespeicherten Themperaturen und schalte es auf AUTO // Variable zur Einstellung des Termostats var modus = 1; // 1 = AUTO; 2 = CEN // schreibe die momentane Temperaturen in die Merker, setze auf CEN und 0 thermometerSetzen(IDthermKueche, modus); setState(IDthermKueche,thempKuecheMerk); thermometerSetzen(IDthermEsszi, modus); setState(IDthermEsszi,thempEssziMerk); thermometerSetzen(IDthermWohnzi, modus); setState(IDthermWohnzi,thempWohnziMerk); }; if(dp.state.val) { // Terassentuer ist "true = offen", dann merke die // momentanen Temperaturen und schalte die Thermostate auf CEN und aus // Variable zur Einstellung des Termostats var modus = 2; // 1 = AUTO; 2 = CEN // schreibe die momentane Temperaturen in die Merker, setze auf CEN und 0 thermometerSetzen(IDthermKueche, modus); thempKuecheMerk = getState(IDthermKueche).val; setState(IDthermKueche,0); thermometerSetzen(IDthermEsszi, modus); thempEssziMerk = getState(IDthermEsszi).val; setState(IDthermEsszi,0); thermometerSetzen(IDthermWohnzi, modus); thempWohnziMerk = getState(IDthermWohnzi).val; setState(IDthermWohnzi,0); }; });
Meinung?
-
Habe gerade schon einen Fehler entdeckt: die Werte in die Merker müssen mit setState und nicht mit „=„ geschrieben werden. Ich falle immer wieder auf sowas rein…
Es muss also statt thempKuecheMerk = getState(IDthermKueche).val; heißen
setState(thempKuecheMerk, getState(IDthermKueche).val);
-
@skorpil sagte: Meinung?
Die Sollwerte 0 würde ich mit einiger Verzögerung nach der Modus-Änderung senden.
-
@paul53 du bist schon wieder auf der richtigen Spur… im log habe ich diese Fehlermeldung
hm-rpc.0 2023-05-09 18:12:33.569 error Cannot call setValue: XML-RPC fault: Failure hm-rpc.0 2023-05-09 18:12:33.568 error xmlrpc -> setValue ["JEQ0553158:2","SETPOINT","javascript.0.BenutzerVariablen.ThempMerkerEsszi"] FLOAT hm-rpc.0 2023-05-09 18:12:33.567 error Cannot call setValue: XML-RPC fault: Failure hm-rpc.0 2023-05-09 18:12:33.566 error xmlrpc -> setValue ["JEQ0552318:2","SETPOINT","javascript.0.BenutzerVariablen.ThempMerkerWohnzi"] FLOAT hm-rpc.0 2023-05-09 18:12:33.562 error Cannot call setValue: XML-RPC fault: Failure hm-rpc.0 2023-05-09 18:12:33.555 error xmlrpc -> setValue ["JEQ0553018:2","SETPOINT","javascript.0.BenutzerVariablen.ThempMerkerKueche"] FLOAT
Bin aber nicht sicher, ob das darauf zurückzuführen ist, oder ob das mit dem (vielleicht untauglichen) Versuch zusammenhängt,
dass der „setpoint“ mit sendto eingestellt werden muss (wie der Mode?)das ist Quatsch. Denn es hat ja eben funktioniert. Ich denke, ich muss mit delayed arbeiten -
@skorpil sagte: Meinung?
Du deklarierst z.B.
thempKuecheMerk
als Datenpunkt-ID - was die Fehlermeldungen erzeugt, aber dann soll sie den Wert des DP enthalten. Ändere Zeilen 27 bis 29 so, dass die Variablen bei Skriptstart den Sollwert enthalten, falls er > 0 ist. Sonst initialisiere sie mit einem typischen Wert.var thempKuecheMerk = getState(IDthermKueche).val; if (thempKuecheMerk < 10) thempKuecheMerk = 20;
-
@paul53 das mache ich morgen und teste.
-
@paul53 ich hatte mir wieder selber die Karten gelegt... und Datenpunkt und Wert des Datenpunkts nicht sauber auseinander gehalten. Und zwar hier (usw.):
const IDthempKuecheMerk = 'javascript.0.BenutzerVariablen.ThempMerkerKueche'/*ThempKueche*/; var thempKuecheMerk = getState(IDthempKuecheMerk).val;
Jetzt ist das Script angepasst und sollte so funktionieren. Zumindest der Lifetest scheint geklappt zu haben. Schaust Du dennoch noch mal drüber bitte?
// ########################################### // Lueften Terassentuer Werte merken // und schreiben // Deklarationen // ########################################### // Datenpunkte erstellen; createState('BenutzerVariablen.ThempMerkerKueche', '', { name: 'ThempKueche', type: 'number' }); createState('BenutzerVariablen.ThempMerkerWohnzi', '', { name: 'ThempWohnzi', type: 'number' }); createState('BenutzerVariablen.ThempMerkerEsszi', '', { name: 'ThempEsszi', type: 'number' }); const IDthermKueche = 'hm-rpc.0.JEQ0553018.2.SETPOINT'/*Thermostat Kueche 1 SETPOINT*/; const IDthermWohnzi = 'hm-rpc.0.JEQ0552318.2.SETPOINT'/*Thermostat Wohnzimmer 1 SETPOINT*/; const IDthermEsszi = 'hm-rpc.0.JEQ0553158.2.SETPOINT'/*Thermostat Esszimmer 1 SETPOINT*/; const IDthempKuecheMerk = 'javascript.0.BenutzerVariablen.ThempMerkerKueche'/*ThempKueche*/; const IDthempWohnziMerk = 'javascript.0.BenutzerVariablen.ThempMerkerWohnzi'/*ThempWohnzi*/; const IDthempEssziMerk = 'javascript.0.BenutzerVariablen.ThempMerkerEsszi'/*ThempEsszi*/; var thempKuecheMerk = getState(IDthempKuecheMerk).val; var thempWohnziMerk = getState(IDthempWohnziMerk).val; var thempEssziMerk = getState(IDthempEssziMerk).val; const terTuerZustand = 'javascript.0.BenutzerVariablen.TerrTuerLueften'/*TerTuer*/; // ########################################### // Programm // ########################################### on(terTuerZustand, function (dp) { if(!dp.state.val) { // Terassentuer ist "false = geschlossen", dann setze die vorher // gespeicherten Themperaturen und schalte es auf AUTO // Variable zur Einstellung des Termostats var modus = 1; // 1 = AUTO; 2 = CEN // schreibe die momentane Temperaturen in die Merker, setze auf CEN und 0 thermometerSetzen(IDthermKueche, modus); setStateDelayed(IDthermKueche,thempKuecheMerk, 500); thermometerSetzen(IDthermEsszi, modus); setStateDelayed(IDthermEsszi,thempEssziMerk, 500); thermometerSetzen(IDthermWohnzi, modus); setStateDelayed(IDthermWohnzi,thempWohnziMerk, 500); }; if(dp.state.val) { // Terassentuer ist "true = offen", dann merke die // momentanen Temperaturen und schalte die Thermostate auf CEN und aus // Variable zur Einstellung des Termostats var modus = 2; // 1 = AUTO; 2 = CEN // schreibe die momentane Temperaturen in die Merker, setze auf CEN und 0 thermometerSetzen(IDthermKueche, modus); setState(IDthempKuecheMerk, getState(IDthermKueche).val); setStateDelayed(IDthermKueche, 6, 500); thermometerSetzen(IDthermEsszi, modus); setState(IDthempEssziMerk, getState(IDthermEsszi).val); setStateDelayed(IDthermEsszi, 6, 500); thermometerSetzen(IDthermWohnzi, modus); setState(IDthempWohnziMerk, getState(IDthermWohnzi).val); setStateDelayed(IDthermWohnzi, 6, 500); }; });
-
@skorpil
Du speicherst zwar die Sollwerte in den Merker-Datenpunkten, liest sie aber nur bei Skriptstart aus - anschließend nicht mehr.
EDIT: Besser so:// ########################################### // Lueften Terassentuer Werte merken // und schreiben // Deklarationen // ########################################### const IDthermKueche = 'hm-rpc.0.JEQ0553018.2.SETPOINT'/*Thermostat Kueche 1 SETPOINT*/; const IDthermWohnzi = 'hm-rpc.0.JEQ0552318.2.SETPOINT'/*Thermostat Wohnzimmer 1 SETPOINT*/; const IDthermEsszi = 'hm-rpc.0.JEQ0553158.2.SETPOINT'/*Thermostat Esszimmer 1 SETPOINT*/; const IDthempKuecheMerk = 'javascript.0.BenutzerVariablen.TempMerkerKueche'/*TempKueche*/; const IDthempWohnziMerk = 'javascript.0.BenutzerVariablen.TempMerkerWohnzi'/*TempWohnzi*/; const IDthempEssziMerk = 'javascript.0.BenutzerVariablen.TempMerkerEsszi'/*TempEsszi*/; const IDterTuerZustand = 'javascript.0.BenutzerVariablen.TerrTuerLueften'/*TerTuer*/; // Datenpunkte erstellen; createState(IDthempKuecheMerk, 6, { name: 'TempKueche', type: 'number' }); createState(IDthempWohnziMerk, 6, { name: 'TempWohnzi', type: 'number' }); createState(IDthempEssziMerk, 6, { name: 'TempEsszi', type: 'number' }); // ########################################### // Programm // ########################################### on(IDterTuerZustand, function (dp) { if(!dp.state.val) { // Terassentuer ist "false = geschlossen", dann setze die vorher // gespeicherten Themperaturen und schalte es auf AUTO // Variable zur Einstellung des Termostats var modus = 1; // 1 = AUTO; 2 = CEN setState(IDthermKueche, getState(IDthempKuecheMerk).val); setState(IDthermEsszi, getState(IDthempEssziMerk).val); setState(IDthermWohnzi, getState(IDthempWohnziMerk).val); setTimeout(function() { thermometerSetzen(IDthermKueche, modus); thermometerSetzen(IDthermEsszi, modus); thermometerSetzen(IDthermWohnzi, modus); }, 500); } else { // Terassentuer ist "true = offen", dann merke die // momentanen Temperaturen und schalte die Thermostate auf CEN und aus // Variable zur Einstellung des Termostats modus = 2; // 1 = AUTO; 2 = CEN // schreibe die momentane Temperaturen in die Merker, setze auf CEN und 0 setState(IDthempKuecheMerk, getState(IDthermKueche).val); setState(IDthempEssziMerk, getState(IDthermEsszi).val); setState(IDthempWohnziMerk, getState(IDthermWohnzi).val); thermometerSetzen(IDthermKueche, modus); thermometerSetzen(IDthermEsszi, modus); thermometerSetzen(IDthermWohnzi, modus); setTimeout(function() { setState(IDthermKueche, 6); setState(IDthermEsszi, 6); setState(IDthermWohnzi, 6); }, 500); } });
-
@paul53 ich wußte nicht, dass man createState auch so verwenden kann:
createState(IDthempEssziMerk, 6, { name: 'ThempEsszi', type: 'number' });
Danke.
Verwendung von if else und das Auslesen der Merker Datenpunkte: ich bin begeistert.
Klasse u., vielen Dank. Ich teste morgen