Frage Javascript "on-id-change"

sveni_lee

das klingt irgendwie nicht so toll. In der CCU werden die Name mit Anwesenheit.XXX_IPhone bezeichnet.
Ich Iabe die Namensgebung so wie in der Fritzbox gewählt, kann man dann besser nachvollziehen.
Klingt aber nicht schön wenn der Bot medet Anwesenheit.XXX_iPhone ist anwesend. :relaxed:

Darum dachte ich mir die Idee mit dem Objekt wäre nicht schlecht...

müss das Objekt humans nicht so aussehen:

var humans = {['hm-rega.0.1857'] : "XXX", ['hm-rega.0.1858'] : "YYY", ['hm-rega.0.1864'] : "ZZZ"};

thewhobox

@sveni_lee sagte in Frage Javascript "on-id-change":

var humans = {['hm-rega.0.1857'] : "XXX", ['hm-rega.0.1858'] : "YYY", ['hm-rega.0.1864'] : "ZZZ"};

Das wird niemals funktionieren.

Der fehler liegt daran wie du an die Namen kommen möchtest.
Ersetze folgendes:

text: humans.dp + " ist abwesend"
// durch
text: humans[dp.id] + " ist abwesend"

Das sollte funktioneren, ist aber nicht getestet.

paul53

@sveni_lee sagte:

das Objekt humans nicht so aussehen:

Nein, die ursprüngliche Form war schon richtig, nur dass über die Datenpunkt-ID ([dp.id]) zugegriffen werden muss und nicht über das Datenpunkt-Objekt (dp).

sveni_lee

okay, also das ganze sieht nun so aus...

var anwesenheit_devices = ["hm-rega.0.1857", "hm-rega.0.1858", "hm-rega.0.1864"];
var online_devices = ["hm-rega.0.1867", "hm-rega.0.1868", "hm-rega.0.1869", "hm-rega.0.1871", "hm-rega.0.1872", "hm-rega.0.1873", "hm-rega.0.1874", "hm-rega.0.1875", "hm-rega.0.1876", "hm-rega.0.1877", "hm-rega.0.1879", "hm-rega.0.1880", "hm-rega.0.1881", "hm-rega.0.1882", "hm-rega.0.1883", "hm-rega.0.1884"];
var str;
var humans = {"hm-rega.0.1857" : "XXX", "hm-rega.0.1858" : "YYY", "hm-rega.0.1864" : "ZZZ"};

on(anwesenheit_devices, function(dp) {

   if(dp.state.val) {
      var str1 = dp.id;

      // Reaktionen auf true
      sendTo("telegram.0", "send", {
        text: humans[dp.id] + " ist anwesend"
      });
   } 
   
   else {

      // Reaktionen auf false
      sendTo("telegram.0", "send", {
        text: humans[dp.id] + " ist abwesend"
      });
   }

});

on(online_devices, function(dp) {
    str = dp.common.name;
   if(dp.state.val) {

      // Reaktionen auf true
      sendTo("telegram.0", "send", {
        text: str.substr(12) + " ist online"
      });
   } 
   
   else {

      // Reaktionen auf false
      sendTo("telegram.0", "send", {
        text: str.substr(12) + " ist offline"
      });
   }

});

Ich warte dann mal bis heute nachmittag jemand nach Hause kommt. ann weiß ich ob es geht.

sveni_lee

jetzt versuche ich grad in dem zusammenhang eien json-datei auszulesen.

