Discussion :

[CliffeCSTL]

Bonjour,

Je récupère une date dans un String. Cette date est renseigner par des utilisateurs d'un programme et est censé respecter ce format : DDMMYYYY.
Le problème c'est qu'il y a souvent des erreurs, l'utilisateur peut potentiellement mettre n'importe quoi.

Comment faire au plus simple pour :
- Vérifier le format de la date
- Vérifier que la date est comprise entre 01/01/1900 et TODAY.

Merci.

[eulbobo]

Je te donne deux pistes :

http://docs.oracle.com/javase/7/docs...ateFormat.html

https://docs.oracle.com/javase/7/doc.../Calendar.html

Normalement, il y a tous les éléments qu'il faut dans la documentation pour t'aider à trouver une solution à ton problème avec ces deux classes !

[CliffeCSTL]

Oui merci. J'ai déjà trouvé des exemples, sur ce forum. Mais je souhaite faire le plus propre et simple possible.

[eulbobo]

Oui merci. J'ai déjà trouvé des exemples, sur ce forum. Mais je souhaite faire le plus propre et simple possible.

Montres-nous comment tu fais pour l'instant, on te dira comment l'améliorer

[CliffeCSTL]

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
String DclDateActe056 = "01012015";
if (DclDateActe056.length() == 8) {     
    try {
        SimpleDateFormat sdf = new SimpleDateFormat("ddMMyyyy");
        Date dateQuest  = sdf.parse(DclDateActe056);
        Date dateNi     = sdf.parse("01011900");
        Date dateActuel = new Date();
 
        if (dateQuest.compareTo(dateActuel) > 0) {
            System.out.println("La date est dans le futur");
        } else if (dateQuest.compareTo(dateNi) < 0 ) {
            System.out.println("La date est avant 1900");
        }
 
    } catch (ParseException e) {
        e.printStackTrace();
    } 
} else {
    System.out.println("Mauvais format de la date, trop de caractère");
}

[eulbobo]

Tester la taille de la chaîne, c'est déjà bien, tu enlèves deux cas pourri !

Pour le reste ben... Ca marche déjà de manière simple et efficace !
Ce que tu pourrais améliorer, c'est les tests sur les dates: l'objet Date à une fonction before() et une fonction after qui serait plus explicite dans ton cas

Code :

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

1
2
3
4
5
6
 
        if (dateQuest.after(new Date())) {
            System.out.println("La date est dans le futur");
        } else if (dateQuest.before(dateNi)){
            System.out.println("La date est avant 1900");
        }

Et sinon, il faut modifier le catch

Code :

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

1
2
3
4
 
} catch (ParseException e) {
            System.out.println("Format de date inconnu, ou date incohérente");
    }

Et c'est tout, y'a pas grand chose à améliorer !

[thelvin]

On pourrait éventuellement :

- ajouter une protection contre le 45 Janvier (45012015), le 29 Février 2011 (29022011), ou le 3 Hexadécembre (03182015).

avec sdf.setLenient(false); .

- éviter de parser la date mini à chaque fois : elle va pas changer d'un appel à l'autre, alors c'est pas très clair comme idée. Plutôt la stocker dans une variable private static final, initialisée dans un bloc static.

- S'intéresser à passer à Java 1.8 et utiliser des LocalDate et tout le toutim au lieu des anciennes Date et leurs bizarreries.

[CliffeCSTL]

Très bien. Merci.