[gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an

crunchip

@kilasat ich stelle das direkt in den Objekten ein, ist einfacher und übersichtlicher, denn wenn du es über Aufzählung machst, verlierst du den Überblick, welche Sensoren du schon wo drin hast, da ja alles "is opened" heisst
einmal der Sensor selbst, erste Zeile und einmal der state, daher steht das in Raum und Funktion 2mal drin

enum.room.xyz und enum.functions.xyz sind bei mir alles klein geschrieben

crunchip

@Negalein @kilasat hatte heute Morgen auch seltsamerweise nur ein [object Object], als meine Haustür offen war. Alles andere wurde richtig angezeigt.
Habe mich dann auf die Suche gemacht.
Sensor aus den enums heraus gelöscht, neu hinzugefügt, ohne Erfolg
Sensor komplett aus dem Mihome-Adapter gelöscht und zugefügt, ohne Erfolg
Sensor aus dem Gateway gelöscht und neu eingerichtet, ohne Erfolg

Habe dann spaßeshalber, den Sensor auf einen anderen Raum gelegt, und siehe da, er wird wieder richtig angezeigt.
Folge dessen, habe ich in meinem Fall dann Raum Flur enum.rooms.flur komplett gelöscht, neu angelegt und Sensoren wieder entsprechend hinzugefügt, läuft wieder

paul53

@Negalein
Die Funktion cacheSelectorState.each() ist so bei Deiner Namensgebung ungeeignet. Ändere mal in

    cacheSelectorState.each(function(id, i) { // Schleife für jedes gefundenen Element *.state im Gewerk Fenster
        var status = getState(id).val; // Zustand *.state abfragen (jedes Element)
        var deviceId = id.substring(0, id.lastIndexOf("."));
        var devicename = getObject(deviceId).common.name; 
        var raumname = getObject(id, "rooms").enumNames[0]; // <---- NEU
        if (logging) {
            log('---Mi----');
            log('Kanal: ' + devicename);
            log('Status: ' + status);
        }
        if (status) { // wenn Zustand = true, dann wird die Anzahl der Fenster hochgezählt
            ++anzahlLichterAn;
            textLichterAn.push(devicename); // Zu Array hinzufügen
            textRaum.push(raumname); // <---- NEU
        }
        ++anzahlLichter; // Zählt die Anzahl der vorhandenen Lichter unabhängig vom Status
    });

Negalein

@paul53 sagte in [gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an:

@Negalein
Die Funktion cacheSelectorState.each() ist so bei Deiner Namensgebung ungeeignet. Ändere mal in

Danke

Nur jetzt dein Script als eigenständiges Script, oder meins von oben erweitern?

cacheSelectorState.each() finde ich in dem von mir alleine nirgends.
Nur mit Inhalt in den Klammern.

paul53

@Negalein sagte:

eigenständiges Script, oder meins von oben erweitern?

Weder noch. Die komplette Funktion cacheSelectorState.each(function(id, i) austauschen.

Negalein

@paul53 sagte in [gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an:

@Negalein sagte:

eigenständiges Script, oder meins von oben erweitern?

Weder noch. Die komplette Funktion cacheSelectorState.each(function(id, i) austauschen.

Sorry für die Fragen. JS ist ein Buch mit 1000 Siegeln für mich.

Also,

/* Status Zahl Fenster
Homematic var cacheSelectorLevel = $('channel[state.id=*.STATE](functions="Fenster")');
Mi var cacheSelectorState = $('state[id=*.state](functions="Fenster")');
{1}
*/
var logging = false;
 
var idAnzahlEin = 'javascript.0.Status.Fenster.Anzahl_auf',
    idAnzahl = 'javascript.0.Status.Fenster.Anzahl',
    idText = 'javascript.0.Status.Fenster.Text',
    idRaum = 'javascript.0.Status.Fenster.Raum', // <---- NEU
    idAnsage = 'javascript.0.Status.Fenster.Ansage';

tauschen in

    cacheSelectorState.each(function(id, i) { // Schleife für jedes gefundenen Element *.state im Gewerk Fenster
        var status = getState(id).val; // Zustand *.state abfragen (jedes Element)
        var deviceId = id.substring(0, id.lastIndexOf("."));
        var devicename = getObject(deviceId).common.name; 
        var raumname = getObject(id, "rooms").enumNames[0]; // <---- NEU
        if (logging) {
            log('---Mi----');
            log('Kanal: ' + devicename);
            log('Status: ' + status);
        }
        if (status) { // wenn Zustand = true, dann wird die Anzahl der Fenster hochgezählt
            ++anzahlLichterAn;
            textLichterAn.push(devicename); // Zu Array hinzufügen
            textRaum.push(raumname); // <---- NEU
        }
        ++anzahlLichter; // Zählt die Anzahl der vorhandenen Lichter unabhängig vom Status
    });

crunchip

@Negalein @paul53 meint diesen Block austauschen

Negalein

Danke @paul53 und @crunchip

Ich bin dank euch einen Schritt weiter!

DP "Text" wird richtig ausgefüllt
DP "Raum" erscheint weiterhin dieses [object Object]

objects - ioBroker (11).png
objects - ioBroker (12).png

enums - ioBroker (2).png
enums - ioBroker (3).png

paul53

@Negalein
Wie sieht es aus, wenn das Gewerk "Fenster" und der Raum "Schlafzimmer" nur dem Datenpunkt und nicht auch dem Kanal zugeordnet werden ?

Negalein

@paul53 sagte in [gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an:

@Negalein
Wie sieht es aus, wenn das Gewerk "Fenster" und der Raum "Schlafzimmer" nur dem Datenpunkt und nicht auch dem Kanal zugeordnet werden ?

gleiches Ergebnis

crunchip

@Negalein probier mal einen anderen Raum, vllt findest so den Fehler, bei mir ging plötzlich der Raum Flur nicht mehr
zur Not auch enum.funktions.fenster auch mal löschen und neu anlegen

paul53

@Negalein
Setze mal logging auf true und tausche die Zeile

            log('Kanal: ' + devicename);

gegen

            log('Raum: ' + raumname);

Was kommt im Log ? Version des JS-Adapters ?

Negalein

@paul53 sagte in [gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an:

Was kommt im Log ? Version des JS-Adapters ?

JS: 4.1.14

Log:

22:23:07.022	info	javascript.0 Stop script script.js.common.Fensterzählen
22:23:07.187	info	javascript.0 Start javascript script.js.common.Fensterzählen
22:23:07.187	info	javascript.0 script.js.common.Fensterzählen: registered 2 subscriptions and 0 schedules
22:23:09.182	info	javascript.0 script.js.common.Fensterzählen: Auslöser Skriptstart
22:23:09.182	info	javascript.0 script.js.common.Fensterzählen: ++++++ Fenster Anzahl ++++
22:23:09.182	info	javascript.0 script.js.common.Fensterzählen: #### Mi #####
22:23:09.182	info	javascript.0 script.js.common.Fensterzählen: ---Mi----
22:23:09.182	info	javascript.0 script.js.common.Fensterzählen: Raum: [object Object]
22:23:09.182	info	javascript.0 script.js.common.Fensterzählen: Status: false
22:23:09.182	info	javascript.0 script.js.common.Fensterzählen: #### DIMMER #####
22:23:09.182	info	javascript.0 script.js.common.Fensterzählen: Text:
22:23:09.182	info	javascript.0 script.js.common.Fensterzählen: Anzahl Fenster: 1 # davon Fenster offen: 0

Negalein

@crunchip sagte in [gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an:

@Negalein probier mal einen anderen Raum, vllt findest so den Fehler, bei mir ging plötzlich der Raum Flur nicht mehr
zur Not auch enum.funktions.fenster auch mal löschen und neu anlegen

hat leider nichts gebracht

paul53

@Negalein
Dann versuche mal das Log so zu ändern:

            log('Raum: ' + JSON.stringify(raumname));

Negalein

@paul53 sagte in [gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an:

@Negalein
Dann versuche mal das Log so zu ändern:
            log('Raum: ' + JSON.stringify(raumname));

[object Objekt] bleibt bestehen

Log:

javascript.0	2019-07-17 22:46:30.497	info	script.js.common.Fensterzählen: Anzahl Fenster: 1 # davon Fenster offen: 0
javascript.0	2019-07-17 22:46:30.497	info	script.js.common.Fensterzählen: Text:
javascript.0	2019-07-17 22:46:30.497	info	script.js.common.Fensterzählen: #### DIMMER #####
javascript.0	2019-07-17 22:46:30.497	info	script.js.common.Fensterzählen: Status: false
javascript.0	2019-07-17 22:46:30.497	info	script.js.common.Fensterzählen: Raum: {"en":"Sleeping room","de":"Schlafzimmer","ru":"Спальня","pt":"Quarto de dormir","nl":"Slaapkamer","fr":"Chambre à coucher","it":"Camera da letto","es":"Dormitor
javascript.0	2019-07-17 22:46:30.497	info	script.js.common.Fensterzählen: ---Mi----
javascript.0	2019-07-17 22:46:30.497	info	script.js.common.Fensterzählen: #### Mi #####
javascript.0	2019-07-17 22:46:30.496	info	script.js.common.Fensterzählen: ++++++ Fenster Anzahl ++++
javascript.0	2019-07-17 22:46:30.496	info	script.js.common.Fensterzählen: Auslösender Schalter: mihome.0.devices.magnet_158d0003139ea7.state: false
javascript.0	2019-07-17 22:46:25.334	info	script.js.common.Fensterzählen: Anzahl Fenster: 1 # davon Fenster offen: 1
javascript.0	2019-07-17 22:46:25.334	info	script.js.common.Fensterzählen: Text: Schlafzimmer
javascript.0	2019-07-17 22:46:25.334	info	script.js.common.Fensterzählen: #### DIMMER #####
javascript.0	2019-07-17 22:46:25.333	info	script.js.common.Fensterzählen: Status: true
javascript.0	2019-07-17 22:46:25.333	info	script.js.common.Fensterzählen: Raum: {"en":"Sleeping room","de":"Schlafzimmer","ru":"Спальня","pt":"Quarto de dormir","nl":"Slaapkamer","fr":"Chambre à coucher","it":"Camera da letto","es":"Dormitor
javascript.0	2019-07-17 22:46:25.333	info	script.js.common.Fensterzählen: ---Mi----
javascript.0	2019-07-17 22:46:25.333	info	script.js.common.Fensterzählen: #### Mi #####
javascript.0	2019-07-17 22:46:25.333	info	script.js.common.Fensterzählen: ++++++ Fenster Anzahl ++++
javascript.0	2019-07-17 22:46:25.332	info	script.js.common.Fensterzählen: Auslösender Schalter: mihome.0.devices.magnet_158d0003139ea7.state: true

paul53

@Negalein
Interessant ! Habe ich so noch nicht beobachtet. Dann ändere mal

        var raumname = getObject(id, "rooms").enumNames[0]; // <---- NEU

in

        var raumname = getObject(id, "rooms").enumNames.de; // <---- NEU

Negalein

@paul53 sagte in [gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an:

@Negalein
Interessant ! Habe ich so noch nicht beobachtet. Dann ändere mal

Danke!
Ich denke jetzt funktioniert es!

objects - ioBroker (13).png

Log:

22:57:26.424	info	javascript.0 Stop script script.js.common.Fensterzählen
22:57:38.435	info	javascript.0 Stop script script.js.common.Fensterzählen
22:57:41.492	info	javascript.0 Start javascript script.js.common.Fensterzählen
22:57:41.492	info	javascript.0 script.js.common.Fensterzählen: registered 2 subscriptions and 0 schedules
22:57:43.487	info	javascript.0 script.js.common.Fensterzählen: Auslöser Skriptstart
22:57:43.487	info	javascript.0 script.js.common.Fensterzählen: ++++++ Fenster Anzahl ++++
22:57:43.487	info	javascript.0 script.js.common.Fensterzählen: #### Mi #####
22:57:43.487	info	javascript.0 script.js.common.Fensterzählen: ---Mi----
22:57:43.487	info	javascript.0 script.js.common.Fensterzählen: Raum: undefined
22:57:43.487	info	javascript.0 script.js.common.Fensterzählen: Status: false
22:57:43.487	info	javascript.0 script.js.common.Fensterzählen: #### DIMMER #####
22:57:43.487	info	javascript.0 script.js.common.Fensterzählen: Text:
22:57:43.487	info	javascript.0 script.js.common.Fensterzählen: Anzahl Fenster: 1 # davon Fenster offen: 0

paul53

@Negalein sagte:

Ich denke jetzt funktioniert es!

Nein, undefined ist nicht das gewünschte Ergebnis.
Habe es gerade bei mir getestet und es funktioniert mit

var raumname = getObject(id, "rooms").enumNames[0];

Negalein

@paul53 sagte in [gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an:

@Negalein sagte:

Ich denke jetzt funktioniert es!

Nein, undefined ist nicht das gewünschte Ergebnis.
Habe es gerade bei mir getestet und es funktioniert mit
var raumname = getObject(id, "rooms").enumNames[0];

komisch!
Mit diesem hat er immer das [object Object] zusätzlich gezeigt.

NEWS

[gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an

Support us

810
Online

31.7k
Users

79.9k
Topics

1.3m
Posts

NEWS

[gelöst] Script "Zählen von Fenstern" zeigt keine offenen Fenster an

Support us

810Online

31.7kUsers

79.9kTopics

1.3mPosts

810
Online

31.7k
Users

79.9k
Topics

1.3m
Posts