mein Rollladenskript und ein Paar Fragen :-)

[tloeffel]

Hallo Zusammen,

ich habe mal eine Frage an einen Experten.

Um zu überprüfen ob alle Rollladen wirklich runter gefahren sind, möchte ich dieses Skript verwenden. Ich benutze zum Checken den Kanal 3 des HM-IP Broll. Zum Schalten muss ich aber den Kanal 4 benutzen. Des Weiteren möchte ich immer nur einen offenen Rollladen ansteuern und dann im Fall das es mehrere gibt das Skript beenden. Das Skript soll über einen Cuxd Timer angesteuert werden bis es keine offenen Rollladen mehr gibt.

Mein funktionierendes Skript habe ich mal angefügt. Nun meine Fragen .

1) Ich habe zwei Methoden ausprobiert (alchy's aus dem Forum (methode1) und meine (methode2)). In alchy's Artikel spricht er von sicherer Zugriffsmethode. Ist meine nicht sicher?
2) Kann man das Skript vereinfachen? Ich denke bestimmt zu kompliziert/unerfahren.
3) In manchen Skripten lese ich von der Methode: IsTypeOf. Wofür soll diese angewendet werden?
4) Kennst Ihr eine bessere Methode die If-Schleife zu verlassen als „quit“ bei der ich das ganze Skript verlasse?

Danke für Eure Zeit und Hilfe.
Gruß Thomas

Code: Alles auswählen

!!Rollladen checken ob unten aus Beispiel im Gewerk Rollladen
!!Version 0.1, tloeffel

!!Gewerk aus dem die Aktoren gelesen werden sollen
var myFunction = dom.GetObject("Rollladen_Behang");
string itemId;

foreach(itemId, myFunction.EnumUsedIDs())
{   var item = dom.GetObject(itemId);
    if (item.State() != "0")
{
!!Die Adresse des anderen Kanals zusammen bauen
      string str = item.Address();
      string address_alt = str.Replace (":3", "");
      string channel_name = "HmIP-RF." # address_alt # ":4.LEVEL";

     var methode2 = dom.GetObject(channel_name);
      var methode1 = (dom.GetObject(ID_DATAPOINTS)).Get(channel_name);

      WriteLine(methode1.State());
      WriteLine(methode2.State());

!!Beide Wege lassen den Rollladen zu fahren
      !methode1.State(0);
      !methode2.State(0);

!!Verwende ich damit bei mehreren offen Rollladen nicht alle auf einmal “zu“ fahren. Aufruf erfolgt per Timer bis alle zu sind.
     quit;
     }
} 

[Xel66]

Ich würde das Pferd andersherum aufzäumen, bzw. mache das bei einem Rollladen, der ab und zu mal nicht mit seiner Rückmeldung zu CCU durchkommt. Nach einem Befehl an einen Rollladen starte ich zeitverzögert ein Script, welches nach dem Zustand schaut. Bei klassischen Homematic nutze ich hierzu den Datenpunkt .WORKING. Es wird infach nach Ablauf der eingestellten Laufzeit geschaut, ob der Datenpunkt noch WAHR ist. Die Behanghöhe muss bei dieser Methode nicht geprüft werden, da die ja dem letzten Befehl entsprechen müsste. Ich sende dann ein STOP und bekomme die aktuelle Behanghöhe zurück. Wurde der ursprüngliche Befehl nicht empfangen, gibt es ja sowieso Kommunikationsstörungsmeldungen. Die Lösung stammt aus dem Forum.

Gruß Xel66

[Gluehwurm]

Ob die Rollläden wirklich zu sind, siehst Du damit eh nicht. Nur ob die Aktoren gefahren sind.

Wenn die Aktoren nicht richtig funktionieren, gibt es eine Fehlermeldung. Oftmals reicht dann ein Skript auch nicht mehr, da die Teile Komm.-Störungen haben. Ansonsten würde ich mal schauen, warum so ein Skript notwendig ist.

Gruß
Bruno

[tloeffel]

Hallo Zusammen,

danke für die schnelle Antworten .
Allerdings finde ich meine Fragen noch nicht wirklich beantwortet.

Lieben Gruß
Thomas

[alchy]

In alchy's Artikel spricht er von sicherer Zugriffsmethode. Ist meine nicht sicher?

welcher Artikel? Es wäre nett, wenn du das auch verlinken würdest, wenn du mich schon anführst.
Du greifst auf einen Datenpunkt zu. Da ist das Risiko doppelter Namensvergabe relativ gering.
Sicherer ist es aber trotzdem.
Interessanter ist es bei z.B. dem Gewerk
var myFunction = dom.GetObject("Rollladen_Behang");
Das ist unsicher und enthält eben potenziell ein großes Risiko, das du den Namen doppelt vergeben haben könntest.
Damit würde dann mit einer 50% Wahrscheinlichkeit dein Script fehlerhaft sein. Hast du den Namen 3fach vergeben....

2) Kann man das Skript vereinfachen? Ich denke bestimmt zu kompliziert/unerfahren.

Das Script ist nur für dich und deine gewünschte Anwendung geschrieben und funktioniert für dich nach deinen Angaben ja so.
Viele Wege führen zum Ziel. Da halt ich mich lieber raus.

3) In manchen Skripten lese ich von der Methode: IsTypeOf. Wofür soll diese angewendet werden?

Prüft ob ein Objekt einen bestimmten Typ implementiert.
HomeMatic-Script Dokumentation - Teil 2: Objektmodell

4) Kennst Ihr eine bessere Methode die If-Schleife zu verlassen als „quit“ bei der ich das ganze Skript verlasse?

Du willst ja immer nur einen Rolladen ansteuern. Wenn du nicht das Script verlassen würdest, würde ja sofort der nächste Datenpunkt auf 0 gesetzt werden.

Alchy

[tloeffel]

In alchy's Artikel spricht er von sicherer Zugriffsmethode. Ist meine nicht sicher?

welcher Artikel? Es wäre nett, wenn du das auch verlinken würdest, wenn du mich schon anführst.
Du greifst auf einen Datenpunkt zu. Da ist das Risiko doppelter Namensvergabe relativ gering.
Sicherer ist es aber trotzdem.
Interessanter ist es bei z.B. dem Gewerk
var myFunction = dom.GetObject("Rollladen_Behang");
Das ist unsicher und enthält eben potenziell ein großes Risiko, das du den Namen doppelt vergeben haben könntest.
Damit würde dann mit einer 50% Wahrscheinlichkeit dein Script fehlerhaft sein. Hast du den Namen 3fach vergeben....


Deine Seite mit der Erklärung wie man Datenpunkte ausliesst .

3) In manchen Skripten lese ich von der Methode: IsTypeOf. Wofür soll diese angewendet werden?


Prüft ob ein Objekt einen bestimmten Typ implementiert.
HomeMatic-Script Dokumentation - Teil 2: Objektmodell


Ich weiss . Aber in welchen Praxisfall braucht man das?

4) Kennst Ihr eine bessere Methode die If-Schleife zu verlassen als „quit“ bei der ich das ganze Skript verlasse?


Du willst ja immer nur einen Rolladen ansteuern. Wenn du nicht das Script verlassen würdest, würde ja sofort der nächste Datenpunkt auf 0 gesetzt werden.

Ich würde gerne wissen ob es einen Anderen Befehl gibt ab "quit" um ein Skript gewollt zu verlassen.

Jetzt ist mir Formatierung total verrutscht sorry. Bin noch ein bisschen unerfahren mit dem Formatieren.
Vielen Dank für die Unterstützung.

Gruß Thomas