GregorianCalendar (vor und zurück)

[Guest]

grüß euch,

viel code für winziges und sehr leichtes problem.
habe mir folgende methoden zurecht gelegt:

  public void setVorTagesDatum(GregorianCalendar calendar, DateFormat af) {
    calendar.roll(calendar.DAY_OF_MONTH, -1);
    Date date = calendar.getTime();
    vorTagesDatum = af.format(date) ;
  }
  public String getVorTagesDatum() {
    return vorTagesDatum;
  }

  public void setNaechsterTagDatum(GregorianCalendar calendar, DateFormat af) {
    calendar.roll(calendar.DAY_OF_MONTH, +1);
    Date date = calendar.getTime();
    naechsterTagDatum = af.format(date) ;
  }
  public String getNaechsterTagDatum() {
    return naechsterTagDatum;
  }

die methoden werden aus dem konstruktor der reihe nach aufgerufen:

    public SystemZeit() {
    calendar = new GregorianCalendar();
    DateFormat af = new SimpleDateFormat("yyyy-MM-dd");   
    setVorTagesDatum(calendar, af);
    setNaechsterTagDatum(calendar, af);

wenn ich nun an irgendeiner stelle des programms das datum des gestrigen tages aufrufe, dann ist alles ok. wenn ich aber den des morgigen tages wissen möchte, dann bekomme ich den heutigen. offensichtlich merkt sich der GregorianCalendar mit der methode .roll an welchem tag er nun gerade ist. und der gestrige + einen tag mehr ergibt eben den heutigen. das würde aber heissen, dass ich bei der methode setNaechsterTagDatum 2 Tage nach vor rollen muss. das erscheint mir irgendwie falsch.
kann man das schlauer machen? bzw. würdet ihr einfach eine 2 setzen?

danke juergen

 

[L-ectron-X]

Ich habe mir den Code jetzt nicht angesehen, aber das könnte daran liegen, dass Calendar alle Tage, Monate Wochen etc. bei 0 beginnt zu zählen, nicht wie der Mensch bei 1. Ich finde das auch irgendwie umständlich.

 

[SebiB90]

ist normal das der heutige tag heraus kommt weil du änderst da ja beim gleichem object die tage also einmal vor und wieder zurück, wie du ja schon gesagt hast. mein tipp: in der methode den calendar clonen und dann erst roll methode benutzen also so:

public void setVorTagesDatum(GregorianCalendar calendar, DateFormat af) { 
    Calendar cal = (Calenadr) calendar.clone();
    cal.roll(calendar.DAY_OF_MONTH, -1); 
    Date date = cal.getTime(); 
    vorTagesDatum = af.format(date) ; 
  }

 

[Guest]

also das mit dem clonen hat leider nicht funktioniert. ist ja auch verständlich, da ja auch beim geklonten objekt die referenzen die gleichen bleiben (glaub ich). also zeigt er auch beim geklinten objekt auf den gestrigen tag.

übringens mit 0 und 1 hat das nichts zu tun. trotzdem danke.

 

[SebiB90]

das ist es ja beim clonen wird ein neues object erzeugt und es wird nicht die referenz auf das alte object weiter gegeben
das müsste eigentlich funktionieren. zeig nochmal den ganzen code.

 

[Guest]

import java.util.GregorianCalendar;
import java.util.Calendar;
import java.text.DateFormat;
import java.text.SimpleDateFormat;
import java.util.Date;

public class SystemZeit {
  public String Datum = null;
  public String Zeit = null;
  public String vorTagesDatum = null;
  public String naechsterTagDatum = null;
  GregorianCalendar calendar;

  public SystemZeit() {
    calendar = new GregorianCalendar();
    DateFormat af = new SimpleDateFormat("yyyy-MM-dd");
    DateFormat df = new SimpleDateFormat("HH-mm-ss");
    setAktuelleZeit(df);
    setAktuellesDatum(calendar, af);
    setVorTagesDatum(calendar, af);
    setNaechsterTagDatum(calendar, af);
  }


  public void setAktuelleZeit(DateFormat df) {
    Date date2 = calendar.getTime();
    Zeit = df.format(date2);
  }
  public String getAktuelleZeit() {
    return Zeit;
  }

  public void setAktuellesDatum(GregorianCalendar calendar, DateFormat af) {
    Date date = calendar.getTime();
    Datum = af.format(date);
  }
  public String getAktuellesDatum() {
    return Datum;
  }

  public void setVorTagesDatum(GregorianCalendar calendar, DateFormat af) {
    calendar.roll(calendar.DAY_OF_MONTH, -1);
    Date date = calendar.getTime();
    vorTagesDatum = af.format(date) ;
  }
  public String getVorTagesDatum() {
    return vorTagesDatum;
  }

  public void setNaechsterTagDatum(GregorianCalendar calendar, DateFormat af) {
    Calendar cal = (Calendar) calendar.clone();
    calendar.roll(cal.DAY_OF_MONTH, +1);
    Date date = calendar.getTime();
    naechsterTagDatum = af.format(date) ;
  }
  public String getNaechsterTagDatum() {
    return naechsterTagDatum;
  }
}

 

[SebiB90]

1. bei setVorTagesDatum() muss du das calendar object auch clonen
2. du musst dann auch das geclonte object benutzen!

 

[Guest]

danke das tut. nun habe ich zum ersten mal ein objekt geklont. nicht schlecht herr specht.