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

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