Frage Javascript "on-id-change"

[thewhobox]

@sveni_lee Das problem ist, dass call_list ein Array ist und kein String.

call_list = call_list.split('\r\n');

Das erstellt ein Array und trennt den String nach jeder neuen Zeile.
Lass das weg, dann bleibt es ein String.
Und ersetze dann folglich:

call_list.unshift(tempObj.date + " " + tempObj.name + " " + tempObj.caller + '\r\n');

durch

call_list = tempObj.date + " " + tempObj.name + " " + tempObj.caller + '\r\n' + call_list;

Oder... was ach geht:

sendTo("telegram.0", "send", {
     text: call_list.join("\r\n");
});

(Dann musst du aber das \r\n in "unshift" weglassen, da sonst zwei Zeilenumbrüche entstehen.)

[sveni_lee]

ja, das klappt...

aber kann ich irgendwie das "missed calls:" an den Anfang?

[thewhobox]

@sveni_lee Ich würde es so machen:

function list_Calls(){
    var jsonstring = getState("tr-064-community.0.calllists.missed.json"/*json*/).val;
    var call_list = [];
    var objects = JSON.parse(jsonstring);

    for(var i = 0; i<objects.length; i++) {
        var tempObj = objects[i];
        call_list.push(tempObj.date + " " + tempObj.name + " " + tempObj.caller);
    }   
    
    call_list = call_list.reverse();
    log(call_list); 

    sendTo("telegram.0", "send", {
        text: "missed calls:\r\n\r\n" + call_list.join("\r\n");
    });
}

list_Calls();

Was es tut:
Es schreibt alle Anrufer in ein Array.
Array umdrehen, damit neue ganz oben stehen.
Array zusammenführen mit einem Zeilenumbruch, davor noch "missed calls" anfügen.

Minimaler aufwand, Liste wird bei jedem Funktionsaufruf aktualisiert, keine unnötigen Parameter.

[sveni_lee]

jepp...
perfect: so klappt es.

[sveni_lee]

gibt es auch einen "schnellen weg" an den vorherigen Wert eines DP zu gelangen?
als ich möchte den aktuellen Wert gern mit dem Wert davor vergleichen...

[thewhobox]

@sveni_lee Den alten Wert bekommst du mit

dp.oldState.val

[sveni_lee]

ich habe mal einen kleinen Test gemacht...

function test() {
   var dp = "hm-rpc.0.xxxxxxxx.1.DIRECTION";
   log("State: " + dp.oldState.val);
}
test();

klappt aber nicht

Log:

javascript.0	2019-06-26 08:59:45.835	error	at ContextifyScript.Script.runInContext (vm.js:59:29)
javascript.0	2019-06-26 08:59:45.835	error	at script.js.common.Telegram.Test:5:1
javascript.0	2019-06-26 08:59:45.835	error	at test (script.js.common.Telegram.Test:3:32)
javascript.0	2019-06-26 08:59:45.835	error	TypeError: Cannot read property 'val' of undefined
javascript.0	2019-06-26 08:59:45.835	error	^
javascript.0	2019-06-26 08:59:45.835	error	log("State: " + dp.oldState.val);
javascript.0	2019-06-26 08:59:45.835	error	script.js.common.Telegram.Test: script.js.common.Telegram.Test:3

[thewhobox]

@sveni_lee Ja, weil deine Variable dp nur ein String ist.....
Ebenso ist "oldState" nur innerhalb des Triggers vorhanden.

[sveni_lee]

okay... ich probiere es mal inner halb eines Triggers.

[sveni_lee]

das hat jetzt schon mal richtig gut geklappt...
danke für die Hilfe!

Wie das immer so ist, kommt ein zum anderen.

kann ich irgendwie eine Zeitschleife einbauen, das der neue wert erst übertragen wird, wenn ein DP, wie in diesem Fall "Festverglasung rechts.WORKING" wieder auf false steht.

ich hatte da an eine "while" Schleife gedacht oder soetwas in der Art.

[thewhobox]

@sveni_lee Ich verstehe nicht ganz was du erreichen willst?
Was meinst du mit Zeitschleife? oder neuer Wert?
Wenn der Adapter den neuen Wert setzt hast du in Javascript keinen Einfluss darauf.

Da musst du dann mit einem eigenen Datenpunkt arbeiten, den du nur setzt, wenn der DP false ist.
Da ist eine schleife unnötig, da du einfach auf den DP triggerst und mit nem if prüfst ob es false ist.

Mal ein Beispiel:

var stateToHold;

