[gelöst] Geräte-Name ermitteln mit JS-Funktion

[Cinimod]

Hey Leute ... ich habe hier ein Blockly, welches mir zu aufwendig erscheint.
Im Rahmen meiner Alarmanlage lasse ich mich informieren wenn ein Fenster/Türkontakt ausgelöst hat wenn die Alarmanlage scharf gestellt ist.
Ich ermittele recht aufwendig welcher der Sensoren der Auslöser war (Bild 1), ich meine mal gesehen zu haben das man auch den Namen der übergeordneten Geräts ermitteln kann, finde es aber nicht mehr wieder. Ich hoffe ich hab mich halbwegs ausgedrückt. Auf Bild 2 den Namen des Überordners welches den Trigger ausgelöst hat würde ich gern ermitteln, also "Türkontakt Haustür" in diesem Beispiel.

Es gibst doch ne in Formel für oder sowas ...

Ich habe schon folgendes Probiert:

liefert aber nicht das gewünschte Ergebnis :

javascript.0
2022-11-13 22:36:09.505	info	script.js.common.Alarmanlage.Alarmanlage_Sensoren2: null

javascript.0
2022-11-13 22:36:09.505	info	script.js.common.Alarmanlage.Alarmanlage_Sensoren2: undefined

[Codierknecht]

@cinimod
Nimm mal "Geräte ID" bzw. "Kanal ID".
Der Name kann ja nix bringen ... steht ja nix drin

[Cinimod]

@codierknecht

Dann hab ich das:

javascript.0
2022-11-13 22:55:40.230	info	script.js.common.Alarmanlage.Alarmanlage_Sensoren2: hmip.0.devices.3014F711A00000D8A99E4F7F.channels.1

javascript.0
2022-11-13 22:55:40.230	info	script.js.common.Alarmanlage.Alarmanlage_Sensoren2: hmip.0.devices.3014F711A00000D8A99E4F7F.channels.1.windowOpen

Ich meine mal gesehen zu haben das auch das möglich ist. Wie gesagt, finde es leider nicht wieder Das steht ja im Bereich NAME

[Codierknecht]

@cinimod
Jepp - ist ja auch das was zu erwarten gewesen wäre.
Ist Dir nicht sprechend genug?
Dann musst Du die Geräte und Kanäle bereits in der CCU mit entsprechenden Namen versehen.
Dann sollte auch "Gerätename" bzw. "Kanalname" sprechende Werte liefern.

Ich sehe gerade: Du hast HmIP? Über die Cloud?
Weiß nicht, ob das da so funktioniert

[paul53]

@cinimod sagte: liefert aber nicht das gewünschte Ergebnis :

Das funktioniert beim HmIP-Adapter nicht, da noch die Ordnerebene "channels" zwischen Device und Channel ist.

[Codierknecht]

@paul53
In der CCU kann ich auch den Kanal benennen. Das würde ja reichen.
Geht das bei HmIP?

[paul53]

@codierknecht sagte: Kanal benennen. Das würde ja reichen.

Das sollte auch bei HmIP funktionieren (habe selbst nur HM-RPC).
Oder man verwendet eine JS-Funktion mit Ergebnis.

[Cinimod]

@paul53

Oder man verwendet eine JS-Funktion mit Ergebnis.

Oh da kenne ich mich garnicht aus ... habe das jetzt so übernommen:

function deviceName(id) { // Datenpunkt-ID
    id = id.substring(0, id.lastIndexOf('.'));
    id = id.substring(0, id.lastIndexOf('.'));
    id = id.substring(0, id.lastIndexOf('.'));
    if(existsObject(id) && getObject(id).type == 'device') return getObject(id).common.name;
}

Weiß nicht ob das richtig ist ... oder richtig eingebunden ... liefert aber auch nicht das gewünschte Ergebnis:

javascript.0
2022-11-14 08:32:40.500	info	script.js.common.Alarmanlage.Alarmanlage_Sensoren2: undefined

javascript.0
2022-11-14 08:32:37.761	info	script.js.common.Alarmanlage.Alarmanlage_Sensoren2: undefined

[Codierknecht]

@cinimod
Du erwartest in der Funktion "deviceName" einen Parameter "id".
Der wird aber im Blockly nicht übergegeben.

Außerdem heißt die aufgerufene Funktion die ich da sehe "deviceName". Im Blockly wird aber eine Funktion "read_out_device" aufgerufen.
Und Du benötigst ein Funktion, die auch ein Ergebnis zurückliefert.
Und zum guten Schluss: Die ID des übergeordneten Gerätes kann man vermutlich auch besser (RegEx?) aus der kompletten ID extrahieren. Damit stehe ich aber auch noch auf Kriegsfuß - vielleicht kann @paul53 da einspringen

