Übergang von Kalenderwochen ins neue Jahr

[Zed]

Hallo,

also mein Problem. Ich habe eine Liste mit Kalenderwochen von 2007 bis 2010.

Ich hab einen funktion die mir das Datum auf den ersten Tag der KW legt. Dann lese ich die KW und das Jahr aus dem Datum

Hier mal ein Beispiel:

KW-52/2007
KW-01/2007
KW-02/2008

Die erste Kalenderwoche im Jahr 2008 fängt am 31.12.2007 an gehört aber zu 2008. Das Jahr ist 2007 sollte aber 2008 sein.

Kann mir jemand verraten wie ich den übergang sauber hinbekomme und zwar immer. Da muss es doch was geben.

public Date getStartDateOfKW(Date aDate) {
	    Calendar theCalendar = getNewSystemCalendarInstance();
	    theCalendar.setTime(aDate);
	    theCalendar.set(Calendar.DAY_OF_WEEK, theCalendar.getFirstDayOfWeek());
	    return theCalendar.getTime();
	}

public Date rollDateKW(Date aDate, int aCount) {
	    Calendar theCalendar = getNewSystemCalendarInstance();
	    theCalendar.setTime(aDate);
	    theCalendar.add(Calendar.WEEK_OF_YEAR,aCount);
	    return theCalendar.getTime();
	}

So erstelle ich meine Liste

 DateUtilities theDateUtil =  DateUtilities.getInstance();
	Date theCurrentDate = theDateUtil.normalizeDate(new Date());
	Date theBeginnDate = theDateUtil.rollDateYear(theCurrentDate, -2);
	theBeginnDate =  theDateUtil.getStartDateOfKW(theBeginnDate);
	Date theEndDate = theDateUtil.rollDateYear(theCurrentDate, 1);
	theEndDate = theDateUtil.getStartDateOfKW(theEndDate);
	List<Date> theList = new ArrayList<Date>();
	
		
	while(theBeginnDate.before(theEndDate)) {
	    int theKW = theDateUtil.getKW(theBeginnDate);
	    int theYear = theDateUtil.getYear(theBeginnDate);

	    ...

	    theList.add(theBeginnDate);
	    theBeginnDate = theDateUtil.rollDateKW(theBeginnDate, +1);
	}
	
	return theList;

thx 4 help

 

[SlaterB]

nochmal von vorne, nix verstanden,

wenn du vom 1.1.2008 auf den Anfang der Woche gehst, landest du anscheinend beim 31.12.2007 oder nicht?
wenn doch, was soll nun passieren?
daraus 31.12.2008 zu machen scheint wenig sinnvoll,

soll eine Woche nach vorne zum vielleicht 6.1.2008 gesprungen werden?
oder worum gehts?

-----

edit: mal wieder zuspät für die weiteren Antworten eingefügt:
wenn du vom Datum 31.12.2007 aus die Aussage haben willst 'dieses Datum gehört zu einer 2008er-Kalenderwoche'
dann scheint es in der Tat notwendig, nicht einfach die Jahreszahl herauszulösen (2007), sondern eine höhere Methode zu schreiben, die sich wieder genau den Verlauf der Kalenderwochen anschaut

 

[Ebenius]

Er will aus einem beliebigen Datum das Jahr ermitteln zu dem die Kalendarwoche gehört.

 

[Zed]

Das ist genau der Punkt.
Ich möchte das wenn eine KW im Jahr 2007 anfängt aber zum Jahr 2008 gehört. Die Jahreszahl 2008 bekommen und nicht 2007. Ich hab gedacht das der Calender evtl. diese Funktionalität irgenwie besitzt. Aber anscheinend bleibt mir wohl nichts anderes über als das selber händisch zu implementieren.

Edit: Ist es so das ein eine KW zum neuen Jahr gehört wenn der Donnerstag im neuen Jahr liegt?

 

[Ebenius]

So sollte's gehen. Solltest Du aber nochmal genau testen:

private static int getYearOfWeekInYear(Date date) {
  final Calendar cal = Calendar.getInstance();
  cal.setTime(date);
  final int weekInYear = cal.get(Calendar.WEEK_OF_YEAR);
  cal.set(Calendar.DAY_OF_WEEK, cal.getActualMinimum(Calendar.DAY_OF_WEEK));
  final int weekStartsInYear = cal.get(Calendar.YEAR);
  cal.set(Calendar.DAY_OF_WEEK, cal.getActualMaximum(Calendar.DAY_OF_WEEK));
  final int weekEndsInYear = cal.get(Calendar.YEAR);

  final int minWeekInYear = cal.getActualMinimum(Calendar.WEEK_OF_YEAR);
  if (weekStartsInYear != weekEndsInYear && weekInYear == minWeekInYear) {
    return weekEndsInYear;
  } else {
    return weekStartsInYear;
  }
}

