Discussion :

[Amara]

Bonjour,

J'ai un petit souci tout bête avec la classe Calendar, je n'ai rien trouvé dans la doc pour m'expliquer ce qui se passe.

Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
GregorianCalendar c = new GregorianCalendar();
c.set(
	Integer.parseInt("2006"),
	Integer.parseInt("7"),
	Integer.parseInt("1")
);
 
System.out.println(c.get(Calendar.DATE) + "-" + c.get(Calendar.MONTH) + "-" + c.get(Calendar.YEAR));
// on prend la date de la veille
c.add(Calendar.DATE, -1);
System.out.println(c.get(Calendar.DATE) + "-" + c.get(Calendar.MONTH) + "-" + c.get(Calendar.YEAR));

Et voici ce que j'obtiens dans la console :

Code :

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

1
2
1-7-2006
31-6-2006

A ma connaissance (on m'aurait menti ?!?) le 31 juin n'existe pas, quelqu'un pourrait-il éclairer ma lanterne et m'indiquer où je fais l'erreur (au final bien sûr j'aimerais obtenir que la veille du 1er juillet est bien le 30 juin...).

Merci.

[in]

je crois qu'il faut utiliser "lenient" pour que les controles de vallidité soient effectués ...

[Amara]

Merci pour ta réponse, malheureusement en faisant setLenient(true) ou setLenient(false) avant d'enlever le jour le résultat reste inchangé.

[mavina]

Salut,

Comme l'a dit in, il existe une fonction pour obliger le calendar à vérifier si la date que tu lui passes est valide :

When a Calendar is lenient, it accepts a wider range of field values than it produces. For example, a lenient GregorianCalendar interprets MONTH == JANUARY, DAY_OF_MONTH == 32 as February 1. A non-lenient GregorianCalendar throws an exception when given out-of-range field settings. When calendars recompute field values for return by get(), they normalize them. For example, a GregorianCalendar always produces DAY_OF_MONTH values between 1 and the length of the month.

La fonction à utiliser est :

Code :

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

monCalendar.setLenient(false);

voilà voilààà

Fred

[Amara]

Merci Mavina, je reposte donc mon code au cas où j'utiliserais mal setLenient :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
GregorianCalendar c = new GregorianCalendar();
c.set(
	Integer.parseInt("2006"),
	Integer.parseInt("7"),
	Integer.parseInt("1")
);
 
System.out.println(c.get(Calendar.DATE) + "-" + c.get(Calendar.MONTH) + "-" + c.get(Calendar.YEAR));
// on prend la date de la veille
c.setLenient(false);
c.add(Calendar.DATE, -1);
System.out.println(c.get(Calendar.DATE) + "-" + c.get(Calendar.MONTH) + "-" + c.get(Calendar.YEAR));

Et le résultat est toujours :

Code :

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

1
2
1-7-2006
31-6-2006

[in]

je tenterai avant le set pour ma part.

mais bon ça change rien surement ...

[dude]

Les mois sont indexés de 0 à 11....
ton 31-6-2006 correspond donc au 31 juillet 2006.

au cas où, pour te convaincre essaye donc :

Code :

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

1
2
3
4
5
6
7
8
9
 
GregorianCalendar c = new GregorianCalendar();
c.set(
	Integer.parseInt("2006"),
	Integer.parseInt("6"),
	Integer.parseInt("1")
);
 
System.out.println(new DateFormatSymbols().getMonths()[c.get(Calendar.MONTH)]);

[Amara]

En effet, en la plaçant juste après la déclaration de c le résultat est identique. Il doit y avoir un truc qui m'échappe car les exemples de la doc semble montrer que c'est bien la façon de faire, j'ai aussi vu un topic où pour trouver le nombre de jours dans un mois il suffisait d'aller au premier du mois suivant puis d'enlever un jour et ça semblait marcher alors que la méthode utilisée est la même...

[mavina]

Re,

Apres quelques tests :

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
import java.util.*;
 
public class TestCalendar
{
	public static void main(String[] args)
	{
 
		GregorianCalendar c = new GregorianCalendar();
		c.setLenient(false);
		c.set(2006,6,31);
 
		System.out.println(c.get(Calendar.DATE) + "-" + c.get(Calendar.MONTH) + "-" + c.get(Calendar.YEAR));
		// on prend la date de la veille
		//c.setLenient(false);
		c.add(Calendar.DATE, -1);
		System.out.println(c.get(Calendar.DATE) + "-" + c.get(Calendar.MONTH) + "-" + c.get(Calendar.YEAR));
	}
}

Ca me renvoie bien

31-6-2006
30-6-2006
Press any key to continue...

Si je mets 32 au lieu de 31 :

Exception in thread "main" java.lang.IllegalArgumentException
at java.util.GregorianCalendar.computeTime(GregorianCalendar.java:1523)
at java.util.Calendar.updateTime(Calendar.java:1577)
at java.util.Calendar.complete(Calendar.java:1114)
at java.util.Calendar.get(Calendar.java:950)
at TestCalendar.main(TestCalendar.java:12)
Press any key to continue...

Donc il y a un bug dans Calendar

Voilà désolé de n'avoir pu t'aider

Fred

[Amara]

Les mois sont indexés de 0 à 11....
ton 31-6-2006 correspond donc au 31 juillet 2006.

au cas où, pour te convaincre essaye donc :

Code :

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

1
2
3
4
5
6
7
8
9
 
GregorianCalendar c = new GregorianCalendar();
c.set(
	Integer.parseInt("2006"),
	Integer.parseInt("6"),
	Integer.parseInt("1")
);
 
System.out.println(new DateFormatSymbols().getMonths()[c.get(Calendar.MONTH)]);

Arf un grand merci à toi Dude le problème était bien là en effet, tu m'enlèves une belle épine du pied pour une erreur vraiment bête... On va dire que c'est parce qu'on est lundi matin...

Merci Mavina d'avoir pris le temps de faire quelques tests pour m'aider dans mon problème.

Merci à vous tous pour votre aide, bonne journée

[mavina]

woups c'est vrai ca m'est sorti de la tête...
Pardon James Gosling d'avoir douté de ton Calendar

Fred

[sozie9372]

A mon humble avis, la méthode add n'est pas la plus commode dans ce cas, il faudrait que tu utilises une méthode plus adaptée à ton utilisation, je te conseille donc d'utiliser la méthode set(Calendar.DATE, -1) sans setLenient...

PS : on a toujours besoin de son binôme, toujours...
[EDIT] Il semblerait que j'eusse été devancé... En plus ma réponse était pas claire, je m'en remets donc à mon labeur

[Amara]

A mon humble avis, la méthode add n'est pas la plus commode dans ce cas, il faudrait que tu utilises une méthode plus adaptée à ton utilisation, je te conseille donc d'utiliser la méthode set(Calendar.DATE, -1) sans setLenient...

PS : on a toujours besoin de son binôme, toujours...
[EDIT] Il semblerait que j'eusse été devancé... En plus ma réponse était pas claire, je m'en remets donc à mon labeur

Il en va sans peine cher partenaire, merci d'avoir consacré quelques temps à mes tracas du lundi matin.