JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.

skorpil

@paul53 jetzt haben wir wieder den Salat. Ich benötighe Programmierhilfe bei der Umstellung von request auf httpGet.

Ich habe folgendes, beispielhafte Script (und noch einige mehr), die alle request nutzen.

// *****************************************
//              mp3 abspielen via AAPI
//******************************************

// *****************************************
//               Definitionen
// *****************************************

// Definition APIclip Wohnzimmer;
const idApiClipTerrasse = "http://192.168.X.XXX:5005/Terrasse/clip/Rolladen.mp3/50";
const idUrlaub = 'hm-rega.0.8359'/*URLAUB Hzg  Wohnb  ALLES (Var )*/;


var AufZu = 'hm-rega.0.1792'/*RollAufZu Logik*/;  

// Ansage spielen
on({id: AufZu, change: 'ne', val: true}, function (data) {
if(getState(idUrlaub).val == 0) {
try {
    require("request")(idApiClipTerrasse).on("error", function (e) {console.error(e);});
        } catch (e) { console.error(e); };
};
});

Der eigentliche Aufruf erfolgt in allen Scripten mit einem Codeschnipsel (den ich mir vor langer Zeit irgendwo heruntergeladen habe):

try {
    require("request")(idApiClipTerrasse).on("error", function (e) {console.error(e);});
        } catch (e) { console.error(e); };

wobei dann jeweils idApiClipTerrasse entsprechend auf eine andere Variable geändert wird.

Wie muß der jetzt geändert werden in httpGet? Ich bin mal wieder zu dumm. Danke für die Hilfe.

paul53

@skorpil sagte: Der eigentliche Aufruf erfolgt in allen Scripten mit einem Codeschnipsel (den ich mir vor langer Zeit irgendwo heruntergeladen habe):

Das ist der Javascript-Code des request-Blockes (Blockly).

skorpil

@paul53 aha. Und wie geht das dann mit httpGet? Ich muß ja umstellen

paul53

@skorpil sagte: wie geht das dann mit httpGet?

httpGet(idApiClipTerrasse, (error, response) => {
    if(error) {
        console.error(error);
        return;
    }        
    // nur falls bisher body ausgewertet wurde:
    const body = response.data;
    // bestehendes Programm
});

skorpil

@paul53 vielen Dank.

skorpil

@paul53 brauche ich das try noch?

try {
httpGet(idApiClipTerrasse, (error, response) => {
    if(error) {
        console.error(error);
        return;
    }        
    // nur falls bisher body ausgewertet wurde:
    const body = response.data;
    // bestehendes Programm
})} catch (e) { console.error(e);}

oder kann das erstzlos wegfallen?

paul53

@skorpil sagte: brauche ich das try noch?

Eher nicht. Die Auswertung von error sollte genügen.

skorpil

@paul53 mit anderen Worten:

try {
    require("request")(idApiClipTerrasse).on("error", function (e) {console.error(e);});
        } catch (e) { console.error(e); };

wird ersetzt durch


httpGet(idApiClipTerrasse, (error, response) => {
    if(error) {
        console.error(error);
        return;
    }        
    });

Richtig?

PS: Sorry, dass ich nochmal nachfrage, aber bevor ich jetzt alle Scripte umbaue, sollte ich alles richtig verstanden haben

paul53

@skorpil sagte: Richtig?

Richtig.
Der erste Parameter (idApiClipTerrasse) darf nur eine URL enthalten - kein Objekt.
Siehe Doku

skorpil

@paul53 alles klar. Nochmals ein großes Danke

skorpil

@paul53 sorry, noch zwei Fragen:

ich habe dieses Script von request auf httpGet entsprechend Deiner Anleitung umgeschrieben:

// ##################################################################

//                      Ansage

// ##################################################################



// ##################################################################
//                      Definitionen
// ##################################################################


// Quellen
var idWetter =           "hm-rega.0.29954";
var idTemperatursensor = 'hm-rpc.0.OEQ1296052.1.TEMPERATURE'/*Außentemperatur:1 TEMPERATURE*/
var idAnsageBuero = 'hm-rega.0.4529'/*AnsageAlexaBuero*/;
var idAnsageSchlafzimmer = 'hm-rega.0.4528'/*AnsageAlexaSchlafzimmer*/;
var idAnsageTerrasse = 'hm-rega.0.4626'/*AnsageAlexaTerrasse*/;
var idAnsageKueche = 'hm-rega.0.4625'/*AnsageAlexaKüche*/;
var idAnsageWohnzimmer = 'hm-rega.0.72055'/*AnsageAlexaWohnzimmer*/;



// ##################################################################
//                      Basiswerte ermitteln
// ##################################################################


function ermitteleAnsagedatum () {
    //Wochentag ermitteln
    var d = new Date ();
    var w = new Array("Sonntag","Montag","Dienstag","Mittwoch","Donnerstag","Freitag","Samstag");
    var wochentag = w[d.getDay()]; 

    //Tagesdatum ermitteln
    var tag = d.getDate();

    //Monat ermitteln
    var month = new Array("Januar","Februar","März","April","Mai","Juni","Juli","August","September","Oktober","November","Dezember");
    var monat = month[d.getMonth()];

    //Jahr ermitteln
    var jahr = d.getFullYear();

    //Stunde ermitteln
    var stunde = d.getHours();

    //Minute ermitteln
    var minute = d.getMinutes();

    //Begrüssung ermitteln
    var Begr = " ";
    if(stunde<=12){Begr = "Guten Morgen"};
    if(stunde>=12 && stunde<=18){Begr = "Guten Tag"};
    if(stunde>18){Begr = "Guten Abend"};
    
    return { // zurückgeben
        'Jahr'      : jahr,
        'Monat'     : monat,
        'Tag'       : tag,
        'Wochentag' : wochentag,
        'Stunde'    : stunde,
        'Minute'    : minute,
        'Begr'      : Begr
    };
}



function ermitteleWetter () { // Wetterbedingungen
    var wetterdaten = getState(idWetter).val;
//    log('Wetter: ' + wetterdaten);
    return(wetterdaten);
}

