Fehlermeldung, die ich nicht verstehe... [gelöst]

mcm1957

@paul53
OK und sorry,
hab nicht überzuckert dass die Meldung schon im Editor kommt und nicht erst zur Laufzeit.

Wüsste aber dann nicht, inwiefern deine Vermutung dass es an den drei nahezu identen Scripts liegen könnte zutreffen könnte. Der Editor kann ja kaum wissen, dass es noch andere Scripte am System gibt - oder ?

Bin mal gespannt ob ein erfahrenerer Scripter da was sieht.

McM

OliverIO

@mcm57

das wäre jetzt auch mein tip.
du hast die variable bereits in einem global-skript definiert.

hat zwar mit dem Fehler nix zu tun, aber diese Anweisungen machen keinen Sinn

        setTimeout(function() {
        }, 1000);

wenn du sie später noch füllen willst, gut,
dann würde aber eine auch reichen.

und noch ein Fehler
du willst doch sicherlich vergleichen?
So weist du aber ein true zu.
zum vergleichen benötigst du ==

    if (data.state.val = true) {

der nachfolgende teil wird dann immer ausgeführt, da eine Zuweisung immer den zugewiesenen wert zurückgibt und daher ist hier immer alles true

und mit dem einrücken stimmt hier was nicht
das macht den code schwer lesbar

so müsste es richtig aussehen


// ##################################
// Deklarationen
// ##################################

var sperre = false;
const IDklingelsensor = 'hm-rpc.0.MEQ0656853.1.PRESS_SHORT' /*Klingelsensor Trepph :1 PRESS SHORT*/;

// ##################################
// Programm
// ##################################

on({
    id: IDklingelsensor
}, function (data) {
    if (data.state.val = true) {

        if (!sperre) {
            sperre = true;

            //Ansagetext
            const text = "Es klingelt an der neuen Haustür unten";
            //log("läuft");

            // Badezimmer
            setState('alexa2.0.Echo-Devices.G090LF09716403QV.Commands.speak-volume' /*Volume to use for speak commands*/, 100)
            setState('alexa2.0.Echo-Devices.G090LF09716403QV.Commands.speak' /*speak*/, text);
            setTimeout(function () {}, 1000);

            // Buero;
            setState('alexa2.0.Echo-Devices.G090XG1215160GQD.Commands.speak-volume' /*Volume to use for speak commands*/, 100)
            setState('alexa2.0.Echo-Devices.G090XG1215160GQD.Commands.speak' /*speak*/, text);
            setTimeout(function () {}, 1000);

            // Kueche;
            setState('alexa2.0.Echo-Devices.G090LF0965231J5Q.Commands.speak-volume' /*Volume to use for speak commands*/, 100)
            setState('alexa2.0.Echo-Devices.G090LF0965231J5Q.Commands.speak' /*speak*/, text);
            setTimeout(function () {}, 1000);

            // Wohnzimmer;
            setState('alexa2.0.Echo-Devices.G090LF09716403QV.Commands.speak-volume' /*Volume to use for speak commands*/, 100)
            setState('alexa2.0.Echo-Devices.G090LF09716403QV.Commands.speak' /*speak*/, text);

            setTimeout(function () {
                sperre = false;
            }, 5000);
        }
    }
});

paul53

@skorpil sagte:

on({id: IDklingelsensor}, function (data) {
    if (data.state.val = true) {

Das ersetze besser durch

on({id: IDklingelsensor, val: true}, function () {

skorpil

@mcm57

Danke für den Hinweis. Richtig, es geht nicht um eine Adapterprogrammierung, sondern um ein Script im Script Adapter.

Ich bin nicht sicher, ob das der Ordner GLOBAL ist. Hier ein Screenshot

Alle meine Scripte stehen nicht unter "common", sondern ich habe im Stammverzeichnis Unterordner angelegt. Vielleicht ist das nicht richtig. Was sollte ich tun?

paul53

@skorpil sagte: im Stammverzeichnis Unterordner angelegt. Vielleicht ist das nicht richtig.

Das ist in Ordnung.

skorpil

@oliverio

Ich habe eben einen Screenshot gepostet. Wenn der Fehler tatsächlich durch Vewendung des "GLOBAL" entsteht, reich es dann, sie in common zu verschieben? Oder was ist Dein Rat?
beim Timeout habe ich vermutlich falsch programmiert. Ich wollte eigentlich nur eine Pause von einer Sekunde einbauen, bevor der nächste Teil aufgerufen wird. Wie wäre das richtig?
ich schaue mir das mit dem "==" an und werde es korrigieren.
...ebenso das Einrücken

Vielen Dank, ich sehe, ich muß viel lernen...

skorpil

@paul53 sagte in Fehlermeldung, die ich nicht verstehe...:

@skorpil sagte:

on({id: IDklingelsensor}, function (data) {
    if (data.state.val = true) {

Das ersetze besser durch

on({id: IDklingelsensor, val: true}, function () {

werde ich mörgen testen. Danke

paul53

@skorpil sagte: eine Pause von einer Sekunde einbauen, bevor der nächste Teil aufgerufen wird. Wie wäre das richtig?

await wait(1000);

skorpil

@oliverio sagte in Fehlermeldung, die ich nicht verstehe...:

das wäre jetzt auch mein tip.
du hast die variable bereits in einem global-skript definiert.

lt. paul53 scheine ich das mit den Ordnern richtig gemacht zu haben. Was ist dann falsch. Bin verwirrt.

skorpil

@paul53

so einfach, daaaanke

paul53

@skorpil sagte: Was ist dann falsch. Bin verwirrt.

Wie schon geschrieben: Ignorieren! Manchmal liefert die Syntaxprüfung des Editors Fehler, die keine sind.

skorpil

Ich werde morgen mal die ganzen Hinweise, für die ich sehr dankbar bin, einbauen und schauen, was passiert. Danach poste ich das überarbeitete Script noch mal. Danke Euch allen.

OliverIO

@skorpil
ja seltsam.
so wie es aussieht hast du den global ordner nicht mal eingeblendet (expertenansicht).
dann wird es so sein, das es ein problem mit der syntaxprüfung ist

wenn ich das skript allerdings bei mir einfüge, wird nicht gemeckert.

skorpil

@oliverio Danke für den Hinweis. Ich wusste nicht mal, dass man in der Expertenansicht global an und ausschaltet. Wieder was gelernt…

skorpil

@oliverio …sehr merkwürdig: jetzt habe ich mal den Expertenmodus aus- und wieder eingeschaltet. Jetzt wird der leere Ordner „global“ angezeigt. Er verschwindet aber auch nicht wieder…sollte er im Expertenmodus zu sehen sein oder umgekehrt?

OliverIO

@skorpil

im expertenmodus sichtbar
ohne sollte er ausgeblendet sein
also das
kopficon grün ist expertenmodus
kopficon weiß ist normal

skorpil

@oliverio

Offenbar spinnt meine Installation. Jetzt ist der Ordner immer da, egal ob Expertenmodus oder nicht.

Auch neu laden der Website bringt keine Veränderung.

Danke für die Hilfe. Ich melde mich morgen….

OliverIO

@skorpil
wenn er leer ist, ist das nicht so schlimm.
mach mal ctrl+reload knopf im browser
dadurch wird der cache geleert.
evtl hängt es daran

skorpil

@paul53 @OliverIO

So, jetzt habe ich das Script entsprechend Euren Vorschlägen angepasst:

die On Anweisung habe ich angepasst (womit vermutlich die If Anweisung mit der falschen Zuweisung "=", statt dem richtigen Vergleich mit "==" entfällt,
das Einrücken ebenfalls
die await Funktion (die mir noch Sorgen macht)


 // ##################################
// Deklarationen
// ##################################
 
var sperre = false;
const IDklingelsensor = 'hm-rpc.0.MEQ0656853.1.PRESS_SHORT' /*Klingelsensor Trepph :1 PRESS SHORT*/;
 
// ##################################
// Programm
// ##################################
 
on({
    id: IDklingelsensor, val: true
}, function () {
 
        if (!sperre) {
            sperre = true;
 
            //Ansagetext
            const text = "Es klingelt an der neuen Haustür unten";
            //log("läuft");
 
            // Badezimmer
            setState('alexa2.0.Echo-Devices.G090LF09716403QV.Commands.speak-volume' /*Volume to use for speak commands*/, 100)
            setState('alexa2.0.Echo-Devices.G090LF09716403QV.Commands.speak' /*speak*/, text);
            await wait(1000);
 
            // Buero;
            setState('alexa2.0.Echo-Devices.G090XG1215160GQD.Commands.speak-volume' /*Volume to use for speak commands*/, 100)
            setState('alexa2.0.Echo-Devices.G090XG1215160GQD.Commands.speak' /*speak*/, text);
            await wait(1000);
 
            // Kueche;
            setState('alexa2.0.Echo-Devices.G090LF0965231J5Q.Commands.speak-volume' /*Volume to use for speak commands*/, 100)
            setState('alexa2.0.Echo-Devices.G090LF0965231J5Q.Commands.speak' /*speak*/, text);
            await wait(1000);
 
            // Wohnzimmer;
            setState('alexa2.0.Echo-Devices.G090LF09716403QV.Commands.speak-volume' /*Volume to use for speak commands*/, 100)
            setState('alexa2.0.Echo-Devices.G090LF09716403QV.Commands.speak' /*speak*/, text);
 
            setTimeout(function () {
                sperre = false;
            }, 5000);
        }
    }
});

Ich bekomme nun eine Fehlermeldung in Zeile 27.

javascript.0 (20957) script.js.Alexa.Überarbeitet compile failed: at script.js.Alexa.Überarbeitet:27

Offenbar habe ich den Hinweis von @paul53 falsch interpretiert, wie man die Pause von 1 Sekunde implementiert. Wie wäre es denn richtig?

paul53

@skorpil sagte: die await Funktion (die mir noch Sorgen macht)

Ach ja: Wenn man await verwendet, muss man die Funktion als "async" deklarieren:

on({id: IDklingelsensor, val: true}, async function () {

NEWS

Fehlermeldung, die ich nicht verstehe... [gelöst]

Support us

748
Online

31.8k
Users

80.0k
Topics

1.3m
Posts

NEWS

Fehlermeldung, die ich nicht verstehe... [gelöst]

Support us

748Online

31.8kUsers

80.0kTopics

1.3mPosts

748
Online

31.8k
Users

80.0k
Topics

1.3m
Posts