Frage Javascript "on-id-change"

sveni_lee

stimmt..
aber die ccu bietet einen state mit allen anwesenden. den könnte man wenn nötig auch auswerten...

ONKYO-AVR,Google-Home,EchoDot,WDR3600,CCU2,Strom-Pi,Gotham-City,VissmannWP-Pi

sveni_lee

@paul53

Ich wollte das ganze jetzt noch etwas verfeinern und dem Datenpunkt einen clearname zuordnen aber das scheint so nicht zu funktionieren.

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 humans = {"hm-rega.0.1857" : "XXX", "hm-rega.0.1858" : "YYY", "hm-rega.0.1864" : "ZZZ"};

on(anwesenheit_devices, function(dp) {
    str = dp.common.name;

   if(dp.state.val) {

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

      // Reaktionen auf false
      sendTo("telegram.0", "send", {
        text: humans.dp + " 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"
      });
   }

});

aber das klappt nicht... "humans.dp" wird nicht zugeordnet.

paul53

@sveni_lee
Weshalb verwendest Du nicht die in der CCU vergebenen Namen ?

       text: dp.common.name + " ist anwesend"

In humans sind die Datenpunkt-IDs die Bezeichner, also

    var str = dp.id;
    ...    
        text: humans[str] + " ist anwesend"

sveni_lee

@paul53

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.

Community Forum

NEWS

Frage Javascript "on-id-change"

Support us

757

32.5k

81.6k

1.3m