Problème avec le type GregorianCalendar

**faLco3** · 28/10/2008, 19h49

Bonjour,
par avance je suis désolé si le sujet a déjà été posté mais je n'ai pas trouvé de réponse à mon problème en cherchant.
Voilà : j'ai déclaré une variable date_naissance de type GregorianCalendar dans une classe "Adherent".
J'ai vu qu'on pouvait construire un objet de ce type avec 3 paramètres :

GregorianCalendar(int year, int month, int date)
Constructs a GregorianCalendar with the given date set in the default time zone with the default locale.

J'ai donc 3 paramètres dans mon constructeur qui me permettent de construire la date de naissance. Ensuite, je surcharge toString pour afficher un adherent comme je veux. Seulement, l'affichage foire complètement.
Est-ce qu'il y a quelque chose qui je fais pas bien ? Peut-être que j'utilise mal le type GregorianCalendar ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
package appliadac;
 
import java.util.ArrayList;
import java.util.GregorianCalendar;
 
public class Adherent {
	private int numero;
	private String nom;
	private String prenom;
	private String ville;
	private GregorianCalendar date_naissance;
	private static ArrayList<Adherent> TousLesAdherents = new ArrayList<Adherent> ();
 
	public Adherent(int numeroN, String nomN, String prenomP, String villeV, int yearY, int monthM, int dayD) {
		numero = numeroN;
		nom = nomN;
		prenom = prenomP;
		ville = villeV;
		date_naissance = new GregorianCalendar(yearY, monthM, dayD);
		TousLesAdherents.add(this);
	}
 
 
	public static void affichertout() {
		for(Adherent A : TousLesAdherents) {
			System.out.println(A);
		}
	}
 
	public String toString() {
		return "N°" + numero + " : " + nom + " " + prenom + " residant a " + ville + " (ne le " + date_naissance + ")";
	}
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
package appliadac;
 
public class Application {
 
	public static void main(String[] args) {
		new Adherent(1, "dupond", "bob", "paris", 89, 1, 03);
		new Adherent(2, "durant", "alfred", "versailles", 90, 7, 21);
		Adherent.affichertout();
	}
}

Voici ce que donne l'exécution :

N°1 : dupond bob residant a paris (ne le java.util.GregorianCalendar[time=?,areFieldsSet=false,areAllFieldsSet=false,lenient=true,zone=sun.util.calendar.ZoneInfo[id="Europe/Paris",offset=3600000,dstSavings=3600000,useDaylight=true,transitions=184,lastRule=java.util.SimpleTimeZone[id=Europe/Paris,offset=3600000,dstSavings=3600000,useDaylight=true,startYear=0,startMode=2,startMonth=2,startDay=-1,startDayOfWeek=1,startTime=3600000,startTimeMode=2,endMode=2,endMonth=9,endDay=-1,endDayOfWeek=1,endTime=3600000,endTimeMode=2]],firstDayOfWeek=2,minimalDaysInFirstWeek=4,ERA=?,YEAR=89,MONTH=1,WEEK_OF_YEAR=?,WEEK_OF_MONTH=?,DAY_OF_MONTH=3,DAY_OF_YEAR=?,DAY_OF_WEEK=?,DAY_OF_WEEK_IN_MONTH=?,AM_PM=0,HOUR=0,HOUR_OF_DAY=0,MINUTE=0,SECOND=0,MILLISECOND=?,ZONE_OFFSET=?,DST_OFFSET=?])
N°2 : durant alfred residant a versailles (ne le java.util.GregorianCalendar[time=?,areFieldsSet=false,areAllFieldsSet=false,lenient=true,zone=sun.util.calendar.ZoneInfo[id="Europe/Paris",offset=3600000,dstSavings=3600000,useDaylight=true,transitions=184,lastRule=java.util.SimpleTimeZone[id=Europe/Paris,offset=3600000,dstSavings=3600000,useDaylight=true,startYear=0,startMode=2,startMonth=2,startDay=-1,startDayOfWeek=1,startTime=3600000,startTimeMode=2,endMode=2,endMonth=9,endDay=-1,endDayOfWeek=1,endTime=3600000,endTimeMode=2]],firstDayOfWeek=2,minimalDaysInFirstWeek=4,ERA=?,YEAR=90,MONTH=7,WEEK_OF_YEAR=?,WEEK_OF_MONTH=?,DAY_OF_MONTH=21,DAY_OF_YEAR=?,DAY_OF_WEEK=?,DAY_OF_WEEK_IN_MONTH=?,AM_PM=0,HOUR=0,HOUR_OF_DAY=0,MINUTE=0,SECOND=0,MILLISECOND=?,ZONE_OFFSET=?,DST_OFFSET=?])

**Deadpool** · 29/10/2008, 00h13

Salut.

C'est pas que ça foire complètement, c'est juste que tu affiches des variables GregorianCalendar et que la méthode toString de GregorianCalendar est très bavarde.

Au cas où, pour un affichage plus propre, dans ta méthode toString(); tu peux tranformer ton objet GregorianCalendar en Date et utiliser un SimpleDateFormat pour formater son affichage.

Exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
 
package appliadac;
 
import java.text.SimpleDateFormat;
import java.util.ArrayList;
import java.util.GregorianCalendar;
 
public class Adherent {
	private int numero;
	private String nom;
	private String prenom;
	private String ville;
	private GregorianCalendar date_naissance;
	private static ArrayList<Adherent> TousLesAdherents = new ArrayList<Adherent> ();
 
//le SimpleDateFormat utilisé pour l'affichage des dates.
	private static final SimpleDateFormat formateur = new SimpleDateFormat("dd/MM/yyyy");
 