Du kannst das hier als Vorlage verwenden:

Spoiler

<xml xmlns="https://developers.google.com/blockly/xml">
 <variables>
   <variable id="dTU{03h%DXju^JS/AxF;">id</variable>
 </variables>
 <block type="procedures_defcustomreturn" id="XQ;HxW_Z2.P#LqOT6{k7" x="-462" y="362">
   <mutation statements="false">
     <arg name="id" varid="dTU{03h%DXju^JS/AxF;"></arg>
   </mutation>
   <field name="NAME">getDeviceNameFromChannelID</field>
   <field name="SCRIPT">cmV0dXJuICdibHViYic7</field>
   <comment pinned="false" h="80" w="160">Beschreibe diese Funktion …</comment>
 </block>
 <block type="on" id="WK^uVQER2gcXe|w0rs=4" x="-462" y="413">
   <field name="OID">Object ID</field>
   <field name="CONDITION">ne</field>
   <field name="ACK_CONDITION"></field>
   <statement name="STATEMENT">
     <block type="debug" id="#zEB*rjBDAi#n01rfdg,">
       <field name="Severity">log</field>
       <value name="TEXT">
         <shadow type="text" id="cN3?;Dd(R/QA!-AJUk|r">
           <field name="TEXT">test</field>
         </shadow>
         <block type="procedures_callcustomreturn" id="+?CRP/=qz/_[6fF1tSzO">
           <mutation name="getDeviceNameFromChannelID">
             <arg name="id"></arg>
           </mutation>
           <value name="ARG0">
             <block type="on_source" id="tn,V`w6$D-(HF{8J}TmA">
               <field name="ATTR">channelId</field>
             </block>
           </value>
         </block>
       </value>
     </block>
   </statement>
 </block>
</xml>

[Cinimod]

@codierknecht in deiner Funktion steht quasi nichts drin ? ist das richtig ?

[Codierknecht]

@cinimod sagte in Geräte-Name ermitteln:

@codierknecht in deiner Funktion steht quasi nichts drin ? ist das richtig ?

Ja, das soll ja nur eine Vorlage sein, die das Prinzip klarstellen soll.
Mit Leben darfst Du die dann selbst füllen

[Cinimod]

@codierknecht

Und genau da bin ich total raus Da brauche ich unbedingt Hilfe, das ist Neuland für mich, aber ich bin interessiert ....
Kannst du es mir vielleicht etwas näher bringen ?

[Codierknecht]

@cinimod
Nimm doch erstmal den Code den Du in Deiner Funktion "deviceName" drin hattest.
So allein vom Lesen müsste die doch schon ein Ergebnis bringen. Ob's das richtige Ergebnis ist, kann ich nicht beurteilen.
Zur Not mal Holzhammer-Debugging: Hinter jede Anweisung den aktuellen Wert ins Log schreiben

[paul53]

@cinimod sagte: Kannst du es mir vielleicht etwas näher bringen ?

In die Funktion gehört nur der innere Teil, die Funktionsdeklaration bildet Blockly:

Es muss der Funktionsparameter id an die Funktion übergeben werden:

An die Funktion muss bei Aufruf die Datenpunkt-ID übergeben werden.

[Codierknecht]

@paul53
Es muss aber eine Funktion mit Ergebnis sein. Sonst ist die hier relativ sinnbefreit

[paul53]

@codierknecht sagte: Es muss aber eine Funktion mit Ergebnis sein.

Du hast natürlich recht. Habe oben korrigiert.

[Cinimod]

@paul53

und die Variable "id" beinhaltete dann das Ergebnis ?

Ach so ne das geht ja nicht, weiß er ja garnicht woraus er das nehmen soll ...

dann so ?

EDIT: da kommt dann das raus:

javascript.0
2022-11-14 12:23:17.361	info	script.js.common.testen.eigene.JS_Funktion_mit_ergebnis: undefined

[paul53]

@cinimod sagte: weiß er ja garnicht woraus er das nehmen soll ...

Habe ich doch geschrieben:

An die Funktion muss bei Aufruf die Datenpunkt-ID übergeben werden.

[Cinimod]

@paul53

Sorry, hatte ich nicht gleich verstanden. Jetzt klappt es wie gewünscht.
Danke euch beiden.

[Codierknecht]

@paul53 @Cinimod
Wenn's dann funktioniert, könnte man die Funktion noch etwas verschlanken:

id = id.split('.',3).join('.'); // die ersten 3 Teil-Strings
if(existsObject(id) && getObject(id).type == 'device') return getObject(id).common.name; // Name aus DP lesen

Da wird das Pferd von der anderen Seite aufgezäumt und man spart sich 2 Durchläufe

OK, ich weiß ... Erbsenzählerei