UPDATE 31.10.: Amazon Alexa - ioBroker Skill läuft aus ?

[paul53]

@Hansi1234 sagte:

State "javascript.0.StatusDoorOpen" not found

Sagt das nicht genug aus ?
Wieso hat das Script mind. 93 Zeilen ?

[Hansi1234]

@paul53
Wenn ich hier einen Code herauskopiere und im Java einfüge, dann füllt er es mit zusätzlichen Zeilen.

Ja das mit dem State not found habe ich gesehen. Wieso legt er diese nicht an?

[v522533]

@Hansi1234

die musst du selbst anlegen.

[Hansi1234]

Sorry, lange her :-)

Geht nun. Danke dir für die Hilfe.

[Homoran]

@Hansi1234 sagte in Fenster Script: Liste der offenen Fenster:

dann füllt er es mit zusätzlichen Zeilen.

das sollte es aber nicht
Warum passiert das bei dir?

Welche Zeilen sind das?

bearbeitest du den Text vorher mit einem Editor, der Zeilenumbrüche einfügt?
Hoffentlich im LINUX-Mode

[paul53]

@Homoran sagte:

das sollte es aber nicht
Warum passiert das bei dir?

Wahrscheinlich nutzt er FF, bei dem Leerzeilen beim Copy eingefügt werden. Mit Chrome passiert es nicht.

[Hansi1234]

jap, ich benutze Firefox. War bei mir schon immer so. Darum hab ich mir nie was dabei gedacht.

[dos1973]

Hi,

ich habe das jetzt auch mal alles nachgebaut und funktioniert echt gut, mir würde noch eine "gekippte Fenster" Funktion gut gefallen. Könnte das gffs. noch jemand ergänzen. Danke

0: closed
1: kipp
2: offen

das ist das gesamte script mit allen Ändeurungen bisher.

function countOpenWindowsDoors() {
    var openWindows = 0;
    var openDoors = 0;
    var listWindows = [];
    var listDoors = [];
 
    // Fenster
    var windows = getObject("enum.functions.windowstatus").common.members;
    for(let i = 0; i < windows.length; i++) {
        var Status = getState(windows[i]).val;
        if ( Status ) { 
            openWindows++;
            let id = windows[i].substring(0, windows[i].lastIndexOf('.')); // Device ID
             listWindows.push(getObject(id).common.name); // Device Name
        }
    }
 
    // Türen
    var doors = getObject("enum.functions.doorstatus").common.members;
    for(let i = 0; i < doors.length; i++) { 
        var Status = getState(doors[i]).val; 
        if ( Status ) { 
            openDoors++;       
            let id = windows[i].substring(0, windows[i].lastIndexOf('.')); // Device ID
             listDoors.push(getObject(id).common.name); // Device Name
        }
    } 
    var openWindowsText = "Kein Fenster geöffnet"; 
    var openDoorsText = "Keine Tür geöffnet";
    if(listWindows.length) openWindowsText = '' + listWindows.join(', ');
    if(listDoors.length) openDoorsText = 'Offene Türen: ' + listDoors.join(', ');
    setState('javascript.0._NEU.enum.StatusWindowOpenText', openWindowsText, true);
    setState('javascript.0._NEU.enum.StatusDoorOpenText', openDoorsText, true);
    setState('javascript.0._NEU.enum.StatusWindowOpen', openWindows, true );
    setState('javascript.0._NEU.enum.StatusDoorOpen', openDoors, true );
}
 
$('state(functions=windowstatus)').on(function(obj) {
    countOpenWindowsDoors();
});
 
$('state(functions=doorstatus)').on(function(obj) {
    countOpenWindowsDoors();
});
 
countOpenWindowsDoors();

[dos1973]

ich habe hier weiterprobiert und habe inzwischen auch meine Lichter hier zählen und auflisten können.

