$data in MyUtils

[Det20]

Hallo,

auch auf die Gefahr hin, gelyncht zu werden: Wie greife ich auf $data in der MyUtils zu? Wenn ich folgendes verwende, stürzt FHEM ab. Aber wie ist der korrekte Aufbau?

Code Auswählen Erweitern

$data{LastRainMsg} = time();

[Thorsten Pferdekaemper]

Hi,
ich habe das mit dem $data schon einmal irgendwo gesehen, aber nicht verstanden, was es soll. Vielleicht kenne ich da aber auch etwas nicht...
Also: Was ist Deiner Meinung nach $data und was erhoffst Du Dir davon, was Du da machst?
Gruß,
   Thorsten

[Det20]

Soweit ich das verstanden habe, ist es ein globaler Variablenstack.

[Markus Bloch]

ich habe das mit dem $data schon einmal irgendwo gesehen, aber nicht verstanden, was es soll. Vielleicht kenne ich da aber auch etwas nicht...

Schau mal hier: https://wiki.fhem.de/wiki/DevelopmentModuleIntro#Wichtige_globale_Variablen_aus_fhem.pl

Wenn ich folgendes verwende, stürzt FHEM ab. Aber wie ist der korrekte Aufbau?

Code Auswählen Erweitern

$data{LastRainMsg} = time();

Das ist vollkommen korrekt. Wo genau in welchem Kontext führst du das aus? Hast du die genaue Fehlermeldung warum FHEM abstürzt?

Gruß
Markus

[Det20]

In einer MyUtils. Finde leider in den Logs nichts außer "PERL WARNING: Use of uninitialized value $data{"LastRainMsg"} in addition (+) at ./FHEM/99_myUtils.pm line 21. ".
Und zack ... Weg ist er. Dachte der ist ev schon initialisiert. Wie initialisiere ich beim Start? Hätte nicht gedacht, dass sowas den FHEM in den Tot reißt.

Code Auswählen Erweitern

sub ItsRain()
{
  # Nur alle 60 Minuten
  if(time()<($data{LastRainMsg}+60*60)) { exit; };
  $data{LastRainMsg} = time();

...

[Markus Bloch]

Das Problem warum die FHEM dabei um die Ohren fliegt liegt höchstwahrscheinlich daran, dass du $data{LastRainMsg} in deiner IF-Bedingung verwendest ohne vorher zu prüfen, ob der Wert überhaupt in %data existiert, geschweige denn definiert ist. Wenn FHEM frisch startet existiert dieser Wert ja noch garnicht und du greifst damit auf einen nicht existierendes Element zu und versuchst damit zu rechnen. undef+60*60 wirft dabei dann einen Perl-Fehler.

Du solltest daher vor deinem if-Konstrukt prüfen ob dein Wert existiert und definiert ist, z.B. via if(defined($data{LastRainMsg})) und festlegen, was zu tun ist, wenn der Wert nicht definiert ist.

Gruß
Markus

[Markus Bloch]

Wie initialisiere ich beim Start?

z.B. mit einem notify:

Code Auswählen Erweitern

define init_LastRainMsg notify global:INITIALIZED {$data{LastRainMsg} = 0}


Ich würde dir aber eher dazu raten, in deiner Funktion eine entsprechende Behandlung zu machen und dort bspw. den Wert initialisiern, sollte er nicht existieren:

Code Auswählen Erweitern

sub ItsRain()
{
  # Init
  $data{LastRainMsg} = 0 unless(defined($data{LastRainMsg}));

  # Nur alle 60 Minuten
  if(time()<($data{LastRainMsg}+60*60)) { exit; };
  $data{LastRainMsg} = time();

[betateilchen]

Ausserdem sind da für meinen Geschmack zu viele Klammern und ich würde die Verarbeitung lieber mit return als mit exit beenden.

Code Auswählen Erweitern


sub ItsRain()
{
  # Nur alle 60 Minuten
  $data{LastRainMsg} //= 0;
  return if( time() < $data{LastRainMsg}+60*60) ;
  $data{LastRainMsg} = time();
...


Damit wird der Wert automatisch beim ersten Aufruf der Funktion mit 0 initialisiert und später regelmäßig aktualisiert.

[Thorsten Pferdekaemper]

Schau mal hier: https://wiki.fhem.de/wiki/DevelopmentModuleIntro#Wichtige_globale_Variablen_aus_fhem.pl

Ah, sorry, ich wäre nie auf die Idee gekommen, dass es so etwas gibt, geschweige denn zur Benutzung propagiert wird.

Ich hätte das gelöst, indem ich an ein geeignetes Device ein Reading hänge und das dann mit ReadingsVal abfrage. Da passiert sowas nicht.
...oder aber direkt ein "at" angelegt, dass das alle 60 Minuten erledigt.

Gruß,
   Thorsten

[Det20]

Code Auswählen Erweitern

$data{LastRainMsg} = 0 unless(defined($data{LastRainMsg}));

Sehr geil, den kannte ich noch nicht. Vielen Dank!

[betateilchen]

Code Auswählen Erweitern

$data{LastRainMsg} = 0 unless(defined($data{LastRainMsg}));

das ist exakt das gleiche wie

Code Auswählen Erweitern

$data{LastRainMsg} //= 0;

nur länger 

[Markus Bloch]

Ah, sorry, ich wäre nie auf die Idee gekommen, dass es so etwas gibt, geschweige denn zur Benutzung propagiert wird.

Naja, es ruft jetzt niemand aktiv dazu auf %data zu nutzen. Es dient dazu definitionsübergreifend Daten zur Laufzeit zu speichern. Definitionen können bspw in %data Extensions für FHEMWEB registrieren.

das ist exakt das gleiche wie

Code Auswählen Erweitern

$data{LastRainMsg} //= 0;

nur länger 

Ich störe mich da immer an den Schrägstrichen und denke dabei immer an Regexp   

[betateilchen]

Ich störe mich da immer an den Schrägstrichen und denke dabei immer an Regexp

Naja, bei regexp kommen Schrägstriche in den allermeisten Fällen erst nach einem = vor, selten davor.
Insofern hab ich damit überhaupt keine Verwexlungsprobleme.