function ermitteleAnsageTemperatur () {
    // Die Außentemperatur ist xx. Das Wetter heute ist xx
    // Einfache Temperaturansage mit SayIt.
    // Variante 1 mittels splitten der Temperatur, damit die Ansage nicht
    // "Es sind 18 Punkt 2 Grad " sagt.

    var temperatursensor = getState(idTemperatursensor).val;
    var temp_string = temperatursensor.toString();
//    log('Temp  ' + temp_string);
    var temp_array = [];
    temp_array = temp_string.split(".");

    // Fange leere Nachkommastellen ab. Das passiert, wenn die Temperatur z. B. 18.0 ist.
    // Es wird dann nur "18" gelesen.
    if (!temp_array[1]) {
        temp_array[1] = "0";
//        log("Die Nach-Kommastelle in temp_array[1] war nicht vorhanden und wird nun fest auf 0 gesetzt.");
    }
    return{
        'Ganzzahl'      : temp_array[0],
        'Nachkommazahl' : temp_array[1]
    };
}

on([idAnsageBuero , idAnsageSchlafzimmer , idAnsageKueche , idAnsageTerrasse , idAnsageWohnzimmer], function(dp) {
   if(dp.state.val) {

    // ###########################################
    //                 ClearQueue
    // ###########################################
   
    // var APIQueueBuero = "http://192.168.0.105:5005/B%C3%BCro%20Ikea/clearqueue";
    // var APIQueueSchlafzimmer = "http://192.168.0.105:5005/Schlafzimmer/clearqueue";
   
    

// ##################################################################
//                          Basisansage
// ##################################################################

    
    var ansagetext = ermitteleAnsagedatum().Begr + ", heute ist " + ermitteleAnsagedatum().Wochentag + " der " + ermitteleAnsagedatum().Tag + "te " + ermitteleAnsagedatum().Monat + ' ' + ermitteleAnsagedatum().Jahr 
            + ". Es ist " + ermitteleAnsagedatum().Stunde + "  Uhr und " + ermitteleAnsagedatum().Minute + "  Minuten." 
            + " Die Aussentemperatur beträgt " + ermitteleAnsageTemperatur().Ganzzahl + "," + ermitteleAnsageTemperatur().Nachkommazahl + " Grad." 
            + " Wetter Bedingungen, " + ermitteleWetter() +" .";
            
            




// ##################################################################
//      Ansagetext zusammenstellen und in Objekt schreiben
// ##################################################################

// Info zum Verzeichnis     /home/brs/node-sonos-http-api-master/static/clips/;
// Definitionen MP3;
// herzlichWillkommen.mp3";
// var herz = "herzlichWillkommen.mp3";
// HundeGebell.mp3;
// var hund = "HundeGebell.mp3";
// Rolladen.mp3;
// var roll = "Rolladen.mp3";

// Definition APIsay Büro Ikea;
var APIsayBuero = "http://192.168.0.105:5005/B%C3%BCro%20Ikea/say/";

// Definition APIsay Schlafzimmer;
var APIsaySchlafzimmer = "http://192.168.0.105:5005/Schlafzimmer/say/";

// Definition APIsay Kueche;
var APIsayKueche = "http://192.168.0.105:5005/K%C3%BCche/say/";

// Definition APIsay Terrasse;
var APIsayTerrasse = "http://192.168.0.105:5005/Terrasse/say/";

// Definition APIsay Wohnzimmer;
var APIsayWohnzimmer = "http://192.168.0.105:5005/Wohnzimmer/say/";

var lautstaerke = 50;



// Ansagetext
        // Basisansage
        var Ansage = ansagetext;
        
        //Geburtstagsansage
        var tempGEBURTSTAGE = "javascript.0.Ansage.GeburtstageHEUTE";
        var AnsageGEBURTSTAGE = getState(tempGEBURTSTAGE).val;
        
        //Muellsansage heute
        var tempMUELLheute = "javascript.0.Ansage.MuellHEUTE";
        var AnsageMUELLheute = getState(tempMUELLheute).val;
        
        //Muellsansage Zukunft
        var tempMUELLzukunft = "javascript.0.Ansage.MuellZUKUNFT";
        var AnsageMUELLzukunft = getState(tempMUELLzukunft).val;

/*log("Tempansage--------------------->" + tempAnsage);
log("Ansage--------------------->" + Ansage);

log("tempGEBURTSTAGE--------------------->" + tempGEBURTSTAGE);
log("AnsageGEBURTSTAGE--------------------->" + AnsageGEBURTSTAGE);

log("tempMUELLheute--------------------->" + tempMUELLheute);
log("AnsageMUELLheute--------------------->" + AnsageMUELLheute);

log("tempMUELLzukunft--------------------->" + tempMUELLzukunft);
log("AnsageMUELLzukunft--------------------->" + AnsageMUELLzukunft);*/



// ####################################################
// Wenn Geburtstag dann mit ansagen
// ####################################################

    if (AnsageGEBURTSTAGE.length > 2) { // wenn der Inhalt des Objektes "AnsageGEBURTSTAGE" größer als 2 Zeichen lang ist, dann ...
        Ansage = Ansage + "Heute haben Geburtstag, " + AnsageGEBURTSTAGE +" .";
        //log("Ansage mit Geburtstag--------------------->" + Ansage);
    }
    
   
// ####################################################
// Wenn Müll dann mit ansagen
// ####################################################

    if (AnsageMUELLheute.length > 2) { // wenn der Inhalt des Objektes "AnsageMUELLheute" größer als 2 Zeichen lang ist, dann ...
        Ansage = Ansage + "  Achtung, heute ist " + AnsageMUELLheute +" ." + "Bitte an die Strasse stellen" + " !";
        //log("Ansage mit Geburtstag und Muell heute--------------------->" + Ansage);
    }
    
// ####################################################
// Wenn Müll ZUKUNFT, dann mit ansagen
// ####################################################

    if (AnsageMUELLzukunft.length > 2) { // wenn der Inhalt des Objektes "AnsageMUELLzukunft" größer als 2 Zeichen lang ist, dann ...
        Ansage = Ansage + "  Achtung, morgen ist " + AnsageMUELLzukunft + " ." + "Bitte MORGEN an die Strasse stellen" + " !";
        //log("Ansage mit Geburtstag und Muell heute und morgen--------------------->" + Ansage);
    }    



// ####################################################
//                  ANSAGE
// ####################################################
   
   var idAnsageBueroWert = getState(idAnsageBuero).val;
   var idAnsageSchlafzimmerWert = getState(idAnsageSchlafzimmer).val;
   var idAnsageKuecheWert = getState(idAnsageKueche).val;
   var idAnsageTerrasseWert = getState(idAnsageTerrasse).val;
   var idAnsageWohnzimmerWert = getState(idAnsageWohnzimmer).val; 

   log("Buero: " + idAnsageBueroWert);
   log("Schlaf: " + idAnsageSchlafzimmerWert);
   log("Kueche: " + idAnsageKuecheWert);
   log("Terrasse: " + idAnsageTerrasseWert);
   log("Wohnzimmer: " + idAnsageWohnzimmerWert);
   
   if(idAnsageBueroWert) {
       
    Ansage = APIsayBuero + Ansage + "/" + lautstaerke;
      
    log("AnsageArbeitszimmer=" + Ansage);
    setState(idAnsageBuero,false);
 
    httpGet(Ansage, (error) => {
    if(error) {
        console.error(error);
        return;
    }        
    });
    
    }
    // Ende If Büro

    if(idAnsageSchlafzimmerWert) {
       
    Ansage = APIsaySchlafzimmer + Ansage + "/" + lautstaerke;
    
    log("AnsageSchlafzimmer=" + Ansage);
    setState(idAnsageSchlafzimmer,false);

    httpGet(Ansage, (error) => {
    if(error) {
        console.error(error);
        return;
    }        
    });
    
    }
    // Ende If Schlafzimmer


     if(idAnsageKuecheWert) {
       
    Ansage = APIsayKueche + Ansage + "/" + lautstaerke;
    
    log("AnsageKuecher=" + Ansage);
    setState(idAnsageKueche,false);

    httpGet(Ansage, (error) => {
    if(error) {
        console.error(error);
        return;
    }        
    });

    }
    // Ende If Kueche


    if(idAnsageTerrasseWert) {
       
    Ansage = APIsayTerrasse + Ansage + "/" + lautstaerke;
    
    log("AnsageTerrasse=" + Ansage);
    setState(idAnsageTerrasse,false);

    httpGet(Ansage, (error) => {
    if(error) {
        console.error(error);
        return;
    }        
    });

    }
    // Ende If Terrasse

    
    if(idAnsageWohnzimmerWert) {

    Ansage = APIsayWohnzimmer + Ansage + "/" + lautstaerke;
    
    log("AnsageWohnzimmer=" + Ansage);
    setState(idAnsageWohnzimmer, false);

    // Definition Zustaende Denon
    const idDenonPower = 'denon.0.zoneMain.powerZone'/*Main Zone Power State*/;
    const idDenonInput = 'denon.0.zoneMain.selectInput'/*Select input*/;
    // const idDenonVolume = 'denon.0.zoneMain.volume'/*Main Volume*/;
    // var alterWertVolume = getState(idDenonVolume).val;

    // Check if the Denon adapter is off 

    if (!getState(idDenonPower).val) { 
    // Turn on the Denon adapter 
    setState(idDenonPower, true);
     log("idDenonPower=" + "true");


    // Set the input to CD
    setStateDelayed(idDenonInput, "CD", 2000);
        log("idDenonInput=" + "CD");
        // setState(idDenonVolume, 50 );
    // Play the announcement delayed by 10 seconds using the Sonos API
    setTimeout(function() {
        log("Ansage wird gespielt");
    httpGet(Ansage, (error) => {
    if(error) {
        console.error(error);
        return;
    }        
    });
    }, 6000);

    // Set the input back to TV
    setTimeout(function() {
    setStateDelayed(idDenonInput, "TV", 2000);
    log("idDenonInput auf TV zurueck");
    // Volume zurücksetzen
    // setState(idDenonVolume, alterWertVolume);
    // Turn off the Denon adapter
    setStateDelayed(idDenonPower, false, 4000);
    log("idDenonPower aus");
    }, 30000);

    }; // Ende If ist AN Z.302
   
    } // Ende If Wohnzimmer Z. 289

    
 } // Ende If aus on Ansage Z. 100
 
}); // Ende ON Zeile 98

