[gelöst] Elegantere Programmierung?

skorpil

@paul53 ich habe Dich hier nicht richtig verstanden.

Die Bilder der Programme aus der Homematic sollten nur demonstrieren, was ich nun in Java Skript programmieren möchte.

Du schreibst von der Hysterese, die Du für besser hältst. Haben wir die jetzt in Deinem letztenJava Skript so abgebildet, wie du es für richtig hältst?

paul53

@skorpil sagte: Haben wir die jetzt in Deinem letztenJava Skript so abgebildet, wie du es für richtig hältst?

Nein, das hat keine Hysterese.

EDIT: Mit 5 % Hysterese:

on(idThermBarKellerFeucht, function (data) {
    if(!getState(idSommer).val) {
        if(data.state.val > 65 && data.oldState.val <= 65) {
            // mit 2 auf CEN setzen, mit 1 auf AUTO 
            if(getState(idThermBarKellerTemp).val < 17 && !getState(heizStatusBar).val) {
                sendTo('hm-rpc.0', 'putParamset', {ID: 'JEQ0553882:2', paramType: 'MASTER', params: {'MODE_TEMPERATUR_REGULATOR': 1}}, res => {
                    log(JSON.stringify(res));
                });
            }
        } else if(data.state.val <= 60 && data.oldState.val > 60) { // 5 % Hysterese
            // 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); 
        }
    }
});

Besser wäre ein eigener Hilfsdatenpunkt, der bei Wertänderung den Modus an den MASTER-Parameter sendet und in dem man den zuletzt gesendeten Modus sieht. Das könnte so aussehen:

// ###########################################
//            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 )*/;
const idModus = '0_userdata.0.Keller.Bar.Thermostat.Modus';
 
 
// ###########################################
//                  Programm
// ###########################################
var mode = getState(idModus).val; // Modus bei Skriptstart

on(idThermBarKellerFeucht, function (data) {
    if(!getState(idSommer).val) {
        if(data.state.val > 65 && getState(idThermBarKellerTemp).val < 17 && !getState(heizStatusBar).val) {
            mode = 1; // AUTO
        } else if(data.state.val <= 60) { // 5 % Hysterese
            mode = 2; // CEN
        }
        setState(idModus, mode, true);
    }
});

on(idModus, function(dp) {
    sendTo('hm-rpc.0', 'putParamset', {ID: 'JEQ0553882:2', paramType: 'MASTER', params: {'MODE_TEMPERATUR_REGULATOR': dp.state.val}}, res => {
        log(res);
    });
    if(dp.state.val == 2) setStateDelayed(idThermBarKellerSetpoint, 15, 12000);
});

skorpil

@paul53 gerade erst gesehen. Wahnsinn! Daaaaaanke. Ich schaue mir das morgen in Ruhe an.

skorpil

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

Besser wäre ein eigener Hilfsdatenpunkt, der bei Wertänderung den Modus an den MASTER-Parameter sendet und in dem man den zuletzt gesendeten Modus sieht. Das könnte so aussehen:

So, bin wieder da und habe diese Variante übernommen. Frage nur: warum ist das besser?

paul53

@skorpil sagte: warum ist das besser?

Man hat einen Datenpunkt, der den Modus anzeigt und über den man den Modus bei Bedarf auch mal manuell (innerhalb der Hysterese) ändern kann.

skorpil

@paul53 ich kriege für diese Zeile


setState(idModus, mode, true);

eine Fehlermeldung:


You are assigning a number to the state "javascript.0.BenutzerVariablen.idModus" which expects a boolean. Please fix your code to use a boolean or change the state type to number. This warning might become an error in future versions.

paul53

@skorpil sagte: which expects a boolean

Ändere den Datenpunkt-Typ von "javascript.0.BenutzerVariablen.idModus" in "number", denn er enthält die Werte 1 oder 2. Erweitere die Objekteigenschaften um common.states:

{
    "1": "AUTO",
    "2": "CEN"
}

skorpil

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

Erweitere die Objekteigenschaften um common.states:

das Script sieht jetzt so aus. Ich mußte mit createState den Datenpunkt anlegen. Sonst ging es nicht. Er steht auf "number". Die genannte Fehlermeldung kommt dennoch.