function countOpenWindowsDoors() {
    var openWindows = 0;
    var openDoors = 0;
    var lights_on = 0;
    var listWindows = [];
    var listDoors = [];
    var listlights = [];
 
    // Lichter
    var windows = getObject("enum.functions.light").common.members;
    for(let i = 0; i < windows.length; i++) {
        var Status = getState(windows[i]).val;
        if ( Status ) { 
            lights_on++;
            let id = windows[i].substring(0, windows[i].lastIndexOf('.')); // Device ID
             listlights.push(getObject(id).common.name); // Device Name
        }
    }

    // Fenster
    var windows = getObject("enum.functions.windowstatus").common.members;
    for(let i = 0; i < windows.length; i++) {
        var Status = getState(windows[i]).val;
        if ( Status ) { 
            openWindows++;
            let id = windows[i].substring(0, windows[i].lastIndexOf('.')); // Device ID
             listWindows.push(getObject(id).common.name); // Device Name
        }
    }
 
    // Türen
    var doors = getObject("enum.functions.doorstatus").common.members;
    for(let i = 0; i < doors.length; i++) { 
        var Status = getState(doors[i]).val; 
        if ( Status ) { 
            openDoors++;       
            let id = windows[i].substring(0, windows[i].lastIndexOf('.')); // Device ID
             listDoors.push(getObject(id).common.name); // Device Name
        }
    } 
    var openWindowsText = "Kein Fenster geöffnet"; 
    var openDoorsText = "Keine Tür geöffnet";
    var lights_onText = "Keine Lampe eingeschaltet";
    if(listWindows.length) openWindowsText = '' + listWindows.join(', ');
    if(listDoors.length) openDoorsText = 'Offene Türen: ' + listDoors.join(', ');
    if(listlights.length) lights_onText = 'Lampen ein: ' + listlights.join(', ');
    setState('0_userdata.0.Wohnung.Status.Text_Fenster_offen', openWindowsText, true);
    setState('0_userdata.0.Wohnung.Status.Text_Tür_offen', openDoorsText, true);
    setState('0_userdata.0.Wohnung.Status.Text_Licht_ein', lights_onText, true);

    setState('0_userdata.0.Wohnung.Status.Fenster_offen', openWindows, true );
    setState('0_userdata.0.Wohnung.Status.Tür_offen', openDoors, true );
    setState('0_userdata.0.Wohnung.Status.Licht_ein', lights_on, true );
}
 
$('state(functions=light)').on(function(obj) {
    countOpenWindowsDoors();
});

$('state(functions=windowstatus)').on(function(obj) {
    countOpenWindowsDoors();
});
 
$('state(functions=doorstatus)').on(function(obj) {
    countOpenWindowsDoors();
});
 
  //  countOpenWindowsDoors();

    if(listlights.length) lights_onText = 'Lampen ein: ' + listlights.join(', ');

listlights.join(', ');

wie kann ich aus dem Komma getrennten eine Zeilenumbruch machen.
<br>
/n
und alles möglich aus dem Internet klappt nicht. Hat jemand einen Tipp.

Danke

[dslraser]

@dos1973

in meinem Batterie Script wird eine Liste für Telegram mit Zeilenumbruch formatiert, vielleicht hilft Dir das.

   setState("0_userdata.0.Telegram.01Batteriealarm.textLOWBAT"/*textLOWBAT*/,textLOWBAT.join(',<br>')); // Schreibt die aktuelle Namen der Geräte mit LOWBAT Meldung

// Telegram Nachricht
function telegram() {
var telegramtext = getState("0_userdata.0.Telegram.01Batteriealarm.textLOWBAT"/*textLOWBAT*/).val.replace(/<br>/g , "\n");  // telegram Liste formatieren, entfernt den HTML Zeilenumbruch und ersetzt mit Zeilenumbruch Telegram
telegramtext = telegramtext.replace(/_DEV:0.LOW_BAT/g, "");
sendTo("telegram", "send", { // Nachricht per Telegram senden
text: "👆Achtung, Batterie fast leer!👆 \n\n" + telegramtext,  
user: 'Heiko'      
});
}

[dslraser]

@dos1973 sagte in Fenster Script: Liste der offenen Fenster:

<br>
/n

Den Schrägstrich anders rum ?

'\n'

[dos1973]

@dslraser

nee, klappt leider nicht. Also den Slash in beide Richtungen geht nicht
auch das obige join(',<br>')); klappt nicht.

Hmm, verzwickt!

[ticaki]

@dos1973

Das hier ist richtig(für telegram, pushover, text, plain email)... aber wo über willst du es ausgeben?

if(listlights.length) lights_onText = 'Lampen ein: ' + listlights.join('\n');

Bei mir sieht das für telegram z.B. so aus:

    var msg = '';
    for (let room in data) msg += room + ': ' + round((data[room].reduce((a, b) => a + b) / data[room].length), 1) + '%\n';
    if (msg) msg = 'Luftfeuchtigkeitsübersicht:\n' + msg;
    else msg = 'Keine Werte gefunden!';
    return msg;
// round() ist eine eigene Funktion zum runden 

[dos1973]

@ticaki sagte in Fenster Script: Liste der offenen Fenster:

@dos1973

Das hier ist richtig(für telegram, pushover, text, plain email)... aber wo über willst du es ausgeben?

if(listlights.length) lights_onText = 'Lampen ein: ' + listlights.join('\n');