das läuft auch, aber es gibt diese Fehlermeldung:

script.js.Skripte_aktiviert.Ansage_API_alle_(mit_Sonos_Api)_mit_Wohnzimmer_mit_HttpGet: timeout of 2000ms exceeded

Die gab es im "alten" Script mit request nicht. Warum? Was ist da falsch?

könnte man den Block

httpGet(Ansage, (error) => {
    if(error) {
        console.error(error);
        return;
    }        
    })

nicht in eine function, entweder im Script selber oder in "globale Scripte" auslagern? Dann muß ich den nicht immer wieder einfügen. Oder?

Danke!

paul53

@skorpil sagte: es gibt diese Fehlermeldung:

Dann setze den Timeout hoch.

httpGet(Ansage, {timeout: 10000}, (error) => {
    if(error) {
        console.error(error);
    }        
});

skorpil

@paul53 das war es! Dankeschön.

und noch bitte zu meiner Frage 2:

könnte man den Block

httpGet(Ansage, {timeout: 10000}, (error) => {
    if(error) {
        console.error(error);
    }        
});

nicht in eine function, entweder im Script selber oder in "globale Scripte" auslagern? Dann muß ich den nicht immer wieder einfügen. Oder?

paul53

@skorpil sagte: nicht in eine function, entweder im Script selber oder in "globale Scripte" auslagern?

Kann man. Dann muss anstelle der 5 Zeilen die Funktion aufgerufen werden. Beispiel:

