Discussion :

[Romz_Java]

Bonjour à tous,

je cherche une méthode simple pour comparer deux dates en ignorant l'heure.
J'ai une méthode existante qui utilise des méthodes dépréciées, que je voudrais réécrire:

Code :

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

1
2
3
4
5
6
7
8
9
private boolean dateBeforeOrEqual(Date date, Date dateRef) {
		GregorianCalendar dateCourante = new GregorianCalendar(date.getYear()+1900, date.getMonth(), date.getDate());
		GregorianCalendar dateReference = new GregorianCalendar(dateRef.getYear()+1900, dateRef.getMonth(), dateRef.getDate());
 
		if (dateCourante.before(dateReference) || dateCourante.equals(dateReference))
			return true;
		else
			return false;
	}

La solution que j'ai trouvée pour le moment est celle là:

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
private boolean dateBeforeOrEqual2(Date date, Date dateRef) {
		GregorianCalendar dateCourante = new GregorianCalendar();
		dateCourante.setTime(date);
		dateCourante.set(Calendar.HOUR_OF_DAY, 0);
		dateCourante.set(Calendar.MINUTE, 0);
		dateCourante.set(Calendar.SECOND, 0);
		dateCourante.set(Calendar.MILLISECOND, 0);
 
		GregorianCalendar dateReference = new GregorianCalendar();
		dateReference.setTime(dateRef);
		dateReference.set(Calendar.HOUR_OF_DAY, 0);
		dateReference.set(Calendar.MINUTE, 0);
		dateReference.set(Calendar.SECOND, 0);
		dateReference.set(Calendar.MILLISECOND, 0);
 
		if (dateCourante.before(dateReference) || dateCourante.equals(dateReference))
			return true;
		else
			return false;
	}

C'est un peu lourd...
Est-ce que vous voyez un meilleur moyen de réécrire cette méthode sans utiliser les méthodes dépréciées de la classe Date?

[aperrin]

yoda time

[adiGuba]

Salut,

yoda time

Je pense que tu veux parler de Joda Time


C'est en effet une bonne idée si on a de gros besoin de gestion des heures/dates...

Mais s'il s'agit d'un besoin épisodique, ajouter une librairie entière peut s'avérer peut pratique... surtout que je trouve la seconde version assez propre, même si j'aurais privilégier Calender.getInstance() et clear() :

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
	private boolean dateBeforeOrEqual(Date date, Date dateRef) {
		Calendar dateCourante = Calendar.getInstance();
		dateCourante.setTime(date);
		dateCourante.clear(Calendar.HOUR_OF_DAY);
		dateCourante.clear(Calendar.MINUTE);
		dateCourante.clear(Calendar.SECOND);
		dateCourante.clear(Calendar.MILLISECOND);
 
		Calendar dateReference = Calendar.getInstance();
		dateReference.setTime(dateRef);
		dateReference.clear(Calendar.HOUR_OF_DAY);
		dateReference.clear(Calendar.MINUTE);
		dateReference.clear(Calendar.SECOND);
		dateReference.clear(Calendar.MILLISECOND);
 
		//return (dateCourante.before(dateReference) || dateCourante.equals(dateReference))
		return dateCourante.compareTo(dateReference) <= 0;
	}

Qu'as-tu à reprocher à cette méthode, si ce n'est sa taille ?

a++

[lunatix]

oui, la deuxième méthode est pas mal, mais lourde.

perso, a chaque fois que je dois manipuler des dates, je m'inclus un petit import (une classe) qui me permet d'écrire des choses comme

Code :

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

Date date = now().clearTime().toDate();

ca me simplifie grandement la vie !

d'ailleurs, j'étais passé a coté de la méthode clear() pour mon clearTime()
je vais améliorer ça (merci adiGuba)

[Romz_Java]

Merci adiGuba,

c'est plus propre comme tu l'écris.

