Jahresrückblick 2025 – unser neuer Blogbeitrag ist online! ✨

[Aphofis]

@paul53
ist es so in Ordnung ?

const idRestzeit = 'Osmose.Restzeit';
const text_ein = ['Filter Max unterschritten, Skimmer & Osmose','\n','wurden wieder eingeschaltet'].join('');
const text_aus = ['Filter Max erreicht, Skimmer & Osmose','\n','wurden abgeschaltet'].join('');

createState(idRestzeit, 300, {type: 'number', unit: 's'});

var Osmosefreigabe = !getState('sonoff.0.Aqua_Float.POWER4').val;
var Schwimmer = getState("sonoff.0.Aqua_Float.POWER1").val;
var Restzeit, timeout_aus, timeout_ein, Magnetventil_Kontrolle;

function Magnetventil_Ein() {
  if (!Magnetventil_Kontrolle && Restzeit > 0 && Osmosefreigabe) {
    Magnetventil_Kontrolle = setInterval(function () {
      if (Restzeit > 0 && Schwimmer && Osmosefreigabe) {
        Restzeit--;
        setState(idRestzeit, 2 * Restzeit, true);
      } else {
        setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false);
        clearInterval(Magnetventil_Kontrolle); 
        Magnetventil_Kontrolle = null;
      }
    }, 2000);
    setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, true);
  }
}
// Skripstart
setTimeout(function() {
    Restzeit = getState(idRestzeit).val / 2;
    if(Schwimmer && Osmosefreigabe) Magnetventil_Ein();
    else setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false);
}, 200);
on({id: 'sonoff.0.Aqua_Float.POWER1', change: "ne"}, function (obj) {
  Schwimmer = obj.state.val;
  if(Schwimmer) Magnetventil_Ein();
 else setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false); 
});

schedule("0 0 * * *", function () {
  Restzeit = 150;
  setState(idRestzeit, 300, true);
  if(Schwimmer) Magnetventil_Ein();
});

on({id: 'sonoff.0.Aqua_Float.POWER4', change: "ne"}, function (obj) {
  if (obj.state.val) {
    timeout_aus = setTimeout(function () {
      Osmosefreigabe = false;  
      setState("sonoff.0.Aqua_Power.POWER3"/*Aqua_Power_Skimmer*/, false);
      setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false);
    }, 5000);
  if (timeout_ein) {
        clearTimeout(timeout_ein); 
        timeout_ein = null;
    }
  } else {
    timeout_ein = setTimeout(function () {
      Osmosefreigabe = true;
      if(Schwimmer) Magnetventil_Ein();                
      setStateDelayed("sonoff.0.Aqua_Power.POWER3"/*Aqua_Power_Skimmer*/, true, 10000);
    }, 10000);
  if (timeout_aus) {
        clearTimeout(timeout_aus); 
        timeout_aus = null;
    }
  }
});

on({id: 'sonoff.0.Aqua_Power.POWER3', change: "ne"}, function (obj) {
  var txt;  
  if (obj.state.val) txt = text_ein;   
  else txt = text_aus;   
  sendTo("telegram.0", "send", {
    text: txt
  });
  log(txt);
  log('Schwimmer: ' + Schwimmer);
});

so

[paul53]

@Aphofis sagte:

ist es so in Ordnung ?

Ja, aber wegen der besseren Lesbarkeit füge vorn in Zeile 35 (vor else) ein Leerzeichen ein; in Zeilen 51 und 61 vor if zwei Leerzeichen ein.

[Aphofis]

@paul53
So besser ?

const idRestzeit = 'Osmose.Restzeit';
const text_ein = ['Filter Max unterschritten, Skimmer & Osmose','\n','wurden wieder eingeschaltet'].join('');
const text_aus = ['Filter Max erreicht, Skimmer & Osmose','\n','wurden abgeschaltet'].join('');

createState(idRestzeit, 300, {type: 'number', unit: 's'});

var Osmosefreigabe = !getState('sonoff.0.Aqua_Float.POWER4').val;
var Schwimmer = getState("sonoff.0.Aqua_Float.POWER1").val;
var Restzeit, timeout_aus, timeout_ein, Magnetventil_Kontrolle;

