[GELÖST] id Übergabe an Function klappt nicht

[mctom]

Hallo zusammen,

hier ein Auszug aus einem Script von mir:

var = idRArbeitenTuer = "hm-rpc.0.IEQ123456.1.LEVEL"/*Rollo-Arbeiten-Tuer.LEVEL*/,
idKArbeitenTuer = "hm-rpc.0.IEQ123456.1.STATE"/*Kontakt-Arbeiten-Tuer.STATE*/,
idRolloStatus = "javascript.0.Rollo.Steuerung.Status"/*Rollo.Steuerung.Status*/;

on({id: idKArbeitenTuer, change: "ne"}, function () {
	kontakt (idKArbeitenTuer,idRArbeitenTuer);
});

function kontakt (namekontakt,namerollo){
    log("kontaktänderung " + namerollo + namekontakt);
    if(getState(namekontakt) === 2 && getState(idRolloStatus) === 2)setState(namerollo,20);
    if(getState(namekontakt) === 0 && getState(idRolloStatus) === 2)setState(namerollo,0);
}

Leider funktioniert das ganze nicht so wie es soll.

Der Aufruf der Function klappt soweit. Allerdings bekomme ich nicht die korrekte Übergabe der IDs hin. Da die Anführungszeichen entfernt werden.

Kann mir jemand helfen und vielleicht einen Tipp geben ?

Gruß

Michael

[apollon77]

Hey,

hat mit den Anführungszeichen nichts zu tun. Ich denke eher das es daran liegt das ein "getState()" ein Objekt zurückgibt und nicht den Wert!!

Du musst getState().val nutzen um den Wert zu bekommen!+

Ingo

[ruhr70]

var = idRArbeitenTuer

Das funktioniert trotz dem Gleichheitszeichen hinter dem var?

Das Gleichheitszeichen gehört nach meiner Meinung nicht nicht da hin.

[apollon77]

Ja, das ist auch falsch. Hatte ich glatt übersehen

[arteck]

var idRArbeitenTuer = "hm-rpc.0.IEQ123456.1.LEVEL"/*Rollo-Arbeiten-Tuer.LEVEL*/;
var idKArbeitenTuer = "hm-rpc.0.IEQ123456.1.STATE"/*Kontakt-Arbeiten-Tuer.STATE*/;
var idRolloStatus = "javascript.0.Rollo.Steuerung.Status"/*Rollo.Steuerung.Status*/;

on({id: idKArbeitenTuer, change: "ne"}, function () {
   kontakt (idKArbeitenTuer,idRArbeitenTuer);
});

function kontakt (namekontakt,namerollo) {
    log("kontaktänderung " + namerollo + namekontakt);
    if (getState(namekontakt) === 2 && getState(idRolloStatus) === 2) {
    	setState(namerollo,20);
    }
    if (getState(namekontakt) === 0 && getState(idRolloStatus) === 2) {
    	setState(namerollo,0);
    }
}

und hinten kein komma am variablendeklarationsende

probier das mal..

[AlCalzone]

@arteck:

und hinten kein komma am variablendeklarationsende `
Deklarationen per Komma trennen ist vollkommen ok, solange hinter dem letzten ein Semikolon steht. Nur das = direkt hinter <u>var</u> darf nicht sein.

[dwm]

var idRArbeitenTuer = "hm-rpc.0.IEQ123456.1.LEVEL"/*Rollo-Arbeiten-Tuer.LEVEL*/;
var idKArbeitenTuer = "hm-rpc.0.IEQ123456.1.STATE"/*Kontakt-Arbeiten-Tuer.STATE*/;
var idRolloStatus = "javascript.0.Rollo.Steuerung.Status"/*Rollo.Steuerung.Status*/;

on({id: idKArbeitenTuer, change: "ne"}, function () {
   kontakt (idKArbeitenTuer,idRArbeitenTuer);
});

function kontakt (namekontakt,namerollo) {
    log("kontaktänderung " + namerollo + namekontakt);
    if (getState(namekontakt).val === 2 && getState(idRolloStatus).val === 2) {
       setState(namerollo,20);
    }
    if (getState(namekontakt).val === 0 && getState(idRolloStatus).val === 2) {
       setState(namerollo,0);
    }
}

… die .val nach den getStates haben noch gefehlt.

Noch ein Tipp ... ich würde das mit mindestens 5s Verzögerung schalten. Sonst kriegt das Rollo immer Zuckungen, wenn jemand kurz mal den Hebel hin- und herdreht.

Also:

var idRArbeitenTuer = "hm-rpc.0.IEQ123456.1.LEVEL"/*Rollo-Arbeiten-Tuer.LEVEL*/;
var idKArbeitenTuer = "hm-rpc.0.IEQ123456.1.STATE"/*Kontakt-Arbeiten-Tuer.STATE*/;
var idRolloStatus = "javascript.0.Rollo.Steuerung.Status"/*Rollo.Steuerung.Status*/;
var RolloTimer = null;

on({id: idKArbeitenTuer, change: "ne"}, function () {
   if (RolloTimer !== null) 
       clearTimeout(RolloTimer);
   RolloTimer = setTimeout( function() {kontakt (idKArbeitenTuer,idRArbeitenTuer);},5000);
});

function kontakt (namekontakt,namerollo) {
    log("kontaktänderung " + namerollo + namekontakt);
    if (getState(namekontakt).val === 2 && getState(idRolloStatus).val === 2) {
       setState(namerollo,20);
    }
    if (getState(namekontakt).val === 0 && getState(idRolloStatus).val === 2) {
       setState(namerollo,0);
    }
    RolloTimer=null;
}

Das ruft dann die Routine für Kontaktänderung erst nach 5s nach der letzten Statusänderung auf.

BTDTBTTS …

Werner

[mctom]

Danke an alle für die Hilfe.

Da hatte ich am Anfang wohl Tomaten auf den Augen.

Die Idee mit den 5s habe ich direkt auch übernommen.

Gruß

Michael