Garten giessen mit Timer ?

[Jason]

Hallo,
ich habe im Vis einen Taster angelegt und darunter ein Feld in dem ich die Zeit einstellen kann +5, +10 Min. usw

leider funktioniert mein Script nicht.
Der Taster macht überhaupt nichts

Habe dazu das Script von @paul53 verwendet und angepasst.

// Script zum Starten einer Pumpe
 
// Datenpunkt-ID
var idTaster = "javascript.0.giesen-taster";
var idPumpe = "hm-rpc.0.0001D3C99CAEXX.3.STATE";
var gieszeit = "javascript.0.zeiten.gieszeit"

// Variablendeklaration 
var timer;
gieszeit = getState("javascript.0.zeiten.gieszeit").val;

function taster() {
    if (timer) {
        clearTimeout(timer);   // löschen zum Retriggern
        timer = null;
    }  
     setState(idPumpe, true);  // Ein
     timer = setTimeout(function () {
         setState(idPumpe, false);
     }, gieszeit * 60000);  // nach 60 Minuten aus
}
 
// Aufruf der Funktion bei Änderung von false auf true
on({id: idTaster, val: true}, taster); 

kann mir jemand sagen, wo hier mein Fehler ist?

Vielen Dank

[0018]

Hallo,

du möchtest bei Tastendruck das Dießen starten und nach einer Nachlaufzeit automatisch beenden?
was genau möchtest du mit +5, +10 usw machen? Die Giesszeit um 5min / 10min erweitern?

[0018]

Im ersten Schritt hast du zweimal die var gieszeit definiert. Zudem würde ich empfehlen ein paar mehr Logs einzubauen, um zu sehen bei welchem Step es hängt. Hab es mal etwas angepasst und es läuft soweit erstmal.

// Script zum Starten einer Pumpe
 
// Datenpunkt-ID
var idTaster = "javascript.0.giesen-taster";
var idPumpe = "hm-rpc.0.0001D3C99CAEXX.3.STATE";
var gieszeit = "javascript.0.zeiten.gieszeit"

// Variablendeklaration 
var timer;

 
function taster() {
    var time = getState(gieszeit).val;
    log ("Taster gestartet")
    log ("Zeitauswahl = " + time)
    if (timer) {
        log ("Timer zurück gesetzt")
        clearTimeout(timer);   // löschen zum Retriggern
        timer = null;
    }  
     setState(idPumpe, true);  // Ein
     timer = setTimeout(function () {
         log ("Pumpe aus nach Zeit")
         setState(idPumpe, false);
     }, time * 600000);  // nach 60 Minuten aus
}
 
// Aufruf der Funktion bei Änderung von false auf true
on({id: idTaster, val: true}, taster); 

Du könntest auch gleich auf den Taster reagieren ohne extra eine Funktion aufzumachen indem du

on({id: idTaster, val: true}, function() {
    ... Rest der Funktion
}

Eine weitere Möglichkeit wäre die Pumpe mit

setStateDelayed(idPumpe, false, gieszeit * 60000); 

verzögert auszuschalten, dann sparst du dir die ganze Timeout-Geschichte.

[Jason]

Vielen Dank... werde ich heute gleich mal probieren.

Zur +5, +10, ist einfach dafür gedacht, das ich mir die Zeit einstelle wie lange ich giessen will. Also 5 Min, 10 Min...usw.

Melde mich sobald ich es umgesetzt habe!