function Magnetventil_Ein() {
  if (!Magnetventil_Kontrolle && Restzeit > 0 && Osmosefreigabe) {
    Magnetventil_Kontrolle = setInterval(function () {
      if (Restzeit > 0 && Schwimmer && Osmosefreigabe) {
        Restzeit--;
        setState(idRestzeit, 2 * Restzeit, true);
      } else {
        setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false);
        clearInterval(Magnetventil_Kontrolle); 
        Magnetventil_Kontrolle = null;
      }
    }, 2000);
    setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, true);
  }
}
// Skripstart
setTimeout(function() {
    Restzeit = getState(idRestzeit).val / 2;
    if(Schwimmer && Osmosefreigabe) Magnetventil_Ein();
    else setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false);
}, 200);
on({id: 'sonoff.0.Aqua_Float.POWER1', change: "ne"}, function (obj) {
  Schwimmer = obj.state.val;
  if(Schwimmer) Magnetventil_Ein();
    else setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false); 
});

schedule("0 0 * * *", function () {
  Restzeit = 150;
  setState(idRestzeit, 300, true);
  if(Schwimmer) Magnetventil_Ein();
});

on({id: 'sonoff.0.Aqua_Float.POWER4', change: "ne"}, function (obj) {
  if (obj.state.val) {
    timeout_aus = setTimeout(function () {
      Osmosefreigabe = false;  
      setState("sonoff.0.Aqua_Power.POWER3"/*Aqua_Power_Skimmer*/, false);
      setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false);
    }, 5000);
     if (timeout_ein) {
        clearTimeout(timeout_ein); 
        timeout_ein = null;
    }
  } else {
    timeout_ein = setTimeout(function () {
      Osmosefreigabe = true;
      if(Schwimmer) Magnetventil_Ein();                
      setStateDelayed("sonoff.0.Aqua_Power.POWER3"/*Aqua_Power_Skimmer*/, true, 10000);
    }, 10000);
     if (timeout_aus) {
        clearTimeout(timeout_aus); 
        timeout_aus = null;
    }
  }
});

on({id: 'sonoff.0.Aqua_Power.POWER3', change: "ne"}, function (obj) {
  var txt;  
  if (obj.state.val) txt = text_ein;   
  else txt = text_aus;   
  sendTo("telegram.0", "send", {
    text: txt
  });
  log(txt);
  log('Schwimmer: ' + Schwimmer);
});

[paul53]

@Aphofis sagte:

So besser ?

Bedingt: Jetzt sind es zwei bzw. je ein Leerzeichen zuviel:

• else gehört linksbündig unter if
• if gehört linksbündig über die zugehörige schließende geschweifte Klammer in Zeilen 54, 64

[Aphofis]

@paul53 Ich schnalle die Struktur nicht so ganz!

const idRestzeit = 'Osmose.Restzeit';
const text_ein = ['Filter Max unterschritten, Skimmer & Osmose','\n','wurden wieder eingeschaltet'].join('');
const text_aus = ['Filter Max erreicht, Skimmer & Osmose','\n','wurden abgeschaltet'].join('');

createState(idRestzeit, 300, {type: 'number', unit: 's'});

var Osmosefreigabe = !getState('sonoff.0.Aqua_Float.POWER4').val;
var Schwimmer = getState("sonoff.0.Aqua_Float.POWER1").val;
var Restzeit, timeout_aus, timeout_ein, Magnetventil_Kontrolle;

function Magnetventil_Ein() {
if (!Magnetventil_Kontrolle && Restzeit > 0 && Osmosefreigabe) {
    Magnetventil_Kontrolle = setInterval(function () {
    if (Restzeit > 0 && Schwimmer && Osmosefreigabe) {
        Restzeit--;
        setState(idRestzeit, 2 * Restzeit, true);
      } 
else {
        setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false);
        clearInterval(Magnetventil_Kontrolle); 
        Magnetventil_Kontrolle = null;
      }
    }, 2000);
    setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, true);
  }
}
// Skripstart
setTimeout(function() {
    Restzeit = getState(idRestzeit).val / 2;
     if(Schwimmer && Osmosefreigabe) Magnetventil_Ein();
    else setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false);
}, 200);
on({id: 'sonoff.0.Aqua_Float.POWER1', change: "ne"}, function (obj) {
  Schwimmer = obj.state.val;
if(Schwimmer) Magnetventil_Ein();
    else setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false); 
});

schedule("0 0 * * *", function () {
  Restzeit = 150;
  setState(idRestzeit, 300, true);
if(Schwimmer) Magnetventil_Ein();
});

