[gelöst] switch / case mit Wert aus state liefert undefined… wieso?

tempestas

Guten morgen,

ich habe in den letzten Tagen etwas an meinem Ansageskript für die morgendliche Begrüßung im Bad rumgespielt.

Bislang war es so, dass im Skript selbst festgelegt wurde, ob man die Terminvorschau nur für heute, bis morgen oder bis einschließlich übermorgen hören wollte.

Das funktionierte auch wunderbar

Skriptextrakt (zusammengekürzt auf die relevanten Bereiche):

! ````
//*******************************************
//*******************************************************************************************************
// Festlegen, ob nur heute [1], heute und morgen [2] oder einschliesslich uebermorgen angesagt wird [3]
//*******************************************************************************************************
! var terminAnsage = 1; // hier festlegen

    // Text kürzen je nach gewählter Selektion
    // ******************************************

    if (termineCount === 0) { 
        inhaltStringText ="Es liegen keine Termine an";

    }
    else {
        switch(terminAnsage) {
            case 1:
                if (terminHeuteCount === 0) { 
                    inhaltStringText ="Es liegen keine Termine an";
                }
                else{
                nthIndex(inhaltStringReplace,"§$%", terminHeuteCount);                // nutzen der Suchfunktion zum Suchen der n-ten sinnlosen Zeichenkette
                inhaltStringText = inhaltStringReplace.slice(0,i_search);
                for (k =0; k < terminHeuteCount; k++) {
                inhaltStringText = inhaltStringText.replace("§$%", "");             // rausnehmen der sinnlosen Zeichen, damit diese nicht mitgesprochen werden
                }
                }
            break;

            case 2:
                if (termineHeuteMorgenCount === 0) { 
                    inhaltStringText ="Es liegen keine Termine an";
                }
                else{
                nthIndex(inhaltStringReplace,"§$%", termineHeuteMorgenCount);            // nutzen der Suchfunktion zum Suchen der n-ten sinnlosen Zeichenkette
                inhaltStringText = inhaltStringReplace.slice(0,i_search);
                for (k =0; k < termineHeuteMorgenCount; k++) {
                inhaltStringText = inhaltStringText.replace("§$%", "");             // rausnehmen der sinnlosen Zeichen, damit diese nicht mitgesprochen werden
                }          
                }                    
            break; 

            case 3:
                if (termineCount === 0) { 
                    inhaltStringText ="Es liegen keine Termine an";
                }
                else{
                nthIndex(inhaltStringReplace,"§$%", termineCount);                // nutzen der Suchfunktion zum Suchen der n-ten sinnlosen Zeichenkette
                inhaltStringText = inhaltStringReplace.slice(0,i_search);
                for (k =0; k < termineCount; k++) {
                inhaltStringText = inhaltStringText.replace("§$%", "");           // rausnehmen der sinnlosen Zeichen, damit diese nicht mitgesprochen werden
                }
                }

        }
    }

! ````

Nun habe ich es abgewandelt, so dass man in VIS via Dropdown diese Auswahl treffen kann.

Den Code habe ich zunächst so abgewandelt bzw ergänzt (wieder relevante Extrakte):

! ````
! createState('Ansage.Terminvorschau','1', force, {
name: 'Termine heute, morgen oder bis einschließlich übermorgen',
type: 'number',
min: '1',
max: '3',
states: '1:heute;2:morgen;3:übermorgen',
});
! // IDs
! var idTerminvorschau = "javascript.0.Ansage.Terminvorschau"/Termine heute, morgen oder bis einschließlich übermorgen/;
! //*******************************************************************************************************
// Festlegen, ob nur heute [1], heute und morgen [2] oder einschliesslich uebermorgen angesagt wird [3]
//*******************************************************************************************************
! var terminAnsage = getState(idTerminvorschau).val;

    [hierunter dann wieder der switch case wie oben]

! ````

Das liefert mir dann "undefined" zurück.

Dann dachte ich, ok, liegt eventuell daran, dass getState noch nicht "durch" ist, wenn die Ansage erfolgen soll und er folglich nicht weiß, welchen Case er nehmen soll.

Dann habe ich es so angewandelt:

