Jahresrückblick 2025 – unser neuer Blogbeitrag ist online! ✨

[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 :grin:

[Codierknecht]

@cinimod
Zum rumprobieren kann man auch das hier gut brauchen: https://playcode.io/javascript
Da fehlen dann zwar die ioBroker-spezifischen Dinge, aber für ein paar Gehversuche mit JS kann man das ganz gut einsetzen.

[paul53]

@codierknecht sagte: die Funktion noch etwas verschlanken:

Das wird mit 3 nicht funktionieren, da noch ein Ordner "devices" dazwischen hängt:
hmip.0.devices.3014F711A0001118A9925C9B

[Codierknecht]

@paul53
Jetzt wirst Du aber kleinlich ... dann sind's halt 4 :grin: :grin: :grin: :grin:

Das wird jeder vielleicht ein bisschen anders sehen, aber ich persönlich finde die so deutlich besser lesbar.
Und wenn man die Funktion dann noch umbenennt in z.B. getDeviceNameFromID kann man auch im Blockly direkt erkennen, was die denn so macht und auch die CleanCode-Fraktion ist zufrieden ;-)

[paul53]

@codierknecht
Beide Versionen sind an den HmIP-Adapter angepasste Versionen.
Eine Version, die mit allen Adaptern (auch Zigbee) funktioniert, würde so aussehen:

id = id.substring(0, id.lastIndexOf('.'));
if(existsObject(id) && getObject(id).type == 'device') return getObject(id).common.name;
id = id.substring(0, id.lastIndexOf('.'));
if(existsObject(id) && getObject(id).type == 'device') return getObject(id).common.name;
id = id.substring(0, id.lastIndexOf('.'));
if(existsObject(id) && getObject(id).type == 'device') return getObject(id).common.name;

[Codierknecht]

@paul53 sagte in [gelöst] Geräte-Name ermitteln mit JS-Funktion:

Beide Versionen sind an den HmIP-Adapter angepasste Versionen.

Jepp - war ja hier auch die Anforderung.

Eine Version, die mit allen Adaptern (auch Zigbee) funktioniert, würde so aussehen:

Auch dazu: :+1:
Ob die Roundtrips dann (spürbare) negative Auswirkungen auf das Laufzeitverhalten haben, kannst Du besser beurteilen als ich.