	public Adherent(int numeroN, String nomN, String prenomP, String villeV, int yearY, int monthM, int dayD) {
		numero = numeroN;
		nom = nomN;
		prenom = prenomP;
		ville = villeV;
		date_naissance = new GregorianCalendar(yearY, monthM, dayD);
		TousLesAdherents.add(this);
	}
 
 
	public static void affichertout() {
		for(Adherent A : TousLesAdherents) {
			System.out.println(A);
		}
	}
 
	public String toString() {
		return "N°" + numero + " : " + nom + " " + prenom + " residant a " + ville + " (ne le " + formateur.format(date_naissance.getTime()) + ")";
               //utilisation du formateur de date
	}
 
 
}

Par contre, fais gaffe avec les GregorianCalendar, pour une raison qui me dépasse, les mois sont comptés de 0 (Janvier) à 11 (Décembre). Il est préférable d'utiliser les constantes de mois définies dans GregorianCalendar pour ne pas se tromper comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
public static void main(String[] args) {
	new Adherent(1, "dupond", "bob", "paris", 89, GregorianCalendar.JANUARY, 03);
	new Adherent(2, "durant", "alfred", "versailles", 90, GregorianCalendar.JULY, 21);
//utilisation des constantes GregorianCalendar
	Adherent.affichertout();
 
}

Ou alors tu retranches 1 au mois dans ta classe.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
public Adherent(int numeroN, String nomN, String prenomP, String villeV, int yearY, int monthM, int dayD) {
		numero = numeroN;
		nom = nomN;
		prenom = prenomP;
		ville = villeV;
		date_naissance = new GregorianCalendar(yearY, monthM-1, dayD);//on retranche 1 au mois
		TousLesAdherents.add(this);
	}

Enfin, si je peux me permettre, ta conception est moyenne car ta classe représente en même temps un adhérent et un conteneur d'adhérent.

Il faudrait mieux séparer ces 2 aspects à mon avis.

Edit : J'ajouterai que dans la prochaine version de Java, il y aura une nouvelle API pour les dates bien plus claire et pratique qui fera son apparition.

Cette API, Joda-Time, est déjà disponible là, si tu veux regarder.

**faLco3** · 29/10/2008, 01h05

Wouah, j'en attendais pas tant !
Merci beaucoup pour cette réponse très complète et très instructive ! Ca va beaucoup m'aider. Je vais jeter un coup d'oeil à Joda-Time, ça a l'air intéressant

Sinon, pour ce qui est de ma classe "Adherent" qui est en même temps un conteneur de plusieurs objets de cette même classe, moi non plus je ne trouve pas ça top mais c'est un projet que je réalise pour mes études et le prof veut qu'on fasse comme ça pour le moment. On apprendra peut-être à améliorer ça plus tard...

En tout cas, merci encore pour tout, c'est très sympa.

Bonne continuation,
Cordialement

**benben02** · 29/10/2008, 10h30

Envoyé par faLco3

e prof veut qu'on fasse comme ça pour le moment. On apprendra peut-être à améliorer ça plus tard...

ce "peut-être" depend-il du budget de l'éducation nationale?

**faLco3** · 31/10/2008, 02h12

lol, non je ne pense pas, cela dépend plutôt du rythme relativement lent imposé par le besoin que tous les étudiants comprennent ce qu'on fait avant de continuer !

Sinon, pour en revenir à la remarque initiale, en continuant mon projet j'ai réfléchi un peu et je me suis dit que finalement c'était peut-être mieux de mettre des listes static pour "sauvegarder" tous les objets d'une class. En effet, j'en ai discuté avec un ami et l'avantage de ce procédé est qu'il permet d'accéder très simplement à "l'ensemble" des objets de la class, puisque la liste est privée. Si on mettait des objets "Liste" pour chaque class, on aurait des tas de méthodes supplémentaires et un nombre de class doublé. Ça semble plus contraignant, alors même si d'un point de vu conceptuel c'est pas très logique que les objets d'un type aient une liste commune qui les contient tous, c'est plus facile à gérer en codant.
Non ?

**benben02** · 31/10/2008, 12h26

si j'ai bien compris

tu veux crées des objets "Liste" pour regrouper tes objets.

Quand tu utilises java c'est pour profiter de la puissance du framework qui vient avec.

Y a tout un framework "Collections" qui serrent à manipuler un ensemble d'objets (de même type-utilises les génériques à partir de java 5

- ou pas).

regarde du côtés du java.util.*

Problème avec le type GregorianCalendar

Collection et Stream Java

Vue hybride

Discussions similaires

Partager

Partager