[gelöst] Elegantere Programmierung?

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

OliverIO

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

@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.

sorry, hab leider auch keine IT-Ausbildung (und so wie ich manche kennen ist das auch nicht immer eine Auszeichnung )

Wie so oft muss man hier mehrere Technologien kennen.
1.) Javascript
2) Eine Bibliothek mit der man http sprechen kann
3) Eine Bibliothek, die OAuth kann
4) und zum Schluss muss man graphQL können bzw. die Basiskonzepte verstehen.
Du hast einen Link zum graphQL Explorer geschickt, der natürlich auch ein Token benötigt. Durch Druck auf den Knopf Demo-Token kann man damit erst mal grundsätzlich arbeiten. Wie das mit Load personal Token application token abgefragt wird und im Hintergrund dann ein AccessToken abgefragt wird, keine Ahnung. Das reagiert nicht so wie ich erwartet hätte

Also zur groben Lösung

Muss ich voraussetzen, sonst wird es zu schwierig
Hier würde ich Axios empfehlen https://www.npmjs.com/package/axios
Hier würde ich ein Aufsatz auf axios empfehlen https://www.npmjs.com/package/axios-oauth-client
In deiner tibber oberfläche müsstest du dir irgendwo ein application token/client-secret erzeugen können. Das wird per request an den server übertragen und du erhälst ein oauth-token zurück, welches du dann bei jeder nachfolgenden Abfrage als http-header mit dem vorsatz BEARER mitschicken musst
Hier unter Authorization ist es erklärt, wohl auch mit Link zur Seite wo du dir einen Token holen kannst.
https://developer.tibber.com/docs/guides/calling-api
Wie man dann Abfragen ausführt steht dann hier:
https://developer.tibber.com/docs/guides/calling-api unter "Performing" Requests
Als curl Befehl ist ein Beispiel aufgeführt, was du an Daten mitgeben musst (Authorization, ContentType und die eigentlichen Daten), das kannst du beim lesen der Axios-Doku fast direkt umsetzen, da sind auch Beispiele für javascript mit dabei.
Als Daten musst du dann ein graphQL-Ausdruck mitschicken(in einem JSON eingepackt als Attribut query, siehe Beispiel), den du in dem Explorer vorher ausprobieren kannst (halt mit Demo-Daten oder du trägst da ein OAuth-token ein, den du mit 3 geholt hast, dann auch mit deinen Daten)
Mit dem Explorer müsstest du sehr Detailliert angeben können welche Daten du zurückbekommen möchtest, leider funktioniert die Autovervollständigung und das laden des Schemas (rechte Seite) nicht richtig. Daher könnte es schwierig werden, die richtigen Angaben zu machen. Da musst mal dort nachfragen. Evtl muss man eingeloggt sein.

Codierknecht

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

sorry, hab leider auch keine IT-Ausbildung

Willkommen im Club

und so wie ich manche kennen ist das auch nicht immer eine Auszeichnung

Manchmal (oft) sogar kontraproduktiv.
Ich habe jedenfalls die Erfahrung gemacht, dass es (in unserem Team) einfacher ist, einem IT-affinen Fachmann das Programmieren beizubringen als einem (studierten) Programmierer unsere Prozesse zu erklären.
Der junge Kollege beißt sich daran seit 3 Jahren die Zähne aus

OliverIO

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

Prozesse zu erklären.
Der junge Kollege beißt sich daran seit 3 Jahren die Zähne aus

Da ich mich schon sehr umfangreich auch mit Prozessmanagement auskenne und schon viele Schulungen durchgeführt habe, kann ich sagen, das es Leute gibt, die es nie lernen werden, da sie nicht in der Lage sind komplexe Problemstellungen in kleinere aufzuteilen, zu abstrahieren und dann noch das richtige Maß der Granularität zu finden.

Dann ist er aber auch kein guter Programmierer? Das ist für mich nämlich ein sehr verwandter Skill.
Am besten mal 4 Wochen in den Fachbereich schicken und ihn dort mitarbeiten lassen

Codierknecht

[Smalltalk]

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

Dann ist er aber auch kein guter Programmierer?

Sagen wir's mal so: Es hält sich in Grenzen
Ich maße mir aber auch nicht an, das beurteilen zu können.
DCOM / COM+ ist inzwischen aber auch reichlich exotisch und Delphi als Sprache leider auch etwas aus der Mode gekommen.

Aber wie im SAP ein Materialstamm aufgebaut ist (A-, B-, C- und D-Segment) und wie daraus anhand von Primärbedarfen über die Stücklistenauflösung Fertigungsaufträge entstehen, sollte man nach 3 Jahren im Schlaf runterbeten können.

Aber lassen wir das.
Mühsam ernährt sich das Eichhörnchen - bis zu meinem Renteneintritt hat er's dann hoffentlich drauf.

NEWS

[gelöst] Elegantere Programmierung?

Support us

870
Online

31.7k
Users

79.8k
Topics

1.3m
Posts

NEWS

[gelöst] Elegantere Programmierung?

Support us

870Online

31.7kUsers

79.8kTopics

1.3mPosts

870
Online

31.7k
Users

79.8k
Topics

1.3m
Posts