Script: Batteriespannungen in Werteliste

[Feyrgas Mortes]

Hallo,

ich habe ein kleines Scripting Problem...
Ich versuche seit geraumer Zeit via Script den "BATTERY_STATE" der HKT's und Wandthermostaten abzufragen und in einer Systemvariablen abzulegen.

Hier erst mal dass Script:

Code: Alles auswählen

var BatDev = dom.GetObject("Batterie");
    string itemID;
    string text = "";

    ! Durch alle angemeldeten Geräte laufen
    foreach(itemID, BatDev.EnumUsedIDs())
    {
    ! Gerätedetails & Schnittstelle holen
    var item = dom.GetObject(itemID);
    var device = dom.GetObject(item.Device());
    var interface = dom.GetObject(item.Interface());

    ! Name, Adresse, Kanal und Batteriespannung auslesen (Kanal4)
    string interface_name = interface.Name();
    string device_address = device.Address();
    device_address = device_address.StrValueByIndex(":", 0);
    string channel_name = interface_name # "." # device_address # ":4.BATTERY_STATE";
    var channel = dom.GetObject(channel_name);
    string batvalue = dom.GetObject(channel_name).Value();
    ! Text zusammensetzen
    string text = text # "- " # item.Name() # " hat: " # batvalue.ToString(1) # " V Spannung" # "\r\n";
!WriteLine(text);
    }
    }
     dom.GetObject("Batteriestatus").State(text);
! WriteLine(Batteriestatus);

Soweit funzt es ganz gut ABER:
1) Die SysVar "Batteriestatus" bleibt leer. Warum?
2) Habe ich das Problem dass manche Geräte Kanal 2 (:2.BATTERY_STATE) verwenden. Wie kann ich dass lösen dass ich NICHT 2x foreach durchlaufen lassen muss?
3) Aktuell hat die SysVar den Typus "Zeichenkette". Wie funktioniert dass ganze mit dem Typus "Wertetabelle"?

Ich möchete ein Programm anlegen welches alle 24 Std. die SysVar mit den Aktuellen Spannungen aktualisiert. Möchte mir die Variable auf dem Startbildschirm meiner CCU2 anzeigen lassen.
Es geht mir nur um das Script, die Programme und Rest ist kein Problem.

Danke im Vorraus
Feyrgas Mortes

[Cash]

Code: Alles auswählen

string text = text # "- " # item.Name() # " hat: " # batvalue.ToString(1) # " V Spannung" # "\r\n";

Nimm dort mal das string weg.

Ich weiß nicht ob es daran liegt aber wenn man in einen Script eine Variable neue deklariert wird der vorige Wert gelöscht. Steht auch so in der Doku. Weiß nicht ob das in einer Schleife auch gilt.

Noch ein anderer Hinweis: Am Anfang ist es vielleicht total spannend den Batteriestatus ablesen zu können aber nach längerer Nutzung? Am Anfang bekam ich täglich eine Mail darüber. Nach ein paar Monaten habe ich es deaktiviert. Im Bedarfsfall kann ich per Button mir die Pushnachricht schicken lassen aber der Bedarf ist bei mir eigentlich nie da. Ich warte bis ich irgendwann die Meldung bekomme das ich die Batterie tauschen soll und danach tausche ich - fertig. Die Warnmeldung kamen bisher immer sehr zeitig so das ich mich darauf verlasse und nicht selber kontrolliere.

[alchy]

Soweit funzt es ganz gut

ernsthaft? Kann ich mir aber nicht wirklich vorstellen.
Ok kann es nicht testen, aber ich habe da so meine theoretischen Zweifel.

Was kriegst du denn als Ausgaben?

Alchy

[Feyrgas Mortes]

ernsthaft? Kann ich mir aber nicht wirklich vorstellen.
Ok kann es nicht testen, aber ich habe da so meine theoretischen Zweifel.

Was kriegst du denn als Ausgaben?

Alchy