// ###########################################
//            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 )*/;

// Datenpunkt erstellen (für andere Scripte);
createState('javascript.0.BenutzerVariablen.FeuchtigkeitsschutzKellerBar', '', {
    name: 'idFeuchtigkeitBarModus',
    type: 'number'
});
const idFeuchtigkeitBarModus = 'javascript.0.BenutzerVariablen.FeuchtigkeitsschutzKellerBar'/**/;

 
// ###########################################
//                  Programm
// ###########################################
var mode = getState(idFeuchtigkeitBarModus).val; // Modus bei Skriptstart

on(idThermBarKellerFeucht, function (data) {
    if(!getState(idSommer).val) {
        if(data.state.val > 65 && getState(idThermBarKellerTemp).val < 17 && !getState(heizStatusBar).val) {
            mode = 1; // AUTO
        } else if(data.state.val <= 60) { // 5 % Hysterese
            mode = 2; // CEN
        }
        setState(idFeuchtigkeitBarModus, mode, true);
    }
});
 
on(idFeuchtigkeitBarModus, function(dp) {
    sendTo('hm-rpc.0', 'putParamset', {ID: 'JEQ0553882:2', paramType: 'MASTER', params: {'MODE_TEMPERATUR_REGULATOR': dp.state.val}}, res => {
        log(res);
    });
    if(dp.state.val == 2) setStateDelayed(idThermBarKellerSetpoint, 15, 12000);
});

Mir ist aber auch nicht klar, WO ich "Erweitere die Objekteigenschaften um common.states:" durchführen muß. Danke für Hilfe

paul53

@skorpil sagte: Die genannte Fehlermeldung kommt dennoch.

Die bezieht sich auf einen anderen Datenpunkt als im gezeigten Skript.

You are assigning a number to the state "javascript.0.BenutzerVariablen.idModus" which expects a boolean.

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

WO ich "Erweitere die Objekteigenschaften um common.states:" durchführen muß.

Unter "OBJEKTDATEN":

  "type": "state",
  "common": {
    "name": "Kellerbar Modus",
    "role": "level.mode",
    "type": "number",
    "states": {
      "1": "AUTO",
      "2": "CEN"
    },
    "min": 1,
    "max": 2,
    "def": 1,
    "read": true,
    "write": true
  },

skorpil

@paul53 so das habe ich jetzt gefunden und wiefolgt geändert:

{
  "common": {
    "name": "idFeuchtigkeitBarModus",
    "role": "level.mode",
    "type": "number",
    "states": {
      "1": "AUTO",
      "2": "CEN"
    },
    "min": 1,
    "max": 2,
    "def": 1,
    "read": true,
    "write": true
  },
  "native": {
    "name": "idFeuchtigkeitBarModus",
    "type": "number",
    "role": "level.mode"
  },
  "type": "state",
  "_id": "javascript.0.BenutzerVariablen.FeuchtigkeitsschutzKellerBar",
  "acl": {
    "object": 1636,
    "state": 1636,
    "owner": "system.user.admin",
    "ownerGroup": "system.group.administrator"
  },
  "from": "system.adapter.admin.0",
  "user": "system.user.admin",
  "ts": 1678544548349
}

Zur Sicherheit hier noch einmal das zugehörige 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 )*/;

// Datenpunkt erstellen (für andere Scripte);
createState('javascript.0.BenutzerVariablen.FeuchtigkeitsschutzKellerBar', '', {
    name: 'idFeuchtigkeitBarModus',
    type: 'number'
});
const idFeuchtigkeitBarModus = 'javascript.0.BenutzerVariablen.FeuchtigkeitsschutzKellerBar'/**/;

 
// ###########################################
//                  Programm
// ###########################################
var mode = getState(idFeuchtigkeitBarModus).val; // Modus bei Skriptstart

on(idThermBarKellerFeucht, function (data) {
    if(!getState(idSommer).val) {
        if(data.state.val > 65 && getState(idThermBarKellerTemp).val < 17 && !getState(heizStatusBar).val) {
            mode = 1; // AUTO
        } else if(data.state.val <= 60) { // 5 % Hysterese
            mode = 2; // CEN
        }
        setState(idFeuchtigkeitBarModus, mode, true);
    }
});
 
