Jahre mit 53 Wochen

[DeddyH]

Ja, mir ging es ja nur darum, dass man die Konstanten für die Wochentage nicht unbedingt neu definieren muss (aber natürlich kann), da diese doch bereits in DateUtils definiert sind. Aber Heiko hat schon recht, mit sprechenden Konstanten ist der Code für Außenstehende einfacher zu verstehen.

[hoika]

Hallo,

jepp, das meinte ich.
Der Anfänger weiss z.B. nicht, dass die Wochentage
mit Sonntag als 0 definiert seind.


*erstmal grillen*


Heiko

[Wolfgang Mix]

@hoika

Zitat aus Post #1:

Zitat:

Jahre mit 53 Wochen

gibt es immer dann, wenn der 1.1. oder der 31.12. des Jahres auf einen Donnerstag fallen.

weiteres Zitat aus Post #1:

Zitat:

Result := (DayOfTheWeek(EncodeDate(y, 1 , 1)) = 4) (DayOfTheWeek(date2)=4) then ...

Wer ein bißchen den Quelltext liest und Finger zum Zählen hat,
der kommt sehr schnell darauf, daß die Zählung bei WeekOfTheYear
bei Montag = 1 beginnt

Gruß und guten Appetit

Wolfgang

[DeddyH]

Hallo Wolfgang,

reg Dich doch nicht auf, es sollte ja keine Kritik sein. Nur wenn diese Funktion irgendwann einmal ohne den zugehörigen Kommentar zitiert wird, ist es doch zugegebenermaßen einfacher zu verstehen, dass der Donnerstag der Casus Knacktus ist, wenn eine sprechende Konstante verwendet wird, meinst Du nicht auch? Es macht den Code ja nicht schlechter, sondern nur schneller nachvollziehbar.

[Wolfgang Mix]

Ich rege mich nicht auf.
Wollte nur klarmachen, daß DayOfTheWeek definitiv mit Montag = 1 startet.
War wohl mit DayOfWeek verwechselt worden.

Gruß

Wolfgang

[DeddyH]

Gerade wegen dieser Verwechslungsgefahr bietet es sich doch an, die 4 gegen DayThursday oder eine selbst definierte Konstante auszutauschen, oder?

[Wolfgang Mix]

Einverstanden
Gruß

Wolfgang

[DeddyH]

Schön, dass wir einmal drüber gesprochen haben