on({id: 'sonoff.0.Aqua_Float.POWER4', change: "ne"}, function (obj) {
if (obj.state.val) {
    timeout_aus = setTimeout(function () {
      Osmosefreigabe = false;  
      setState("sonoff.0.Aqua_Power.POWER3"/*Aqua_Power_Skimmer*/, false);
      setState("sonoff.0.Aqua_Control.POWER1"/*Osmose_Magnetventil*/, false);
    }, 5000);
    if (timeout_ein) {
        clearTimeout(timeout_ein); 
        timeout_ein = null;
    }
  } else {
    timeout_ein = setTimeout(function () {
      Osmosefreigabe = true;
    if(Schwimmer) Magnetventil_Ein();                
      setStateDelayed("sonoff.0.Aqua_Power.POWER3"/*Aqua_Power_Skimmer*/, true, 10000);
    }, 10000);
    if (timeout_aus) {
        clearTimeout(timeout_aus); 
        timeout_aus = null;
    }
  }
});

on({id: 'sonoff.0.Aqua_Power.POWER3', change: "ne"}, function (obj) {
  var txt;  
 if (obj.state.val) txt = text_ein;   
else txt = text_aus;   
  sendTo("telegram.0", "send", {
    text: txt
  });
  log(txt);
  log('Schwimmer: ' + Schwimmer);
});

[paul53]

@Aphofis sagte:

Ich schnalle die Struktur nicht so ganz!

Die vorherige Version war besser !

[Aphofis]

@paul53
Na toll

[paul53]

@Aphofis
Nimm diese:
Osmose.js.txt

[Aphofis]

@paul53
Da fehlt dann nur

 log('Schwimmer: ' + Schwimmer); 

[paul53]

@Aphofis sagte:

Da fehlt dann nur

Wo ?

[Aphofis]

@paul53
In der txt die ich nehmen sollte oder ist der Log da schon irgendwo drin?

[paul53]

@Aphofis sagte:

ist der Log da schon irgendwo drin?

Zeile 36.

[Aphofis]

@paul53 Das hab ich wohl nicht so ganz gefunden!
Wird wohl auch noch eine Zeit dauern, bis ich JS lesen kann

[Aphofis]

@paul53
Bis jetzt macht die Osmose Steuerung was sie soll!
Wird nicht zu viel dosiert und Filter Max wurde seid gestern Abend nicht erreicht

[paul53]

@Aphofis sagte:

Wird nicht zu viel dosiert

Wenn vorher zuviel dosiert wurde, dann war die Abfrage des Schwimmers alle 2 Sekunden zu langsam. Jetzt wird direkt auf den Schwimmer reagiert.

[Aphofis]

@paul53
klingt toll aber heute wurde wieder bis filter max true war dosiert.
bringt es was den wert auf 500 ms zu stellen???

[paul53]

@Aphofis sagte:

bringt es was den wert auf 500 ms zu stellen???

Nein, das Magnetventil wird direkt vom Trigger des Schwimmers geschlossen. Schneller geht es per Software (JS) nicht. Die Frage ist, ob das setState() wirklich zuverlässig das Ventil steuert oder es bei zuviel Verkehr im 2,4-GHz-Band ins Leere läuft, bzw. ob jedes Schwimmer-Signal zuverlässig in ioBroker ankommt.
Wenn der Schwimmer noch einen zweiten potentialfreien Kontakt hat, würde ich ihn in Reihe (UND-Verknüpfung) zwischen Sonoff-Aktor und Magnetventil schalten.

[Aphofis]

@paul53 nein leider nicht es sind normale Schwimmer mit einem pull-up Widerstand dazwischen mehr nicht

Kann man nicht die sende Intervalle vom ESP8266 verkürzen !?

[paul53]

@Aphofis sagte:

Kann man nicht die sende Intervalle vom ESP8266 verkürzen !?

Keine Ahnung, mit dem WLAN-Zeugs kenne ich mich nicht aus, da ich es nicht verwende.

[Aphofis]

@paul53
Naja ich kenne mich eigntl damit aus!
Nur vllt liegt es an was anderem.
hatte jetzt 2 liter wasser raus Filter max ist false aber Skimmer wird nicht wieder eingeschaltet. Relais ist aus und im Sonoff andapter ist er auch false
Das meine ich damit wenn ich sage Blockly ist mir lieber! da ich dann selbst guchen kann. Bei einem JS weiß ich nicht wie ich die einzelnen Datenpunkte kontrollieren soll.