[GELÖST] Skript gibt [objekt Objekt] aus

[Codierknecht]

@fischi87 sagte in Skript gibt [objekt Objekt] aus:

Hast zu meinen anderen Fragen eine Idee?

Ich mag mich täuschen, aber ich lerne ja selbst noch gerne dazu.
Kann es sein, dass da der Scope nicht passt und die Variable textUpdateAn innerhalb der anonymen Funktion für cacheSelectorState.each nicht bekannt ist?
Was passiert, wenn Du die als globale Variable deklarierst?

Vielleicht kann @OliverIO hier Licht ins Dunkel bringen?

Was mir noch aufgefallen ist:
Du schreibst
++variable
Ich persönlich finde das etwas unüblich.
Diese Notation kenne ich nur, wenn man eine Variable verwendet und sie vor der Verwendung inkrementiert werden soll.
Will man einfach nur hochzählen, verwendet man üblicherweise
variable++
Ist zwar semantisch das gleiche, liest sich aber irgendwie besser.

[ticaki]

@mcu sagte in Skript gibt [objekt Objekt] aus:

@fischi87

Sehr wahrscheinlich sollte es so sein?

var name = getObject(id).common.name 
/* besser */
var name = obj.common.name
/* obj wurde ja schon gesetzt */

Warum devicename der nicht genutzt wird?

ne so macht man das nicht. getObject wird mit callback verwendet.

async function main() 
{
  const obj = await getObjectAsync(id);
  let oName = ''
  if (obj && obj.common) {
    oName = obj.common.name
  }
}

Man testet auch die Keys die man benutzen will - wenn da einer von nicht existiert gibts sonst einen Javascript Adapter neustart oder wenn nicht in einer async funktion einen Fehler.

ach und name ist als Variable ungünstig. Da steht AFAIK der scriptname drin.

[fischi87]

@ticaki

Mit deiner Antwort bin ich etwas überfordert muss ich sagen. Das Skript gibt mir ja alle shelly Geräte aus die ein Update verfügbar haben nur halt den Namen nicht.

Ist dein Ansatz jetzt der neuen JavaScript Version geschuldet?

Komm auch mit deinem Code Schnipsel nicht so klar. Füg ich die Funktion hinzu oder ändere ich diese ab oder wie meinst du das?

[fischi87]

@codierknecht

Bin auch noch einiges am lernen. Hatte mir nur gedacht das ich ein funktionieren Skript nur anpasse, deswegen wundert es mich dass es so nicht funktioniert.

Kenn das mit ++variable dies anfang an deswegen liest sich das für mich besser

[Codierknecht]

@fischi87
Du liest z.B. in Zeile 51 den Namen über getObject(id).
In Zeile 53 willst Du den gleich weiter verarbeiten.

getObject() liest aber asynchron.
Das heißt, in Zeile 53 ist name noch gar nicht besetzt. Das asynchrone Lesen ist da nämlich mit ziemlicher Sicherheit noch gar nicht fertig.

