Discussion :

[xduris]

Bonjour, j'ai une fonction que j'ai ecrit et qui marche mais qui ne me plait pas.
Cette fonction prend un date et retourne un chaine de caractère reprennant la date de la manière suivante AAAAMMJJ.
Je vous soumet ma méthode.
Je n'aime pas le fait d'utiliser une date (Est ce qu'il y a mieux?)
Je n'aime pas le fait d'utiliser des méthodes dépréciés (getYear ...)
Je n'aime pas mon utilisation de la classe String et la concatenation.
J'ai un soucis de décalage dans la date (year+1900, month+1, day+3)
Merci pour votre regard avisé.

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
    // formater AAAAMMJJ
    private static String Date2String (Date date){
 
        String string = new String();
 
 
        Integer year = new Integer(date.getYear() + 1900);
        Integer month = new Integer (date.getMonth() + 1);
        Integer day = new Integer (date.getDay() + 3);
 
        String stringYear = new String(year.toString());
        String stringMonth;
        String stringDay;
 
 
        if (month < 10){
            stringMonth = new String("0" + month.toString());
        }
        else{
            stringMonth = new String(month.toString());
        }
 
        if (day < 10){
            stringDay = new String("0" + day.toString());
        }
        else{
            stringDay = new String(day.toString());
        }
 
        string = stringYear + stringMonth + stringDay;
 
        return string;
 
    }

[Duc Lebowski]

Code :

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

1
2
3
Date d = Calendar.getInstance().getTime();
DateFormat df = new SimpleDateFormat("yyyyMMdd");
System.out.println(df.format(d));

=> 20080206

[DevTeam_]

Sans code deprecated :

Code :

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

1
2
3
4
5
6
7
 
private static String Date2String(Date date) {
  Calendar cal = new GregorianCalendar();
  cal.setTimeInMillis(date.getTime());
  SimpleDateFormat df = new SimpleDateFormat("yyyyMMdd");
  return df.format(cal.getTime());
}

EDIT : grilled, et en mieux

[adiGuba]

Salut,

Je n'aime pas le fait d'utiliser une date (Est ce qu'il y a mieux?)

Qu'y a-t-il de gênant à une utiliser une Date pour représenter une date ?

Je n'aime pas mon utilisation de la classe String et la concatenation.

Ici cela ne pose pas de problème particulier...


Sinon j'aurais rajouté :

• Je n'aime pas le nom de la méthode qui commence par une majuscule.
• Je n'aime pas la duplication inutile des String avec les new String()

Et surtout : je n'aime pas quand on oublie de consulter la docs ou les FAQs, car tu serais surement tomber sur SimpleDateFormat :

Code :

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

1
2
3
4
    private static String toString (Date date) {
    	SimpleDateFormat sdf = new SimpleDateFormat("yyyyMMdd");
    	return sdf.format(date);
    }

a++

[vincent63]

Bonjour,

Qu'y a-t-il de gênant à une utiliser une Date pour représenter une date ?

Dans le cas présent, cela ne pose pas de problème, mais il me semble qu'il est quand même largement déconseillé d'utiliser les object Date lorsque cela n'est pas indispensable. Si je ne me trompe pas la recommandation au sujet des dates est de leur préférer l'utilisation de long.

Bon ok, c'était juste pour chipotter un peu.

[xduris]

Salut,


Qu'y a-t-il de gênant à une utiliser une Date pour représenter une date ?


Ici cela ne pose pas de problème particulier...


Sinon j'aurais rajouté :

• Je n'aime pas le nom de la méthode qui commence par une majuscule.
• Je n'aime pas la duplication inutile des String avec les new String()

Et surtout : je n'aime pas quand on oublie de consulter la docs ou les FAQs, car tu serais surement tomber sur SimpleDateFormat :

Code :

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

1
2
3
4
    private static String toString (Date date) {
    	SimpleDateFormat sdf = new SimpleDateFormat("yyyyMMdd");
    	return sdf.format(date);
    }

a++

Je suis désolé de t'embeter encore mais ca me retourne un java.text.SimpleDateFormat@d547ad40

[vincent63]

java.text.SimpleDateFormat@d547ad40

Tu dois faire un toString() sur ton SimpleDateFormat.
Tu es sûr d'avoir bien renvoyé sdf.format(date) et non pas simplement sdf?

[DevTeam_]

Tu dois faire un toString() sur ton SimpleDateFormat.
Tu es sûr d'avoir bien renvoyé sdf.format(date) et non pas simplement sdf?

Ca ne compilerait même pas vu que la méthode est censée retourner un String

[vincent63]

Ca ne compilerait même pas vu que la méthode est censée retourner un String

J'suis peut-être allé un peu vite dans mon explication. Au temps pour moi, je recommence.

Ne sachant pas comment le code fourni a été récupéré et réutilisé, (méthode ou pas méthode), et voyant la trace, j'en conclu que c'est le toString() du SimpleDateFormat qui a été utlisé (soit explicitement, soit implicitement). Donc que ce n'est pas le String renvoyé par sdf.format(date).

[xduris]

Tu dois faire un toString() sur ton SimpleDateFormat.
Tu es sûr d'avoir bien renvoyé sdf.format(date) et non pas simplement sdf?

Merci pour ta réponse et merci à tout le monde

La solution que j'ai retenu est

Code :

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

1
2
3
4
 
Date date = new Date();
SimpleDateFormat simpleDateFormat = new SimpleDateFormat("yyyyMMdd");
fichier = new FileWriter(this.chemin + "/fichier" + simpleDateFormat.format(date) + ".txt");