Du liest nur den Status der ersten Variable ein (ankleide_low_bat).
Ich hab im Moment kein IPS vor mir, aber ich meine, dass du mit GetValueBoolean nur eine einzelne Variable abfragen kannst.
Im Prinzip hast du Recht. Nur die umgekehrte Logik besteht darin, dass, wenn ich das nun richtig weiss, keine intakten, sondern leere Batterien gemeldet werden.
Also stimmt RWNs Script nun so… Es genügt also wenn eine einzelne Batterie leer meldet um eine Message auszulösen. Ich denke das ist es was er will.
es war doch ein Fehler im Skript: vor der Klammer fehlte der Funktionsaufruf
So sollte es funktionieren (mit positiver Logik):
// Voraussetzung:
// XXX_low_bat == true, wenn Batterie schwach
// XXX_low_bat == false, wenn Batterie OK
$status = GetValueBoolean("ankleide_low_bat") or
GetValueBoolean("bad_low_bat") or
GetValueBoolean("buero_low_bat") or
GetValueBoolean("kueche_low_bat") or
GetValueBoolean("Tuerkontakt_Haustuer_low_bat") or
GetValueBoolean("Tuerkontakt_Keller_low_bat") or
GetValueBoolean("TF_Wohnzimmer_low_bat");
if ($status == false)
{
SetValueString("Batterie_voll", "Status OK");
echo "OK";
}
else
{
echo "leer";
}
Habt Vielen Dank ihr 3 für die Super schnelle Hilfe.
Habe wieder was gelernt und das ich wichtig.
Werde das ganze jetzt noch mit array´s ausstatten, so das genaue Meldungen erscheinen. Ich denke, wenn ich nicht weiter komme werde ich noch mal fragen.
Könnte hier mal jemand drüber sehen, ob man das eventuell auch eleganter lösen kann oder ob es so OK ist.
Ist ja schon eine Menge Code für ein paar Meldungen und es kommen noch einige hinzu. Funktion ist einwandfrei.
Achso, es werden einzelne Meldungen und eine Sammelmeldung ausgegeben.
Danke
<?
/*
*******************************
IP-SYMCON Event Scripting
*******************************
File : Batterie_Status.ips.php
Trigger :
Interval :
*/
$status_Ankleide = GetValueBoolean("ankleide_low_bat");
$status_bad = GetValueBoolean("bad_low_bat");
$status_buero = GetValueBoolean("buero_low_bat");
$status_kueche = GetValueBoolean("kueche_low_bat");
$status_Haustuer = GetValueBoolean("Tuerkontakt_Haustuer_low_bat");
$status_Keller = GetValueBoolean("Tuerkontakt_Keller_low_bat");
$status_Wohnzimmer = GetValueBoolean("TF_Wohnzimmer_low_bat");
$Status=($status_Ankleide or $status_bad or $status_buero or $status_kueche or $status_Haustuer or $status_Keller or $status_Wohnzimmer);
if (!$Status)
{
SetValueString("Batterie_Meldung", "Batterie Status OK");
}
if (!$status_Ankleide)
{
SetValueString("FHT_Ankleide", "FHT Ankleide
Batterie Status
OK");
}
if (!$status_bad)
{
SetValueString("FHT_Bad", "FHT Bad
Batterie Status
OK");
}
if (!$status_buero)
{
SetValueString("FHT_Buero", "FHT Büro
Batterie Status
OK");
}
if (!$status_kueche)
{
SetValueString("FHT_Kueche", "FHT Küche
Batterie Status
OK");
}
if (!$status_Haustuer)
{
SetValueString("TFK_Haustuer", "TFK Haustür
Batterie Status
OK");
}
if (!$status_Keller)
{
SetValueString("TFK_Kellertuer", "TFK Kellertür
Batterie Status
OK");
}
if (!$status_Wohnzimmer)
{
SetValueString("TF_Wohnzimmer", "TF Wohnzimmer
Batterie Status
OK");
}
if ($Status)
{
SetValueString("Batterie_Meldung", "Batterie leer ");
}
if ($status_Ankleide)
{
SetValueString("FHT_Ankleide", "FHT Ankleide
Batterie leer");
}
if ($status_bad)
{
SetValueString("FHT_Bad", "FHT Bad
Batterie leer");
}
if ($status_buero)
{
SetValueString("FHT_Buero", "FHT Büro
Batterie leer");
}
if ($status_kueche)
{
SetValueString("FHT_Kueche", "FHT Küche
Batterie leer");
}
if ($status_Haustuer)
{
SetValueString("TFK_Haustuer", "TFK Haustür
Batterie leer");
}
if ($status_Keller)
{
SetValueString("TFK_Kellertuer", "TFK Kellertür
Batterie leer");
}
if ($status_Wohnzimmer)
{
SetValueString("TF_Wohnzimmer", "TF Wohnzimmer
Batterie leer");
}
?>
Vielleicht ist das der falsche Weg. HJH schreibt bestimmt kurzen und effektiven Code - Keine Frage. Aber verstehst du ihn auch? Denk immer daran, dass du in einem Jahr oder zwei vielleicht mal was ändern willst oder einen Fehler suchst.
Ich versuche hier im Forum, grade bei Anfängern, möglicht einfachen, übersichtlichen Code zu posten und wundere mich immer wieder was für komprimierte Code-Klötze, grade von den Laien, bevorzugt werden.
HJH, Olli und auch ich haben kein Problem damit den Code zu lesen, zu interpretieren und zu verstehen. Kannst du das auch noch, wenn du zuhause mal allein davor sitzt? Denk bitte daran, dass dir das Forum vielleicht nicht immer zur Verfügung steht wenn du mal ein Problem mit diesem Code hast. Wichtig ist, dass du verstehst was in deinem Server, also in deinem Haus, vorgeht. Sonst baust du dir ein System, dass du hinterher selbst nicht bedienen kannst.
Du musst schlimmstenfalls auch allein noch mit dem Code klarkommen wenn du dich ein Jahr oder länger überhauptnicht mehr mit PHP beschäftigt hast. Denk da bitte drüber nach.
Ich muss gestehen, dass ich den Code bewusst unkommentiert gepostet habe, einfach um eine Reaktion zu provozieren.
Mir ist beim Schreiben des Codes durchaus klar geworden, dass mit jedem weiteren Schritt in Richtung Kompaktheit die Verständlichkeit irgendwann auf der Strecke bleibt.
Der Schlüssel zum Code liegt hier im Verständnis von PHP-Arrays.
Der Rest ist eigentlich harmlos.
aber im Allgemeinen ist die Vorgehensweise, Einsteigern PHP nach Deiner Methode nahe zu bringen, sicher die bessere Wahl.
Finde es ein super vorbild wie der ganse thread sich zusammen gebaut hat.
Und dann letztlich mit arrays arbeiten… super !
Wer sich schon mal das ganse durchgelesen hat, bekommt die array-sache sicher mit.
Nur die kombination durch und trigger und array wird es schwierig machen um es zu kapieren.
Deshalb: (wenn HJH es zulasst)
//lege ein array an die beeinhaltet:
// die verknupfte low-bat variable <- hierdurch wird dieses script getriggert
// die nahmen vom derre ausgabe-string
// und wie wir den raum heissen möchten im status tekst.
$raeume = array( "ankleide_low_bat" => array("FHT Ankleide", "FHT_Ankleide" ),
"bad_low_bat" => array("FHT Bad", "FHT_Bad" ),
"buero_low_bat" => array("FHT Büro", "FHT_Buero" ),
"kueche_low_bat" => array("FHT Küche", "FHT_Kueche" ),
"Tuerkontakt_Haustuer_low_bat" => array("TFK Haustür", "TFK_Haustuer" ),
"Tuerkontakt_Keller_low_bat" => array("TFK Kellertür", "TFK_Kellertuer"),
"TF_Wohnzimmer_low_bat" => array("TF Wohnzimmer", "TF_Wohnzimmer" )
);
//gib mir den status von die variable die mich (dieser script) getriggert hat (true/false)
$status = GetValueBoolean($IPS_VARIABLE);
//lese im array wie der statusvariable heisst
$var = $raeume[$IPS_VARIABLE][1];
//lese im array wie ich der raum genannt habe
$meldung1 = $raeume[$IPS_VARIABLE][0];
//
: starte auf eine neue linie
$meldung2 = "
Batterie Status
";
//schnell condition: gleicht if ($status) { $meldung3="OK"; } else { $meldung3="leer";}
$meldung3 = ($status) ? "OK" : "leer";
//schiebe alles zuruck im statusvariable
SetValueString($var, $meldung1. $meldung2. $meldung3);
Und was HJH vergessen ist in sein eifer
// setzen vom algemeine meldung
//wen es schon einem lehre batterie gibt
if (!$IPS_VARIABLE){
//nur bei false-wert
SetValueString("Batterie_Meldung", "Batterie leer");
} else {
//nachsehen ob es noch andere lehre batterien gibt
//anders muss die algemeine meldung zuruckgesetst werden.
// init
$result = false;
// foreach = durchlaufe jedem wert des array
// und nehme den einzelne element nahme in $key (ist gleich aus die variable-nahmen)
// und schiebe in $value die beide werte rein (also ist $value eigentlich jetzt ein array)
// $value wird hier nicht gebraucht
foreach($raume as $key => $value){
//lese alle variable und addiere derren status
//lauft nur bei true deshalb oben erst den $result=false init
if (GetValueBoolean($key)) $result = $result or true;
}
//bei $result true (alle batterien sind ok) neue algemeine botschaft setzen
if ($result) SetValueString("Batterie_Meldung","Batterie OK");
}
//Fertig
Bitte um strukturiertes ubernehmen im wiki als beispiel in die newbie-ecke (Tony’s Handbuch)