! ````
! //**********************************************************************************************************************************
// Festlegen, ob nur heute [1], heute und morgen [2] oder einschliesslich uebermorgen angesagt wird [3] --> wird via VIS festgelegt
//**********************************************************************************************************************************
!
var terminAnsage; <--- aus dem Ansagebereich rausgeholt, damit überall verfügbar

on({id: idTerminvorschau, change: "ne"}, function(){
checkTermine();
});

function checkTermine(){
terminAnsage = getState(idTerminvorschau).val;
if(debug) log("Terminvorschau wurde auf "+terminAnsage+" Tage eingestellt");
}

! [hierunter dann wieder der switch case wie oben]
! ````

Das Log liefert auch wunderbar die richtig eingestellte Anzahl der Tage, allein der switch case Teil liefert wieder "undefined".

Ich verstehe es nicht. Wieso funktioniert es, wenn ich

terminAnsage = 1
````eingebe (oder 2 oder 3), aber wenn die Zahl aus einem State gezogen wird, dann mag das Skript nicht.

Habs vor lauter Verwirrung schon als String versucht, hat auch nicht geholfen.

w3schools hat mir keine neuen Erkenntnisse geliefert, was ich falsch mache.

Also bitte ich hier die Könner um Hilfe und Erleuchtung

Danke euch vorab

apollon77

Das State gibt es? Auch genau mit diesem Namen?

Was gibt denn ein "console.log(JSON.stringify(getState(idTerminvorschau)));" aus?

Du führst das Skript auch in JavaScript-Instanz 0 aus?

BuZZy