Mein Test-Code dazu sieht so aus:

public static void main(String[] args) throws ParseException {
  printTestYearsAndWeeks("2007-12-26");
  printTestYearsAndWeeks("2007-12-27");
  printTestYearsAndWeeks("2007-12-28");
  printTestYearsAndWeeks("2007-12-29");
  printTestYearsAndWeeks("2007-12-30");
  printTestYearsAndWeeks("2007-12-31");
  printTestYearsAndWeeks("2008-01-01");
  printTestYearsAndWeeks("2008-01-02");
  printTestYearsAndWeeks("2008-01-03");
  printTestYearsAndWeeks("2008-01-04");
  printTestYearsAndWeeks("2008-01-05");
  printTestYearsAndWeeks("2008-01-06");
}

private static void printTestYearsAndWeeks(String dateStr)
      throws ParseException {
  final Date date = new SimpleDateFormat("yyyy-MM-dd").parse(dateStr);
  final Calendar cal = Calendar.getInstance();
  cal.setTime(date);
  System.out.println(dateStr
        + ": Year="
        + cal.get(Calendar.YEAR)
        + " WeekOfYear="
        + cal.get(Calendar.WEEK_OF_YEAR)
        + " WeekInYear="
        + getYearOfWeekInYear(date));
}

// EDIT:

2007-12-26: Year=2007 WeekOfYear=52 WeekInYear=2007
2007-12-27: Year=2007 WeekOfYear=52 WeekInYear=2007
2007-12-28: Year=2007 WeekOfYear=52 WeekInYear=2007
2007-12-29: Year=2007 WeekOfYear=52 WeekInYear=2007
2007-12-30: Year=2007 WeekOfYear=1 WeekInYear=2008
2007-12-31: Year=2007 WeekOfYear=1 WeekInYear=2008
2008-01-01: Year=2008 WeekOfYear=1 WeekInYear=2008
2008-01-02: Year=2008 WeekOfYear=1 WeekInYear=2008
2008-01-03: Year=2008 WeekOfYear=1 WeekInYear=2008
2008-01-04: Year=2008 WeekOfYear=1 WeekInYear=2008
2008-01-05: Year=2008 WeekOfYear=1 WeekInYear=2008
2008-01-06: Year=2008 WeekOfYear=2 WeekInYear=2008

Ebenius

 

[SlaterB]

oder so:

    private static int getYearOfWeekInYear2(Date date)
    {
        final Calendar cal = Calendar.getInstance();
        cal.setTime(date);
        boolean strangeWeek = cal.get(Calendar.WEEK_OF_YEAR) < 3 
                              && cal.get(Calendar.MONTH) == 11;
        return cal.get(Calendar.YEAR) + (strangeWeek ? 1 : 0);
    }

 

[Zed]

/**
	 * Setzt den Tag der KW auf Donnerstag.
	 * @param aDate Beliebiges Datum
	 * @return Date Es wird immer ein Datum mit dem Wochentag Donnerstag zurückgegeben
	 * 
	 */
	public Date getThursdayByDateInKW(Date aDate) {
	    Calendar theCalendar = getNewSystemCalendarInstance();
	    theCalendar.setTime(aDate);
	    // Setzt das Datum auf Donnerstag
	    theCalendar.set(Calendar.DAY_OF_WEEK, Calendar.THURSDAY);
	    return theCalendar.getTime();
	}

Diese Methode scheint mein Problem zu lösen. Da ich nur die KW und das Jahr brauche. So ist mir der Wochentag des Datums egal. Ist schon fast zu einfach. Hab mal wieder um 5 Ecken gedacht.

Wie ich sehe denkt ihr auch um paar Ecken . Thx für eure Mühe.

 

[Ebenius]

oder so:

    private static int getYearOfWeekInYear2(Date date)
    {
        final Calendar cal = Calendar.getInstance();
        cal.setTime(date);
        boolean strangeWeek = cal.get(Calendar.WEEK_OF_YEAR) < 3 
                              && cal.get(Calendar.MONTH) == 11;
        return cal.get(Calendar.YEAR) + (strangeWeek ? 1 : 0);
    }

Das ist zwar möglich aber nicht gut! Meine Implementierung verwendet absichtlich so wenig Wissen wie möglich. Dein Beispiel funktioniert eben nicht mit Kalendern die den UNDECIMBER unterstützen.

