V0.9 Final - Sommerzeitumstellung

[lembi]

...ich bring nichtmal ein 'Hallo Welt' in C# zusammen

was macht das denn?
begin_ = 0x030000 | (begin << 8) | 1;
end_ = 0x0A0000 | (end << 8) | 1;

[Campus]

was macht das denn?
begin_ = 0x030000 | (begin << 8) | 1;
end_ = 0x0A0000 | (end << 8) | 1;

Das erzeugt eine Zahl aus dem Monat, dem Tag des Monats und der Stunde.
0x03 == 3 == Maerz
0x0A == 10 == Oktober
Die Monate werden dann auf das dritte Byte geschoben, daher zu
0x030000 == 196608
0x0A0000 == 655360
Da sieht man, dass die unteren zwei Bytes (0000) komplett unbesetzt sind.
Dann werden die Tage des Monats auf das zweite Byte kopiert
0x030000 | (begin << 8) == 0x031E00 == 204288 (wenn der Beginn am 30.03., also am 30, also 0x1E ist)
0x0A0000 | (end << 8) == 0x0A1A00 == 662016 (wenn das Ende am 26.10., also am 26, also 0x1A ist)
Dann noch die Stunde der Uhrzeit (in dieser Rechnung immer 1, sollte eigentlich 2 oder 3 sein) auf das letzte Byte
0x030000 | (begin << 8) | 1 == 0x031E00 | 1 == 0x031E01 == 204289
0x0A0000 | (end << 8) | 1 == 0x0A1A00 | 1 == 0x0A1A01 == 662017

Die gleiche Rechnung wird dann für das heutige Datum und die momentane Stunde gemacht um dann zu checken, ob dieser Wert zwischen den gerade berechneten Werten liegt, denn dann wird Sommerzeit angenommen.
Das gefaehrliche an der Rechnung sind die Monatskonstanten, bei denen man immer checken muss, ob die Werte 0 oder 1-basiert sind. Die Rechnung
now_ = (pUtc->month << 16) | (pUtc->day << 8) | pUtc->hour;
geht davon aus, dass Monat und Tag des pUtc-Pointers 1-basiert sind.

[Admin]

Campus hat meine Funktion verstanden.

Zu deinem Bedenken mit dem 1. basierten Monat / Tag: Da ich auch den Rest programmiert habe, bin ich mir da relativ sicher.

Dann noch die Stunde der Uhrzeit (in dieser Rechnung immer 1, sollte eigentlich 2 oder 3 sein) auf das letzte Byte

Hier habe ich extra UTC verwendet, dass macht es viel einfacher. Denn die Zeitumstellung passiert immer um 1 Uhr Nachts (UTC).
Die aktuelle Uhrzeit im Winter ist dann UTC+1, bzw. im Sommer UTC+2.

(Vor Robonect habe ich mir da auch nie Gedanken drüber gemacht )

[lembi]

Ja, danke für die Erklärung. Mit dieser Bit shifterei komm ich nie klar
Das mit den Monaten war mir auch aufgefallen, hab aber dann mal unterstellt das es schon richtig sein wird.

Code: Alles auswählen

for (begin = 31; RtcDayOfWeek(pUtc->year, 3, begin) != 6; begin--)

würde bei 0 basierter Zählung im April suchen und käme auf den 30. ... hätte also noch gar keine Umstellung passieren dürfen.

Stunde "1" sollte auch passen, nachdem ja mit UTC gearbeitet wird. Im Winter haben wir (hier zumindest) Local-time = UTC+1, im Sommer UTC+2
UTC 01:00 Uhr ist also bei uns zur Umstellung 2 Uhr und entsprechend sollte das beim Vergleich begin_ <= now_ zur richtigen Zeit zutreffen.


Seh sonst auch nix...

Code: Alles auswählen

RtcDayOfWeek(pUtc->year, 3, begin) 

Nur ne Vermutung... Sicher das in year das aktuelle Jahr steht und das die Reihenfolge der übergebenen Variablen korrekt ist?

[Mattieu]

Kenn mich mit C# auch nicht wirklich aus.

Aber sehe ich das richtig, dass die Funktion einen Boolschen Wert liefert und per Definition uint8_t zurückgibt? Passt das?

uint8_t RtcIsSummer(tTimeDate* pUtc)
{
[...]
return (begin_ <= now_) && (now_ <= end_);
}

Gruß,
Martin

[Admin]

Ist wird ja auch nicht in C# programmiert, sondern in C. Und c kennt kein bool.