UPDATE 31.10.: Amazon Alexa - ioBroker Skill läuft aus ?

[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 :-(

[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!