@ticaki meint das vermutlich so (Code ab Zeile 49):

    cacheSelectorState.each(function (id, i) { 
        const obj = await getObjectAsync(id);
        if (obj && obj.common) {
            let oName = obj.common.name;
        }

[fischi87]

@codierknecht

Okay werde ich probieren.

Nur als Verständnis, ist das schon immer so weil ich meine hier vor paar Jahren das lichter Skript kopiert zu haben was genauso aufgebaut ist und funktioniert, deswegen erschließt sich mir dies im Moment nicht. Kann ja sein das sich bei version Änderung etwas geändert hat?

Wie gesagt das lichter, Fenster und lowbat Skript laufen immer noch ohne Probleme und da werden mir auch die Namen angezeigt.

Würde es gerne verstehen.

[Codierknecht]

@fischi87 sagte in Skript gibt [objekt Objekt] aus:

vor paar Jahren

So lange bin ich noch nicht dabei.
Ich kenne es nur so - also dass asynchron gelesen wird.
Das kann dann ohne await funktionieren, muss aber nicht. In der Regel tut es das nicht.

An den ganzen asynchronen Kram muss man sich erstmal gewöhnen. Ich habe da auch so meine liebe Not.

[fischi87]

@codierknecht

In deinen erwähnten Code schreibst du:

let oName = obj.common.name;

Kann ich denn dann oName außerhalb dieser Funktion nutzen? Dachte immer let kann man nur innerhalb der deklarierten Funktion nutzen?

[Codierknecht]

@fischi87 sagte in Skript gibt [objekt Objekt] aus:

Dachte immer let kann man nur innerhalb der deklarierten Funktion nutzen?

Da dürftest Du Recht haben.
Scope in JS ist auch eine meiner Baustellen

Mach halt ein var draus, wenn Du es brauchst.

[fischi87]

@codierknecht

Okay wollte es nur wissen nicht das ich da was falsches gelernt hab

[haus-automatisierung]

@codierknecht sagte in Skript gibt [objekt Objekt] aus:

Mach halt ein var draus, wenn Du es brauchst.

var braucht man nie immer nur let und const.

Ansonsten kann common.name natürlich auch ein Objekt sein, wenn es in mehrere Sprachen übersetzt ist. Also z.B. common.name.de wäre dann die Deutsche Übersetzung.

[fischi87]

@codierknecht

also sollte ich es so versuchen?

    cacheSelectorState.each(function (id, i) { 
        const obj = await getObjectAsync(id);

        if (obj && obj.common) {
            var oName = obj.common.name;
           var státus = getState(id).val;
        }

         if (status) {
               ++anzahlUpdateAn;
               textUpdateAn.push(Oname);
         }

[fischi87]

@haus-automatisierung

wie jetzt man bruacht nie var?

[haus-automatisierung]

@fischi87 sagte in Skript gibt [objekt Objekt] aus:

also sollte ich es so versuchen?

Bitte nicht Der Scope von status ist absolut blöde so.

cacheSelectorState.each(function (id, i) {
    let anzahlUpdateAn = 0;
    const obj = await getObjectAsync(id);

    if (obj && obj.common) {
        const oName = typeof obj.common.name === 'object' ? obj.common.name.de : obj.common.name;

        if (getState(id).val) {
            anzahlUpdateAn++;
            textUpdateAn.push(oName);
        }
    }
)};

Und achte darauf, Variablen RICHTIG wieder zu verwenden. Du schreibst oName und unten Oname. Das sind zwei komplett verschiedene Dinge. Genau wie státus

[haus-automatisierung]

@fischi87 sagte in Skript gibt [objekt Objekt] aus:

wie jetzt man bruacht nie var?

Na einfach NIE var ... schreiben. Siehst Du ja an deinem Code. Du deklarierst innerhalb des if ... eine Variable (z.B. oName) und möchtest diese außerhalb wieder verwenden. Das ist kein schöner Code und sollte man nie so machen.

Brauchst Du diese außerhalb, musst diese auch vorher außerhalb mit let deklarieren (falls diese geändert werden sollen). Ansonsten mit const arbeiten.

Google: JavaScript variable scopes https://www.w3schools.com/js/js_scope.asp

[Codierknecht]

@haus-automatisierung

Und achte darauf, Variablen RICHTIG wieder zu verwenden

Ähämmm ... räusper

textUpdateAn.push(oName);

[haus-automatisierung]

@codierknecht sagte in Skript gibt [objekt Objekt] aus:

Ähämmm ... räusper

Danke, im Forum programmieren ist so eine Sache

[Codierknecht]

@haus-automatisierung
Nur interessehalber:
Wie verwendest Du den Inkrement-Operator in einem solchen Fall:

++anzahlUpdateAn;

Vor oder nach der Variablen?

@fischi87
Es ist halt ein himmelweiter Unterschied, ob man das so
Schlechtes Beispiel. Siehe unten.

for (var i = 0; i < 100; i++)

oder so

for (var i = 0; i < 100; ++i)

verwendet.

[haus-automatisierung]

@codierknecht sagte in Skript gibt [objekt Objekt] aus:

Vor oder nach der Variablen?

Wenn der alleine in einer Zeile steht ist das völlig egal. Dann aber meistens hinter der Variable. Ist ja nur entscheidend, wenn man das direkt als Parameter übergibt o.ä.

@codierknecht sagte in Skript gibt [objekt Objekt] aus:

@fischi87
Es ist halt ein himmelweiter Unterschied, ob man das so

In dem Fall nicht. Beide gehen von 0 bis 99.

In einer for-Schleife mache ich es immer dahinter. Macht in deinem Beispiel aber auch keinen Unterschied bei der Schleife (weil es ja alleine steht). Hier wäre es ein Unterschied:

let i = 0;
do {
    console.log(i);
} while (++i < 100);
// 0 bis 99

let i = 0;
do {
    console.log(i);
} while (i++ < 100);
// 0 bis 100

[Codierknecht]

@haus-automatisierung
Asche auf mein Haupt. Schlechtes Beispiel