Calendar-Prüfung funktioniert nicht korrekt

[Moch]

Diese Methode soll prüfen, wie viele Sonn- und Samstag in einem Zeitintervall enthalten sind. Leider gibt die Methode im Intervall vom 01.01.2012 - 31.01.2012 genau zwei mal Samstag aus und das wars. Mag da mal jemand rüberschauen und mir sagen, wo der Fehler liegt?

Hier die Methode:

			/**
			 * Korrigiert die Anzahl der Urlaubstage um Wochenendtage
			 * @param anfang Timestamp: Ist das Anfangsdatum des Antrages
			 * @param ende Timestamp: Ist das Enddatum des Antrages
			 * @param origLaenge Integer: Ist die bisher berechnete Anzahl Urlaubstage
			 * @return Integer: Korrigierte Menge an Urlaubstagen;
			 */
			private int correctWeekend(Timestamp anfang, Timestamp ende, int origLaenge) {
				Calendar beg = Calendar.getInstance();
				Calendar run = Calendar.getInstance();
				Calendar end = Calendar.getInstance();
				beg.set(anfang.getYear(), anfang.getMonth(), anfang.getDay());
				run.set(anfang.getYear(), anfang.getMonth(), anfang.getDay());
				end.set(ende.getYear(), ende.getMonth(), ende.getDay());
				
				while (run.before(end)){
					if(run.DAY_OF_WEEK == Calendar.SATURDAY){
						origLaenge--;

                                                System.out.println("DEBUG: SAMSTAG");
					}
					if(run.DAY_OF_WEEK == Calendar.SUNDAY){
						origLaenge--;
                                                System.out.println("DEBUG: SONNTAG");
					}
					run.add(Calendar.DAY_OF_MONTH, 1);
				}
				return origLaenge;
			}

Grüße
Moch

 

[Michael...]

Auch wenn es jetzt zu gehen scheint: Verwendest Du java.sql.Timestamp? Wenn ja, dann würde ich mir noch mal die Methoden getDate() vs. getDay() anschauen.

Unabhängig davon könnte man die enthaltene Anzahl an Samstagen und Sonntagen auch anhand der Anzahl der Tage und dem Start oder Enddatum berechnen, ohne von sich von Tag zu Tag zu hangeln.

 

[Moch]

Das Problem hat sich leider geändert (siehe Edit).
Läuft also nicht korrekt.

Ja, habe eben mal reingeschaut und tatsächlich Day zu Date geändert.

Hm, den Ansatz mit der Gesamtzahl der Tag habe ich übersehen... mal wieder zu kompliziert gedacht. Ist nur dieses Mist-Framework, das wir nutzen müssen schuld dran^^ - da muss man kompliziert denken.

Dennoch würde mich der Fehler in der Geschichte interessieren. Warum gibt das Ding jetzt mit Date eine endlose Zahl am Samstagen aus?

Hier nochmal der aktuelle Quellcode

			/**
			 * Korrigiert die Anzahl der Urlaubstage um Wochenendtage
			 * @param anfang Timestamp: Ist das Anfangsdatum des Antrages
			 * @param ende Timestamp: Ist das Enddatum des Antrages
			 * @param origLaenge Integer: Ist die bisher berechnete Anzahl Urlaubstage
			 * @return Integer: Korrigierte Menge an Urlaubstagen;
			 */
			private int correctWeekend(Timestamp anfang, Timestamp ende, int origLaenge) {
				Calendar beg = Calendar.getInstance();
				Calendar run = Calendar.getInstance();
				Calendar end = Calendar.getInstance();
				beg.set(anfang.getYear(), anfang.getMonth(), anfang.getDate());
				run.set(anfang.getYear(), anfang.getMonth(), anfang.getDate());
				end.set(ende.getYear(), ende.getMonth(), ende.getDate());
				System.out.println("CCD: " + run.toString());
				
				while (run.before(end)){
					if(run.DAY_OF_WEEK == Calendar.SATURDAY){
						origLaenge--;
						System.out.println("SAMSTAG");
					}
					if(run.DAY_OF_WEEK == Calendar.SUNDAY){
						origLaenge--;
						System.out.println("SONNTAG");
					}
					run.add(Calendar.DAY_OF_MONTH, 1);
				}
				return origLaenge;
			}

Ergebnis um 01.01. bis zum 31.01.:
Alle Tage sind Samstage -.-

 

[Gast2]

		System.out.println(Calendar.DAY_OF_WEEK);
		System.out.println(Calendar.SATURDAY);

Fällt dir was auf?

An den aktuellen Tag des Kalenders kommst du mit der Methode get().

 

[Moch]

Ah, vielen Dank.
Nur bleibt mir noch eine Frage über die Methode get().
Da steht, dass ich das entsprechende field angeben muss.
Nun, ich habe rausgefunden, dass get(1) das Jahr zurückgibt. Welches "Field" ist denn konkret für den Wochentag da? - Die Methodenbeschreibung ist da doch etwas sparsam mit Informationen

Edit: Sorry, habs jetzt, denke ich. Das denke ist offenbar so langsam mit einem dauer-wait() belegt -.-

 

[Gast2]

Es gibt in der Klasse ne ganze Menge Konstanten, z.b. auch:

static int DAY_OF_WEEK
Field number for get and set indicating the day of the week.

Du kommst also per

get(Calendar.DAY_OF_WEEK)

an den Tag des Monats.

 

[Moch]

Ich habs mittlerweile auch herausgefunden, als ich genau diesen Teil der JavaDoc gelesen hatte, aber trotzdem vielen, vielen Dank!
Ich weiß echt nicht: Ich sitze seit Freitag fast ununterbrochen daran, dieses Projekt zu einem sinnigen Abschluss zu bekommen. Irgendwie hat der Denkprozess ziemlich nachgelassen.