function ansagen(url) {
    httpGet(url, {timeout: 10000}, (error) => {
        if(error) {
            console.error(error);
        }        
    });
}

// Aufruf
ansagen(Ansage);

skorpil

@paul53 suuuupi

skorpil

@paul53 ich benötige bei einem auf httpGet umgestellten Script bitte noch einmal Deine Hilfe (alle anderen funktionieren wie gewünscht)

Ich tippe darauf, dass das Problem bei folgendem Script mit den timeouts zusammenhängt. Das ist das httpGet Script:

// ##################################################################

//        Erst Ansage, dann Radio ein u. nach 45 min aus

// ##################################################################



// ##################################################################
//                      Definitionen Ansage
// ##################################################################


// Quellen
var idWetter =           "hm-rega.0.29954";
var idTemperatursensor = 'hm-rpc.0.OEQ1296052.1.TEMPERATURE'/*Außentemperatur:1 TEMPERATURE*/
// var idSayVar = "hm-rega.0.38101";//Variable Sayit  <= Alt, wenn Steuerung via CCU Variable;
var idBM = 'hm-rpc.0.JEQ0266058.1.MOTION'/*BewMldr Treppenhaus:1 MOTION*/

// Definition APIsay Büro Ikea;
// var APIsayBuero = "http://192.168.0.105:5005/B%C3%BCro%20Ikea/say/";

// Definition APIsay Schlafzimmer;
// var APIsaySchlafzimmer = "http://192.168.0.105:5005/Schlafzimmer/say/";

// Definition APIsay Kueche;
var APIsayKueche = "http://192.168.0.105:5005/K%C3%BCche/say/";

// Definition APIsay Terrasse;
// var APIsayTerrasse = "http://192.168.0.105:5005/Terrasse/say/";

// Definition Lautstärke;
    var lautstaerke = "50";

var aufstehen = false;

// ####################################################
//          Funktionen (Basiswerte ermitteln)
// ####################################################

function ermitteleAnsagedatum () {
        //Wochentag ermitteln
        var d = new Date ();
        var w = new Array("Sonntag","Montag","Dienstag","Mittwoch","Donnerstag","Freitag","Samstag");
        var wochentag = w[d.getDay()]; 

        //Tagesdatum ermitteln
        var tag = d.getDate();

        //Monat ermitteln
        var month = new Array("Januar","Februar","März","April","Mai","Juni","Juli","August","September","Oktober","November","Dezember");
        var monat = month[d.getMonth()];

        //Jahr ermitteln
        var jahr = d.getFullYear();

        //Stunde ermitteln
        var stunde = d.getHours();

        //Minute ermitteln
        var minute = d.getMinutes();

        //Begrüssung ermitteln
        var Begr = " ";
        if(stunde<=12){Begr = "Guten Morgen"};
        if(stunde>=12 && stunde<=17){Begr = "Guten Tag"};
        if(stunde>17){Begr = "Guten Abend"};
    
        return { // zurückgeben
            'Jahr'      : jahr,
            'Monat'     : monat,
            'Tag'       : tag,
            'Wochentag' : wochentag,
            'Stunde'    : stunde,
            'Minute'    : minute,
            'Begr'      : Begr
        };
    }


function ermitteleWetter () { // Wetterbedingungen
        var wetterdaten = getState(idWetter).val;
        //    log('Wetter: ' + wetterdaten);
        return(wetterdaten);
    }

function ermitteleAnsageTemperatur () {
    // Die Außentemperatur ist xx. Das Wetter heute ist xx
    // Einfache Temperaturansage mit SayIt.
    // Variante 1 mittels splitten der Temperatur, damit die Ansage nicht
    // "Es sind 18 Punkt 2 Grad " sagt.

    var temperatursensor = getState(idTemperatursensor).val;
    var temp_string = temperatursensor.toString();
    //    log('Temp  ' + temp_string);
    var temp_array = [];
    temp_array = temp_string.split(".");

    // Fange leere Nachkommastellen ab. Das passiert, wenn die Temperatur z. B. 18.0 ist.
    // Es wird dann nur "18" gelesen.
    if (!temp_array[1]) {
    temp_array[1] = "0";
    //log("Die Nach-Kommastelle in temp_array[1] war nicht vorhanden und wird nun fest auf 0 gesetzt.");
    }
    return{
        'Ganzzahl'      : temp_array[0],
        'Nachkommazahl' : temp_array[1]
        };
    }

// ##########################################################
//         ANSAGE (GETRIGGERT via BewMldr)
//  EINMALIG im Zeitraum 6:00 - 10:00, wenn BewMldr
// ##########################################################


