[patch] fhem.pl - Konstanten für das Rechnen mit Zeiten

[betateilchen]

Hallo Rudi,

da man beim Rechnen mit Zeiten bestimmte Multiplikatoren immer wieder benötigt, wäre es schön, wenn diese in FHEM als Konstanten zur Verfügung stünden. In Programmcode finde ich das Arbeiten mit Zahlenwerten immer unübersichtlich (von Speichereffizienz ganz abgesehen)

Deshalb schlage ich folgenden Patch vor.

Code Auswählen Erweitern


Index: fhem.pl
===================================================================
--- fhem.pl     (revision 18475)
+++ fhem.pl     (working copy)
@@ -270,6 +270,11 @@
use vars qw(@intAtA);           # Internal timer array
use vars qw(@structChangeHist); # Contains the last 10 structural changes

+use constant {
+  DAYSECONDS  => 86400,
+  HOURSECONDS =>  3600,
+  MINSECONDS  =>    60
+};

$selectTimestamp = gettimeofday();
$cvsid = '$Id$';


[CoolTux]

Finde ich sehr gut.

[justme1968]

ich würde MINUTESECONDS statt MINSECONDS vorschlagen.

ist zwar etwas länger aber nicht mit minimum verwechselbar und die anderen beiden sind ja auch ausgeschrieben.
falls das zu lang ist lieber SECONDS bei allen dreien zu SEC verkürzen.

[betateilchen]

ich würde MINUTESECONDS statt MINSECONDS vorschlagen.

nix dagegen, die Länge des Namens ist mir ziemlich egal, solange sie unter 255 Zeichen bleibt 

[justme1968]

das bekommen wir hin

[Christoph Morrison]

Darf ich anmerken, dass Tage nicht immer exakt 86400 Sekunden lang sind?
Wenn man damit leben kann, bietet Time::Seconds (Kernmodul) auch bereits entsprechende Konstanten.

[rudolfkoenig]

Habs eingecheckt.

Darf ich anmerken, dass Tage nicht immer exakt 86400 Sekunden lang sind?

Aus Sicht von FHEM ist das aber egal, die Bibliotheken ignorieren die Schaltsekunde, wenn es um das Konvertieren der Zeit seit 1970 geht.

[betateilchen]

Habs eingecheckt.

Danke.