[gelöst] Elegantere Programmierung?

[skorpil]

@haus-automatisierung Frage: da ich gerne Deklarationen verwende, der Übersichtlichkeit halber: was müsste ich noch ändern:

wenn ich

var IDAusloeser ='hm-rega.0.39533'/Anwesend/;

nutze, wie müßze dann diese Zeile aussehen

on({id: 'hm-rega.0.39533'}, (data) => {

Ich frage, um zu lernen

[haus-automatisierung]

@skorpil

const IDAusloeser = 'hm-rega.0.39533';

on({id: IDAusloeser}, (data) => {

});

[skorpil]

So jetzt passiert genau das, was @paul53 angesprochen hat. Offenbar wird die SV jedesmal, wenn sich die Personenzahl ändert, aktualisiert. Und das führt dann im Ergebnis dazu. daß ich jedesmal eine pushover Nachricht erhalte, wenn sich die Personenzahl ändert. Das ist nicht gewollt.

Ich sollte nur dann eine Nachricht erhalten, wenn die SV auf Abwesend oder Anwesend geschaltet wird, also bei VERÄNDERUNG und nicht bei Aktualisierung. Da kommt nun wohl doch Pauls Vorschlag zum Tragen!

[paul53]

@skorpil sagte: Da kommt nun wohl doch Pauls Vorschlag zum Tragen!

Nur, was den Trigger betrifft:

on(IDAusloeser, (data)  => { // triggert bei Wertänderung

[skorpil]

@paul53 was ich jetzt noch nicht begriffen habe:

wann reagiert die on Anweisung auf Aktualisierung und wann auf Wertveränderung?

[paul53]

@skorpil sagte: wann reagiert die on Anweisung auf Aktualisierung und wann auf Wertveränderung?

Wenn das Muster ein ID-String ist, reagiert sie auf Wertänderungen. Wenn das Muster ein Objekt ist, reagiert sie auf Aktualisierung, wobei das durch weitere Attribute eingeschränkt werden kann.
Beispiel: Anstelle der String-ID-Version kann man auch schreiben:

on({id: IDAusloeser, change: 'ne'}, (data) => { // triggert bei Wertänderung

[skorpil]

@paul53 Dankeschön. Letze Frage: wie erkenne ich ein Objekt und wie den String? Es ist verwirrend.. ich bin auf diese Unterscheidung schon x mal reingefallen und verstehe es nicht richtig. Vlt. bin ich zu doof.

[paul53]

@skorpil sagte: wie erkenne ich ein Objekt und wie den String?

Siehe hier

[skorpil]

Ich muss doch noch einmal nachfragen:

// ##################################
// Deklarationen
// ##################################

const IDAusloeser ='hm-rega.0.43334'/*FnstrGaesteUntLinks lang OFFEN*/;


// ##################################
// Programm
// ##################################

on({id: IDAusloeser}, function (data) {

if (data.state.val) {

Kann ich mir nicht eigentlich die "If" Abfrage sparen und das direkt in der "On" Anweisung erledigen? Als sowas wie

on(IDAusloeser.state.val == true, function (data) {

Aber so, wie ich es mir hier ausgedacht habe, funktioniert es nicht.

[paul53]

@skorpil sagte: Kann ich mir nicht eigentlich die "If" Abfrage sparen und das direkt in der "On" Anweisung erledigen?

Ja, wenn nur auf val = true getriggert werden soll.

on({id: IDAusloeser, val: true}, function (data) {

Beschreibung

[skorpil]

@paul53 merci

[skorpil]

So, jetzt bin ich bei einem Fall angekommen, wo ich nicht mehr eine HM Systemvariable mit den Werten true/ false als trigger habe, sondern den Zustand eines Gerätes, hier der opt. Tür/ Fenster Kontakt. Ich möchte eine Nachricht erhalten, nur, wenn ich ANwesend bin

const IDAusloeser ='hm-rpc.1.0000DD89B24B3E.1.STATE'/*TFK opt  Sensor Terrasse:1 STATE*/;
const IDAnwesend = 'hm-rega.0.39533'/*Anwesend*/;

// ##################################
// Programm
// ##################################

on({id: IDAusloeser, val: true} , function (data) {

Der TFK kann die zustände offen u. geschlossen annehmen; CLOSED = 0 oder OPEN = 1. Wie muß ich da den Auslöser setzen?

[haus-automatisierung]

@skorpil Na auch über val...

// CLOSED
on({id: IDAusloeser, val: 0} , function (data) {
});

// OPEN
on({id: IDAusloeser, val: 1} , function (data) {
});

[skorpil]

Ich bekomme von meinem Automower diverse Fehlermeldungen, auf die ich aber nicht alle reagieren möchte. Vielmehr nur auf Werte < 1000 sowie auf 1038 und 1052.

const IDAusloeser ='hm-rega.0.35569'/*Automower Status-Code*/;
var IDstatusCode = getState(IDAusloeser).val;


on(IDAusloeser , function (data) {

    if (IDstatusCode < 1000 || IDstatusCode == 1038  || IDstatusCode == 1052) {

Frage 1: Ist die "If" Anweisung so richtig?

Frage 2: muß ich den Umweg über die getState Anweisung und die Definition einer Variablen (hier: IDstatusCode) gehen? Oder könnte ich in der "If" Anweisung direkt mit der IDAusloeser arbeiten? Ich bin immer unsicher, wie und wann das mit ".val" funktioniert.

Frage 3: benötige ich überhaupt die If Anweisung oder ließe sich das auch direkt in der On Anweisung lösen?

Dank Euch schön.

PS: warum ich dauernd frage: ich stelle gerade meine HM Programme auf iobroker um.

[haus-automatisierung]

@skorpil Du solltest Dich erstmal mit den Grundlagen der Programmierung beschäftigen glaube ich. Also unabhängig vom ioBroker.

1. Die Variable IDstatusCode wird beim Script-Start EINMALIG mit dem Wert aus dem Datenpunkt gefüllt. Danach nie wieder.
2. Diesen Wert (was auch immer zum Script-Start da drin stand) wird dann jedes Mal verglichen, wenn sich der Wert ändert. Aber der neue Wert wird halt nirgendwo berücksichtigt.

Du willst wahrscheinlich sowas machen:

const IDAusloeser ='hm-rega.0.35569'/*Automower Status-Code*/;
 
on(IDAusloeser , function (data) {
    const IDstatusCode = getState(IDAusloeser).val;
 
    if (IDstatusCode < 1000 || IDstatusCode == 1038  || IDstatusCode == 1052) {

Schlauer wäre es dann aber, den neuen Wert aus dem Parameter zu übernehmen:

const IDAusloeser ='hm-rega.0.35569'/*Automower Status-Code*/;
 
on(IDAusloeser , function (data) {
    const IDstatusCode = data.state.val;
 
    if (IDstatusCode < 1000 || IDstatusCode == 1038  || IDstatusCode == 1052) {

[paul53]

@skorpil sagte: Ist die "If" Anweisung so richtig?

Ja, aber die Variable IDstatusCode wird nicht aktualisiert. Außerdem enthält die Variable keine ID, was man beim Bezeichner berücksichtigen sollte.

const IDAusloeser ='hm-rega.0.35569'/*Automower Status-Code*/;
var statusCode = getState(IDAusloeser).val;
  
on(IDAusloeser, function (data) {
    statusCode = data.state.val;
    if (statusCode < 1000 || statusCode == 1038  || statusCode == 1052) {

[skorpil]

@haus-automatisierung sagte in [gelöst] Elegantere Programmierung?:

Du solltest Dich erstmal mit den Grundlagen der Programmierung beschäftigen

Das stimmt. Gibt es irgendwo einen Videokurs zu Javascript Programmierung im Zusammenhang mit iobroker? Deinen ioBroker-Master-Kurs habe ich erworben, aber da geht es meist um Blockly. Und damit tue ich mich schwer.

[haus-automatisierung]

@skorpil Am Ende schiebst Du mit Blockly nur fertigen JavaScript-Code zusammen. Wenn Du Blockly verstehst, und dann zu JavaScript umschaltest, dann siehst Du ja sofort was daraus generiert wird.

Also am Ende ist für Blockly als auch für JavaScript das gleiche grundlegende Verständnis notwendig. Nur dass Du bei Blockly keine syntaktischen Fehler machen kannst. Semantisch kann man genauso "Mist bauen".

Über eine JavaScript-Erweiterung habe ich schon nachgedacht - aber wo fängt man da an? Wie man an diesem Beispiel gut sieht, müsste man bei den absoluten Basics starten, welche erstmal nichts mit ioBroker an sich zu tun haben. Und ich möchte ja keine Kurse für Softwareentwickler erschaffen. Daher fehlt mir da etwas die Abgrenzung und deswegen gibt es aktuell "nur" Blockly im Kurs.

[Codierknecht]

@haus-automatisierung sagte in [gelöst] Elegantere Programmierung?:

Also am Ende ist für Blockly als auch für JavaScript das gleiche grundlegende Verständnis notwendig. Nur dass Du bei Blockly keine syntaktischen Fehler machen kannst. Semantisch kann man genauso "Mist bauen".

Dem würde ich mich anschließen.
Versuch's zunächst mit Blockly. Viel einfacher kann man kaum einsteigen, da es Dir viele Dinge abnimmt und - wie bereits gesagt - Fehler mit der Syntax ausschließt.

Nach meiner Erfahrung haben die meisten Anfänger Probleme damit, das grundlegende Verständnis aufzubauen wie ein "if-then-else" funktioniert und wie man passende Bedingungen mit "and/or" verküpft. Da werden die meisten Fehler gemacht, oder - um den Begriff zu verwenden - "Mist gebaut". Eine Klammer an der falschen Stelle - zack - völlig anderes Verhalten.
Da gleich mit JS (und womöglich Objektorientierung) einzusteigen könnte schwierig werden. Da würde ein Basic-Dialekt womöglich einfacher sein. War ja ursprünglich genau dafür ersonnen. Macht aber heute keiner mehr.

[haus-automatisierung]

@codierknecht sagte in [gelöst] Elegantere Programmierung?:

Da gleich mit JS (und womöglich Objektorientierung) einzusteigen könnte schwierig werden.

Naja, OOP braucht man nun wirklich nicht. Es reicht schon, wenn man sich mit Variablen, Datentypen (number, boolean, string, object), (anonymen) Funktionen / Callbacks und einfachen Vergleichen beschäftigt. Dann kann man 99% erschlagen.

Dafür gibt es ja ohne Ende guten Content auf YouTube und co. Wie gesagt - unabhängig vom ioBroker. Eventuell auch einfach mal ein paar Beispiele auf https://playcode.io durchspielen. Variablen deklarieren, Funktionen aufrufen, usw.

https://www.codewars.com finde ich auch ganz cool. Dort bekommt man Probleme gestellt und muss diese dann möglichst elegant lösen.

Wenn man das hier durcharbeitete, weiß man schon mehr als genug und muss dann nur noch das Wissen auf den ioBroker anwenden: https://github.com/jonasschmedtmann/complete-javascript-course