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

[crunchip]

@Negalein lösch mal deine aufzählung raum und fenster komplett.
Starte den java adapter neu und dann füg mal nur ein fenster hinzu so wie bei mir und starte dann java nochmal neu.

[Negalein]

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

@Negalein lösch mal deine aufzählung raum und fenster komplett.
Starte den java adapter neu und dann füg mal nur ein fenster hinzu so wie bei mir und starte dann java nochmal neu.

kommt auch wieder nur dieses [object Object]

[crunchip]

@Negalein seltsam, mihome eventuell noch neu starten, falls du namen geändert hast. Und dann nochmal java.
Das einzige was bei dir anders ist, ich habe fenster und tür getrennt in den enums und kleingeschrieben
Hast du auch mal das fenster auf und zu gemacht?

[Negalein]

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

@Negalein seltsam, mihome eventuell noch neu starten, falls du namen geändert hast. Und dann nochmal java.
Das einzige was bei dir anders ist, ich habe fenster und tür getrennt in den enums und kleingeschrieben

hat auch nichts gebracht.

Ich teste morgen mal mit fenster

[kilasat]

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

@Negalein dem state selbst fehlt noch die raumzuweisung
Wenn ich nur übergeordnet den sensor zuweise, funktioniert die Abfrage bei mir zumindest nicht

Was heisst hier genau dem state selber? Machst du die Zuweisungen bei den Aufzählungen oder bei den Objekten? Müssen alle Objekte sowohl Raum als auch Funktion haben oder nur der State?

[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]

[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));