//Speichere den letzen Helligkeitswert in einer Variable
on({id: "hue.0.Light_0.bri", change: "ne", ack: true}, (dp) => {
    stateToHold = dp.state.val;
});

//Triggere wenn Lampe geschaltet wird
on({id: "hue.0.Light_0.on", change: "ne", ack: true}, (dp) => {
    //wenn Lampe ausgeschaltet wird, schreibe letzt Helligkeit in einen eigenen Datenpunkt
    if(dp.state.val == false) {
        setState("mein.eigener.datenpunkt.lastbri", stateToHold, true);
    }
});

[sveni_lee]

okay, da hatte ich mich undeutlich ausgedrückt.

ich trigger zur den Level eines Rollladen. wenn der Wert sich ändert wird mir das gemeldet.
Das Problem ist aber, das der Wert sich solange ändert bis der Rolladen die Endposition erreicht hat.
Ich möchte die Nachricht erst haben, wenn die endposition erreicht ist.
Es gibt den DP "Festverglasung rechts.WORKING" der geht auf true solange der Rollladen in Bewegung ist.
Ich wollte nun den DP als auslöser verwenden um die nachricht abzusetzen...

[paul53]

@sveni_lee sagte:

DP "Festverglasung rechts.WORKING" der geht auf true solange der Rollladen in Bewegung ist.
Ich wollte nun den DP als auslöser verwenden

const idFestRechtsWork = getIdByName('Festverglasung rechts.WORKING');
const idFestRechtsLevel = getIdByName('Festverglasung rechts.LEVEL');

var rechtslevel = getState(idFestRechtsLevel).val; // Initialisierung bei Skriptstart

on(idFestRechtsLevel, function(dp) { // Triggern bei Wertänderung
    rechtslevel = dp.state.val;
});

on(idFestRechtsWork, function(dp) {
   if(!dp.state.val) { // Änderung auf false
      // Nachricht mit Auswertung von rechtslevel
   }
});

[sveni_lee]

wie könnte ich mir anzeigen lassen, welcher state getriggert hat?

var Rolllaeden = ["hm-rpc.0.xxxx.1.LEVEL"/*Terrassentür links.LEVEL*/, "hm-rpc.0.yyyy.1.LEVEL"/*Terrassentür rechts.LEVEL*/, "hm-rpc.0.zzzz.1.LEVEL"/*Festverglasung links.LEVEL*/, "hm-rpc.0.rrrr.1.LEVEL"/*Festverglasung rechts.LEVEL*/];


on(Rolllaeden, function(dp) {...

[paul53]

@sveni_lee sagte:

welcher state getriggert hat?

on(Rolllaeden, function(dp) {
   log('Trigger: ' + dp.common.name);
});

[sveni_lee]

ich kann das grad nicht testen. gibt das den State "hm-rpc.0.xxxx.1.LEVEL" oder den Namen "Terrassentür links.LEVEL" aus?

[paul53]

@sveni_lee
Den Namen "Terrassentür links.LEVEL".

[sveni_lee]

komme ich auch irgendwie an den State ran? Also an "hm-rpc.0.xxxx.1.LEVEL".

[thewhobox]

@sveni_lee

dp.id

[sveni_lee]

so, es hat zwar etwas gedauert aber ich habe es hinbekommen.

var Rolllaeden = ["hm-rpc.0.WWWW.1.WORKING"/*Terrassentür links.WORKING*/,
                    "hm-rpc.0.XXXX.1.WORKING"/*Terrassentür rechts.WORKING*/, 
                    "hm-rpc.0.YYYY.1.WORKING"/*Festverglasung links.WORKING*/, 
                    "hm-rpc.0.ZZZZ.1.WORKING"/*Festverglasung rechts.WORKING*/];

var start_position;
var end_position;
                    
on(Rolllaeden, function(dp) {
   
   if(dp.state.val) {
       
       start_position = getState((dp.id).replace('WORKING', 'LEVEL')).val;

   }
   
   if(!dp.state.val) {
       
       end_position = getState((dp.id).replace('WORKING', 'LEVEL')).val;
       
       if(end_position > start_position)
       
       sendTo("telegram.0", "send", {
        text: dp.common.name.replace(".WORKING", "") + " geöffnet auf: " + end_position + "%"
      });
      
        if(end_position < start_position)
       
       sendTo("telegram.0", "send", {
        text: dp.common.name.replace(".WORKING", "") + " geschlossen auf: " + (100 - end_position) + "%"
      });
      
   }
    
});

Danke für eure hilfe...