Bitte Programmierhilfe

ticaki

@skorpil
ganz am Anfang dieses Topics wurde gesagt das warten in while schleifen nicht "gut" ist, und hier wartest du ja nicht mal. Die Schleife läut in unter 1 Millisekunde durch.

Das hier könnte funktioniert:
EDIT: mehr zusammengefasst, kürzer und nen logikfehler raus:

// ************Deklarationen********;
const KlimAut = 'hm-rega.0.9668'; // KlimaAutomatic
const AktTemperaturChan = 'hm-rpc.0.JEQ0709626.1.TEMPERATURE'; // Thermostat Wohnzimmer 0 TEMPERATURE
const Stromwert = 'hm-rpc.0.OEQ1544610.2.POWER'; // Zw Stecker 3 Wohnzimmer mM(WERTE):2 POWER
const MAX_RETRY = 3;
let automaticActive = false; // Prüfvariable für Klimaautomatik
 
// Sollwert 23 Wohnzimmer ................
const Sollwert = 23;
 
// Erlaubten Schwankungsbereich der Temperatur setzen 
const maxSoll = Sollwert + 1; 
const minSoll = Sollwert - 1;
 
// Schaltbefehle 
const klimEin = 'broadlink2.0.RM:Broadlink-RMPROPLUS-f4-3c-2b.L.Klima Weiden EIN'; // Klima Weiden EIN
const klimAus = 'broadlink2.0.RM:Broadlink-RMPROPLUS-f4-3c-2b.L.Klima  Weiden AUS'; // Klima  Weiden AUS
 
/********************* */
const maxPower = 100
var retrys = 0;
var target_state = getState(Stromwert).val > maxPower ? klimEin : klimAus
var current_state = target_state
/********************* */
//****************Programm*********;
on({ id: KlimAut, change: 'any' }, function (obj) {
    const automaticOn = obj.state.val; // Wert der Systemvariable KlimAut (true/false)
    
    if (automaticOn && !automaticActive) {
        automaticActive = true; // Klimaautomatik aktivieren
        checkTemperature(getState(AktTemperaturChan).val);
    } else if (!automaticOn && automaticActive) {
        retrys = 0
        automaticActive = false; // Klimaautomatik deaktivieren
        sendIRCommand(klimAus); // Klimaanlage ausschalten
    }
});
 
on({ id: AktTemperaturChan, change: 'ne', ack: true }, function (obj) {
    if (automaticActive) checkTemperature(obj.state.val)
});
 
function checkTemperature(val) {
    const currentTemperature = val;
 
    if (currentTemperature > maxSoll) {
        sendIRCommand(klimEin);
    } else if (currentTemperature < minSoll) {
        sendIRCommand(klimAus);
    }
}
 
function sendIRCommand(val) {
    target_state = val
    retrys = 0;
}
on({id:Stromwert, change:'ne'}, function(obj){
    if (obj.state.val > maxPower) current_state = klimEin
    else current_state = klimAus
})
setInterval(function(){
    if (target_state == current_state || ++retrys > MAX_RETRY) return
    setState(target_state, true)
},10000)

EDIT: stell die 10000 auf die Reaktionsgeschwindigkeit deiner Geräte ein, also x für das senden des Befehls + Y für die längste Zeit die es braucht bis ein Stromwert geliefert wird. Maßeinheit ist Millisekunden.

skorpil

@ticaki Dankeschön. Wo war der Logikfehler, den Du entdeckt hast?

Und das mit den while schleifen habe ich offenbar übersehen. Ich kann es auch nicht finden.

Aber Du hast recht, es ist gut, wenn ich eine Reaktionszeit für das Warten auf die Meldung des Zwischen Steckers eine Verzögerung einbaue.

ticaki

@skorpil
Ich hatte in der ersten Version einen Logikfehler, dann hab ich meine Teil vollständig in deinen integriert, bei beiden alles mehr zusammengefasst. Das sollte so laufen. Und die Reaktionszeit bezog sich auf das von mir gepostet Skript die 10000 am Schluß muß größer als die Reaktionszeit von allen Komponenten sein. Das ist nicht nur ein Beispiel :)

skorpil

@ticaki vieeeelen Dank. Ich habe das so übernommen. Es läuft!

skorpil

@ticaki kannst Du mir erklären, was diese Anweisung

var target_state = getState(Stromwert).val > maxPower ? klimEin : klimAus

macht?

Einen Fehler habe ich schon entdeckt. Das Script schaltet NICHT aus, wenn KlimAut aus aus gesetzt wird.