[{"id":290,"type":"2","caller":"","called":"SIP: 700000","callednumber":"700000","name":"Unbekannt","numbertype":"sip","device":"","port":"-1","date":"24.06.19 14:48","duration":"0:00","count":"","path":"","sym":"x","external":""},
 {"id":289,"type":"2","caller":"","called":"SIP: 700000","callednumber":"700000","name":"Unbekannt","numbertype":"sip","device":"","port":"-1","date":"24.06.19 14:47","duration":"0:00","count":"","path":"","sym":"x","external":""},
 {"id":288,"type":"2","caller":"","called":"SIP: 700000","callednumber":"700000","name":"Unbekannt","numbertype":"sip","device":"","port":"-1","date":"24.06.19 14:46","duration":"0:00","count":"","path":"","sym":"x","external":""},
 {"id":287,"type":"2","caller":"","called":"SIP: 700000","callednumber":"700000","name":"Unbekannt","numbertype":"sip","device":"","port":"-1","date":"24.06.19 14:45","duration":"0:00","count":"","path":"","sym":"x","external":""},
 {"id":286,"type":"2","caller":"","called":"SIP: 700000","callednumber":"700000","name":"Unbekannt","numbertype":"sip","device":"","port":"-1","date":"24.06.19 14:43","duration":"0:00","count":"","path":"","sym":"x","external":""},
 {"id":285,"type":"2","caller":"","called":"SIP: 700000","callednumber":"700000","name":"Unbekannt","numbertype":"sip","device":"","port":"-1","date":"24.06.19 14:42","duration":"0:00","count":"","path":"","sym":"x","external":""},
 {"id":284,"type":"2","caller":"","called":"SIP: 700000","callednumber":"700000","name":"Unbekannt","numbertype":"sip","device":"","port":"-1","date":"24.06.19 14:40","duration":"0:00","count":"","path":"","sym":"x","external":""},
 {"id":283,"type":"2","caller":"","called":"SIP: 700000","callednumber":"700000","name":"Unbekannt","numbertype":"sip","device":"","port":"-1","date":"24.06.19 14:38","duration":"0:00","count":"","path":"","sym":"x","external":""},
 {"id":282,"type":"2","caller":"","called":"SIP: 700000","callednumber":"700000","name":"Unbekannt","numbertype":"sip","device":"","port":"-1","date":"24.06.19 14:33","duration":"0:00","count":"","path":"","sym":"x","external":""},
 {"id":281,"type":"2","caller":"","called":"SIP: 700000","callednumber":"700000","name":"Unbekannt","numbertype":"sip","device":"","port":"-1","date":"24.06.19 14:29","duration":"0:00","count":"","path":"","sym":"x","external":""}]

jetzt hatte ich folgendes probiert um zu sehen ob etwas augegeben wird:

    var objects = JSON.parse(jsonstring);
    log(objects.id);

aber wird es natürlich nicht. Ich denke ich müsste mit einer Schleife arbeiten aber irgendwie hab ich da grad keine idee...

thewhobox

@sveni_lee
Bei dir ist objects ein Array/Liste.
Du musst, wie du gesagt hast, mit einer Schleife alle Einträge durchgehen.

for(var i = 0; i<objects.length; i++) {
    var tempObj = objects[i];
    log(tempObj.id);
}

sveni_lee

ja, das hat funktioniert.

ich hab das ganze jetzt mal versuht in ein scrpt zu packen: am ende möchte ich per telegram einen button drücken und dann bekomme ich die entsprechende Liste.

var jsonstring = getState("tr-064-community.0.calllists.missed.json"/*json*/).val;

function list_Calls(jsonstring, call_list){

    call_list = call_list.split('\r\n');
    var objects = JSON.parse(jsonstring);

    for(var i = 0; i<objects.length; i++) {
        var tempObj = objects[i];
        
        call_list.unshift(tempObj.date + " " + tempObj.name + " " + tempObj.caller + '\r\n');
    }   
    
    log(call_list); 
        
     sendTo("telegram.0", "send", {
     text: call_list
     });
       
}

list_Calls(jsonstring, "missed calls:");

wenn ich das so mache bekomme ich eine Fehlermeldung von Telegram:

telegram.0	2019-06-25 12:06:59.838	error	Cannot send message [chatId - xxxxxxx]: Error: ETELEGRAM: 400 Bad Request: message text is empty
javascript.0	2019-06-25 12:06:59.812	info	script.js.common.Telegram.Abfrage_Calls: registered 0 subscriptions and 0 schedules
javascript.0	2019-06-25 12:06:59.812	debug	sendTo "send" to system.adapter.telegram.0 from system.adapter.javascript.0
javascript.0	2019-06-25 12:06:59.811	info	,missed calls:
javascript.0	2019-06-25 12:06:59.811	info	,24.06.19 14:48 Unbekannt
javascript.0	2019-06-25 12:06:59.811	info	,24.06.19 14:47 Unbekannt
javascript.0	2019-06-25 12:06:59.811	info	,24.06.19 14:46 Unbekannt
javascript.0	2019-06-25 12:06:59.811	info	,24.06.19 14:45 Unbekannt
javascript.0	2019-06-25 12:06:59.811	info	,24.06.19 14:43 Unbekannt
javascript.0	2019-06-25 12:06:59.811	info	,24.06.19 14:42 Unbekannt
javascript.0	2019-06-25 12:06:59.811	info	,24.06.19 14:40 Unbekannt
javascript.0	2019-06-25 12:06:59.811	info	,24.06.19 14:38 Unbekannt
javascript.0	2019-06-25 12:06:59.811	info	,24.06.19 14:33 Unbekannt

lt. Log wird die Variable "call_list" ja erzeugt.

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
   }
});

Community Forum

NEWS

Frage Javascript "on-id-change"

Support us

637

32.5k

81.6k

1.3m