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

tempestas

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

639

32.7k

82.6k

1.3m