Und nicht nur das: kannst Du mir bitte, damit ich das verstehe und lerne auch erklären, wie jetzt Dein Teil mit meinem genau zusammenarbeitet?

ticaki

@skorpil sagte in Bitte Programmierhilfe:

wenn KlimAut aus aus gesetzt wird.**

Und nicht nur das

on({ id: KlimAut, change: 'any' }, function (obj) {
    const automaticOn = obj.state.val; // Wert der Systemvariable KlimAut (true/false)
    
    if (automaticOn && !automaticActive) {

Also wenn Klima aus war und du sie einschaltest passiert das

        automaticActive = true; // Klimaautomatik aktivieren
        checkTemperature(getState(AktTemperaturChan).val);

und wenn Klima An war und du schaltest sie aus das:

    } else if (!automaticOn && automaticActive) {
        retrys = 0
        automaticActive = false; // Klimaautomatik deaktivieren
        sendIRCommand(klimAus); // Klimaanlage ausschalten
    }
});

Und was passiert wenn die Anlage an ist und du das Skript neustartest? hehehe

// das oben ersetzten
let automaticActive = getState(KlimAut).val

ticaki

Erklärung mache ich gesondert:

Mein Code stellt ein gesonderter Teil dar, der nicht sychron läuft. Wenn du also die Klima an schaltest, kann es bis zur Intervalllänge dauern bis was passiert, beim Testen nervig, im normalen Betrieb wayne. :)

Der Stromverbrauch wird überwacht und in einer Variable current_state (aktueller Zustand) wird festgehalten ob das Gerät an oder aus ist.
Dein Code sendet an meinen Befehle was du gerne hättest, das wird in target_state also als Wunsch/Zielzustand gespeichert, dann wird alle X Sekunden geprüft ob der Zielzustand erreicht wurde und wenn nicht wird die Fernbedienung ausgelöst. MAX_RETRY begrenzt die Anzahl der Versuche.

EDIT: hab doch noch einen Fehler gefunden

setInterval(function(){
    if (target_state != current_state && ++retrys < MAX_RETRY) setState(target_state, true)   
},10000)

Vorher hat retrys nicht hochgezählt, wenn der Schaltvorgang erfolgt war. Das führt dazu dass das Skript das Gerät umschaltet wenn du es manuell bedienst.
Jetzt zählt es hoch und nach MAX_RETRY * 10000 (oder was du da stehen hast) macht das skript nichts mehr.

skorpil

@ticaki merci

paul53

@ticaki sagte: das wird in target_state also als Wunsch/Zielzustand gespeichert

Die Variablen target_state und current_state enthalten Datenpunkt-IDs. Die Bezeichnung "state" ist unglücklich gewählt, wie man an der Formulierung sieht.

ticaki

@paul53
Ja, hatte vorher ON und OFF, aber das ist doch nur mehr code :)

skorpil

Danke Euch beiden @paul53 @ticaki

ticaki

@skorpil
hmpf habs falsch rum gepostet:

setInterval(function(){
    if (++retrys < MAX_RETRY && target_state != current_state ) setState(target_state, true)   
},10000)

skorpil

ich habe jetzt ganz viel getestet. Das mit der Prüfung des Stromverbrauchs und die daran anschließende erneute Sendung der IR Steuerbefehle (Broadlink) funktioniert nicht zuverlässig. Das hängt damit zusammen, dass an dem Zwischenstecker auch weitere Geräte (z.B.TV) hängen, und es mir nicht gelingt, einen zuverlässigen Entscheidungswert zu definieren. Die Klimaanlae (Spittgerät) alleine liefert schon unterschiedliche Werte, jenachdem, ob nur das Innengerät aktiv ist oder eben das Außengerät zusätzlich.

Ich habe mich nun entschlossen, auf die Stromverbrauchsmessung komplett zu verzichten, zumal die Sendung Broadlink IR Befehle und Schaltung der Anlage - unidirektional - zuverlässig klappt.

Mein Script ist jetzt:

// #########################################################################
//    Deklarationn
// #########################################################################

const KlimAut = 'hm-rega.0.9668'/*KlimaAutomatic*/;
const AktTemperaturChan = 'hm-rpc.0.JEQ0709626.1.TEMPERATURE'/*Thermostat Wohnzimmer 0 TEMPERATURE*/;

// Schaltbefehle  
const klimEin = 'broadlink2.0.RM:Broadlink-RMPROPLUS-f4-3c-2b.L.Klima Weiden EIN'/*Klima Weiden EIN*/;
const klimAus = 'broadlink2.0.RM:Broadlink-RMPROPLUS-f4-3c-2b.L.Klima  Weiden AUS'/*Klima  Weiden AUS*/;

