JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.

[paul53]

@skorpil sagte: verstehe die Frage nicht!

Im ersten Skript werden die beiden DP auf Wertänderung geprüft und schalten die SV ein und nach 2 Minuten aus.
Im zweiten Skript werden die gleichen DP abhängig vom Wert der SV ein- und ausgeschaltet. Das führt zu einer Trigger-Schleife: Das Licht lässt sich nicht ausschalten!
Schalten die beiden Aktoren überhaupt ein Außenlicht? "Schlafzimmer" und "Hinter TV" deuten nicht darauf hin.

EDIT: Haben die beiden Schalter eine Schalterwippe oder einen Taster? Falls Taster, gibt es sicherlich auch den DP "PRESS_SHORT", der als Trigger für das erste Skript besser geeignet wäre.

[skorpil]

@paul53 sagte in JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.:

Haben die beiden Schalter eine Schalterwippe oder einen Taster?

Es sind beides (leider) Schalterwippen und nicht Taster.

Im ersten Skript werden die beiden DP auf Wertänderung geprüft und schalten die SV ein und nach 2 Minuten aus.

Im zweiten Skript werden die gleichen DP abhängig vom Wert der SV ein- und ausgeschaltet. Das führt zu einer Trigger-Schleife: Das Licht lässt sich nicht ausschalten!

Es ist etwas komplizierter und ich habe Dir noch ein Script unterschlagen. Ich versuche mal zu erklären:

• ich wollte die SV "Aussenbeleuchtung" eigentlich nur Merker nutzen
• die beiden Schalter "Schlafzimmer" und "Hinter TV" schalten nur einen Teil des Lichts aussen
• es sollen aber weitere Lichtquellen "Lichtsch alter Eingang" + "Lichtsch Haustür" + "Lichtsch Alter Eingang (BewMeldAuff)" + "MülleimerLicht" geschaltet werden, wenn einer der beiden Schalter betätigt wird (beide Schalter schalten die gleiche Quelle und sind quasi Wechselschalter)
• oder der Bewegungsmelder auslöst (das Script hatte ich bisher unterschlagen):

// ########################################################################
//                           Deklarationen
// (wenn BewMldrGarten Bewegung erkennt und es ist Nacht, dann Variable Aussenbeleuchtung schalten)
// ########################################################################

const IDAussenbeleuchtung = 'hm-rega.0.1620'/*Aussenbeleuchtung*/;
// const idTag = 'javascript.0.variables.isDayTime';
const idBewMldrGarten = 'hm-rpc.0.LEQ0657464.1.MOTION'/*Bew Mldr Garten:1 MOTION*/
const idBewMldrGartenHell = 'hm-rpc.0.LEQ0657464.1.BRIGHTNESS'/*Bew Mldr Garten:1 BRIGHTNESS*/ 
// log(getState(idBewMldrGartenHell).val);

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

on({id: idBewMldrGarten, change: 'ne', val: true}, function () {
    // if(!getState(idTag).val) {
    if(getState(idBewMldrGartenHell).val < 80) {
        // Aktionen bei Bewegung nachts
        setState(IDAussenbeleuchtung, true);
        };
});

Jetzt hoffe ich, daß Du mir folgen kannst aufgrund der Erklärung. Ich hoffe, ich habe das sauber formuliert

[paul53]

@skorpil sagte: beide Schalter schalten die gleiche Quelle und sind quasi Wechselschalter

Das musst du näher erläutern: Sind die beiden Aktoren parallel geschaltet und schalten direkt das gleiche (Teil-)Außenlicht?
Was bedeutet dann Wechselschalter? Eine Wechselschaltung per Hardware lässt sich nur mit zwei Wechslern realisieren. Ob das Licht dann an ist, muss durch ein Exklusiv-Oder (ein Aktor ein, der andere aus) bzw. Äquivalenz (beide Aktoren ein oder aus) ermittelt werden.

[skorpil]

@paul53 sagte in JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.:

Das musst du näher erläutern: Sind die beiden Aktoren parallel geschaltet und schalten direkt das gleiche (Teil-)Außenlicht?

Gerade bringst Du mich in Verlegenheit. Ich weiß es nicht. Ich muß mir das nachher bei Dunkelheit ansehen. Ich glaube aber, ja!

Was bedeutet dann Wechselschalter? Eine Wechselschaltung per Hardware lässt sich nur mit zwei Wechslern realisieren.

Ich schaue das heute abend an und melde mich wieder.

Ob das Licht dann an ist, muss durch ein Exklusiv-Oder (ein Aktor ein, der andere aus) bzw. Äquivalenz (beide Aktoren ein oder aus) ermittelt werden.

Das konnte ich nur durch in Augenscheinnahme ermitteln. Da aber nach 2 Minuten eh alles ausgeschaltet wird, war das nicht schlimm.

In der CCU hatte ich das früher mit diesen Scripten realisiert, und das hat funktioniert:

[paul53]

@skorpil
Im Falle der Parallelschaltung der beiden Aktoren müsste das Programm im ersten Skript so geändert werden:

