NEWS
[gelöst] Elegantere Programmierung?
-
Ein Mysterium, ich brauche Hilfe:
mit diesem Script rufe ich die Funktion ermitteleDatum() auf. Das funktioniert auch gut!
function ermitteleDatum() { var d= new Date(); //Tagesdatum ermitteln var day = new Array("00","01","02","03","04","05","06","07","06","09","10","11","12","13","14","15","16","17","18","19","20","21","22","23","24","25","26","27","28","29","30","31"); var t = day[d.getDate().toString()]; //Monat ermitteln var month = new Array("01","02","03","04","05","06","07","08","09","10","11","12"); var m = month[d.getMonth().toString()]; //Jahr ermitteln var j = d.getFullYear().toString(); var datum= t+"."+m+"."+j+" "; //log("datum: " + datum); return (datum); } var heute = ermitteleDatum(); log("HEUTE:" + heute)
Das log zeigt:
13:42:13.009 info javascript.0 (767) script.js.Test.Geburtstag_Alter: HEUTE:14.02.2023
Nun möchte ich die var "heute" weiter verwenden, um das Alter einer Person zu ermitteln.
Zur Sicherheit habe ich noch einmal eingefügt:
log("Datum= " + heute); heute = "14.02.2023"; log("Datum= " + heute); const date2 = new Date(heute.split(".").reverse().join("-")); log("date2 = " + date2);
Ergebnis im log:
13:57:04.272 info javascript.0 (767) script.js.Test.Geburtstag_Alter: Datum= 14.02.2023 13:57:04.272 info javascript.0 (767) script.js.Test.Geburtstag_Alter: Datum= 14.02.2023 13:57:04.272 info javascript.0 (767) script.js.Test.Geburtstag_Alter: date2 = Tue Feb 14 2023 01:00:00 GMT+0100 (Mitteleuropäische Normalzeit)
Also, alles richtig.
Nun nehme ich "14.02.2023" raus:
log("Datum= " + heute); // heute = "14.02.2023"; log("Datum= " + heute);
und erhalte die Fehlermeldung:
13:58:04.155 info javascript.0 (767) script.js.Test.Geburtstag_Alter: Datum= 14.02.2023 13:58:04.155 info javascript.0 (767) script.js.Test.Geburtstag_Alter: Datum= 14.02.2023 13:58:04.155 info javascript.0 (767) script.js.Test.Geburtstag_Alter: date2 = Invalid Date
Ich raffe es nicht. Was ist da falsch? Die Variable ist ein String, ebenso wie das eingegebene Datum.
-
@skorpil sagte: das Alter einer Person zu ermitteln.
let datum = inhalt[i]._object.rrule.origOptions.dtstart; let alter = new Date().getFullYear() - new Date(datum).getFullYear();
-
@paul53 sagte in [gelöst] Elegantere Programmierung?:
let datum = inhalt[i]._object.rrule.origOptions.dtstart; let alter = new Date().getFullYear() - new Date(datum).getFullYear();
Super, vielen Dank. Jetzt bin ich einen großen Schritt weiter. Nur, eins klappt nicht:
/* ####################################################### Kalenderevent auswerten ----> nur Geburtstage sucht im iCal Adapter nach events (heute) ######################################################*/ // Datenpunkt festlegen createState('Ansage.GeburtstageHEUTE', '', { name: 'Geburtstage ansagen', type: 'string' }); createState('Ansage.Alter', '', { name: 'Alter ansagen', type: 'number' }); var tempAnsage = 'Ansage.GeburtstageHEUTE'; var tempAlter = 'Ansage.Alter'; //Datum als String ermitteln; function ermitteleDatum() { var d= new Date(); //Tagesdatum ermitteln var day = new Array("00","01","02","03","04","05","06","07","06","09","10","11","12","13","14","15","16","17","18","19","20","21","22","23","24","25","26","27","28","29","30","31"); var t = day[d.getDate().toString()]; //Monat ermitteln var month = new Array("01","02","03","04","05","06","07","08","09","10","11","12"); var m = month[d.getMonth().toString()]; //Jahr ermitteln var j = d.getFullYear().toString(); var datum= t+"."+m+"."+j+" "; //log("datum: " + datum); return (datum); } function pruefeKalender() { var inhalt = getState('ical.0.data.table').val; var heute = ermitteleDatum(); //log("HEUTE:" + heute) ; try{ var ereignisse = JSON.stringify(inhalt, null, 2); // Ausgabe als String // log(ereignisse); var ereignisheute = '', // Liste (kommasepariert) ereignisheute_zeilen = ''; // Liste (mit Zeilenumbruch, zB für Anzeige in VIS) for(var i = 0; i <inhalt.length; i++) { // alle Events durchgehen if ( (inhalt[i].date.indexOf(heute) != -1) || (inhalt[i].date.indexOf('Heute') != -1) ) { // Strings Datum oder relatives Datum (nicht nicht) gefunden var ereignis = inhalt[i].event; ereignis = ereignis.replace(',',''); // Komma im Namen ersetzen var komma = (i>0) ? ', ' : ''; ereignisheute = ereignisheute + komma + ereignis; ereignisheute = ereignisheute.replace('Geburtstag von ', ''); // "Geburtstag von " löschen // Alter ermitteln let geburt = inhalt[i]._object.rrule.origOptions.dtstart; let alter = new Date().getFullYear() - new Date(geburt).getFullYear(); log("Alter= " + alter); } } // Aufbereitung für die Ansage (falls vorhanden, wird letztes Komma durch und ersetzt) var lastkomma = ereignisheute.lastIndexOf(','); // letztes Komma in der Reihe if (lastkomma != -1) { var vorn = ereignisheute.slice(0,lastkomma);// lastkomma geändert in Null var hinten = ereignisheute.slice(lastkomma+1, ereignisheute.length); ereignisheute = vorn + ' und' + hinten; } log("ereignisheute= " + ereignisheute); log("Alter= " + alter); // Ende Aufbereitung für die Ansage setState(tempAnsage, ereignisheute); setState(tempAlter, alter); log('Geburtstage: ' + ereignisheute); log('Alter: ' + alter); } catch (fehler_try) { log('Fehler beim Kalenderevent einlesen ' + fehler_try); } } // bei Aktualisierung des Kalenders on ({id:'ical.0.events.Geburtstag', change: 'any'}, function(data) { pruefeKalender(); }); //bei Skriptstart schedule("40 2 * * *", function () { //log("===>Wird einmal am Tag ausgelöst"); pruefeKalender(); }); pruefeKalender()
Es ist noch immer ein Fehler drin: in Zeile 62 wird das log korrekt ausgeführt. In Zeile 75 aber nicht. Cannot find alter. Warum nicht?
-
@skorpil sagte: In Zeile 75 aber nicht.
In Zeile 75 ist die lokale Variable
alter
nicht bekannt. Du musst sie global deklarieren, also außerhalb einer Funktion.var alter; function xyz() { //... alter = new Date().getFullYear() - new Date(geburt).getFullYear(); }
-
@paul53 Dankeschön. Habe die Variable außerhalb definiert. Jetzt bekomme ich nur noch eine WARNUNG. Wo ist da das Problem?
Script:
/* ####################################################### Kalenderevent auswerten ----> nur Geburtstage sucht im iCal Adapter nach events (heute) ######################################################*/ // Datenpunkte festlegen createState('Ansage.GeburtstageHEUTE', '', { name: 'Geburtstage ansagen', type: 'string' }); createState('Ansage.Alter', '', { name: 'Alter ansagen', type: 'string' }); // Deklarationen var tempAnsage = 'Ansage.GeburtstageHEUTE'; var tempAlter = 'Ansage.Alter'; var alter; //Funktion Datum als String ermitteln; function ermitteleDatum() { var d= new Date(); //Tagesdatum ermitteln var day = new Array("00","01","02","03","04","05","06","07","06","09","10","11","12","13","14","15","16","17","18","19","20","21","22","23","24","25","26","27","28","29","30","31"); var t = day[d.getDate().toString()]; //Monat ermitteln var month = new Array("01","02","03","04","05","06","07","08","09","10","11","12"); var m = month[d.getMonth().toString()]; //Jahr ermitteln var j = d.getFullYear().toString(); var datum= t+"."+m+"."+j+" "; //log("datum: " + datum); return (datum); } function pruefeKalender() { var inhalt = getState('ical.0.data.table').val; var heute = ermitteleDatum(); //log("HEUTE:" + heute) ; try{ var ereignisse = JSON.stringify(inhalt, null, 2); // Ausgabe als String // log(ereignisse); var ereignisheute = '', // Liste (kommasepariert) ereignisheute_zeilen = ''; // Liste (mit Zeilenumbruch, zB für Anzeige in VIS) for(var i = 0; i <inhalt.length; i++) { // alle Events durchgehen if ( (inhalt[i].date.indexOf(heute) != -1) || (inhalt[i].date.indexOf('Heute') != -1) ) { // Strings Datum oder relatives Datum (nicht nicht) gefunden var ereignis = inhalt[i].event; ereignis = ereignis.replace(',',''); // Komma im Namen ersetzen var komma = (i>0) ? ', ' : ''; ereignisheute = ereignisheute + komma + ereignis; ereignisheute = ereignisheute.replace('Geburtstag von ', ''); // "Geburtstag von " löschen // Alter ermitteln let geburt = inhalt[i]._object.rrule.origOptions.dtstart; alter = new Date().getFullYear() - new Date(geburt).getFullYear(); log("Alter= " + alter); } } // Aufbereitung für die Ansage (falls vorhanden, wird letztes Komma durch und ersetzt) var lastkomma = ereignisheute.lastIndexOf(','); // letztes Komma in der Reihe if (lastkomma != -1) { var vorn = ereignisheute.slice(0,lastkomma);// lastkomma geändert in Null var hinten = ereignisheute.slice(lastkomma+1, ereignisheute.length); ereignisheute = vorn + ' und' + hinten; } log("ereignisheute= " + ereignisheute); log("Alter= " + alter); // Ende Aufbereitung für die Ansage setState(tempAnsage, ereignisheute); setState(tempAlter, alter); log('Geburtstage: ' + ereignisheute); log('Alter: ' + alter); } catch (fehler_try) { log('Fehler beim Kalenderevent einlesen ' + fehler_try); } } // bei Aktualisierung des Kalenders on ({id:'ical.0.events.Geburtstag', change: 'any'}, function(data) { pruefeKalender(); }); //bei Skriptstart schedule("40 2 * * *", function () { //log("===>Wird einmal am Tag ausgelöst"); pruefeKalender(); }); pruefeKalender()
Warnung, die ich nicht vertstehe:
14:03:40.782 warn javascript.0 (767) at pruefeKalender (script.js.Skripte_aktiviert.Test_Teil_1_-_Geburtstage_mit_Alter:84:5) 14:03:40.782 warn javascript.0 (767) at script.js.Skripte_aktiviert.Test_Teil_1_-_Geburtstage_mit_Alter:106:1 14:03:40.783 warn javascript.0 (767) at script.js.Skripte_aktiviert.Test_Teil_1_-_Geburtstage_mit_Alter:107:3
-
@skorpil sagte: Warnung, die ich nicht vertstehe:
Vermutung: Es wird in Zeile 84 eine Zahl (alter) in einen Datenpunkt vom Typ "string" geschrieben.
Schau im Tab "Protokolle" nach weiteren Informationen zu den Warnungen. -
@paul53 sagte in [gelöst] Elegantere Programmierung?:
@skorpil sagte: Warnung, die ich nicht vertstehe:
Vermutung: Es wird in Zeile 84 eine Zahl (alter) in einen Datenpunkt vom Typ "string" geschrieben.
Schau im Tab "Protokolle" nach weiteren Informationen zu den Warnungen.Du bist ein Fuchs! Das war es! Danke!
-
@paul53 So, und jetzt funktioniert, dank Eurer Hilfe, das Auslesen der Geburtstage aus dem Kalender. Nun werden die Person UND das Alter der Person über Sonos oder alternativ auch über Alexa angesagt. Wenn jemand das Script haben möchte, nur zu, gerne melden.
-
@skorpil
Nun sollte das Endergebnis der vielen Mühe doch hier publiziert werden.
Meine Hochachtung vor der Umsetzung. -
@martybr kommt!
-
Wunschgemäß zeige ich hier jetzt mal meine Scripte, die ich mit Eurer Hilfe erstellt habe. Sie dienen dazu, Wetterdaten, Uhrzeit und Wochentag, Geburtstage mit Alter sowie die Abholtermine für den Müll in Sonos oder Alexa anzusagen. Es sind insgesamt 5 Scripte. Sicher sind die alle nicht optimal programmiert. Die Füchse unter Euch können das sicher effizienter und eleganter. Aber: sie funktionieren! Benötigt werden ein Kalender mit Geburtstagen und Abholterminen für den Müll, der ical sowie die Alexa, Broadlink und iot- adapter sowie die Sonos API.
Script 1 ermittelt aus dem Kalender Geburtstage und Alter und legt die Texte in einem Datenpunkt ab, auf den später andere Scripte zugreifen können. Die Scripte, die der "Ermittlung" dienen werden nur einmal täglich nachts ausgeführt:
Script 2 ermittelt den Abholtermin des Mülls für den heutigen Tag:
Script 3 ermittelt den Abholtermin für den nächsten Tag:
Script 4 dient der ANSAGE der Ereignisse über SONOS, abhängig von einem Bewegungsmelder in einem definierten Raum, und spielt danach auf der Sonos Anlage einen definierten Radiosender für eine definierte Zeit ab:
Script 5 macht das Gleiche wie Script 4, nur reagiert es auf einen definierten Alexa Sprachbefehl (wobei bei einem Sonos Gerät vorher noch der Verstärker eingeschaltet wird):
Script 6 dient der Ansage über ein bestimmtes Alexagerät (hier: im Badezimmer):
Viel Spaß beim Anpassen und verwenden. Für Rückfragen stehe ich jederzeit gerne zur Verfügung.
-
@skorpil
Super. Ich werde mir deine Scripte mal am Wochenende intensiv anschauen. Das kann ich bestimmt eine Menge lernen.
Vielen Dank -
Haben eigentlich die letzten Beiträge noch irgendetwas mit "eleganterer Programmierung" zu tun?
-
@andreas-5 Nein! Du hast Recht. Aber ich finde es sinnvoll, das Ergebnis hier zu zeigen.
-
nochmal eine Frage nach der besten Programmierung:
Es geht um den Feuchtigkeitsschutz in meiner Kellerbar. Ziel:
Sollte die Feuchtigkeit im Keller > 65% übersteigen, soll die Heizung in diesem Raum eingeschaltet werden. Nebenbedingungen: nur, wenn die Temperatur < 17 Grad, der Heizungsstatus in diesem Raum ausgeschaltet ist und es nicht Sommer ist.Ich habe dazu folgende ON Anweisung angelegt:
on({id: idThermBarKellerFeucht, change: 'ne'}, function (data) { if(data.state.val > 65 && !heizStatusBar && getState(idThermBarKellerTemp).val < 17 && !idSommer) {
Meine Frage dazu: sollte ich bereits in der ON Anweisung statt "change: ne" mit "val > 65" arbeiten? Oder ist das egal und ich lasse es, wie es ist?
-
@skorpil sagte: bereits in der ON Anweisung statt "change: ne" mit "val > 65" arbeiten?
Das macht keinen merkbaren Unterschied.
if(data.state.val > 65 && !heizStatusBar && getState(idThermBarKellerTemp).val < 17 && !idSommer) {
Ist
idSommer
nicht eine Konstante(ID)? Wo wirdheizStatusBar
gesetzt? -
@paul53 okay, danke. Hier mal das komplette Script:
// ########################################### // Deklarationen // ########################################### const heizStatusBar = 'hm-rega.0.23436'/*Heizstatus (BAR)*/; const idThermBarKellerTemp ='hm-rpc.0.JEQ0553882.1.TEMPERATURE'/*Thermostat Bar Keller 0 TEMPERATURE*/; const idThermBarKellerSetpoint = 'hm-rpc.0.JEQ0553882.2.SETPOINT'/*Thermostat Bar Keller 1 SETPOINT*/ const idThermBarKellerFeucht = 'hm-rpc.0.JEQ0553882.1.HUMIDITY'/*Thermostat Bar Keller 0 HUMIDITY*/; const idSommer = 'hm-rega.0.11457'/*SOMMER Hzg Wohnb ALLES (Var )*/; // ########################################### // Programm // ########################################### on({id: idThermBarKellerFeucht, change: 'ne'}, function (data) { if(data.state.val > 65 && !heizStatusBar && getState(idThermBarKellerTemp).val < 17 && !idSommer) { // ist true // mit 2 auf CEN setzen, mit 1 auf AUTO sendTo('hm-rpc.0', 'putParamset', {ID: 'JEQ0553882:2', paramType: 'MASTER', params: {'MODE_TEMPERATUR_REGULATOR': 1}}, res => { log(JSON.stringify(res)); }); } else { // ist false // mit 2 auf CEN setzen, mit 1 auf AUTO sendTo('hm-rpc.0', 'putParamset', {ID: 'JEQ0553882:2', paramType: 'MASTER', params: {'MODE_TEMPERATUR_REGULATOR': 2}}, res => { log(JSON.stringify(res)); }); setStateDelayed(idThermBarKellerSetpoint, 15, 12000); } });
Zu Deinen Fragen:
- idSommer kann die Werte EIN/ AUS annehmen und kommt aus der rega.
- heizStatusBar wird in der homematic gesetzt, auch EIN/AUS
Aber Deine Fragen haben bestimmt einen Hintergrund. Was ist falsch bei mir?
-
@skorpil sagte: Aber Deine Fragen haben bestimmt einen Hintergrund.
So ist es. Wenn es Datenpunkt-IDs sind, müssen sie mittels getState(id) abgefragt werden.
if(data.state.val > 65 && !getState(heizStatusBar).val && getState(idThermBarKellerTemp).val < 17 && !getState(idSommer).val) {
-
@paul53 sagte in [gelöst] Elegantere Programmierung?:
if(data.state.val > 65 && !getState(heizStatusBar).val && getState(idThermBarKellerTemp).val < 17 && !getState(idSommer).val) {
merci!
-
Was will mir denn diese Meldung im log im Zusammenhang mit den zuletzt genannten Scripten sagen?
script.js.HM_Skripte.Feuchtigkeitsschutz_Joerg_Keller: {"result":"","error":null}