[gelöst] Frage zum Script "Zählen von Fenstern"

Gwoony

Hallo Andy,

danke. Ich habe es kopiert und die Funktion "Fensterkontakt" hinzugefügt.

Leider kommt nur die vier Variablen Anzahl Fenster = und Offene Fenster 0. Wenn ich Fenster auf und zu machen, passt gar nichts.

im Script steht auch nur am Anfang was von Fenster, danach kommen sofort "Gewerke Licht". Passt dies script so wirklich?

Kannst du weiterhelfen?

function checkDevices(obj) {
    // Setzt die Zähler vor dem Durchlauf aller Elemente *.STATE im Gewerk Licht auf 0
    var anzahlLichterAn = 0;
    var anzahlLichter = 0;
    var textLichterAn = [];
    var textRaum = [];
    if (logging) {
        log('++++++ Fenster Anzahl ++++ ');
        log('#### Mi ##### ');
    }
    cacheSelectorState.each(function(id, i) { // Schleife für jedes gefundenen Element *.STATE im Gewerk Licht
        var obj = getObject(id);
        var name = getObject(id).common.name;
        var status = getState(id).val; // Zustand *.STATE abfragen (jedes Element)
        var devicename = name.substring(0, name.indexOf(".STATE")); //.state aus Text entfernen
        var raumname = getObject(id, "rooms"); // <---- NEU

        // ########## Abfrage ob Virtueller Kanal? VK oder : im Namen)
        // if ( (devicename.search(/(CUX)|(K2)|(K3)|(VK)|(:)/ig) == -1) ) { 
        if (logging) {
            log('---Mi----');
            log('Kanal: ' + name);
            log('Status: ' + status);
        }
        if (status) { // wenn Zustand = true, dann wird die Anzahl der Lichter hochgezählt
            ++anzahlLichterAn;
            textLichterAn.push(devicename)

andyb

Sorry, kann ich nicht.

Das Skrpit ist von Pix irgendwo hier aus dem Forum. Hab es nur soweit angepasst das es meine Fenster zählt und in die Variable schreibt.

Bei mir läuft es so….

Edit: hab gerade noch gesehen, deine function ist "fensterkontakte" klein geschrieben. Wenn du das im Skript überall änderst sollte es gehen.

Gwoony

Dann probiere ich dies zu Hause <emoji seq="1f60a">😊</emoji>

Gwoony

Tut leider immer noch nicht… nun ja

tempestas

~~@Kueppert:~~

Hallo andy,

was hast du genau geändert im Skript, damit deine Fensterkontakte von HM und xiaomi gezählt werden?? Ich habe das Skript von paul kopiert und lediglich das "Suchwort" auf Fensterkontakt geändert (vorher schon Fenster probiert).

Dann habe ich meinen Fensterkontakten eben genau diese "Funktion" unter Objekte zugewiesen. Es werden bei mir aber weder die HM- noch die Xiaomi-Sensoren gezählt :(

Kannst du mir ggf. dein Skript hier schicken? Oder siehst du ggf. einen Fehler bei mir? So lng ist der Javaskript ja nicht, aber ich finde keinen Fehler…

Fenster_zaehlen1.PNG

Fenster_zaehlen2.PNG

! ````
/* System Zahl Fenster
! zählt die Fenster in allen Räumen und meldet die offenen Fenster namentlich
Daten kommen vom Gewerk 'Fenster'
*/

function fensterstatus(zustand) {
var meldung;
switch (zustand) {
case 1:
meldung = 'RHS gekippt';
break;
case 2:
meldung = 'RHS offen';
break;
case true:
meldung = 'TFK offen';
break;
default:
meldung = 'geschlossen';
break;
}
return(meldung);
}

createState('zählen_Fenster.anzahlFenster', { // wenn benötigt: Anzahl der vorhandenen Fenster
type: 'number',
min: 0,
def: 0,
role: 'value'
});
createState('zählen_Fenster.anzahlFensterauf', { // Anzahl der Fenster, die auf sind als Variable unter Javascript.0 anlegen
type: 'number',
min: 0,
def: 0,
role: 'value'
});
createState('zählen_Fenster.textFensterauf', { // Anzahl der offenen Fenster und deren Namen als Variable unter Javascript.0 anlegen
type: 'string',
def: ' ',
role: 'value'
});

var hmFenster = $('channelstate.id=*.STATE');
var miFenster = $('stateid=*.state');

function countFenster() {
// Setzt die Zähler vor dem Durchlauf aller Elemente *.STATE im Gewerk Fenster auf 0
var anzahlFenster = 0;
var anzahlFensterauf = 0;
var textFensterauf = [];
log('Auslösender Aktor: ' + obj.id + ': ' + obj.newState.val);              // Info im Log, welcher Zustand sich geändert hat

hmFenster.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 obj    = getObject(id);
    var name = getObject(id).common.name;
    var devicename = name.substring(0, name.indexOf(".STATE"));             //.state aus Text entfernen
    if (status /*TFK*/ || status === 1 || status === 2 /*RHS*/) {  // wenn Zustand offen, dann wird die Anzahl der Fenster hochgezählt
         ++anzahlFensterauf;
         textFensterauf.push(devicename + ' (' + fensterstatus(status) + ')');  // Name und Zustand zum Array hinzufügen
    }                
    log('Fenster #' + (i+1) + ': ' + devicename + ' ' + fensterstatus(status)/* + ' (' + status + ' / ' + typeof status + ')'*/);
    ++anzahlFenster;                                                        // Zählt die Anzahl der vorhandenen Fenster unabhängig vom Status
}); 

miFenster.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 obj    = getObject(id);
    var name   = getObject(id).common.name;
    if (status /*TFK*/ || status === 1 || status === 2 /*RHS*/) {  // wenn Zustand offen, dann wird die Anzahl der Fenster hochgezählt
         ++anzahlFensterauf;
         textFensterauf.push(name + ' (' + fensterstatus(status) + ')');  // Name und Zustand zum Array hinzufügen
    }                
    log('Fenster #' + (i+1) + ': ' + name + ' ' + fensterstatus(status)/* + ' (' + status + ' / ' + typeof status + ')'*/);
    ++anzahlFenster;                                                        // Zählt die Anzahl der vorhandenen Fenster unabhängig vom Status
}); 

// Schleife ist durchlaufen. Im Log wird der aktuelle Status (Anzahl, davon an) ausgegeben
    log("Text: " + textFensterauf);
    log("Anzahl Fenster: " + anzahlFenster + " - davon Fenster auf: " +  anzahlFensterauf);

// die ermittelten Werte werden als javascript.0\. Variable in ioBroker gespeichert (z.B. für die Verarbeitung in VIS)
setState("zählen_Fenster.textFensterauf", textFensterauf.join(',
')); // Schreibt die aktuelle Namen der offenen Fenster
setState("zählen_Fenster.anzahlFensterauf", textFensterauf.length); // Schreibt die aktuelle Anzahl der offenen Fenster
setState("zählen_Fenster.anzahlFenster", anzahlFenster); // Schreibt die aktuelle Anzahl der vorhandene Elemente im Gewerk Fenster
}

hmFenster.on(countFenster); // bei Zustandänderung *.STATE im Gewerk Fenster

miFenster.on(countFenster); // bei Zustandänderung *.state im Gewerk Fenster

// Variable für Ansage aufbereiten
createState('zählen_Fenster.textFensteraufAnsage', {
type: 'string',
def: ' ',
role: 'value'
});
// Anzahl der Fenster, die auf sind, für Ansage aufbereitet
var idQuelle = 'javascript.0.zählen_Fenster.textFensterauf',
idAnsage = 'javascript.0.zählen_Fenster.textFensteraufAnsage';

on(idQuelle, function (obj) {
var text = obj.state.val;
text = text.replace(/RHS/g, 'Drehgriff');
text = text.replace(/TFK/g, 'Reedkontakt');
text = (text.length > 1) ? 'Geöffnete Fenster: ' + text : 'Alle Fenster sind verschlossen';
setState(idAnsage, text);
});
Danke und Grüße,Thorsten `  

Hallo Thorsten,

schau mal hier, das hatte ich mal für den user dali3 geschrieben und imho hat es funktioniert (kann ich nicht testen, habe keine solche Kontakte).

Wir haben es damals auch NICHT mit dem cache selector hinbekommen. Workaround war ein Array.

! ```
`// Skript zum prüfen, wieviele Fenster offen sind
// Lösung für Xiaomi Kontakte, die nicht via $-Selector ansprechbar zu sein scheinen
// Status: 4.1.2018
! // ############ ALLGEMEINE SKRIPTEINSTELLUNGEN ###############
! // debugging
! var debug = false;
! // states
! createState('Fenster.Total', 0);
createState('Fenster.Offen', 0);
! // ################## Fenster Kontakte #####################
! // Variablen / IDs deklarieren
! var idFenster1 = "mihome.0.devices.magnet_158d0001dacfc1.state"/Is opened Fenster Küche offen/, // ids eintragen, geht rechts oben im JS Editor, "id einfügen". Dort auf den "open" state verlinken.
idFenster2 = "mihome.0.devices.magnet_158d0001dacfc7.state"/Is opened Fenster Wohnzimmer offen/; // du kannst die Variablen auch "FensterKueche" usw nennen, es muss nicht "idFenster1" sein
// idFenster3 = [HIER DIE ID DEINER MAGNETEN EINFÜGEN] , // die IDs kannst du beliebig benennen. Müssen aber nachfolgend entsprechend eingetragen sein
// idFensterN = [HIER DIE ID DEINER MAGNETEN EINFÜGEN] ; // die letzte ID muss mit einem Semikolon beendet werden, dazwischen Kommas
! var arrFenster = [idFenster1, idFenster2]; // hier musst du die gleichen Namen eintragen, wie du direkt hierüber eingestellt hast
! // Variablen

var countOffen = 0;
countTotal =arrFenster.length;

! // ######## FUNKTIONEN ########
! on(arrFenster, function(obj){
! var status = obj.state.val;

    if(status){
        ++countOffen; 
        countOffen= Math.min(arrFenster.length, countOffen);
        if(debug) log("Es sind "+countOffen +" Fenster offen");
    }
    else{
        --countOffen;
        countOffen = Math.max(0, countOffen);
        if(debug) log("Es sind "+countOffen +" Fenster offen");
    }

! setState("javascript.0.Fenster.Offen", countOffen);
setState("javascript.0.Fenster.Total", countTotal);
});

// ############# BEI SKIRPTSTART DIE COUNTER RICHTIG SETZEN ###################

! for (var i = 0; i< arrFenster.length; i++) {
var objekt = arrFenster[i];
var value = getState(objekt).val;
if(debug) log(arrFenster[i] + " ist " +value);
if(value) ++countOffen;
setState("javascript.0.Fenster.Offen", countOffen);
}`[/i][/i]

stimezo

Hallo tempestas,

vielen Dank für Dein Script. Das läuft bei mir hervorragend.

Ist es viel arbeit, die Namen der offenen Fenster in einen Datenpunkt

schreiben zu lassen?

Danke und Gruß

Christian

tempestas

~~@stimezo:~~

Hallo tempestas,

vielen Dank für Dein Script. Das läuft bei mir hervorragend.

Ist es viel arbeit, die Namen der offenen Fenster in einen Datenpunkt

schreiben zu lassen?

Danke und Gruß

Christian `

Hallo Christian,

sorry für die Verspätung, ich schaue kaum noch ins Forum.

ungetestet hier zusammengetippert:

`// Skript zum prüfen, wieviele Fenster offen sind
// Lösung für Xiaomi Kontakte, die nicht via $-Selector ansprechbar zu sein scheinen
// Status: 4.1.2018

// ############ ALLGEMEINE SKRIPTEINSTELLUNGEN ###############

// debugging

var debug = false;

// states

createState('Fenster.Total', 0);
createState('Fenster.Offen', 0);
createState('Fenster.Text', "", true);

// ################## Fenster Kontakte #####################

// Variablen / IDs deklarieren

var    idFenster1 = "mihome.0.devices.magnet_158d0001dacfc1.state"/*Is opened Fenster Küche offen*/,   // ids eintragen, geht rechts oben im JS Editor, "id einfügen". Dort auf den "open" state verlinken. 
       idFenster2 = "mihome.0.devices.magnet_158d0001dacfc7.state"/*Is opened Fenster Wohnzimmer offen*/;               // du kannst die Variablen auch "FensterKueche" usw nennen, es muss nicht "idFenster1" sein
       // idFenster3 = [HIER DIE ID DEINER MAGNETEN EINFÜGEN] ,               // die IDs kannst du beliebig benennen. Müssen aber nachfolgend entsprechend eingetragen sein
       // idFensterN = [HIER DIE ID DEINER MAGNETEN EINFÜGEN] ;               // die letzte ID muss mit einem Semikolon beendet werden, dazwischen Kommas

var arrFenster = [idFenster1, idFenster2];               // hier musst du die gleichen Namen eintragen, wie du direkt hierüber eingestellt hast

// Variablen     

var countOffen = 0;
var countTotal =arrFenster.length;
var arrText = [];

// ######## FUNKTIONEN ########

on(arrFenster, function(obj){

        var status = obj.state.val;
		arrText =[];

        if(status){
            ++countOffen; 
            countOffen= Math.min(arrFenster.length, countOffen);
            if(debug) log("Es sind "+countOffen +" Fenster offen");

			arrText.push(obj.common.name);
        }
        else{
            --countOffen;
            countOffen = Math.max(0, countOffen);
            if(debug) log("Es sind "+countOffen +" Fenster offen");
        }

    	setState("javascript.0.Fenster.Offen", countOffen);
	setState("javascript.0.Fenster.Text", arrText.join(',
'));     
    });  

 // ############# BEI SKIRPTSTART DIE COUNTER RICHTIG SETZEN ###################

for (var i = 0; i< arrFenster.length; i++) {
    var objekt = arrFenster[i];
    var value = getState(objekt).val;

	var name = getObject(objekt).common.name;
	arrText.push(name);

    if(debug) log(arrFenster[i] + " ist " +value);
    if(value) ++countOffen;

    setState("javascript.0.Fenster.Offen", countOffen);

}
    	setState("javascript.0.Fenster.Total", countTotal);
	setState("javascript.0.Fenster.Text", arrText.join(',
'));` [/i][/i]

stimezo

Hallo tempestas,

herzlichen Dank, das klappt schon fast.

Es wird immer das zuletzt geöffnete Fenster in den Datenpunkt geschieben.

Wenn ein zweites geöffnet wird wird der DP mit dessen Namen überschrieben.

Könnte mann das mit einer Liste lösen?

Vielen Dank und viele Grüße

Christian

tempestas

Probier mal bitte diese Version

`// Skript zum prüfen, wieviele Fenster offen sind
// Lösung für Xiaomi Kontakte, die nicht via $-Selector ansprechbar zu sein scheinen
// Status: 4.1.2018

// ############ ALLGEMEINE SKRIPTEINSTELLUNGEN ###############

// debugging

var debug = false;

// states

createState('Fenster.Total', 0);
createState('Fenster.Offen', 0);
createState('Fenster.Text', "", true);

// ################## Fenster Kontakte #####################

// Variablen / IDs deklarieren

var    idFenster1 = "mihome.0.devices.magnet_158d0001dacfc1.state"/*Is opened Fenster Küche offen*/,   // ids eintragen, geht rechts oben im JS Editor, "id einfügen". Dort auf den "open" state verlinken. 
       idFenster2 = "mihome.0.devices.magnet_158d0001dacfc7.state"/*Is opened Fenster Wohnzimmer offen*/;               // du kannst die Variablen auch "FensterKueche" usw nennen, es muss nicht "idFenster1" sein
       // idFenster3 = [HIER DIE ID DEINER MAGNETEN EINFÜGEN] ,               // die IDs kannst du beliebig benennen. Müssen aber nachfolgend entsprechend eingetragen sein
       // idFensterN = [HIER DIE ID DEINER MAGNETEN EINFÜGEN] ;               // die letzte ID muss mit einem Semikolon beendet werden, dazwischen Kommas

var arrFenster = [idFenster1, idFenster2];               // hier musst du die gleichen Namen eintragen, wie du direkt hierüber eingestellt hast

// Variablen     

var countOffen = 0;
var countTotal =arrFenster.length;
var arrText = [];

// ######## FUNKTIONEN ########

on(arrFenster, function(){
	arrText = [];
	arrFenster.forEach(function(obj) {

	var status = getState(obj).val;

        if(status){
            ++countOffen; 
            countOffen= Math.min(arrFenster.length, countOffen);
            if(debug) log("Es sind "+countOffen +" Fenster offen");

			arrText.push(obj.common.name);
        }
        else{
            --countOffen;
            countOffen = Math.max(0, countOffen);
            if(debug) log("Es sind "+countOffen +" Fenster offen");
        }

		setState("javascript.0.Fenster.Offen", countOffen);
		setState("javascript.0.Fenster.Text", arrText.join(',
'));     
	}
});  

 // ############# BEI SKIRPTSTART DIE COUNTER RICHTIG SETZEN ###################

for (var i = 0; i< arrFenster.length; i++) {
    var objekt = arrFenster[i];
    var value = getState(objekt).val;

	var name = getObject(objekt).common.name;
	arrText.push(name);

    if(debug) log(arrFenster[i] + " ist " +value);
    if(value) ++countOffen;

    setState("javascript.0.Fenster.Offen", countOffen);

}
    setState("javascript.0.Fenster.Total", countTotal);
	setState("javascript.0.Fenster.Text", arrText.join(',
'));` [/i][/i]

stimezo

Vielen Dank tempestas,

da bekomme ich einen Syntax:

Viele Grüße

Christian

paul53

~~@stimezo:~~

da bekomme ich einen Syntax: `
Es fehlt ein Punkt vor common:

	var name = getObject(objekt).common.name;

tempestas

Danke, paul :)

stimezo

@paul53,

@tempestas

vielen Dank für Eure Unterstützung.

mit dem Punkt ist der zweite Fehler verschwunden.

Der erste leider noch nicht :-)

tempestas

jetzt aber (?!?):

`// Skript zum prüfen, wieviele Fenster offen sind
// Lösung für Xiaomi Kontakte, die nicht via $-Selector ansprechbar zu sein scheinen
// Status: 4.1.2018

// ############ ALLGEMEINE SKRIPTEINSTELLUNGEN ###############

// debugging

var debug = false;

// states

createState('Fenster.Total', 0);
createState('Fenster.Offen', 0);
createState('Fenster.Text', "", true);

// ################## Fenster Kontakte #####################

// Variablen / IDs deklarieren

var    idFenster1 = "mihome.0.devices.magnet_158d0001dacfc1.state"/*Is opened Fenster Küche offen*/,   // ids eintragen, geht rechts oben im JS Editor, "id einfügen". Dort auf den "open" state verlinken. 
       idFenster2 = "mihome.0.devices.magnet_158d0001dacfc7.state"/*Is opened Fenster Wohnzimmer offen*/;               // du kannst die Variablen auch "FensterKueche" usw nennen, es muss nicht "idFenster1" sein
       // idFenster3 = [HIER DIE ID DEINER MAGNETEN EINFÜGEN] ,               // die IDs kannst du beliebig benennen. Müssen aber nachfolgend entsprechend eingetragen sein
       // idFensterN = [HIER DIE ID DEINER MAGNETEN EINFÜGEN] ;               // die letzte ID muss mit einem Semikolon beendet werden, dazwischen Kommas

var arrFenster = [idFenster1, idFenster2];               // hier musst du die gleichen Namen eintragen, wie du direkt hierüber eingestellt hast

// Variablen     

var countOffen = 0;
var countTotal =arrFenster.length;
var arrText = [];

// ######## FUNKTIONEN ########

on(arrFenster, function(){
	arrText = [];
	arrFenster.forEach(function(obj) {

	var status = getState(obj).val;

        if(status){
            ++countOffen; 
            countOffen= Math.min(arrFenster.length, countOffen);
            if(debug) log("Es sind "+countOffen +" Fenster offen");

			arrText.push(obj.common.name);
        }
        else{
            --countOffen;
            countOffen = Math.max(0, countOffen);
            if(debug) log("Es sind "+countOffen +" Fenster offen");
        }

		setState("javascript.0.Fenster.Offen", countOffen);
		setState("javascript.0.Fenster.Text", arrText.join(',
'));     
		}
	})
);  

 // ############# BEI SKIRPTSTART DIE COUNTER RICHTIG SETZEN ###################

for (var i = 0; i< arrFenster.length; i++) {
    var objekt = arrFenster[i];
    var value = getState(objekt).val;

	var name = getObject(objekt).common.name;
	arrText.push(name);

    if(debug) log(arrFenster[i] + " ist " +value);
    if(value) ++countOffen;

    setState("javascript.0.Fenster.Offen", countOffen);

}
    setState("javascript.0.Fenster.Total", countTotal);
	setState("javascript.0.Fenster.Text", arrText.join(',
'));` [/i][/i]

stimezo

Hallo tempestas,

syntax ist weg :-)

Bekomme in zeile 73 eine Warnung.

und folgendes im log:

javascript.0	2018-06-05 12:34:41.535	error	at Object. <anonymous>(script.js.common.fensterfinal:41:13)
javascript.0	2018-06-05 12:34:41.533	error	at Array.forEach (native)
javascript.0	2018-06-05 12:34:41.532	error	at script.js.common.fensterfinal:50:27
javascript.0	2018-06-05 12:34:41.529	error	Error in callback: TypeError: Cannot read property 'name' of undefined
javascript.0	2018-06-05 12:34:25.163	error	at Object. <anonymous>(script.js.common.fensterfinal:41:13)
javascript.0	2018-06-05 12:34:25.162	error	at Array.forEach (native)
javascript.0	2018-06-05 12:34:25.160	error	at script.js.common.fensterfinal:50:27
javascript.0	2018-06-05 12:34:25.154	error	Error in callback: TypeError: Cannot read property 'name' of undefined
javascript.0	2018-06-05 12:34:03.026	info	script.js.common.fensterfinal: registered 2 subscriptions and 0 schedules
javascript.0	2018-06-05 12:34:02.976	info	Start javascript script.js.common.fensterfinal</anonymous></anonymous>

vielen Dank für Deine Hilfe und viele Grüße

Christian

paul53

In Zeile 73 ist nur eine Warnung, dass der Bezeichner name schon vorher einmal global deklariert wurde: Lass var weg oder verwende let.

Der Fehler in Zeile 50 rührt daher, dass obj kein Objekt, sondern eine Datenpunkt-ID ist. Korrigiere bitte Zeilen 41 bis 43:

	arrFenster.forEach(function(id) {
		var obj = getObject(id);	
		var status = getState(id).val;

stimezo

Du bist der Beste.

Funktioniert super.

Viele Dank und viele Grüße

Christian

wocwm

Moin zusammen,

hört sich auf jeden Fall sehr Interessant an das Ganze. Bin gerade neu in die Materie eingestiegen.

Ist dieses Skript auch verwendbar in Verbindung mit ALEXA?

Der Anwendungsfall soll folgender sein:

Ich zieh die Jacke an, beim Schuheanziehen sage ich: "ALEXA, ich verlasse dich jetzt!"

Alexa soll die Fenster zählen und mir sagen welche gerade noch offen sind.

Ist das so umsetzbar?!

Danke Community

amsernat

@wocwm
das würde mich auch interessieren.

Community Forum

NEWS

[gelöst] Frage zum Script "Zählen von Fenstern"

Support us

778

32.6k

82.3k

1.3m