on(idBM, function(dp) {
   
   log("Motion ausgelöst");
   
   if(dp.state.val && compareTime('6:00', '10:00', 'between') && !aufstehen) {
      aufstehen = true;

      log("Motion ausgelöst, in der Zeit und aufstehen falsch");

    // ##########################################################
    //         ANSAGE Verögerung 10 sec
    // ##########################################################

    setTimeout(function() {

      var ansagetext = ermitteleAnsagedatum().Begr + ", heute ist " + ermitteleAnsagedatum().Wochentag + " der " + ermitteleAnsagedatum().Tag + "te " + ermitteleAnsagedatum().Monat + ' ' + ermitteleAnsagedatum().Jahr 
                + ". Es ist " + ermitteleAnsagedatum().Stunde + "  Uhr und " + ermitteleAnsagedatum().Minute + "  Minuten." 
                + " Die Aussentemperatur beträgt " + ermitteleAnsageTemperatur().Ganzzahl + "," + ermitteleAnsageTemperatur().Nachkommazahl + " Grad." 
                + " Wetter Bedingungen, " + ermitteleWetter() +" .";



// ##################################################################
//      Ansagetext zusammenstellen und in Objekt schreiben
// ##################################################################


    // Ansagetext
        // Basisansage
        var Ansage = ansagetext;
        
        //Geburtstagsansage
        var tempGEBURTSTAGE = "javascript.0.Ansage.GeburtstageHEUTE";
        var AnsageGEBURTSTAGE = getState(tempGEBURTSTAGE).val;
        
        //Muellsansage heute
        var tempMUELLheute = "javascript.0.Ansage.MuellHEUTE";
        var AnsageMUELLheute = getState(tempMUELLheute).val;
        
        //Muellsansage Zukunft
        var tempMUELLzukunft = "javascript.0.Ansage.MuellZUKUNFT";
        var AnsageMUELLzukunft = getState(tempMUELLzukunft).val;

    /*log("Tempansage--------------------->" + tempAnsage);
    log("Ansage--------------------->" + Ansage);

    log("tempGEBURTSTAGE--------------------->" + tempGEBURTSTAGE);
    log("AnsageGEBURTSTAGE--------------------->" + AnsageGEBURTSTAGE);

    log("tempMUELLheute--------------------->" + tempMUELLheute);
    log("AnsageMUELLheute--------------------->" + AnsageMUELLheute);

    log("tempMUELLzukunft--------------------->" + tempMUELLzukunft);
    log("AnsageMUELLzukunft--------------------->" + AnsageMUELLzukunft);*/



    // ####################################################
    // Wenn Geburtstag dann mit ansagen
    // ####################################################

    if (AnsageGEBURTSTAGE.length > 2) { // wenn der Inhalt des Objektes "AnsageGEBURTSTAGE" größer als 2 Zeichen lang ist, dann ...
        Ansage = Ansage + "Heute haben Geburtstag, " + AnsageGEBURTSTAGE +" .";
        //log("Ansage mit Geburtstag--------------------->" + Ansage);
    }
    
   
    // ####################################################
    // Wenn Müll dann mit ansagen
    // ####################################################

    if (AnsageMUELLheute.length > 2) { // wenn der Inhalt des Objektes "AnsageMUELLheute" größer als 2 Zeichen lang ist, dann ...
        Ansage = Ansage + "  Achtung, heute ist " + AnsageMUELLheute +" ." + "Bitte an die Strasse stellen" + " !";
        //log("Ansage mit Geburtstag und Muell heute--------------------->" + Ansage);
    }
    
    // ####################################################
    // Wenn Müll ZUKUNFT, dann mit ansagen
    // ####################################################

    if (AnsageMUELLzukunft.length > 2) { // wenn der Inhalt des Objektes "AnsageMUELLzukunft" größer als 2 Zeichen lang ist, dann ...
        Ansage = Ansage + "  Achtung, morgen ist " + AnsageMUELLzukunft + " ." + "Bitte MORGEN an die Strasse stellen" + " !";
        //log("Ansage mit Geburtstag und Muell heute und morgen--------------------->" + Ansage);
    }    

    //Ansage
    Ansage = APIsayKueche + Ansage + "/" + lautstaerke;       
    
    ansagenSonos(Ansage);//Funktion mit HttpGet in Global

   }, 15000);
   // Ende timeout/ Verzögerung


//****************************************************************************************** */


//************************************************
//Javascript zum Schalten von Sonos via BewMldr
//************************************************

        /* Das folgende sind nur Befehle, die ich mir aufgehoben habe: 
        // var idSonosfav = getState("sonos.0.root.192_168_0_63.favorites_list").val;      //speichern des aktuellen Senders
        // var idSonosplay = getState ("sonos.0.root.192_168_0_63.state_simple").val;     //Status des Speakers
        // laut = getState('sonos.0.root.' + raum + '.volume').val;
        // log(idSonosfav);
        // log(idSonosplay);*/


//*******************************************
//              Definitionen für Radio
//*******************************************

    // *******************************************************************************  
    // Info zum Verzeichnis     /home/brs/node-sonos-http-api-master/static/says/;
    // Definitionen MP3;
    // herzlichWillkommen.mp3";
    // var herz = "herzlichWillkommen.mp3";
    // HundeGebell.mp3;
    // var hund = "HundeGebell.mp3";
    // Rolladen.mp3;
    // var roll = "Rolladen.mp3";

    //AUFRUF:
            //Beispiel: http://192.168.0.105:5005/B%C3%BCro%20Ikea/favorite/RTL - Die besten Hits aller Zeiten

    
    // IDsonosAufr = "http://192.168.0.105:5005/"<------hier richtige IP eintragen
    
    // Definition STRUKTUR
    // Achtung, am Ende der Einzelteile des Aufrufs jeweils ein "7"
    // IDsonosAufr + IDsonosRaum + IDsonosFunktion + IDsonosSender + laut
        
        //Favorites bzw. Sender
        //IDsonosSender = "104.6 RTL Weihnachtsradio/"
        //IDsonosSender = "RTL - Die besten Hits aller Zeiten/"
        //IDsonosSender = "RTL - Die besten Hits aller Zeiten/"
        //IDsonosSender = "RTL Deutschlands Hit-Radio/"

        //Definition Räume
        // Definition raum Büro Ikea;
        // IDsonosRaum ="B%C3%BCro%20Ikea/";

        // Definition raum Schlafzimmer;
        // IDsonosRaum = "Schlafzimmer/";

        // Definition raum Kueche;
        // IDsonosRaum = "K%C3%BCche/";

        // Definition raum Terrasse;
        // IDsonosRaum = "Terrasse/";

        // Definition Funktionen
        // IDsonosFunktion = "say/"
        // IDsonosFunktion = "clip/"
        // IDsonosFunktion = "favorite/"
        // IDsonosFunktion = "pause"; <-------ohne "/"

 

    //STOP
    //BEISPIEL: http://192.168.0.105:5005/B%C3%BCro%20Ikea/pause
    //STRUKTUR: IDsonosAufr + IDsonosRaum + IDsonosFunktion

    // das ist der Aufruf
    // ansagenSonos(radioAbspiel);
    // Funktion mit HttpGet in Global

    // Anweisung setzen LAUTSTAERKE
    // var laut = "8";//<------hier gewünschte LAUTSTÄRKE eintragen
    
    // *******************************************************************************  

var IDsonosAufr = "http://192.168.0.105:5005/";
var IDsonosRaum = "K%C3%BCche/";




    // ************************************************  
    // Auswahl Radiosender
        // Sender muß in Favoriten vorhanden sein
        // momentan: 104.6 RTL Weihnachtsradio, RTL Deutschlands Hit-Radio, RTL - Die besten Hits aller Zeiten
    // ************************************************
    
var IDsonosSender = "RTL - Die besten Hits aller Zeiten/";
// var radioAbspiel = IDsonosAufr + IDsonosRaum + IDsonosFunktion + IDsonosSender;  

setTimeout(function() {
    // ************************************************
    //  nach 60 sekunden starten
    // ************************************************

// Befehle ausführen

    // ************************************************
    // Befehl zum play
    // ************************************************



var IDsonosFunktion = "favorite/";
var radioAbspiel = IDsonosAufr + IDsonosRaum + IDsonosFunktion + IDsonosSender;

ansagenSonos(radioAbspiel);//Funktion mit HttpGet in Global

// Lautstaeke einstellen 
var IDsonosFunktion = "volume/";
var laut = "8";
var radioAbspiel = IDsonosAufr + IDsonosRaum + IDsonosFunktion + laut;

ansagenSonos(radioAbspiel);//Funktion mit HttpGet in Global

// Ende lautstaerke

 

 }, 60000);
   // Ende timeout für Start

setTimeout(function() {
    // ************************************************
    // Befehl zum stop nach 45 Minuten
    // ************************************************

var IDsonosFunktion = "pause";
var radioAbspielStop = IDsonosAufr + IDsonosRaum + IDsonosFunktion; 

log("radioAbspielStop--------------------->" + radioAbspielStop);

ansagenSonos(radioAbspielStop);//Funktion mit HttpGet in Global

}, 2700000);
// Ende timeout für Stop


   }
   // Ende If
   
});


    // ####################################################
    //     var "aufstehen" zurücksetzen auf false um 3 Uhr
    // ####################################################


