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

tempestas

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.

NEWS

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

Support us

754
Online

31.8k
Users

79.9k
Topics

1.3m
Posts

NEWS

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

Support us

754Online

31.8kUsers

79.9kTopics

1.3mPosts

754
Online

31.8k
Users

79.9k
Topics

1.3m
Posts