Berechnung im Ausführungsteil notify - HILFE - ich kapier es nicht

[betateilchen]

Code Auswählen Erweitern

$Aut=round($Aut,2);fhem("setreading Verbrauchsmeter Autarkie $Aut")

[Stonemuc]

Auch wenn ich noch was davor habe? Weil dann gibt er mir nen Klammerfehler aus....

Code Auswählen Erweitern

defmod Autarkie at +*00:30:00 {my$Aut=((ReadingsNum("Wechselrichter","Daily.Energy",0) - ReadingsNum("Stromzaehler_Einspeisung","Tageseinspeisung",0))/(ReadingsNum("Stromzaehler_Verbrauch","Tagesverbrauch",0))*100);; fhem("setreading Verbrauchsmeter Autarkie $Aut")}


[MadMax-FHEM]

Wie wäre es denn, wenn du den Fehler psoten würdest?

Und ich denke hier fehlt ein Leerzeichen?

my$Aut=... vs. my $Aut=...

Und: warum so elendsviele Klammern?

Und: wie gibst du das ein? Jedes mal KOMPLETT SO in  FHEMWeb-cmd ODER per Klick auf DEF? ODER Raw-Def? -> wegen der Strichpunkte... z.B. hier https://forum.fhem.de/index.php?topic=98634.0

Ja, einfach NACHDEM du $Aut korrekt berechnet hast einen "Einschub" (von betateilchen) mit dem Runden und DANACH dann das set/setreading das du schon hast...

Pseudocode (weil mir deine Klammern zu viel sind):

Code Auswählen Erweitern


my $Aut=Berechnung;; $Aut=round($Aut,2);; fhem("set...")


Gruß, Joachim

[betateilchen]

Naja, mit der Klammersetzung scheinst Du ja grundsätzlich auf Kriegsfuß zu stehen.

Grundsätzlich gilt bei Klammern nicht "viel hilft viel". Klammern sollte man genau da setzen, wo sie notwendig sind, und nicht "auf Verdacht".
Klammern können in Deinem Konstrukt aus zwei Gründen notwendig sein: Entweder, weil sie zu einer aufgerufenen perl Funktion z.B. ReadingsNum() gehören ODER weil sie mathematisch notwendig sind. Eigentlich müsste das Ganze so aussehen, wenn ich Dein Vorhaben richtig interpretiere:

Code Auswählen Erweitern


my $Aut=( ReadingsNum("Wechselrichter","Daily.Energy",0) - ReadingsNum("Stromzaehler_Einspeisung","Tageseinspeisung",0) ) / ReadingsNum("Stromzaehler_Verbrauch","Tagesverbrauch",0) * 100;


[Stonemuc]

Wie wäre es denn, wenn du den Fehler psoten würdest?

Und ich denke hier fehlt ein Leerzeichen?

my$Aut=... vs. my $Aut=...

Und: warum so elendsviele Klammern?

Und: wie gibst du das ein? Jedes mal KOMPLETT SO in  FHEMWeb-cmd ODER per Klick auf DEF? ODER Raw-Def? -> wegen der Strichpunkte... z.B. hier https://forum.fhem.de/index.php?topic=98634.0

Ja, einfach NACHDEM du $Aut korrekt berechnet hast einen "Einschub" (von betateilchen) mit dem Runden und DANACH dann das set/setreading das du schon hast...

Pseudocode (weil mir deine Klammern zu viel sind):

Code Auswählen Erweitern


my $Aut=Berechnung;; $Aut=round($Aut,2);; fhem("set...")


Gruß, Joachim

Die Berechnung funktioniert mit meinen Klammern und dem my$Aut ohne Leerzeichen korrekt. Es kommt ja der richtige Wert raus - auch händisch nachgerechnet. Das Ändern mache ich immer über die RAW-def.
Dank deinem "Pseudocode" konnte ich es jetzt nachvollziehen. Ich muss also immer nach jedem "Abschnitt" ein doppel ;; machen...daran hat es bei mir gelegen.

[betateilchen]

Typischer Fall von Beratungsresistenz...

[Stonemuc]

Ne..so soll es ja nicht sein, ich füge das Leerzeichen ein und versuch es mal mit deinem Vorschlag mit weniger Klammern...solte mathematisch ja auch klappen