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

[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

[paul53]

@codierknecht sagte: getObject() liest aber asynchron.

getObject(id) liest synchron, wenn keine Callback-Funktion übergeben wird.

[Codierknecht]

@paul53
Ich sag' ja: An dem synchron/synchron Kram verzweifle ich noch

[haus-automatisierung]

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

An dem synchron/synchron Kram verzweifle ich noch

Ich verwende einfach immer die ...Async-Funktionen. Dann kann ich sicher sein, dass ich ein Promise zurückbekomme und kann damit dann halt mit await oder .then() weitermachen... Callbacks sind ja irgendwie retro und führen nur zu stark verschachteltem Code.

[Codierknecht]

@haus-automatisierung sagte in Skript gibt [objekt Objekt] aus:

Ich verwende einfach immer die ...Async-Funktionen

Ich ja mittlerweile auch.

Was aber auch daran liegen mag, dass ich das ganze Konzept noch immer nicht verinnerlicht habe.
Ich komme halt aus der Windows-Welt. Und da muss man Klimmzüge machen, um etwas asynchron hinzukriegen und nicht, um etwas synchron ablaufen zu lassen.

[fischi87]

@paul53

also müsste es so laufen wie im ersten post? jetzt bin ich verwirrt!

[paul53]

@fischi87 sagte also müsste es so laufen wie im ersten post?

Nicht ganz (Zeilen 50, 51)

        var obj = getObject(id);
        var name = getObject(id).common;

Richtig:

        var name = getObject(id).common.name;
        if(typeof name == 'object') name = name.de;

[haus-automatisierung]

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

also müsste es so laufen wie im ersten post?

War wohl etwas viel Offtopic hier, aber fehlt Dir hier? https://forum.iobroker.net/topic/72999/skript-gibt-objekt-objekt-aus/17

[fischi87]

@paul53

Sind denn da die Fehlermeldung bei .push und .sort auch behoben oder liegt die dann an einem anderen fehler?

Danke dir erstmal.

Ich glaube die Antwort auf die Frage nach dem „wie ist es richtig“ bzw „wie ist es am korrektesten“ wird man wohl nie beantworten können?

[paul53]

@fischi87
Zur Überschrift: Zeile 68

    if (logging) log("Text: " + textUpdateAn);

zeigt "Text: [objekt Objekt]", da textUpdateAn ein Array enthält. Entweder

    if (logging) log("Text: " + JSON.stringify(textUpdateAn));

oder

    if (logging) log(textUpdateAn);

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

Sind denn da die Fehlermeldung bei .push und .sort auch behoben

Ich sehe im ersten Post keinen Grund, weshalb es nicht funktionieren sollte.

EDIT: Durch falsch gewählte Variablenbezeichner (textUpdateAn) kann man sich auch selbst verwirren.

[haus-automatisierung]

@fischi87 Habe mir jetzt das ganze Script nochmal angeschaut. Dein Problem ist .each und async. Das geht so nicht.

const logging = true;
 
const idbutton = 'shelly.0.info.update';
 
const idAnzahlAn = 'javascript.0.Status.Shelly.UpdateFirmware.An';
const idAnzahl = 'javascript.0.Status.Shelly.UpdateFirmware.Anzahl';
const idText = 'javascript.0.Status.Shelly.UpdateFirmware.Text';
const iddatenpunkt = 'javascript.0.Status.Shelly.UpdateFirmware.State';
 
createState(idAnzahl, { // wenn benötigt: Anzahl der vorhandenen Lichter
    type: 'number',
    name: 'Anzahl aller Geräte',
    min: 0,
    def: 0,
    role: 'value'
});
createState(idAnzahlAn, { // Anzahl der Lichter, die auf sind als Variable unter Javascript.0 anlegen
    type: 'number',
    name: 'Anzahl Update',
    min: 0,
    def: 0,
    role: 'value'
});
createState(idText, { // Anzahl der brennenden Lichter und deren Namen als Variable unter Javascript.0 anlegen
    type: 'string',
    name: 'Eingeschaltete Update',
    desc: 'Namen der Update Geräte',
    def: ' ',
    role: 'value'
});
createState(iddatenpunkt, {
    type: 'boolean',
    name: 'State',
    role: '',
    read: true,
    write: false,
    def: false
});

const cacheSelectorState = $('shelly.0.*[state.id=*.firmware]');

async function checkDevices() {
    let anzahlUpdateAn = 0;
    let anzahlUpdate = 0;
    const textUpdateAn = [];
 
    for (const id of cacheSelectorState) {
        const obj = await getObjectAsync(id);
        const oName = typeof obj.common.name === 'object' ? obj.common.name.de : obj.common.name;
        const state = await getStateAsync(id).val;
 
        if (state.val) { 
            anzahlUpdateAn++;
            textUpdateAn.push(oName); 
        }
        anzahlUpdate++;
 
        setState(iddatenpunkt, anzahlUpdateAn > 0, true);
    }
 
    textUpdateAn.sort();
 
    if (logging) log(`Text: ${JSON.stringify(textUpdateAn)}`);
    if (logging) log(`Anzahl Geräte: ${anzahlUpdate} # davon: ${anzahlUpdateAn} Updaten`);

    setState(idText, textUpdateAn.join(',<br>')); 
    setState(idAnzahlAn, textUpdateAn.length); 
    setState(idAnzahl, anzahlUpdate); 
 
    if (anzahlUpdateAn > 0 ) {
        sendTo('telegram.0', {
            text: `+++ Geräte Update +++\n\n + ${textUpdateAn.join(', ')}`,
            reply_markup: {
                keyboard: [['System', 'Heizung'], ['Licht', 'Strom'], ['Multimedia', 'Szenen'], ['Home']],
                resize_keyboard: true,
                one_time_keyboard: true
            }
        });
    }
}

// Trigger
cacheSelectorState.on(checkDevices);

function main() {
    setTimeout(() => {
        if (logging) {
            log('Auslöser Skriptstart');
        }

        checkDevices();
    }, 2000);
}

function button() { 
    setTimeout(() => {
        setState(idbutton, false);
    }, 200);
}

on(idbutton, (dp) => {
    if (dp.state.val) {
        button();
    }
});

main(); // Skriptstart-Auslöser

[paul53]

@haus-automatisierung sagte: Problem ist .each und async.

Ich kann im ersten Post kein async entdecken.

[fischi87]

@haus-automatisierung

Soviele Wege

Das ist echt viel Input. Hab noch paar solche Skripte die eigentlich laufen aber wenn ich das jetzt sehe mit 100% Wahrscheinlichkeit auch nicht richtig sind.

Danke euch erstmal, werd heut Abend Erfolg oder Misserfolg vermelden!

[fischi87]

@paul53

Was meinst du selber verwirren? Ich möchte die Geräte ausgeben die ein verfügbares Update haben. Wie meinst du das? Wie im Skript lass ich mir das auch als Telegram Nachricht senden, da bekomme ich auch nur [objekt Objekt] ausgegeben. Also nicht nur im log?!

[fischi87]

@haus-automatisierung

Wofür ist dieses Zeichen? Bzw was besagt oder besser wo ist der Unterschied ohne?

=>

[paul53]

@fischi87 sagte: Telegram Nachricht senden, da bekomme ich auch nur [objekt Objekt]

Ja, weil auch dort ein Array in textUpdateAn übergeben wird.

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

Was meinst du selber verwirren?

Wenn eine Variable textUpdateAn bezeichnet ist, erwartet man einen String und kein Array - wie die Verwendung für Telegram zeigt.

[fischi87]

@paul53

Wie würdest du diese benennen damit man weiß es ist ein Array? Nur zum Verständnis

[paul53]

@fischi87 sagte: Wie würdest du diese benennen damit man weiß es ist ein Array?

Siehe Funktion checkDevices(). Ersatz für Zeilen 42 bis 86 (erster Post):

const idsUpdate = $('shelly.0.*.firmware'); 
setState(idAnzahl, idsUpdate.length, true); 
 
function checkDevices() {
    const arrUpdate = [];
    idsUpdate.each(function (id) { 
        if(getState(id).val) { 
            let name = getObject(id).common.name;
            if(typeof name == 'object') name = name.de;
            arrUpdate.push(name); 
        }
    });

    arrUpdate.sort();
    const anzahlUpdateAn = arrUpdate.length;
    setState(iddatenpunkt, anzahlUpdateAn > 0, true);
    const textUpdateAn = arrUpdate.join(', ');
 
    if (logging) log("Text: " + textUpdateAn);
    if (logging) log(`Anzahl Geräte: ${idsUpdate.length} # davon: ${anzahlUpdateAn} Updaten`);
    setState(idText, arrUpdate.join(',<br>'), true); 
    setState(idAnzahlAn, anzahlUpdateAn, true); 
 
    if (anzahlUpdateAn > 0 ) {
        sendTo('telegram.0', {
            text:   '+++ Geräte Update +++' +
                '\n' +
                '\n' + textUpdateAn,
            reply_markup: {
                keyboard: [['System','Heizung'],['Licht','Strom'],['Multimedia','Szenen'],['Home']],
                resize_keyboard: true,
                one_time_keyboard: true
            }
        });
    }
}

In Zeile 17 wird das Array in einen Text gewandelt.