Der andere Lösungsvorschlag des OP ist auch nicht gut. Was ist mit Kalendern die andere Regeln für den Anfang der Kalenderwoche haben? Der Wochenanfang ist auch im Englischen anders als im Deutschen...

Das Jahr umfasst mindestens 52 durchnummerierte Kalenderwochen (KW), wobei es bei den Wochen-Nummerierungen verschiedene Variationen gibt. Je nach angewandter Regel ist die erste Woche des Jahres
jene, in die der 1. Januar fällt (USA, Excel-Funktion „Kalenderwoche“)
die erste Woche, in die mindestens vier Tage des neuen Jahres fallen (DIN 1355 / ISO 8601)
die erste vollständige Woche des Jahres (selten)

Grüße, Ebenius

 

[Ebenius]

So. Bei mir gab's nen Bug, set geht immer vorwärts. Jetzt stimmt's:

private static int getYearOfWeekInYear(Date date) {
  final Calendar cal = Calendar.getInstance();
  cal.setTime(date);
  final int weekInYear = cal.get(Calendar.WEEK_OF_YEAR);
  final int maxDayOfWeek = cal.getActualMaximum(Calendar.DAY_OF_WEEK);
  cal.set(Calendar.DAY_OF_WEEK, maxDayOfWeek);
  final int weekEndsInYear = cal.get(Calendar.YEAR);
  cal.add(Calendar.DAY_OF_WEEK, cal.getActualMinimum(Calendar.DAY_OF_WEEK)
        - cal.getActualMaximum(Calendar.DAY_OF_WEEK));
  final int weekStartsInYear = cal.get(Calendar.YEAR);

  final int minWeekInYear = cal.getActualMinimum(Calendar.WEEK_OF_YEAR);
  if (weekStartsInYear != weekEndsInYear && weekInYear == minWeekInYear) {
    return weekEndsInYear;
  } else {
    return weekStartsInYear;
  }
}

Ebenius

 

[Ebenius]

Und der dritte Post in Folge, sorry! Aber ist alles etwas länger.

Slater, bei Dir ist auch ein Bug.

2010-12-30: Year=2010 WeekOfYear=52 WeekInYear=2010
2010-12-31: Year=2010 WeekOfYear=52 WeekInYear=2010
2011-01-01: Year=2011 WeekOfYear=52 WeekInYear=2011
2011-01-02: Year=2011 WeekOfYear=52 WeekInYear=2011

Zed, nimm meinen letzten Code, der sollte funktionieren.

Ebenius

 

[SlaterB]

und der Vollständgkeithalber nochmal die elegante Variante:

    private static int getYearOfWeekInYear2(Date date)
    {
        final Calendar cal = Calendar.getInstance();
        cal.setTime(date);
        boolean lowWeekOfYear = cal.get(Calendar.WEEK_OF_YEAR) < 3;
        boolean strangeWeek = lowWeekOfYear == (cal.get(Calendar.MONTH) > 5);
        return cal.get(Calendar.YEAR) + (strangeWeek ? lowWeekOfYear ? 1 : -1 : 0);
    }

 

[Ebenius]

Slater, testest Du Deinen Code manchmal?

2008-01-14: Year=2008 WeekOfYear=3 WeekInYear=2007 Expected=2008
2008-01-15: Year=2008 WeekOfYear=3 WeekInYear=2007 Expected=2008
2008-01-16: Year=2008 WeekOfYear=3 WeekInYear=2007 Expected=2008
2008-01-17: Year=2008 WeekOfYear=3 WeekInYear=2007 Expected=2008
2008-01-18: Year=2008 WeekOfYear=3 WeekInYear=2007 Expected=2008
2008-01-19: Year=2008 WeekOfYear=3 WeekInYear=2007 Expected=2008
2008-01-20: Year=2008 WeekOfYear=3 WeekInYear=2007 Expected=2008
2008-01-21: Year=2008 WeekOfYear=4 WeekInYear=2007 Expected=2008
2008-01-22: Year=2008 WeekOfYear=4 WeekInYear=2007 Expected=2008
2008-01-23: Year=2008 WeekOfYear=4 WeekInYear=2007 Expected=2008
2008-01-24: Year=2008 WeekOfYear=4 WeekInYear=2007 Expected=2008
2008-01-25: Year=2008 WeekOfYear=4 WeekInYear=2007 Expected=2008
2008-01-26: Year=2008 WeekOfYear=4 WeekInYear=2007 Expected=2008
2008-01-27: Year=2008 WeekOfYear=4 WeekInYear=2007 Expected=2008
2008-01-28: Year=2008 WeekOfYear=5 WeekInYear=2007 Expected=2008
2008-01-29: Year=2008 WeekOfYear=5 WeekInYear=2007 Expected=2008
2008-01-30: Year=2008 WeekOfYear=5 WeekInYear=2007 Expected=2008