var terminAnsage = parseInt(getState(idTerminvorschau).val);
switch(terminAnsage) { ..... bla bla und so weiter
````:lol:

Gruß

tempestas

Hi,

danke für deine Rückmeldung.

Ja, den State gibt es. Das Log liest ja auch diesen State aus (deshalb habe ich das getestet).

Ja, alles unter javascript.0 Instanz

Warum json.stringify?

Arbeitet case switch nur mit strings?

tempestas

~~@BuZZy:~~

var terminAnsage = parseInt(getState(idTerminvorschau).val);

Gruß `

Danke, teste ich sofort. Frage: wieso parseInt? Dachte, state als Number sollte dafür sorgen, dass es nicht notwendig ist?

apollon77

~~@tempestas:~~

Warum json.stringify?

Arbeitet case switch nur mit strings? `

Nein aber wenn der "state.val" undefined ist dann will ich wissen wie das gesamte State Objekt aussieht was zurückgegeben wird weil dort ggf eine Fehlermeldfung steht!

Deswegen auch nur console.log Sollst ja nicht das benutzen, ich will die Ausgabe sehen

tempestas

Buzzy: hat funktioniert, danke dir!

Wäre ich im Leben nicht drauf gekommen.

Bitte sag mir, dass du beruflich was mit IT machst, sonst habe ich das Gefühl, ich bin total verblödet (meine Lernkurve im vgl zu deiner :oops:

apollon77

Was kommt den raus? immer 0?

Wenn der state.val===undefined ist kommt das immer eine 0 raus … in Meinen Augen also auch nicht unbedingt was DFu willst ... wie gesagt: wenn state.val===undefined

BuZZy

~~@tempestas:~~

Bitte sag mir, dass du beruflich was mit IT machst, sonst habe ich das Gefühl, ich bin total verblödet (meine Lernkurve im vgl zu deiner :oops: ` Muss dich leider enttäuschen.. Bin gelernter KFZ Mechatroniker - habe der KFZ Branche aber den Rücken gekehrt und arbeite jetzt in einem Betrieb der ESG (Einscheiben Sicherheitsglas) herstellt.. Sitze hier im Büro und steuere die Produktion… :lol:

Aber: kleiner Wehrmutstropfen.. Ich kümmere mich hier in der Firma auch um die IT.... So nebenbei.. :lol:

~~@apollon77:~~

Was kommt den raus? immer 0? ` Denke nicht, da er weder 0 noch undefined als Case im Switch block angegeben hat. Sonst hätte er (hoffe ich) nicht geschrieben dass es funktioniert.

Gruß

tempestas

Apollon:

parseInt hats gelöst.

@ Buzzy: ok, nun fühle ich mich minderbemittelt :oops: :lol:

Aber Frage: wie bist du drauf gekommen, dass es parseInt sein muss? Dachte, mit der deklaration des States als number wäre das Thema durch?

parseInt hätte ich genutzt (bzw nutze ich in Skripten), wenn ich Zahlen aus strings extrahieren will

Edit:

Apollon:

dein Hinweis liefert das hier:

{"val":"\n3","ack":false,"ts":1516956173260,"q":0,"from":"system.adapter.cloud.0","lc":1516956173260}
````(hatte beim Testen jetzt die drei eingestellt)

\n ist ein Zeilenumbruch imho. Das erklärt dann, warum parseInt das Thema löst. Wie der \n da rein kommt weiß ich aber nicht

Edit 2:

Das erklärt mir jetzt auch, warum im Log immer stand:

javascript.0 2018-01-26 08:16:28.464 info 2 Tage eingestellt
javascript.0 2018-01-26 08:16:28.464 info script.js.common.Ansage_Morgens: Terminvorschau wurde auf


Seltsamerweise habe ich das aber nur, wenn ich mich per .pro einlogge.

Gestern abend daheim war das Log ganz normal in einer Zeile geschrieben.

Edit 3:

Ahh klar, der Zeilenumbruch wird wohl aus dem VIS jqui select widget kommen. Dort stehen unter Werte ja

1;

2;

3;

verstanden! (hoffe ich)

BuZZy

~~@tempestas:~~

Aber Frage: wie bist du drauf gekommen, dass es parseInt sein muss? Dachte, mit der deklaration des States als number wäre das Thema durch?

parseInt hätte ich genutzt (bzw nutze ich in Skripten), wenn ich Zahlen aus strings extrahieren will `

Wenn du````
log(typeof getState(idTerminvorschau).val);


Obwohl der Datenpunkt als number im ioBroker definiert wurde.

Deswegen musst du ihn in ein integer umwandeln..

Hättest du case "1" ... case "2" ... gemacht hätte es wahrscheinlich auch funktioniert - ohne es vorher mit parseInt umzuwandeln.. ;) :)

Gruß

tempestas

ok danke.

Hier sind aber auch Füchse am Werk.

Gruß ebenfalls aus dem nördlichen Speckgürtel

BuZZy

Ah warte mal, hab mir dein Script nochmal angesehen..````
createState('Ansage.Terminvorschau','1', { ... });

createState('Ansage.Terminvorschau',1, { ... });

(OHNE ' ' um den Wert 1 herum)..

Dann hast du die 1 als Number drin stehen! ;)

Jetzt ist wieder die Frage was schreibt VIS in den Datenpunkt.. Wahrscheinlich einen String, ich vermute da kommt auch der Zeilenumbruch \n her.. Also ist der Weg mit parseInt wohl unumgänglich.. (Weiß nicht ob man im VIS irgendwo einstellen kann dass nur number eingetragen werden soll im Datenpunkt..) :D

Gruß

tempestas

~~@BuZZy:~~

Ah warte mal, hab mir dein Script nochmal angesehen..````
createState('Ansage.Terminvorschau','1', { ... });
createState('Ansage.Terminvorschau',1, { ... });
(OHNE ' ' um den Wert 1 herum)..

Dann hast du die 1 als Number drin stehen! ;)

Jetzt ist wieder die Frage was schreibt VIS in den Datenpunkt.. Wahrscheinlich einen String, ich vermute da kommt auch der Zeilenumbruch \n her..

Gruß `  

Hi,

das ist ein C&P Fehler. Ich hatte zwischenzeitlich zum Testen mal auf string gestellt und dann die '' gesetzt. Vorher war es nur Number und ohne ''

Den Grund habe ich nun denke ich entdeckt, steht paar Postings weiter oben; Apollons Hinweis hat mich auf die Spur der Erklärung gebracht, dein Posting hats gelöst.

NEWS

[gelöst] switch / case mit Wert aus state liefert undefined… wieso?

Support us

842
Online

31.7k
Users

79.8k
Topics

1.3m
Posts

NEWS

[gelöst] switch / case mit Wert aus state liefert undefined… wieso?

Support us

842Online

31.7kUsers

79.8kTopics

1.3mPosts

842
Online

31.7k
Users

79.8k
Topics

1.3m
Posts