var timer = null;
on([IDLichtschalterAussenSchlaf, IDLichtschalterHinterTV], function () { // triggert bei Wertänderung eines DP
    // beide Aktoren sind parallel geschaltet 
    if(getState(IDLichtschalterAussenSchlaf).val || getState(IDLichtschalterHinterTV).val) {
        setState(IDAussenbeleuchtung, true);
        clearTimeout(timer); // Verlängerung um 2 Minuten, wenn innerhalb der Zeit erneute Schalterbetätigung "Ein"
        timer = setTimeout(function() {
            setState(IDAussenbeleuchtung, false);
        }, 120000);
    }
});

[paul53]

@skorpil sagte: Bewegungsmelder auslöst (das Script hatte ich bisher unterschlagen):

Das Programm ändere besser so:

var timer = null; 
on(idBewMldrGarten, function (dp) {
    if(getState(idBewMldrGartenHell).val < 80) {
        if(dp.state.val) {
            clearTimeout(timer); // erneute Bewegung erkannt
            setState(IDAussenbeleuchtung, true);
        } else timer = setTimeout(function() {
            setState(IDAussenbeleuchtung, false);
        }, 120000);
    }
});

[skorpil]

@paul53 mache ich, danke

[skorpil]

@paul53 zu meinem Verständnis: warum?

[paul53]

@skorpil sagte: warum?

Du schaltest bei Bewegung nur ein.
Das Ausschalten sollte mit den dritten Skript erfolgen? Dann passiert es, dass ausgeschaltet wird, weil der Timeout (im 3. Skript) abgelaufen ist und erst danach bei erneuter Bewegung wieder eingeschaltet wird. Besser ist es, den Ausschalt-Timeout bei erneuter Bewegung zu stoppen, damit das Licht an bleibt.

[skorpil]

@paul53 das macht Sinn. Danke.

[skorpil]

@paul53 wg. des Verlustes der bereits angepassten Scripte und der Wiederherstellung habe ich so ein bisschen die Übersicht verloren. Was ich aber schon sagen ist, dass das die SV Außenbeleuchtung nicht schaltet. Erstmal Danke für all die Hilfe heute. Ich werde morgen noch einmal die Scripte, die ich jetzt habe, überarbeiten und dann etwas dazu sagen.

[skorpil]

@paul53 So, jetzt habe ich getestet und die drei Scripte angepasst, wie Du es vorgeschlagen hast. Leider funktioniert das eigentliche "Schaltscript" nicht:

// ########################################################################
//                           Deklarationen
// (wenn im Schlafzimmer oder hinter TV geschaltet wird, dann Variable Aussenbeleuchtung schalten)
// ########################################################################

const IDAussenbeleuchtung = 'hm-rega.0.1620'/*Aussenbeleuchtung*/;

const IDSchlafAussen = 'hm-rpc.0.NEQ0274762.1.STATE'/*Lichtschltr  Schlafzimmer Aussenbeleuchtung 1 STATE*/;
const IDAlteHaustuer = 'hm-rpc.0.NEQ0133869.1.STATE'/*Lichtsch  alter Eingang STATE*/;
const IDHaustuerBew = 'hm-rpc.0.SEQ2044883.1.STATE'/*Lichtsch Haustür (Bewe Mldr ) STATE*/;
const IDHinterTV = 'hm-rpc.0.MEQ0000333.1.STATE'/*Lichtsch Hinter TV Aussenbeleuchtung 1 STATE*/;
const IDAlterEingangBew = 'hm-rpc.0.JEQ0093945.1.STATE'/*Lichtsch Alter Eingang (BewMeldAuff) STATE*/;
const IDMuellEimerLicht = 'hm-rpc.0.LEQ0877661.1.STATE'/*MülleimerLicht:1 STATE*/

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

on({id: IDAussenbeleuchtung, change: 'ne'}, function (data) {
    
    if(data.state.val) {
    // ist true;
    setState(IDSchlafAussen, true);
    setState(IDAlteHaustuer, true);
    setState(IDHaustuerBew, false);
    setState(IDHinterTV, true);
    setState(IDAlterEingangBew, false);
    setStateDelayed(IDHaustuerBew, true, 1000);
    setStateDelayed(IDAlterEingangBew, true, 1500);
    setStateDelayed(IDMuellEimerLicht, true, 2000);
    
    } else {
    // is false;    
    setState(IDSchlafAussen, false);
    setStateDelayed(IDHinterTV, false, 1000);
    setStateDelayed(IDAlteHaustuer, false, 2000);
    setStateDelayed(IDMuellEimerLicht, false, 3000);
    }
});
    

Es reagiert nicht auf die Änderung der SV Aussenbeleuchtung. Die anderen Scripte setzen die IDAussenbeleuchtung auf true.

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

const IDAusloeser ='hm-rega.0.1620'/*Aussenbeleuchtung*/;
// const IDAnwesend = 'hm-rega.0.39533'/*Anwesend*/;
const titel = "Aussenbeleuchtung:";
const sound = 'siren';
const prio = 1;

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