schedule('0 3 * * *', function() {
   aufstehen = false;
});

hier ist das zugehörige globale Script resp Function:

function ansagenSonos(url) {
    httpGet(url, {timeout: 30000}, (error) => {
        if(error) {
            console.error(error);
        }        
    });
}

Das Merkwürdige ist, es löst nicht aus. Ich habe mir zum debuggen schon logs eingefügt:

log("Motion ausgelöst");

und

log("Motion ausgelöst, in der Zeit und aufstehen falsch");

Es kommt aber nur "Motion ausgelöst", also wird die if Aweisung nicht zutreffend ausgelöst.

Dieses "alte" Script mit "request" funktioniert dagegen perfekt:

// ##################################################################

//        Erst Ansage, dann Radio ein u. nach 45 min aus

// ##################################################################



// ##################################################################
//                      Definitionen Ansage
// ##################################################################


// Quellen
var idWetter =           "hm-rega.0.29954";
var idTemperatursensor = 'hm-rpc.0.OEQ1296052.1.TEMPERATURE'/*Außentemperatur:1 TEMPERATURE*/
// var idSayVar = "hm-rega.0.38101";//Variable Sayit  <= Alt, wenn Steuerung via CCU Variable;
var idBM = 'hm-rpc.0.JEQ0266058.1.MOTION'/*BewMldr Treppenhaus:1 MOTION*/

// Definition APIsay Büro Ikea;
// var APIsayBuero = "http://192.168.0.105:5005/B%C3%BCro%20Ikea/say/";

// Definition APIsay Schlafzimmer;
// var APIsaySchlafzimmer = "http://192.168.0.105:5005/Schlafzimmer/say/";

// Definition APIsay Kueche;
var APIsayKueche = "http://192.168.0.105:5005/K%C3%BCche/say/";

// Definition APIsay Terrasse;
// var APIsayTerrasse = "http://192.168.0.105:5005/Terrasse/say/";

// Definition Lautstärke;
    var lautstaerke = "50";

var aufstehen = false;

// ####################################################
//          Funktionen (Basiswerte ermitteln)
// ####################################################

function ermitteleAnsagedatum () {
        //Wochentag ermitteln
        var d = new Date ();
        var w = new Array("Sonntag","Montag","Dienstag","Mittwoch","Donnerstag","Freitag","Samstag");
        var wochentag = w[d.getDay()]; 

        //Tagesdatum ermitteln
        var tag = d.getDate();

        //Monat ermitteln
        var month = new Array("Januar","Februar","März","April","Mai","Juni","Juli","August","September","Oktober","November","Dezember");
        var monat = month[d.getMonth()];

        //Jahr ermitteln
        var jahr = d.getFullYear();

        //Stunde ermitteln
        var stunde = d.getHours();

        //Minute ermitteln
        var minute = d.getMinutes();

        //Begrüssung ermitteln
        var Begr = " ";
        if(stunde<=12){Begr = "Guten Morgen"};
        if(stunde>=12 && stunde<=17){Begr = "Guten Tag"};
        if(stunde>17){Begr = "Guten Abend"};
    
        return { // zurückgeben
            'Jahr'      : jahr,
            'Monat'     : monat,
            'Tag'       : tag,
            'Wochentag' : wochentag,
            'Stunde'    : stunde,
            'Minute'    : minute,
            'Begr'      : Begr
        };
    }


function ermitteleWetter () { // Wetterbedingungen
        var wetterdaten = getState(idWetter).val;
        //    log('Wetter: ' + wetterdaten);
        return(wetterdaten);
    }