// EDIT: Und "eleganter" ist der Code nicht. Er unterstützt eben nur den gregorianischen Kalender ordentlich. Ein Kalender der der Meinung ist, die Kalender-Wochen im Mai/Juni zu beginnen, würde damit nicht funktionieren. Ich hab schon Gründe, warum ich die ganzen furchtbaren getActualXXX()-Methoden und so verwende!

BTW: Meinen Code habe ich eben gegen den Gregorianischen Kalender der nächsten 200 Jahre im Deutschen Raum getestet (natürlich nach heutigen Regeln ).

Ebenius

 

[SlaterB]

tja, diesmal gewiss zu wenig, zum Glück passt du auf

    private static int getYearOfWeekInYear2(Date date)
    {
        final Calendar cal = Calendar.getInstance();
        cal.setTime(date);
        int weekOfYear = cal.get(Calendar.WEEK_OF_YEAR);
        int month = cal.get(Calendar.MONTH);

        boolean strangeWeek = (weekOfYear < 3 && month > 7) || (weekOfYear > 45 && month < 3);
        return cal.get(Calendar.YEAR) + (strangeWeek ? weekOfYear < 3 ? 1 : -1 : 0);
    }

 

[Ebenius]

Jetzt besteht's meinen Trivial-Test.

Immernoch offen ist die Frage, warum Dein Ansatz eleganter sein soll. Er ist nicht besser lesbar, und kann mit Kalender-Korrekturen nicht umgehen. Ich zitiere mal wieder:

Jahresbeginn [Bearbeiten]

Parallel zur Kalenderreform, nicht gleichzeitig mit ihr, wurde der Jahresbeginn offiziell auf den 1. Januar verschoben, der sich aufgrund seines Namens (Ianua [lat.] = Tür), der Wintersonnenwende und Christi Geburt als Neujahrstag anbietet; außerdem hatte sich darin die römische Tradition bewahrt. Im Mittelalter hatte das Jahr ansonsten an unterschiedlichen Tagen begonnen, darunter Weihnachten, Ostern und Mariä Verkündigung. Dennoch hatte die gregorianische Reform einen Einfluss darauf, weil in der päpstlichen Bulle eine Liste mit den neuen Namenstagen der Heiligen beilag, die die restlichen Feiertage 1582 bis zum 31. Dezember und die des ganzen folgenden, neu aufgeteilten Jahres (und aller zukünftigen) aufführte. Damit kam es zu Überschneidungen von 11 Tagen (Gregorianischer/Julianischer Kalender) und gleichzeitig einem Jahr (zwischen Neujahr und Ostern): „am 10./21. Februar 1750/1751“.

Ebenius

 

[SlaterB]

du hast Recht, streiche bitte 'elegant' aus unser aller Gedächtnis,
manchmal kann es auch übertreiben..

 

[Zed]

Danke für die Mühe hat euch wohl echt spass gemacht das Problem
Ich nehme auf alle Fälle die funktionen in mein Codebook auf. Schon sehr nice aber der Calender sollte die KW eigentlich von haus aus richtig ausgeben auch mit dem Übergang.

Aber wenn man sicher ist das der Calender GregorianCalendar bleibt ist meine Lösung eigentlich auch nicht verkehrt

 

[Ebenius]

Deine Lösung ist in Amerika falsch! Da liegt der erste Januar nämlich immer in KW 1. Weiter oben ein Auszug aus der Wiki beschreibt das doch.

// EDIT: Der Calendar macht doch alles richtig. Du fragst nach WEEK_IN_YEAR und er gibt die richtige Antwort. Und wenn Du nach YEAR fragst, gibt er auch die richtige Antwort.

 

[Zed]

Mir ist schon bewusst das das es in America falsch ist. Eine andere Möglichkeit wäre anhand der Locale zu bestimmen ob man in USA oder Europa ist. Was ich aber eher als Flickwerk bezeichnen würde.

Aber die Software kommt nicht nach America.

Der Calender macht es schon richtig was ihm aber fehlt ist ne Konstante YEAROFWEEK

 

[Ebenius]

Benutz doch meinen Code. Der macht's richtig, egal in welcher Locale. :roll:

 

[Zed]

Mach ich doch auch