das klappt nicht.

ausgegeben wird es im DP

setState('0_userdata.0.Wohnung.Status.Text_Licht_ein', lights_onText, true);

[ticaki]

@dos1973
merkwürdig und wie ist es mit

if(listlights.length) {
 lights_onText = 'Lampen ein: \n ' 
 listlights.forEach((a) => { lights_onText += a + '\n';}
}

\n ist auf jeden Fall korrekt um Zeilenumbrüche im Datenpunkt zu haben. Ist bei mir zumindest so.

EDIT oder liegt es vielleicht daran das DP's Zeilenumbrüche nicht im Datenfeld darstellen?

[dos1973]

dein Vorschlag klappt nicht, bzw.das Script funktioniert dann nicht, bzw. bringt Fehler.

	2020-03-28 23:35:18.465	info	(22746) Stop script script.js.1_Überwachung_Wohnung.Fenster_Licht_Status
javascript.0	2020-03-28 23:34:32.025	error	(22746) at process._tickCallback (internal/process/next_tick.js:68:7)
javascript.0	2020-03-28 23:34:32.025	error	(22746) at promise.then (/opt/iobroker/node_modules/standard-as-callback/built/index.js:19:49)
javascript.0	2020-03-28 23:34:32.025	error	(22746) at tryCatcher (/opt/iobroker/node_modules/standard-as-callback/built/utils.js:11:23)
javascript.0	2020-03-28 23:34:32.025	error	(22746) at client.get (/opt/iobroker/node_modules/iobroker.js-controller/lib/states/statesInRedis.js:580:17)
javascript.0	2020-03-28 23:34:32.025	error	(22746) at adapter.getForeignState (/opt/iobroker/node_modules/iobroker.javascript/main.js:1055:17)
javascript.0	2020-03-28 23:34:32.025	error	(22746) at createProblemObject (/opt/iobroker/node_modules/iobroker.javascript/main.js:1464:17)
javascript.0	2020-03-28 23:34:32.025	error	(22746) at prepareScript (/opt/iobroker/node_modules/iobroker.javascript/main.js:1411:37)
javascript.0	2020-03-28 23:34:32.024	error	(22746) at compile (/opt/iobroker/node_modules/iobroker.javascript/main.js:1188:28)
javascript.0	2020-03-28 23:34:32.024	error	(22746) at Object.createScript (vm.js:277:10)
javascript.0	2020-03-28 23:34:32.024	error	(22746) at new Script (vm.js:83:7)
javascript.0	2020-03-28 23:34:32.024	error	(22746) SyntaxError: missing ) after argument list
javascript.0	2020-03-28 23:34:32.024	error	(22746) ^
javascript.0	2020-03-28 23:34:32.024	error	(22746) listlights.forEach((a) => { lights_onText += a + '\n';}
javascript.0	2020-03-28 23:34:32.024	error	at script.js.1_Überwachung_Wohnung.Fenster_Licht_Status:50
javascript.0	2020-03-28 23:34:32.024	error	(22746) script.js.1_Überwachung_Wohnung.Fenster_Licht_Status compile failed:

der Zeilenumbruch wird nicht angenommen, ich habe schon einen DP im VIS um das zu prüfen.

[ticaki]

@dos1973
jup in meinem Vorschlag hab ich wieder ); vergessen, passiert mir häufiger.

listlights.forEach((a) => { lights_onText += a + '\n';});

Aber VIS ist nicht Datenpunkt: schreib mal von hand

1<br>2<br>3
oder
1<br/>2<br/>3
oder 
1\n2\n3
oder
1\r\n2\r\n3

in den Datenpunkt. Eines davon wird gehen.

[dos1973]

ich bekomme immer mehr das Gefühl dass ich in einem DP gar kein Zeilenumbruch haben kann??

auch wenn ich das manuell Eingebe gehts nicht?
ALSO im VIS wird einfach Fließtext ausgegeben, der in dem DP aufgenommen ist

[dos1973]

ich habs gefunden - mein Fehler "natürlich"

 if(listlights.length) lights_onText = 'Lampen ein: ' + listlights.join((',<br>'));

aber in VIS hatte ich einen Basic-String genommen, damit gehts natürlich nicht.
Jetzt mit einem HTML Widget gehts.

;-)

[J-Lind]

@dos1973 said in Fenster Script: Liste der offenen Fenster:

aber in VIS hatte ich einen Basic-String genommen, damit gehts natürlich nicht.
Jetzt mit einem HTML Widget gehts.

Hi Dos1973
wie trägst du den datenpunkt in dem HTML widget ein?
Gruss Juergen