var timer = null;
on(IDAusloeser, function (dp) {

    if(!timer) {
        timer = setTimeout(function() {
            timer = null;
        }, 5000);    
            // if (getState(IDAnwesend).val) {
                var message = dp.state.val ? 'EIN!' : 'AUS!'; 
                pushoverSend(message, titel, sound, prio);                                
        //};         
    
    };        
});

Das obige "Puschover Script" reagiert, wenn ich die IDAussenbeleuchtung auf true bzw. false setze, nicht aber das Schaltscript Was ist falsch?

[paul53]

@skorpil sagte: Was ist falsch?

Ich kann keinen Fehler erkennen. Das Skript ist aktiv?
Weshalb setzt du die Bewegungsmelder erst auf false und kurz darauf auf true?

[skorpil]

@paul53 die „Bewegungsmelder“ sind eigentlich Schalter, die mit den Bewegungsmeldern (keine smarte) gekoppelt sind. Durch das kurze ein und wieder ausschalten bleiben die verbundenen Leuchten so lange an, wie im Bewegungsmelder vorgegeben.

[ticaki]

hier stand quatsch

[skorpil]

@ticaki das finde ich lustig! Gute Nacht und danke!

[ticaki]

@skorpil

Ich hatte das nur überflogen und beim wieder holten lesen ist mir mein Irrtum aufgefallen.

IMHO solltest du die on() gleich schreiben, wenn du das gleiche erwartest.

on(IDAusloeser, function...
on({id: IDAussenbeleuchtung, change: 'ne'}, function...

Das sollte zwar das gleiche sein, wenn sie sich aber unterschiedlich verhalten würde ich es mal mit gleich versuchen

[skorpil]

@paul53 sagte in JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.:

Das führt zu einer Trigger-Schleife: Das Licht lässt sich nicht ausschalten!

Gestern abend hatte ich den GAU. Ich hatte das Schaltscript geändert, weil es ja nicht geschaltet hat, und zwar von

on({id: IDAussenbeleuchtung, change: 'ne'}, function (data) {

auf

on(IDAussenbeleuchtung, true}, function (data) {

Dann hat das gesamte System verrückt gespielt. Das log lief mit Fehlermeldungen voll. Der iobroker stellte seine Arbeit ein. Ich mußte das System stoppen (Gottseidank habe ich ja die VirtualBox

Ich habe mir jetzt überlegt, das Script komplett neu aufzusetzen und auf die Aufteilung auf vier Scripte zu verzichten. Was will ich erreichen?

Wenn einer der beiden Schalter "Außenschlaf" oder "HinterTV" oder der Bewegungsmelder im Garten (bei Helligkeit < 80 oder die CCU SV Aussenbeleuchtung ausgelöst wird, soll das hier ablaufen:

{
    // ist true;
    setState(IDSchlafAussen, true);
    setState(IDAlteHaustuer, true);
    setState(IDHaustuerBew, false);
    setState(IDHinterTV, true);
    setState(IDAlterEingangBew, false);
    setStateDelayed(IDHaustuerBew, true, 1000);
    setStateDelayed(IDAlterEingangBew, true, 1500);
    setStateDelayed(IDMuellEimerLicht, true, 2000);
    
    } else {
    // is false;    
    setState(IDSchlafAussen, false);
    setStateDelayed(IDHinterTV, false, 1000);
    setStateDelayed(IDAlteHaustuer, false, 2000);
    setStateDelayed(IDMuellEimerLicht, false, 3000);
    }

Ich habe überlegt, das mit der folgenden ON Anweisung zu schalten.

on([IDLichtschalterAussenSchlaf, IDLichtschalterHinterTV, IDAussenbeleuchtung, IDBewMldrGarten], function (dp) {

Dabei müssen im folgenden zwei Fälle Unterschieden werden, je nachdem, ob das Script vom Bewegungsmelder im Garten oder von einem der anderen drei Trigger ausgelöst wird. Das GartenBewegungsMelder Script war ja eigentlich mit dem Timer schon fertig.

Nun weiß ich nicht, wie ich diese beiden Fälle sinnvoll in das Script integriere. Oder vielleicht doch zwei Scripte für GartenBew einerseits und alle anderen andererseits?

Hier bitte ich noch einmal um Hilfe. Danke.

[paul53]

@skorpil sagte: on(IDAussenbeleuchtung, true}, function (data) {

Das ist falsch. Richtig

on(IDAussenbeleuchtung, function (data) {

Ist das Skript so überhaupt gestartet?

@skorpil sagte in JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.:

Ich habe überlegt, das mit der folgenden ON Anweisung zu schalten.

Das macht das Skript unnötig kompliziert. Man kann alles in ein Skript bringen, aber besser mit getrennten Triggern.

@skorpil sagte in JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.:

oder die CCU SV Aussenbeleuchtung ausgelöst wird

Auch nach 2 Minuten aus?

[skorpil]

@paul53 sagte in JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.:

Auch nach 2 Minuten aus?

ja

Ist das Skript so überhaupt gestartet?

ja, mit den beschriebenen Folgen

Das macht das Skript unnötig kompliziert. Man kann alles in ein Skript bringen, aber besser mit getrennten Triggern.

je einfacher, je besser, Danke für Deine Hilfe