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

[fischi87]

Hallo Leute ich habe mal das Lichter Skript etwas angepasst um Firmware Update benachritigungen zu bekommen von meinen Shelly Autoren. Leider gibt das Skript mir im textUpdateAn nur: [objekt Objekt] aus, kann mir da jemand Helfen?

des Weiteren eine Frage, bei

textUpdate.push(name)

bekomme ich diesen Fehler angezeigt:

Property 'push' does not exist on type 'any[]'.

sowohl bei .sort und auch bei .lengh

 var 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, {
name: "State",
role: "",
type: "boolean",
read: true,
write: true,
desc: "Manuell erzeugt",
def: false
});


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

function checkDevices(obj) {
    var anzahlUpdateAn = 0;
    var anzahlUpdate = 0;
    var textUpdateAn = [];

    cacheSelectorState.each(function (id, i) { 
        var obj = getObject(id);
        var name = getObject(id).common;
        var status = getState(id).val;
        var devicename = name;

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

        var state = false;
        if(anzahlUpdateAn > 0) state = true;
        setState(iddatenpunkt, state, true);
    });

    textUpdateAn.sort();

    if (logging) log("Text: " + 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,
                reply_markup: {
                keyboard: [['System','Heizung'],['Licht','Strom'],['Multimedia','Szenen'],['Home']],
                resize_keyboard: true,
                one_time_keyboard: true
                }
        });
    }
}

// Trigger
cacheSelectorState.on(function(obj) { 
    if (logging) log('Auslösendes Gerät: ' + obj.id + ': ' + obj.state.val); 
    checkDevices();
});

function main() {
    setTimeout(function(){
        if (logging) log('Auslöser Skriptstart');
        checkDevices();
    }, 2000);
};

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

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

main(); // Skriptstart-Auslöser

danke euch Grüße

[MCU]

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

[fischi87]

@mcu

Danke für deine Antwort, werde es probieren.

Hast zu meinen anderen Fragen eine Idee?

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