[gelöst] Lichter zählen Script läuft nicht

lobomau

Bei mir ist irgendwas kaputt. Liegt nicht am Script (funktionierte mal)

Ich hatte hier die Hoffnung, dass die neueste Version was ändert, aber das Problem liegt irgendwo anders :roll:

javascript.0	2018-10-09 16:28:47.509	error	TypeError: Cannot read property 'common' of undefined
javascript.0	2018-10-09 16:28:47.509	error	^
javascript.0	2018-10-09 16:28:47.509	error	} else if (objects[id].common) {
javascript.0	2018-10-09 16:28:47.508	error	script.js.common.Licht.Licht-check: /opt/iobroker/node_modules/iobroker.javascript/javascript.js:1252
javascript.0	2018-10-09 16:28:47.403	info	Start javascript script.js.common.Licht.Licht-check

paul53

Editiere die Datei /opt/iobroker/node_modules/iobroker.javascript/javascript.js und ändere Zeile 1252 in

                                } else if (objects[id] && objects[id].common) {

Anschließend die JS-Instanz neu starten, damit die Änderung wirksam wird.

lobomau

~~@paul53:~~

Editiere die Datei /opt/iobroker/node_modules/iobroker.javascript/javascript.js und ändere Zeile 1252 in
                                } else if (objects[id] && objects[id].common) {
Anschließend die JS-Instanz neu starten, damit die Änderung wirksam wird. `
Grad getestet und es funktioniert Vielen Dank!!!

Bleibt die Frage… warum ist der Fehler nur bei mir gewesen? Ich habe noch nie in dieser Datei was geändert :roll:

paul53

@lobomau:

warum ist der Fehler nur bei mir gewesen? `

Ist er nicht. Ich habe die Änderung auch vorgenommen.

wendy2702

Hi,

wollte gerade das Script von Paul testen aber meine HUE Lampen werden mir nicht aufgezählt/angezeigt.

Als Beispiel mal diese Lampe:

Wenn ich die z.B per App einschalte ändert sich der Status aber das Script zeigt mir keine Änderung:

Hatte vorher das alte script in Benutzung und da hat es noch funktioniert aber halt mit Ausgabe des kompletten Namens.

Muss ich, wenn ich das neue Script verwenden möchte noch etwas an meiner HUE Lampen Zuordnung ändern?

paul53

@wendy2702:

meine HUE Lampen werden mir nicht aufgezählt/angezeigt.

role stimmt nicht überein, kann auch weggelassen werden.

const hueon   = $('state[id=*.on](functions=Licht)');

wendy2702

Hallo Paul,

danke für die schnelle Antwort. Das funktioniert natürlich.

Allerdings schreibt er mir den Namen der Lampe die an ist z.B. so:

15:22:43.772	[info]	javascript.0 script.js.Licht.Lichter_zaehlen_Paul53: registered 31 subscriptions and 0 schedules
15:22:44.268	[info]	javascript.0 script.js.Licht.Lichter_zaehlen_Paul53: Auslöser Skriptstart
15:22:44.268	[info]	javascript.0 script.js.Licht.Lichter_zaehlen_Paul53: Text: s_hue.Stehlampe
15:22:44.268	[info]	javascript.0 script.js.Licht.Lichter_zaehlen_Paul53: Anzahl Lichter: 30 # davon Lichter an: 1

Ist das so richtig oder liegt es an meiner Namensgebung der HUEs?

paul53

@wendy2702:

liegt es an meiner Namensgebung der HUEs?

Ja. Du solltest die Zahl der Zeichen, die vorn weggelassen werden sollen, an Deine Namensgebung anpassen.

wendy2702

DANKE!

Geändert und es klappt.

McMoneysack

Hallo zusammen,

anscheinend bin ich wirklich zu blöd.

Ich habe jetzt die letzten funktionierenden Codezeilen von Seite 4 genommen.

Dann an meine Loxone Installation angepasst, wie beschrieben.

Mein Code sieht wie folgt aus:

/* Status Zahl Lichter

zählt die Lichter in allen Räumen und meldet die Eingeschalteten namentlich
Daten kommen vom Gewerk "Licht"
*/
const logging = true;

const idAnzahlEin = 'javascript.0.Status.Lichter.Anzahl_ein',
      idAnzahl = 'javascript.0.Status.Lichter.Anzahl',
      idText = 'javascript.0.Status.Lichter.Text',
      idRaum = 'javascript.0.Status.Lichter.Raum', // <---- NEU
      idAnsage = 'javascript.0.Status.Lichter.Ansage';

// Ab hier nix mehr ändern
createState(idAnzahl, { // wenn benötigt: Anzahl der vorhandenen Lichter
    type: 'number',
    name: 'Anzahl aller Lichter',
    min: 0,
    def: 0,
    role: 'value'
});
createState(idAnzahlEin, { // Anzahl der Lichter, die auf sind als Variable unter Javascript.0 anlegen
    type: 'number',
    name: 'Anzahl der eingeschalteten Lichter',
    min: 0,
    def: 0,
    role: 'value'
});
createState(idText, { // Anzahl der brennenden Lichter und deren Namen als Variable unter Javascript.0 anlegen
    type: 'string',
    name: 'Eingeschaltete Lichter',
    desc: 'Namen der eingeschalteten Lichter',
    def: ' ',
    role: 'value'
});
createState(idAnsage, {
    type: 'string',
    name: 'Eingeschaltete Lichter (Ansage)',
    desc: 'Namen der eingeschalteten Lichter (für Ansage aufbereitet)',
    def: ' ',
    role: 'value'
});
createState(idRaum, { // Räume, in denen Lichter brennen // <---- NEU
    type: 'string',
    name: 'Räume mit eingeschalteten Lichter',
    desc: 'Namen der Räume, in denen Lichter eingeschaltet sind',
    def: ' ',
    role: 'value'
});

// HM Schalter, HM Dimmer und Hue Schalter im Gewerk Licht
const hmSTATE = $('device[state.id=*.active](functions=Beleuchtung)');
const hmLEVEL = $('device[state.id=*.position](functions=Beleuchtung)');
const hueon   = $('state[id=*.on](functions=Beleuchtung)[role=switch.light]');

var anzahlLichterAn,
    anzahlLichter,
    textLichterAn,
    textRaum;

function checkLicht(id, vorn, hinten) {
    var name = getObject(id).common.name;
    var devicename = name.substring(vorn, name.indexOf(hinten)); //Zeichen vorn, State hinten aus Text entfernen
    var raum = getObject(id, "rooms").enumNames[0]; 
    if (getState(id).val) { // wenn Zustand != false / 0, dann wird die Anzahl der Lichter hochgezählt
        ++anzahlLichterAn;
        textLichterAn.push(devicename); // Zu Array hinzufügen
        if(textRaum.indexOf(raum) == -1) textRaum.push(raum); // zu Array hinzu, wenn noch nicht vorhanden
    }
    ++anzahlLichter; // Zählt die Anzahl der vorhandenen Lichter unabhängig vom Status
}

function checkDevices() {
    // Setzt die Zähler vor dem Durchlauf aller Elemente *.STATE im Gewerk Licht auf 0
    anzahlLichterAn = 0;
    anzahlLichter = 0;
    textLichterAn = [];
    textRaum = [];
    hmSTATE.each(function(id, i) { // Schleife für jedes gefundenen Element *.STATE im Gewerk Licht
	    checkLicht(id, 0, '.STATE');
    });

    hueon.each(function(id, i) { // Schleife für jedes gefundenen Element *.on im Gewerk Licht
	    checkLicht(id, 6, '.on');
    });

    hmLEVEL.each(function(id, i) { // Schleife für jedes gefundenen Element *.LEVEL im Gewerk Licht
	    checkLicht(id, 0, '.LEVEL');
    });

    // Array mit Lichternamen sortieren
    textLichterAn.sort();
    textRaum.sort(); // <---- NEU
    // Schleife ist durchlaufen. Im Log wird der aktuelle Status (Anzahl, davon an) ausgegeben
    if (logging) log("Text: " + textLichterAn);
    if (logging) log("Anzahl Lichter: " + anzahlLichter + " # davon Lichter an: " + anzahlLichterAn);
    // die ermittelten Werte werden als javascript.0\. Variable in ioBroker gespeichert (z.B. für die Verarbeitung in VIS)
    //setState(idText, textLichterAn.join(',
')); // Schreibt die aktuelle Namen der eingeschalteten Lichter
    setState(idText, textLichterAn.join(', ')); // Schreibt die aktuelle Namen der eingeschalteten Lichter
    setState(idAnzahlEin, textLichterAn.length); // Schreibt die aktuelle Anzahl der eingeschalteten Lichter
    setState(idAnzahl, anzahlLichter); // Schreibt die aktuelle Anzahl der vorhandene Elemente im Gewerk Licht
    // setState(idRaum, textRaum.join(',
')); // Räume, in denen Licht brennt // <---- NEU
    setState(idRaum, textRaum.join(', ')); // Räume, in denen Licht brennt // <---- NEU
}
// Trigger
hmSTATE.on(function(obj) { // bei Zustandänderung *.STATE im Gewerk Licht
    if (logging) log('Auslösender Schalter: ' + obj.id + ': ' + obj.state.val); // Info im Log, welcher Zustand sich geändert hat
    checkDevices();
});
hueon.on(function(obj) { // bei Zustandänderung *.on von HUE Lampen im Gewerk Licht
    if (logging) log('Auslösende HUE: ' + obj.id + ': ' + obj.state.val); // Info im Log, welcher Zustand sich geändert hat
    checkDevices();
});
hmLEVEL.on(function(obj) { // bei Zustandänderung *.LEVEL im Gewerk Licht
    if (logging) log('Auslösender Dimmer: ' + obj.id + ': ' + obj.state.val); // Info im Log, welcher Zustand sich geändert hat
    checkDevices();
});

// Skriptstart
setTimeout(function() {
    if (logging) log('Auslöser Skriptstart');
    checkDevices();
}, 500);

// Aufbereitung für Ansage
function strip_tags(data) {
    var rueckgabe = data.replace(/( |<([^>]+)>)/ig, " ");
    return (rueckgabe);
}

on(idText, function(obj) {
    var text = obj.state.val;
    text = (getState(idAnzahl).val > 0) ? 'Lichter eingeschaltet: ' + strip_tags(text) : 'Alle Lichter sind ausgeschaltet';
    setState(idAnsage, text);
});

Da mein Gewerk Beleuchtung heißt, habe ich das an den 4 Stellen geändert.

In den Objekten bekomme ich angezeigt, wieviele Lampen an sind. Alexa sagt auch artig, wenn keine an sind, aber sobald eine Lampe eingeschaltet wird, bleibt das Feld text für die Lampen leer. Schalte ich 2 Lampen ein, bekomme ich 2 Punkte in das Feld, bei 3 Lampen 3 Punkte und Alexa sagt nix dazu.

Was zum Henker mache ich falsch?

Danke für Eure Geduld und Hilfe.

Liebe Grüße

Mc

paul53

@McMoneysack:

sobald eine Lampe eingeschaltet wird, bleibt das Feld text für die Lampen leer. Schalte ich 2 Lampen ein, bekomme ich 2 Punkte in das Feld, bei 3 Lampen 3 Punkte

Wie sehen die Namen der Datenpunkte für die Beleuchtung aus (Screenshot der Objekte) ? Das, was als Text eingetragen wird, ist aus den Datenpunktnamen gebildet.

McMoneysack

Danke Paul!

Hab mal ein paar Screenshots gemacht:

Interessant, sind alle Lichter wieder aus, zeigt er trotzdem noch an:

DIe Lichter Objekte sehen so aus:

Und wenn ich mehrere Lichter anhabe eben so:

paul53

Ich sehe keinen Datenpunkt, dessen ID auf .on, .active oder .position endet.

McMoneysack

~~@paul53:~~

Ich sehe keinen Datenpunkt, dessen ID auf .on, .active oder .position endet. `

Danke für den Hinweis, hier mit "active", sofern es das richtige "Ende" ist.

paul53

Vom Namen des Datenpunktes xxx.AI1.active sehe ich leider nur einen Teil. Muss am Ende etwas entfernt werden ?

Wie sieht es bei den anderen Datenpunkte (mit .on, .position) aus ?

McMoneysack

Danke für deine Mühe Paul.

Ich habe nun alle Daten im Screenshot einer Lampe.

Die Datenpunktendungen active und postion hab ich von dir übernommen wo würde ich die sonst finden?

Brauchst du noch was?

paul53

Dann ändere mal:

    hmSTATE.each(function(id, i) { // Schleife für jedes gefundenen Element *.active im Gewerk Licht
	    checkLicht(id, 0, ': active');
    });

und

    hmLEVEL.each(function(id, i) { // Schleife für jedes gefundenen Element *.position im Gewerk Licht
	    checkLicht(id, 0, ': position');
    });

McMoneysack

Javascript Adapter neugestartet, nach den Änderungen.

Leider keine Veränderung in der Ausgabe.

paul53

~~@McMoneysack:~~

Leider keine Veränderung in der Ausgabe. `
Dann bau mal ein Log ein:

function checkLicht(id, vorn, hinten) {
    var name = getObject(id).common.name;
    log('DP: ' + name);
    var devicename = name.substring(vorn, name.indexOf(hinten)); //Zeichen vorn, State hinten aus Text entfernen
    log('Gerät: ' + devicename);

McMoneysack

Folgendes schmeißt er raus:

Nochmals Danke für deine Zeit

NEWS

[gelöst] Lichter zählen Script läuft nicht

Support us

774
Online

31.8k
Users

80.0k
Topics

1.3m
Posts

NEWS

[gelöst] Lichter zählen Script läuft nicht

Support us

774Online

31.8kUsers

80.0kTopics

1.3mPosts

774
Online

31.8k
Users

80.0k
Topics

1.3m
Posts