on(idFeuchtigkeitBarModus, function(dp) {
    sendTo('hm-rpc.0', 'putParamset', {ID: 'JEQ0553882:2', paramType: 'MASTER', params: {'MODE_TEMPERATUR_REGULATOR': dp.state.val}}, res => {
        log(res);
    });
    if(dp.state.val == 2) setStateDelayed(idThermBarKellerSetpoint, 15, 12000);
});

und da ich noch ein Geräte im Einsatz habe in einem anderen Zimmer:

{
  "common": {
    "name": "idFeuchtigkeitJoergModus",
    "type": "number",
    "role": "level.mode",
    "states": {
      "1": "AUTO",
      "2": "CEN"
    },
    "min": 1,
    "max": 2,
    "def": 1,
    "read": true,
    "write": true
  },
  "native": {
    "name": "idFeuchtigkeitJoergModus",
    "type": "number",
    "role": "level.mode"
  },
  "type": "state",
  "_id": "javascript.0.BenutzerVariablen.FeuchtigkeitsschutzJoergKeller",
  "acl": {
    "object": 1636,
    "state": 1636,
    "owner": "system.user.admin",
    "ownerGroup": "system.group.administrator"
  },
  "from": "system.adapter.admin.0",
  "user": "system.user.admin",
  "ts": 1678544841925
}

Ist das dann so in Ordnung? Denn mit den OBJEKTDATEN jabe ich mich bislang noch nie befasst!

paul53

@skorpil sagte: Ist das dann so in Ordnung?

Ja, allerdings würde ich keine Namen verwenden, die mit "id" beginnen.

skorpil

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

@skorpil sagte: Ist das dann so in Ordnung?

Ja, allerdings würde ich keine Namen verwenden, die mit "id" beginnen.

Dankeschön. Zur Aufklärung für mich: wann ist ID richtig? In welchen Fällen nutzt man ID in JavaScript? Das habe ich noch nicht verstanden!

paul53

@skorpil sagte: In welchen Fällen nutzt man ID in JavaScript?

Ich verwende id für den Bezeichner einer Konstanten oder Variablen, die eine Objekt-ID enthält.

skorpil

@paul53 kannst Du hier helfen

https://forum.iobroker.net/topic/63796/javascript-zum-auslesen-der-tibber-api

paul53

@skorpil sagte: kannst Du hier helfen

Nicht wirklich. Es gibt doch schon zwei Adapter.

skorpil

@paul53 die funktionieren aber wohl beide nicht. Mir geht es ja nur darum, einen einzelnen Datenpunkt. auszulesen. Ich wollte da so machen, wir Du es hier beschrieben hast:

https://forum.iobroker.net/topic/38775/efergy-api-auslesen-und-in-wert-umwandeln/2

Ich tue mich aber schwer mit den Authentifizierung.

Hier ist das ja alles schön dokumentiert:

https://developer.tibber.com/explorer

paul53

@skorpil sagte: die funktionieren aber wohl beide nicht.

Wenn es IT-Profis nicht hinbekommen: Wie soll ich es dann ohne IT-Ausbildung hinbekommen?

skorpil

@paul53 waaaas!? Du ohne IT Ausbildung? Ich glaube es nicht Ich hätte meinen Kopf darauf verwettet, Du hättest Informatik studiert.

Nun gut, dann probiere ich weiter eine Lösung für Tibber zu finden. Und wenn ich was habe, melde ich mich hier.

Homoran

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

Ich hätte meinen Kopf darauf verwettet,

ab mit dem Kopf!
(die Herz-Königin in Alice im Wunderland)

skorpil

@homoran ich schrieb „hätte“. Daher bleibt er dran. Liebe Grüße

NEWS

[gelöst] Elegantere Programmierung?

Support us

810
Online

30.5k
Users

76.8k
Topics

1.2m
Posts

NEWS

[gelöst] Elegantere Programmierung?

Support us

810Online

30.5kUsers

76.8kTopics

1.2mPosts

810
Online

30.5k
Users

76.8k
Topics

1.2m
Posts