JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.

skorpil

@paul53 Test erfolgreich! Vielen Dank für die Hilfe!

skorpil

@paul53 ich habe wieder ein "unlösbares" Problem und bitte um Hilfe:

Ich habe drei Programme:

Dieses schaltet den Boiler abends um 22:30 aus und morgens um 06:30 eingeschaltet werden

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

const IDZwBoilerKueche = 'hm-rpc.0.LEQ0197823.1.STATE'/*Zw -Stecker Küche:1 STATE*/;
const BoilerSchalter = 'javascript.0.BenutzerVariablen.BoilerProgrammSchalter'/*Boiler AnAus*/;

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

    schedule({hour: 6, minute: 30}, function () {
    //Wird um 06:30 eingeschaltet, wenn der Boilerschalter true;
    if(getState(BoilerSchalter).val == true) { 
    setState(IDZwBoilerKueche, true);
    };
    });

    schedule({hour: 22, minute: 30}, function () {
    //Wird um 22:30 AUSgeschaltet, wenn der Boilerschalter true;
    if(getState(BoilerSchalter).val == true) { 
    setState(IDZwBoilerKueche, false);
    };
    });

Dieses Programm schaltet die Geräte im Urlaub bzw. wenn Sommer oder Winter ist, wobei nur der Boiler von Interesse ist, denn der schaltet nicht richtig (erkläre ich später):

// ###########################################
//         Urlaub und Sommer Schaltungen
//                Deklarationen
// ###########################################

const urlaub = 'hm-rega.0.8359'/*URLAUB Hzg  Wohnb  ALLES (Var )*/;
const sommer = 'hm-rega.0.11457'/*SOMMER Hzg  Wohnb  ALLES (Var )*/;

const IDthermKueche = 'hm-rpc.0.JEQ0553018.2.SETPOINT'/*Thermostat Kueche 1 SETPOINT*/;
const IDthermDieleTreppenhaus = 'hm-rpc.0.JEQ0552520.2.SETPOINT'/*Thermostat Diele Treppenhaus 1 SETPOINT*/;
const IDthermEsszimmer = 'hm-rpc.0.JEQ0553158.2.SETPOINT'/*Thermostat Esszimmer 1 SETPOINT*/;
const IDthermWohnzimmer = 'hm-rpc.0.JEQ0552318.2.SETPOINT'/*Thermostat Wohnzimmer 1 SETPOINT*/;

const IDanwesend = 'hm-rega.0.39533'/*Anwesend*/;
const IDstaubsauger = 'hm-rega.0.29769'/*Staubsauger ein/aus*/;
const IDspringbrunnenGanz = 'hm-rega.0.55701'/*SpringBruGanzEinAus*/;

const IDheizStatusBuegelEltern = 'hm-rega.0.13752'/*Heizstatus (Buegel Eltern ges )*/;
const IDheizStatusBuero = 'hm-rega.0.14236'/*Heizstatus (Buero)*/;
const IDheizStatusBar = 'hm-rega.0.23436'/*Heizstatus (BAR)*/;
const IDheizStatusBritta = 'hm-rega.0.14235'/*Heizstatus (Britta)*/;
const IDheizStatusJoerg = 'hm-rega.0.4107'/*Heizstatus (JOERG)*/;
const IDheizStatusGaeste = 'hm-rega.0.3388'/*Heizstatus (Gaestetrakt)*/;

const IDheizHauptOben = 'hm-rpc.0.SEQ3263796.1.STATE'/*HeizungsHauptschalter OBEN:1 STATE*/;
const IDheizHauptUnten = 'hm-rpc.0.JEQ0097730.1.STATE'/*HeizungsHauptschalter:1 STATE*/;

const IDklimaBuero = 'hm-rpc.0.OEQ1975895.1.STATE'/*Zw Stkr KlimaBuero:1 STATE*/;
const IDklimaSchaf = 'hm-rpc.0.OEQ1975873.1.STATE'/*ZW Stkr KlimaSchlafzimer:1 STATE*/;

const IDKeyMatic = 'hm-rpc.0.JEQ0466522.1.STATE'/*KeyMatic Haustuere neu STATE*/;