function ermitteleAnsageTemperatur () {
    // Die Außentemperatur ist xx. Das Wetter heute ist xx
    // Einfache Temperaturansage mit SayIt.
    // Variante 1 mittels splitten der Temperatur, damit die Ansage nicht
    // "Es sind 18 Punkt 2 Grad " sagt.

    var temperatursensor = getState(idTemperatursensor).val;
    var temp_string = temperatursensor.toString();
    //    log('Temp  ' + temp_string);
    var temp_array = [];
    temp_array = temp_string.split(".");

    // Fange leere Nachkommastellen ab. Das passiert, wenn die Temperatur z. B. 18.0 ist.
    // Es wird dann nur "18" gelesen.
    if (!temp_array[1]) {
    temp_array[1] = "0";
    //log("Die Nach-Kommastelle in temp_array[1] war nicht vorhanden und wird nun fest auf 0 gesetzt.");
    }
    return{
        'Ganzzahl'      : temp_array[0],
        'Nachkommazahl' : temp_array[1]
        };
    }

// ##########################################################
//         ANSAGE (GETRIGGERT via BewMldr)
//  EINMALIG im Zeitraum 6:00 - 10:00, wenn BewMldr
// ##########################################################


on(idBM, function(dp) {
   if(dp.state.val && compareTime('6:00', '10:00', 'between') && !aufstehen) {
      aufstehen = true;

    // ##########################################################
    //         ANSAGE Verögerung 10 sec
    // ##########################################################

    setTimeout(function() {

      var ansagetext = ermitteleAnsagedatum().Begr + ", heute ist " + ermitteleAnsagedatum().Wochentag + " der " + ermitteleAnsagedatum().Tag + "te " + ermitteleAnsagedatum().Monat + ' ' + ermitteleAnsagedatum().Jahr 
                + ". Es ist " + ermitteleAnsagedatum().Stunde + "  Uhr und " + ermitteleAnsagedatum().Minute + "  Minuten." 
                + " Die Aussentemperatur beträgt " + ermitteleAnsageTemperatur().Ganzzahl + "," + ermitteleAnsageTemperatur().Nachkommazahl + " Grad." 
                + " Wetter Bedingungen, " + ermitteleWetter() +" .";



// ##################################################################
//      Ansagetext zusammenstellen und in Objekt schreiben
// ##################################################################


    // Ansagetext
        // Basisansage
        var Ansage = ansagetext;
        
        //Geburtstagsansage
        var tempGEBURTSTAGE = "javascript.0.Ansage.GeburtstageHEUTE";
        var AnsageGEBURTSTAGE = getState(tempGEBURTSTAGE).val;
        
        //Muellsansage heute
        var tempMUELLheute = "javascript.0.Ansage.MuellHEUTE";
        var AnsageMUELLheute = getState(tempMUELLheute).val;
        
        //Muellsansage Zukunft
        var tempMUELLzukunft = "javascript.0.Ansage.MuellZUKUNFT";
        var AnsageMUELLzukunft = getState(tempMUELLzukunft).val;

    /*log("Tempansage--------------------->" + tempAnsage);
    log("Ansage--------------------->" + Ansage);

    log("tempGEBURTSTAGE--------------------->" + tempGEBURTSTAGE);
    log("AnsageGEBURTSTAGE--------------------->" + AnsageGEBURTSTAGE);

    log("tempMUELLheute--------------------->" + tempMUELLheute);
    log("AnsageMUELLheute--------------------->" + AnsageMUELLheute);

    log("tempMUELLzukunft--------------------->" + tempMUELLzukunft);
    log("AnsageMUELLzukunft--------------------->" + AnsageMUELLzukunft);*/



    // ####################################################
    // Wenn Geburtstag dann mit ansagen
    // ####################################################

    if (AnsageGEBURTSTAGE.length > 2) { // wenn der Inhalt des Objektes "AnsageGEBURTSTAGE" größer als 2 Zeichen lang ist, dann ...
        Ansage = Ansage + "Heute haben Geburtstag, " + AnsageGEBURTSTAGE +" .";
        //log("Ansage mit Geburtstag--------------------->" + Ansage);
    }
    
   
    // ####################################################
    // Wenn Müll dann mit ansagen
    // ####################################################

    if (AnsageMUELLheute.length > 2) { // wenn der Inhalt des Objektes "AnsageMUELLheute" größer als 2 Zeichen lang ist, dann ...
        Ansage = Ansage + "  Achtung, heute ist " + AnsageMUELLheute +" ." + "Bitte an die Strasse stellen" + " !";
        //log("Ansage mit Geburtstag und Muell heute--------------------->" + Ansage);
    }
    
    // ####################################################
    // Wenn Müll ZUKUNFT, dann mit ansagen
    // ####################################################

    if (AnsageMUELLzukunft.length > 2) { // wenn der Inhalt des Objektes "AnsageMUELLzukunft" größer als 2 Zeichen lang ist, dann ...
        Ansage = Ansage + "  Achtung, morgen ist " + AnsageMUELLzukunft + " ." + "Bitte MORGEN an die Strasse stellen" + " !";
        //log("Ansage mit Geburtstag und Muell heute und morgen--------------------->" + Ansage);
    }    

    //Ansage
    Ansage = APIsayKueche + Ansage + "/" + lautstaerke;       
    
    try {
    require("request")(Ansage).on("error", function (e) {console.error(e);});
        } catch (e) { console.error(e); };

   }, 15000);
   // Ende timeout/ Verzögerung


//****************************************************************************************** */


//************************************************
//Javascript zum Schalten von Sonos via BewMldr
//************************************************

        /* Das folgende sind nur Befehle, die ich mir aufgehoben habe: 
        // var idSonosfav = getState("sonos.0.root.192_168_0_63.favorites_list").val;      //speichern des aktuellen Senders
        // var idSonosplay = getState ("sonos.0.root.192_168_0_63.state_simple").val;     //Status des Speakers
        // laut = getState('sonos.0.root.' + raum + '.volume').val;
        // log(idSonosfav);
        // log(idSonosplay);*/


//*******************************************
//              Definitionen für Radio
//*******************************************

    // *******************************************************************************  
    // Info zum Verzeichnis     /home/brs/node-sonos-http-api-master/static/says/;
    // Definitionen MP3;
    // herzlichWillkommen.mp3";
    // var herz = "herzlichWillkommen.mp3";
    // HundeGebell.mp3;
    // var hund = "HundeGebell.mp3";
    // Rolladen.mp3;
    // var roll = "Rolladen.mp3";

    //AUFRUF:
            //Beispiel: http://192.168.0.105:5005/B%C3%BCro%20Ikea/favorite/RTL - Die besten Hits aller Zeiten

    
    // IDsonosAufr = "http://192.168.0.105:5005/"<------hier richtige IP eintragen
    
    // Definition STRUKTUR
    // Achtung, am Ende der Einzelteile des Aufrufs jeweils ein "7"
    // IDsonosAufr + IDsonosRaum + IDsonosFunktion + IDsonosSender + laut
        
        //Favorites bzw. Sender
        //IDsonosSender = "104.6 RTL Weihnachtsradio/"
        //IDsonosSender = "RTL - Die besten Hits aller Zeiten/"
        //IDsonosSender = "RTL - Die besten Hits aller Zeiten/"
        //IDsonosSender = "RTL Deutschlands Hit-Radio/"

        //Definition Räume
        // Definition raum Büro Ikea;
        // IDsonosRaum ="B%C3%BCro%20Ikea/";

        // Definition raum Schlafzimmer;
        // IDsonosRaum = "Schlafzimmer/";

        // Definition raum Kueche;
        // IDsonosRaum = "K%C3%BCche/";

        // Definition raum Terrasse;
        // IDsonosRaum = "Terrasse/";

        // Definition Funktionen
        // IDsonosFunktion = "say/"
        // IDsonosFunktion = "clip/"
        // IDsonosFunktion = "favorite/"
        // IDsonosFunktion = "pause"; <-------ohne "/"

 

    //STOP
    //BEISPIEL: http://192.168.0.105:5005/B%C3%BCro%20Ikea/pause
    //STRUKTUR: IDsonosAufr + IDsonosRaum + IDsonosFunktion

    // das ist der Aufruf
    //try {
    //    require("request")(abspielSchlaf).on("error", function (e) {console.error(e);});
    //        } catch (e) { console.error(e); };

    // Anweisung setzen LAUTSTAERKE
    // var laut = "8";//<------hier gewünschte LAUTSTÄRKE eintragen
    
    // *******************************************************************************  

var IDsonosAufr = "http://192.168.0.105:5005/";
var IDsonosRaum = "K%C3%BCche/";




    // ************************************************  
    // Auswahl Radiosender
        // Sender muß in Favoriten vorhanden sein
        // momentan: 104.6 RTL Weihnachtsradio, RTL Deutschlands Hit-Radio, RTL - Die besten Hits aller Zeiten
    // ************************************************
    
var IDsonosSender = "RTL - Die besten Hits aller Zeiten/";
// var radioAbspiel = IDsonosAufr + IDsonosRaum + IDsonosFunktion + IDsonosSender;  

setTimeout(function() {
    // ************************************************
    //  nach 60 sekunden starten
    // ************************************************

// Befehle ausführen

    // ************************************************
    // Befehl zum play
    // ************************************************



var IDsonosFunktion = "favorite/";
var radioAbspiel = IDsonosAufr + IDsonosRaum + IDsonosFunktion + IDsonosSender;

try {
        require("request")(radioAbspiel).on("error", function (e) {console.error(e);});
            } catch (e) { console.error(e); };

// Lautstaeke einstellen 
var IDsonosFunktion = "volume/";
var laut = "8";
var radioAbspiel = IDsonosAufr + IDsonosRaum + IDsonosFunktion + laut;

try {
        require("request")(radioAbspiel).on("error", function (e) {console.error(e);});
            } catch (e) { console.error(e); };
// Ende lautstaerke

 

 }, 60000);
   // Ende timeout für Start

setTimeout(function() {
    // ************************************************
    // Befehl zum stop nach 45 Minuten
    // ************************************************

var IDsonosFunktion = "pause";
var radioAbspielStop = IDsonosAufr + IDsonosRaum + IDsonosFunktion; 

log("radioAbspielStop--------------------->" + radioAbspielStop);

try {
        require("request")(radioAbspielStop).on("error", function (e) {console.error(e);});
            } catch (e) { console.error(e); };

}, 2700000);
// Ende timeout für Stop


   }
   // Ende If
   
});


    // ####################################################
    //     var "aufstehen" zurücksetzen auf false um 3 Uhr
    // ####################################################


