[gelöst] - Learning JS - ein erster Versuch

[Codierknecht]

@oliverio
Den Verdacht hatte ich auch schon - hilft nix
Hab's dann wieder zu "error" zurück geändert, weil das so in fast allen Dokus verwendet wird.
Der Scope ist ja auch ein völlig anderer als das "error" in der Callback.
Es wird ja auch gemeckert, dass die gar nicht deklariert wäre. Dann müsste ich aber try-catch völlig falsch verstanden haben.

Die 3 Punkte im Callback meckern nur an, dass hier keine Typdeklaration vorhanden ist.
Da wir hier JS und kein TS haben, habe ich das mal geflissentlich ignoriert

P.S.: Es meckert auch nur VSCode - im Editor im ioB sieht's OK aus

[OliverIO]

@codierknecht

Evtl gilt hier dies
https://stackoverflow.com/questions/68240884/error-object-inside-catch-is-of-type-unknown

[Codierknecht]

Jetzt habe ich das zum Testen mal im ioB laufen lassen.
Dabei stolpere ich über das valueArray.length.
Wenn die Zeile drin ist, bringt mit das Log ein "undefined".
Kommentiere ich das aus, läuft das Script wie gewünscht und schreibt auch die erwartbaren Werte in die DP.

Interessant: Das Error-Handling funktioniert wie erwartet. Alles tacko

[OliverIO]

@codierknecht
kannst du bitte den source posten?

[Codierknecht]

@oliverio Voilá

Spoiler

const execPython = 'python /opt/SunnyPortal/SunnyPortal.py 192.168.178.20';
const dpPrefix = '0_userdata.0.SunnyPortal.';
const arraySize = 6;
const position = {"watt": 1, "today": 2, "total": 3};

createState(dpPrefix + 'Watt', 0, {name: 'Aktuelle Leistung', type: 'number', unit: 'W'});
createState(dpPrefix + 'Heute', 0, {name: 'Produziert heute', type: 'number', unit: 'kWh'});
createState(dpPrefix + 'Gesamt', 0, {name: 'Produziert gesamt', type: 'number', unit: 'kWh'});

function parseValue(value) {
  var result = parseFloat(value);
  if (!isNaN(result)) {
    return result;
  } else {
    throw 'Value <' + value + '> could not be converted';  
  }
}

function writeValues(watts, today, total) {
  setState(dpPrefix + 'Watt', watts, true);
  setState(dpPrefix + 'Heute', today, true);
  setState(dpPrefix + 'Gesamt', total, true);
};

function getData() {
  try {
//    exec(execPython, function (error, result) {
//      if (!error) {
var result = 'X):100):150.55):200.88):Y):Z):blubb):blah';    
        log(result);
        var valueArray = result.split('):');
        if (valueArray.length != 6) throw "Result does not match expected length";
        writeValues(parseValue(valueArray[position.watt]), parseValue(valueArray[position.today]), parseValue(valueArray[position.total]));
//      } else {
//        throw error;
//      }
//    });
  } catch(err) {
    log('Error: ' + err.message);
  }
}

getData();

//schedule('*/15 5-22 * * *', getData);

[OliverIO]

@codierknecht sagte in Learning JS - ein erster Versuch:

Da du

throw "Result does not match expected length"

machst wird an das catch auch nur der Text übergeben und nicht ein Error-Objekt.
Daher ist err.message undefined.
Korrekt wäre ein generisches Error-Objekt zu erzeugen

throw Error("Result does not match expected length")

Dann klappt das auch mit dem Stacktrace bei der Fehlerausgabe, falls man den Fehler im catch den Fehler nach Prüfung dann doch rethrown möchte mit

throw error

[Codierknecht]

@oliverio
Danke - kaum macht man's richtig, klappt's auch.

Hab's im Eingangspost nachgezogen.

[Codierknecht]

@codierknecht
Ich denke, mit dem aktuellen Ergebnis kann ich zufrieden sein.
"Da wirst'e alt wie 'ne Kuh und lernst immer noch dazu".

Und falls @jprisum-de hier mitliest:
Natürlich ist das Script jetzt deutlich umfangreicher als das ursprüngliche.
Hat aber einige entscheidende Vorteile:

• Es ist vor allem lesbarer! Es ist damit intuitiv verständlich und leichter zu warten und zu erweitern.
• Fehlerbehandlung: Mögliche Fehler werden abgefangen und mit
  einer aussagekräftige Meldung im Log protokolliert.

Ich mach' dann hier mal zu ...

[Homoran]

@codierknecht sagte in [gelöst] - Learning JS - ein erster Versuch:

Ich mach' dann hier mal zu ...

kannst du nicht

[Codierknecht]

@homoran
Achtung Goldwaagenalarm. Virtuell natürlich