Probleme mit Calendar

[mjest1c]

Hallo,

ich habe ein kleines Problem mit den Datums berechnungen und muss gestehen das ich bei dem Calendar Objekt (Java Doc) nocht nicht ganz durchblicke...

Bei dem Projekt geht es um Lotto Zahlen und ich möchte nur eine Methode die herausfindet ob an dem Tag eine Ziehung ist oder nicht.
Leider gibt sie alle möglichen Daten zurück jedoch nicht die Tage an denen eine Ziehung ist.

Ein teil der Ausgabe:
14.2.1991
15.2.1991
16.2.1991


17.2.1991
18.2.1991
19.2.1991
20.2.1991
21.2.1991
22.2.1991
23.2.1991


24.2.1991
25.2.1991
26.2.1991
27.2.1991
28.2.1991
29.2.1991
30.2.1991


31.2.1991
1.3.1991
2.3.1991
3.3.1991
4.3.1991
5.3.1991
6.3.1991


7.3.1991
8.3.1991
9.3.1991
10.3.1991
11.3.1991
12.3.1991
13.3.1991


14.3.1991
15.3.1991
16.3.1991
17.3.1991
18.3.1991
19.3.1991
20.3.1991


21.3.1991
22.3.1991
23.3.1991
24.3.1991
25.3.1991
26.3.1991
27.3.1991


28.3.1991
29.3.1991
30.3.1991
1.4.1991
2.4.1991
3.4.1991
4.4.1991


5.4.1991
6.4.1991
7.4.1991
8.4.1991
9.4.1991
10.4.1991
11.4.1991


12.4.1991
13.4.1991
14.4.1991
15.4.1991
16.4.1991
17.4.1991
18.4.1991


19.4.1991
20.4.1991
21.4.1991
22.4.1991
23.4.1991
24.4.1991
25.4.1991

Methoden aufruf:

		do
		{
			System.out.print("\n\n");
		} while(this.calc_dates.next());

Methode die Probleme macht:

	/*
	 * Move the now attribute 
	 * returns false otherwise
	 */
	public boolean next()
	{
		Calendar present = Calendar.getInstance();
		for (int i = 0; i <= 6; i++)
		{
			System.out.println(this.now.get(Calendar.DATE)+"."+this.now.get(Calendar.MONTH)+"."+this.now.get(Calendar.YEAR));
			this.now.add(Calendar.DATE, 1);

			if (this.now.after(present))
			{
				return false;
			}			
			if (this.now.get(Calendar.DAY_OF_WEEK) == this.now.SUNDAY)
			{
				return true;
			}
			if (this.mi && this.now.get(Calendar.DAY_OF_WEEK) == this.now.WEDNESDAY)
			{
				return true;
			}
			//System.out.println(this.now.DAY_OF_WEEK);
			//System.out.println(this.now.SUNDAY);
		}
		return false;
	}

Die komplette Klasse:

package lotto;

import java.util.Calendar;
import java.util.GregorianCalendar;

public class calc_dates
{
	private Calendar now = Calendar.getInstance();

	public boolean mi = false;
	public boolean zz = false;
	public boolean sz = false;
	
	public calc_dates()
	{
		Calendar c = GregorianCalendar.getInstance();
		c.set(1955, 10, 9);
		this.set_date(c);
	}	
	public calc_dates(Calendar now)
	{
		this.set_date(now);
	}
	public calc_dates(int year, int month, int day)
	{
		Calendar c = GregorianCalendar.getInstance();
		c.set(year, month, day);
		this.set_date(c);
	}
	
	public void set_date(Calendar now)
	{
		this.now = now;
		Calendar compare_date = Calendar.getInstance();
		
		compare_date.set(1956, 06, 17);		
		if (this.now.after(compare_date) || this.now.equals(compare_date))
		{
			this.zz = true;
		}
		
		compare_date.set(1986, 04, 19);
		if (this.now.after(compare_date) || this.now.equals(compare_date))
		{
			System.out.println("mi is true jetzt!!!!!!!!!!!!");
			this.mi = true;
		}
		
		compare_date.set(1991, 12, 7);
		if (this.now.after(compare_date) || this.now.equals(compare_date))
		{
			this.sz = true;
		}	
	}
	