const IDboiler = 'hm-rpc.0.LEQ0197823.1.STATE'/*Zw -Stecker Küche:1 STATE*/;
const ProgrSchalterBoiler = 'javascript.0.BenutzerVariablen.BoilerProgrammSchalter'/*Boiler AnAus*/;
const IDtechiSat = 'hm-rpc.2.CUX2801002.12.STATE'/*TerchniSat Tasmota 184 (HM-LC-Sw1-Pl CUX2801002:12) STATE*/;

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

    // *********************************************
    // ausführen nur, wenn Urlaub AUS und Sommer AUS
    // *********************************************

on([urlaub, sommer], function () { // triggert bei Wertänderung eines DP
   if(getState(urlaub).val == 0 && getState(sommer).val == 0) {

    // Variable zur Einstellung des Thermometers
    var modus = 1; // 1 = AUTO; 2 = CEN

    // Modus des Thermometers via global function thermometerSetzen u. Temperatur einstellen
    // Kueche
    thermometerSetzen(IDthermKueche, modus);
    setState(IDthermKueche,22);

    // DieleTreppenhaus
    thermometerSetzen(IDthermDieleTreppenhaus, modus);
    setState(IDthermDieleTreppenhaus,19);

    // Esszimmer
    thermometerSetzen(IDthermEsszimmer, modus);
    setState(IDthermEsszimmer,22);

    // Wohnzimmer
    thermometerSetzen(IDthermWohnzimmer, modus);
    setState(IDthermWohnzimmer,22);

    // Anwesend einschalten
    setState(IDanwesend, true);

    // Staubsauger ein (0 = ein)
    setState(IDstaubsauger, 0);

    // Springbrunnen ein
    setState(IDspringbrunnenGanz, true);

    // HeizStati
    setState(IDheizStatusBuegelEltern, 1);
    setState(IDheizStatusBuero, 1);
    setState(IDheizStatusBar, 0);
    setState(IDheizStatusBritta, 0);
    setState(IDheizStatusJoerg, 0);
    setState(IDheizStatusGaeste, 0);

    // HeizHauptschalter AUS
    setState(IDheizHauptUnten, true);
    setState(IDheizHauptOben, true);

    // Klima AUS
    setState(IDklimaBuero, false);
    setState(IDklimaSchaf, false);

    // TechniSat ein
    setState(IDtechiSat, true);
    
   } // Ende IF Urlaub AUS - Sommer AUS

    // *********************************************
    // ausführen nur, wenn Urlaub AUS und Sommer EIN
    // *********************************************

if(getState(urlaub).val == 0 && getState(sommer).val == 1) {

  setState(IDheizHauptOben, true);
  setState(IDheizHauptUnten, false);

  // Anwesend einschalten
    setState(IDanwesend, true);

  // Staubsauger ein (0 = ein)
    setState(IDstaubsauger, 0);

  // Springbrunnen ein
    setState(IDspringbrunnenGanz, true); 
    
  // Klima AUS
    // setState(IDklimaBuero, true);
    // setState(IDklimaSchaf, true);

    // TechniSat ein
    setState(IDtechiSat, true);

    
    } // Ende IF Urlaub AUS - Sommer ein



    // *********************************************
    // ausführen nur, wenn Urlaub EIN und Sommer AUS
    // *********************************************

if(getState(urlaub).val == 1 && getState(sommer).val == 0) {

// Variable zur Einstellung des Thermometers
    var modus = 2; // 1 = AUTO; 2 = CEN

    // Modus des Thermometers via global function thermometerSetzen u. Temperatur einstellen
    // Kueche
    thermometerSetzen(IDthermKueche, modus);
    setState(IDthermKueche,15);

    // DieleTreppenhaus
    thermometerSetzen(IDthermDieleTreppenhaus, modus);
    setState(IDthermDieleTreppenhaus,15);

    // Esszimmer
    thermometerSetzen(IDthermEsszimmer, modus);
    setState(IDthermEsszimmer,15);

    // Wohnzimmer
    thermometerSetzen(IDthermWohnzimmer, modus);
    setState(IDthermWohnzimmer,15);

    // Haustuere verriegeln
    setState(IDKeyMatic, false);

    // Boilder Kueche aus
    setState(IDboiler, false);
    setState(ProgrSchalterBoiler, false);

    // Anwesend einschalten
    setState(IDanwesend, false);

    // Staubsauger AUS (0 = EIN, 1 = AUS)
    setState(IDstaubsauger, 1);

    // Springbrunnen AUS
    setState(IDspringbrunnenGanz, false);

     // HeizStati
    setState(IDheizStatusBuegelEltern, 0);
    setState(IDheizStatusBuero, 0);
    setState(IDheizStatusBar, 0);
    setState(IDheizStatusBritta, 0);
    setState(IDheizStatusJoerg, 0);
    setState(IDheizStatusGaeste, 0);

    // HeizHauptschalter AUS
    setState(IDheizHauptUnten, false);
    setState(IDheizHauptOben, false);

    // Klima AUS
    setState(IDklimaBuero, false);
    setState(IDklimaSchaf, false);

    // TechniSat aus
    setState(IDtechiSat, false);

    } // Ende IF Urlaub EIN und Sommer AUS, 

    // *********************************************
    // ausführen nur, wenn Urlaub EIN und Sommer EIN
    // *********************************************

if(getState(urlaub).val == 1 && getState(sommer).val == 1) {

    // HeizHauptschalter AUS
    setState(IDheizHauptUnten, false);
    setState(IDheizHauptOben, false);

    // Boilder Kueche aus
    setState(IDboiler, false);
    setState(ProgrSchalterBoiler, false);

     // Anwesend einschalten
    setState(IDanwesend, false);

    // Klima AUS
    setState(IDklimaBuero, false);
    setState(IDklimaSchaf, false);

    // Staubsauger AUS (0 = EIN, 1 = AUS)
    setState(IDstaubsauger, 1);

    // Springbrunnen AUS
    setState(IDspringbrunnenGanz, false);

    // Haustuere verriegeln
    setState(IDKeyMatic, false);

    // TechniSat aus
    setState(IDtechiSat, false);

    } // Ende IF Urlaub EIN und Sommer EIN




}); //Ende ON