// Sollwert 22 Wohnzimmer ................
const Sollwert = 22.5;
//Erlaubten Schwankungsbereich der Temperatur setzen 
const maxSoll = Sollwert + 0.5; 
const minSoll = Sollwert - 0.5;

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

 
on(KlimAut, function (dp) {
    if(!dp.state.val) setState(klimAus, true);
    else if(getState(AktTemperaturChan).val > maxSoll) setState(klimEin, true);
});

 
on(AktTemperaturChan, function(dp) {
    // Aktuelle Temperatur von Wohnzimmer ermitteln  
    let AktTemperatur = dp.state.val;
    // log(AktTemperatur);
    if(getState(KlimAut).val) {
        // Bei aktueller Temperatur > Sollwert + 0.5 Aktor ausschalten
        // Bei aktueller Temperatur < Sollwert - 0.5 Aktor einschalten
        if (AktTemperatur > maxSoll) setState(klimEin, true);
        else if (AktTemperatur < minSoll) setState(klimAus, true);
    }
});

Das funtioniert gut.

skorpil

Das Problem ist aber noch, dass bei jeder Aktualisierung der Themperatur, wenn sie nicht im definierten Bereich liegt, ein neues KlimaEin Kommando gesendet wird. Da die Klimaanlage einen Quittungston bei Sendung der IR Befehle von sich gibt, ist das nervig. Daran muss ich noch arbeiten. @paul53 kannst Du mir hier etwas unter die Arme greifen?

ticaki

@skorpil

on(AktTemperaturChan, function(dp) {
    if(getState(KlimAut).val) {
        // Bei aktueller Temperatur > Sollwert + 0.5 Aktor ausschalten
        // Bei aktueller Temperatur < Sollwert - 0.5 Aktor einschalten
        if (obj..state.val > maxSoll && obj.oldstate.val <= maxSoll) setState(klimEin, true);
        else if (obj.state.val < minSoll && obj.oldstate.val >= minSoll) setState(klimAus, true);
    }
});

skorpil

@ticaki wow! Danke. Ich glaube, ich habe nicht gut formuliert. Bei JEDER TemperaturÄnderung sendet das Script den IR Befehl. Es soll aber nur jeweils EINMAL senden, wenn die Grenzwerte über- resp unterschritten werden. Ich hatte mir überlegt, dazu eine variable zu definieren….

ticaki

@skorpil
Ah hab was übersehen

    if(!dp.state.val) setState(klimAus, true);
    else if(getState(AktTemperaturChan).val > maxSoll) setState(klimEin, true);

der else Zweig muß natürlich genauso aussehen. Das senden den Befehl genau einmal. Ah und der doppelte '.' ist natürlich falsch

paul53

@ticaki sagte: der doppelte '.' ist natürlich falsch

... und obj. --> dp.

skorpil

@ticaki @paul53

ich bin jetzt wieder verwirrt und raffe es nicht. Wie muß der Block

on(AktTemperaturChan, function(dp) {
    if(getState(KlimAut).val) {
        // Bei aktueller Temperatur > Sollwert + 0.5 Aktor ausschalten
        // Bei aktueller Temperatur < Sollwert - 0.5 Aktor einschalten
        if (obj..state.val > maxSoll && obj.oldstate.val <= maxSoll) setState(klimEin, true);
        else if (obj.state.val < minSoll && obj.oldstate.val >= minSoll) setState(klimAus, true);
    }
});

RICHTIG aussehen? Oben wird "obj.oldstate.val" empfohlen, im zweiten Vorschlag ist oldstate weg. Könntet ihr bitte diesen ScriptBlock nocheinmal so einstellen, wie er richtig wäre, um die Bfehle jeweils nur einmal und nicht bi jeder Temperturänderung zu senden? Ich wäre uch zu großem Dank verbunden. Und Entschuldigung, daß ich nicht folgen konnte.

paul53

@skorpil sagte: RICHTIG aussehen?

on(AktTemperaturChan, function(dp) {
    if(getState(KlimAut).val) {
        // Bei aktueller Temperatur > Sollwert + 0.5 Aktor ausschalten
        // Bei aktueller Temperatur < Sollwert - 0.5 Aktor einschalten
        if (dp.state.val > maxSoll && dp.oldState.val <= maxSoll) setState(klimEin, true);
        else if (dp.state.val < minSoll && dp.oldState.val >= minSoll) setState(klimAus, true);
    }
});

Community Forum

NEWS

Bitte Programmierhilfe

Support us

750

32.5k

81.8k

1.3m