Effectivement c'est la taille de la méthode qui me gêne un peu vu qu'il faut une bonne dizaine de lignes pour comparer deux pauvres dates

Aucun moyen d'optimiser ça?

[Romz_Java]

oui, la deuxième méthode est pas mal, mais lourde.

perso, a chaque fois que je dois manipuler des dates, je m'inclus un petit import (une classe) qui me permet d'écrire des choses comme

Code :

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

Date date = now().clearTime().toDate();

ca me simplifie grandement la vie !

d'ailleurs, j'étais passé a coté de la méthode clear() pour mon clearTime()
je vais améliorer ça (merci adiGuba)

Donc tu as ta propre classe Date avec un clearTime() c'est ça?

[lunatix]

non, justement, ce qui me gene dans joda, c'est le remplacement de Date.
j'ai un gros builder qui wrappe Date et Calendar. Un genre de classe Util.

http://code.google.com/p/datedsl/ si tu veux voir. mais cette version est incomplète et pas finie. mais ca donne une bonne idée

[Romz_Java]

ok merci pour le lien

ça fait quoi l'import static exactement?

Code :

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

import static com.unimondes.dsl.DateDsl.*;

[Deadpool]

ok merci pour le lien

ça fait quoi l'import static exactement?

Code :

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

import static com.unimondes.dsl.DateDsl.*;

http://java.developpez.com/faq/java/..._import_static

Dispo seulement à partir de Java 5.

[lunatix]

ouép.

en clair ca permet d'ecrire

Code :

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

1
2
3
4
Date date = date().withYear(2008)
                        .withMonth(Calendar.JANUARY)
                        .withDayOfMonth(15)
                        .toDate();

plutôt que

Code :

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

1
2
3
4
Date date = DateDsl.date().withYear(2008)
                        .withMonth(Calendar.JANUARY)
                        .withDayOfMonth(15)
                        .toDate();

[Romz_Java]

ok, si je comprends bien ça importe les méthodes static, comme son nom l'indique quoi

(Moi je suis en java 4 de toute façon)

Sinon personne ne voit comment optimiser ma pauvre méthode en restant dans l'api standard?

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
	private boolean dateBeforeOrEqual(Date date, Date dateRef) {
		Calendar dateCourante = Calendar.getInstance();
		dateCourante.setTime(date);
		dateCourante.clear(Calendar.HOUR_OF_DAY);
		dateCourante.clear(Calendar.MINUTE);
		dateCourante.clear(Calendar.SECOND);
		dateCourante.clear(Calendar.MILLISECOND);
 
		Calendar dateReference = Calendar.getInstance();
		dateReference.setTime(dateRef);
		dateReference.clear(Calendar.HOUR_OF_DAY);
		dateReference.clear(Calendar.MINUTE);
		dateReference.clear(Calendar.SECOND);
		dateReference.clear(Calendar.MILLISECOND);
 
		//return (dateCourante.before(dateReference) || dateCourante.equals(dateReference))
		return dateCourante.compareTo(dateReference) <= 0;
	}

[lunatix]

ah bah non. ta méthode est tres bien.
Tout ce que tu peux faire, c'est te faire une classe utilitaire avec pour pouvoir la réutiliser partout.

[Romz_Java]

Ok je m'en tiens là alors

Merci à tous!

[lekonquerant]

Bonjour,

ma réponse arrive très tard mais bon, à toute fin utile:
pour faire cette comparaison, j'utilise

Code :

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

DateUtils.isSameDay(currentDay, changeDay)

où currentDay et changeDay sont de type java.util.Date. Cette méthode provient de la librairie org.apache.commons.lang

[thelvin]

Hello,

Cette méthode provient de la librairie org.apache.commons.lang

Si on l'a déjà intégrée à son projet, pourquoi pas. Mais sinon, quitte à intégrer une bibliothèque visant à se simplifier la vie, joda-time me semble une meilleure idée.