Würde es tadellos funktionieren, hätte ich wohl kaum meine Frage gestellt.
Was nicht geht bzw. schön währe wenn es funktioniert (wenn machbar), habe ich ja geschrieben^^
Die Variable "text" enthält, leider nur, den letzten Thermostat/HKT der foreach-Schleife und die SysVar bleibt leer.

Code: Alles auswählen

string text = text # "- " # item.Name() # " hat: " # batvalue.ToString(1) # " V Spannung" # "\r\n";

Nimm dort mal das string weg.

Ich weiß nicht ob es daran liegt aber wenn man in einen Script eine Variable neue deklariert wird der vorige Wert gelöscht. Steht auch so in der Doku. Weiß nicht ob das in einer Schleife auch gilt.

Noch ein anderer Hinweis: Am Anfang ist es vielleicht total spannend den Batteriestatus ablesen zu können aber nach längerer Nutzung? Am Anfang bekam ich täglich eine Mail darüber. Nach ein paar Monaten habe ich es deaktiviert. Im Bedarfsfall kann ich per Button mir die Pushnachricht schicken lassen aber der Bedarf ist bei mir eigentlich nie da. Ich warte bis ich irgendwann die Meldung bekomme das ich die Batterie tauschen soll und danach tausche ich - fertig. Die Warnmeldung kamen bisher immer sehr zeitig so das ich mich darauf verlasse und nicht selber kontrolliere.

Danke für den Tipp, aber hat nicht geholfen.
Eine Mail oder Push-Nachricht möchte ich gar nicht bekommen, will sie nur auf der Startseite der CCU2 angezeigt bekommen

[alchy]

Entschuldige, wollte dir weder etwas einreden noch wollte ich dich stören.

Alchy

[Feyrgas Mortes]

Entschuldige, wollte dir weder etwas einreden noch wollte ich dich stören.

Alchy

Keine Angst, du störst weder noch redest du mir was ein
Gegenteil, ich bin sogar Dankbar um Kritik/Hilfe von euch

[dtp]

Nimm dort mal das string weg.

@Feyrgas Mortes: In eine Schleife gehören keine Variablendeklarationen hinein. Besser und sauberer ist es, solche Deklarationen grundsätzlich an den Beginn eines Skripts zu stellen. Zudem enthält Dein Skript eine geschlossene, geschweifte Klammer, zu der es kein Gegenstück gibt. Das Skript kann so also gar nicht funktionieren.

Versuch's mal so:

Code: Alles auswählen

var BatDev = dom.GetObject("Batteriebetrieb"); 
var item; 
string itemID; string text = ""; string channel_name; string batvalue; 

! Durch alle angemeldeten Geräte laufen
foreach(itemID, BatDev.EnumUsedIDs()){
  !Gerätedetails & Schnittstelle holen
  item = dom.GetObject(itemID);
    channel_name = dom.GetObject(item.DPByHssDP("BATTERY_STATE"));
    if(channel_name){
      batvalue = dom.GetObject(channel_name).Value();
 
      ! Text zusammensetzen
      text = text#"Spannung an "#item.Name()#": "#batvalue.ToString(1)#" V\n";
  }
}
WriteLine(text);

(dom.GetObject(ID_SYSTEM_VARIABLES).Get("Batteriestatus")).State(text);

Gruß,

Thorsten

[alchy]

will sie nur auf der Startseite der CCU2 angezeigt bekommen

Hab ich doch glatt oben vergessen.
Da das Script nun dank dtp zumindest augenscheinlich lauffähig ist, bitte die Variable so NICHT zur Startseite hinzufügen.
Den Grund dafür kannst du >> H I E R << nachlesen.

Alchy

[klana]

Hi,

das Thema interessiert mich auch.
Ich habe das Script von dtp ausprobiert und bekomme nur die HeizkörperThermostate und die Wandthermostate ausgegeben.
In Batteriebetrieben sind aber noch viel mehr (Fensterkontakte, usw.).
Warum werden die nicht angezeigt bzw müssen die Batteriestände hier irgend wie anders ausgelesen werden?

Gruß
Klana

[alchy]

Nicht alle Geräte die Batteriebetrieben sind, bieten auch die Möglichkeit die Spannung auszugeben.

Alchy