	/*
	 * Move the now attribute 
	 * returns false otherwise
	 */
	public boolean next()
	{
		Calendar present = Calendar.getInstance();
		for (int i = 0; i <= 6; i++)
		{
			System.out.println(this.now.get(Calendar.DATE)+"."+this.now.get(Calendar.MONTH)+"."+this.now.get(Calendar.YEAR));
			this.now.add(Calendar.DATE, 1);

			if (this.now.after(present))
			{
				return false;
			}			
			if (this.now.get(Calendar.DAY_OF_WEEK) == this.now.SUNDAY)
			{
				return true;
			}
			if (this.mi && this.now.get(Calendar.DAY_OF_WEEK) == this.now.WEDNESDAY)
			{
				return true;
			}
			//System.out.println(this.now.DAY_OF_WEEK);
			//System.out.println(this.now.SUNDAY);
		}
		return false;
	}
	
	public Calendar get_date()
	{
		return this.now;
	}
	
	public String get_date(boolean us_format)
	{
		if (us_format)
		{
			return this.now.get(Calendar.YEAR)+"-"+this.now.get(Calendar.MONTH)+"-"+this.now.get(Calendar.DATE);
		}
		else
		{
			return this.now.get(Calendar.DATE)+"."+this.now.get(Calendar.MONTH)+"."+this.now.get(Calendar.YEAR);
		}
	}	
}

Was mache ich falsch??

 

[SlaterB]

next() durchläuft eine Woche, gibt alles aus, am Sonntag wird true zurückgegeben,
die aufrufende Schleife gibt dann Zeilenumbrüche aus und es geht von vorne los

das ist was passiert, an sich kein Problem, was du stattdessen haben willst ist unbekannt

---

schreibe Klassen groß, ohne _,
schreibe Calendar.SUNDAY statt this.now.SUNDAY

 

[mjest1c]

Ich hätte gerne das er an jedem Sonntag die Zeilenumbüche ausgiebt, immer dann wenn eine Ziehung ist eben.
Wenn man mal im Kalender schaut sieht man das der 16.2.1991, 23.2.1991, ... keine Sonntage sind.

 

[SlaterB]

schau dir doch an, wie die Schleife und next() aufgebaut sind,
der Sonntag ist der 17.2., dann wird true zurückgegeben, \n\n kommt
und erst im nächsten next()-Durchlauf wird dann wirklich 17.2. per System.out.println() ausgegeben

und im Moment ist an jedem Sonntag Abbruch, unabhängig von irgendwelchen Bedingungen,
wenn es nur bestimmte Sonntage sein sollen, dann bau doch entsprechenden Code ein,
beim Mittwoch hast du ja die 'mi'-Variable, auch wenn ich nicht wirklich deren Grund verstehe

wie du siehst erkläre ich dir nur wiederholt, was dein Programm macht, obwohl du das auch selber herausfinden könntest,
ich werde da nix zum Lotto neu einbauen, selbst wenn ich wüßte worum es geht

 

[Andi_CH]

Ich (und auch der Calendar) haben keine Ahnung von Lotto. Schon gar nicht davon an welchen Tagen Ziehugnen sind - ausserdem gibt es hunderte von Lottos.

Formuliere doch bitte deine Requirements päziser.
Welche Tage sollen nun ausgegeben werden?

Alle Sonntage?
Was soll an den anderen Tagen ausgegeben werden?

Du wirst sehen, wenn du es präzise genug formuliert hast und wir es verstehen kommst du vermutlich schon selbst auf die richtig Idee.

 

[mjest1c]

Erst einmal Danke für eure Zeit und Mühe!
Ich muss zugeben ich habe den Thread etwas hoppla hopp gestartet.
Ich versuche nun etwas Präziser zu werden!