schedule('0 3 * * *', function() {
   aufstehen = false;
});

Ich finde einfach den Fehler nicht. Würdest Du bitte mal drüberschauen?

paul53

@skorpil sagte: Es kommt aber nur "Motion ausgelöst",

Dann wird getriggert. Ich kann keinen Fehler erkennen. Logge mal aufstehen bevor es auf true gesetzt wird.

skorpil

@paul53 Danke. okay, ich werde aufstehen loggen

Die quasi ineinander geschachtelten timeouts, in der globalen Function
"{timeout: 30000}"
und in Zeile 207 im Script von 15 Sekunden
" }, 15000)"
sind okay?

paul53

@skorpil sagte: sind okay?

Die Timeouts haben nichts miteinander zu tun. Das Timeout in httpGet verzögert nicht, sondern gibt vor, wie lange gewartet wird, bis mit einer Fehlermeldung abgebrochen wird.

skorpil

@paul53 feedback ist ja bekanntlich breakfast for champions: es geschehen noch Zeiten und Wunder. Ich hab keine Ahnung, warum. Aber heute Morgen hat das Script so ausgelöst, wie ich es gewünscht habe . Ich habe gestern nur die logs für „aufstehen“ hinzugefügt, um heute Morgen zu sehen, wie sich das Skript verhält und, Überraschung, es funktioniert. Manchmal habe ich den Eindruck, iobroker spielt ein Spiel mit mir. Nochmals vielen Dank für die Hilfe und ein schönes Wochenende.

NEWS

JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.

Support us

1.4k
Online

30.7k
Users

77.2k
Topics

1.2m
Posts

NEWS

JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.

Support us

1.4kOnline

30.7kUsers

77.2kTopics

1.2mPosts

1.4k
Online

30.7k
Users

77.2k
Topics

1.2m
Posts