Und dieses schaltet in Abhängigkeit davon, ob der Heizungshauptschalter ein- oder ausgeschaltet ist, den Boiler ein und aus:

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

const IDAnwesend = 'hm-rega.0.39533'/*Anwesend*/;
const IDBoiler = 'hm-rpc.0.LEQ0197823.1.STATE'/*Zw -Stecker Küche:1 STATE*/;
const IDUrlaub = 'hm-rega.0.8359'/*URLAUB Hzg  Wohnb  ALLES (Var )*/;
const IDHeizungHptUnten = 'hm-rpc.0.JEQ0097730.1.STATE'/*HeizungsHauptschalter:1 STATE*/

// Datenpunkt erstellen (für andere Scripte);
createState('BenutzerVariablen.BoilerProgrammSchalter', '', {
    name: 'Boiler AnAus',
    type: 'boolean'
});

// ProgrSchalter standardmaessig auf true (damit Zeitsteuerung aktiv)
    const ProgrSchalter = 'javascript.0.BenutzerVariablen.BoilerProgrammSchalter'/*Boiler AnAus*/;
    setState(ProgrSchalter, true);



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

on({id: IDAnwesend, change: 'ne'}, function () {
        
    // wenn ABwesend und Urlaub AUS (nur prüfen) , dann Boiler AUS und Zeitsteuerung Boiler AUS;
    if(getState(IDUrlaub).val == 0 && getState(IDAnwesend).val == false && getState(IDHeizungHptUnten).val == true) {    
    setState(IDBoiler,false);
    setState(ProgrSchalter, false);
    };

    // wenn ANwesend und Urlaub AUS (nur prüfen) und Boiler Keller ist AUS, dann Boiler EIN Zeitsteuerung Boiler EIN;
    if(getState(IDUrlaub).val == 0  && getState(IDAnwesend).val == true && getState(IDHeizungHptUnten).val == false) {    
    setState(IDBoiler,true);
    setState(ProgrSchalter, true);
    };
});