Ich möchte das das Attribut now (Calendar Objekt) immer entweder auf Mittwoch oder Sonntag "gestellt" ist.
Die Methode naechsteZiehung() soll dies bewerkställigen. Diese Methode geht jeden Tag durch und überprüft ob der Wochentag (DAY_OF_WEEK) WEDNESDAY oder SUNDAY ist.
Das now Attribut springt im Prinzip zum nächsten MI oder Sonntag.

Hier mal ein kompakterer Code meines Versuchs:

import java.util.Calendar;

public class Main 
{
	private static Calendar now = Calendar.getInstance();
	
	public static void main(String[] args) 
	{
		// Erste Ziehung war 9.10.1955
		now.set(1955, 10, 9);
		do
		{
			// Mach irgendwas mit this.now
			
		} while(naechsteZiehung());
	}
	
	public static boolean naechsteZiehung()
	{
		Calendar present = Calendar.getInstance();
		// Maximal 4 Tage MI bis SO
		for (int i = 1; i <= 4; i++)
		{
			// Now einen Tag hochzaehlen
			now.add(Calendar.DATE, 1);

			// Nur Tage aus der vergangenheit
			if (now.after(present))
			{
				System.out.println("Alle Tage bis heute wurden durchlaufen!");
				return false;
			}
			
			if (now.get(Calendar.DAY_OF_WEEK) == Calendar.SUNDAY)
			{
				System.out.println(now.get(Calendar.DATE)+"."+now.get(Calendar.MONTH)+"."+now.get(Calendar.YEAR));
				return true;
			}
			if (now.get(Calendar.DAY_OF_WEEK) == Calendar.WEDNESDAY)
			{
				System.out.println(now.get(Calendar.DATE)+"."+now.get(Calendar.MONTH)+"."+now.get(Calendar.YEAR));
				return true;
			}
		}
		return false;
	}
}

Edit:

Fast hätte ich vergessen die Ausgabe der Konsole zu Posten:

13.10.1955: Donnerstag
16.10.1955: Sonntag
20.10.1955: Donnerstag
23.10.1955: Sonntag
....

 

[SlaterB]

hast du dir die ganze Liste mal angeschaut?
es gibt dort Monate 0, 1, 2, .. 11,
0 ist Januar, 1 ist Februar, .., 10 ist November, 11 ist Dezember aus Java-Calendar-Sicht

 

[Niki]

vielleicht ist ja das deine Lösung

	public static Calendar getNextWednesdayOrSunday(){
		Calendar cal = Calendar.getInstance();
		while(cal.get(Calendar.DAY_OF_WEEK) != Calendar.WEDNESDAY &&
			cal.get(Calendar.DAY_OF_WEEK) != Calendar.SUNDAY){
			cal.add(Calendar.DAY_OF_MONTH, 1);
		}
		return cal;

	}

 

[mjest1c]

hast du dir die ganze Liste mal angeschaut?
es gibt dort Monate 0, 1, 2, .. 11,
0 ist Januar, 1 ist Februar, .., 10 ist November, 11 ist Dezember aus Java-Calendar-Sicht

Oh verdammt, hätte ich mir echt mal genauer anschauen sollen ...
Gibt es da eine Funktion die mir das abnimmt oder muss ich das selber abfangen?

vielleicht ist ja das deine Lösung
public static Calendar getNextWednesdayOrSunday(){
Calendar cal = Calendar.getInstance();
while(cal.get(Calendar.DAY_OF_WEEK) != Calendar.WEDNESDAY &&
cal.get(Calendar.DAY_OF_WEEK) != Calendar.SUNDAY){
cal.add(Calendar.DAY_OF_MONTH, 1);
}
return cal;

}

Danke, sieht echt viel übersichtlicher aus als meine Methode!

 

[SlaterB]

mit SimpleDateFormat das Date calendar.getTime() formatieren