Optimisation de fonction

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:

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; }

Code:

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

=> 20080206

Sans code deprecated :

Code:

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 :aie:

Salut,

Citation:

Envoyé par xduris

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 ?

Citation:

Envoyé par xduris

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:

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

Bonjour,

Citation:

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. :D

Citation:
Envoyé par adiGuba
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:

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

Citation:

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?

Citation:

Envoyé par vincent63

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 8O

Citation:

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. :D

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).

Citation:

Envoyé par vincent63

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:

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