Zusätzlich gibt es die "Frostschutzüberwachung", die den HeizungsHauptschalter betätigt, wenn es kälter wird als 14 Grad:

// **************** DEKLARATIONEN **********************
const IDBarKellerThemp = 'hm-rpc.0.JEQ0553882.1.TEMPERATURE'/*Thermostat Bar Keller 0 TEMPERATURE*/;
const IDUrlaub = 'hm-rega.0.8359'/*URLAUB Hzg  Wohnb  ALLES (Var )*/;
const IDSommer = 'hm-rega.0.11457'/*SOMMER Hzg  Wohnb  ALLES (Var )*/;
const IDHeizHptKeller = 'hm-rpc.0.JEQ0097730.1.STATE'/*HeizungsHauptschalter:1 STATE*/;
var ein = getState(IDHeizHptKeller).val;
// *****************************************************

// **************** Programm ***************************
on(IDBarKellerThemp, function (data) {
    if(!getState(IDSommer).val && getState(IDUrlaub).val) {
        if(data.state.val < 14) ein = true;
        else if(data.state.val >= 15) ein = false; // Hysterese 1 K
        if(ein != getState(IDHeizHptKeller).val) setState(IDHeizHptKeller, ein); // Duty cycle!
    }
});

Jetzt war ich in Urlaub. Und leider schaltete des Boiler Programm (1) offenbar den Boiler ein und aus, obwohl ich "Urlaub" gesetzt hatte. Ich tippe darauf, das das passiert, wenn der Frostschutz den HeizungsHauptschalter betätigt.

Jetzt sehe ich den Wald vor lauter Bäumen nicht mehr. Ziel ist es jedenfalls, dass dass Programm der Zeitsteuerung (1) nur läuft, wenn ich nicht im Urlaub bin.

Konst Du mal draufschauen?

paul53

@skorpil sagte: draufschauen?

Man sollte einen Datenpunkt nur in einem Skript schreiben und in diesem Skript alle Abhängigkeiten realisieren.
Welche Funktion(en) hat der DP "javascript.0.BenutzerVariablen.BoilerProgrammSchalter"?

paul53

@skorpil sagte: Dieses schaltet den Boiler abends um 22:30 aus und morgens um 06:30 eingeschaltet werden

Ich würde dieses Skript so erweitern, dass nur noch in diesem Skript der Boiler geschaltet wird:

// ###########################################
//            Deklarationen
// ###########################################
const IDZwBoilerKueche = 'hm-rpc.0.LEQ0197823.1.STATE'/*Zw -Stecker Küche:1 STATE*/;
const idBoilerSchalter = 'javascript.0.BenutzerVariablen.BoilerProgrammSchalter'/*Boiler AnAus*/;

// ##################################
//              Programm
// ##################################
var BoilerSchalter = getState(idBoilerSchalter).val;

on(idBoilerSchalter, function(dp) {
    BoilerSchalter = dp.state.val;
    let ein = BoilerSchalter && compareTime('6:30', '22:30', 'between');
    if(ein != getState(IDZwBoilerKueche).val) setState(IDZwBoilerKueche, ein);
});

schedule({hour: 6, minute: 30}, function () {
    //Wird um 06:30 eingeschaltet, wenn der Boilerschalter true;
    if(BoilerSchalter) setState(IDZwBoilerKueche, true);
});
 
schedule({hour: 22, minute: 30}, function () {
    //Wird um 22:30 AUSgeschaltet, wenn der Boilerschalter true;
    if(BoilerSchalter) setState(IDZwBoilerKueche, false);
});

Die weiteren Abhängigkeiten wirken nur noch auf den "Boilerschalter".

skorpil

@paul53 daaaanke. Das war der richtige Tip.

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

NEWS

JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.

Support us

657
Online

31.7k
Users

79.8k
Topics

1.3m
Posts

NEWS

JavaScript funktioniert nicht wie gewünscht! Bitte Hilfe.

Support us

657Online

31.7kUsers

79.8kTopics

1.3mPosts

657
Online

31.7k
Users

79.8k
Topics

1.3m
Posts