JpHugo
Messages postés3Date d'inscriptionjeudi 5 janvier 2006StatutMembreDernière intervention 8 avril 2006
-
8 avril 2006 à 10:57
cs_Julien39
Messages postés6414Date d'inscriptionmardi 8 mars 2005StatutModérateurDernière intervention29 juillet 2020
-
13 mars 2011 à 15:50
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
cs_Julien39
Messages postés6414Date d'inscriptionmardi 8 mars 2005StatutModérateurDernière intervention29 juillet 2020371 13 mars 2011 à 15:50
Je suis d'accord avec iubito, ca ne sert à rien de réinventer la roue. Mais bon passons sur ce point.
Ton code n'est pas très bien écrit, je pense en particulier a tes interminables suites de if :
if(_mm==1)
_xM = 365;
if(_mm==2)
_xM = 365- 31;
if(_mm==3)
_xM = 365-(31+28);
if(_mm==4)
_xM = 365-(31+28+31);
if(_mm==5)
_xM = 365-(31+28+31+30);
if(_mm==6)
_xM = 365-(31+28+31+30+31);
if(_mm==7)
_xM = 365-(31+28+31+30+31+30);
if(_mm==8)
_xM = 365-(31+28+31+30+31+30+31);
if(_mm==9)
_xM = 365-(31+28+31+30+31+30+31+31);
if(_mm==10)
_xM = 365-(31+28+31+30+31+30+31+31+30);
if(_mm==11)
_xM = 365-(31+28+31+30+31+30+31+31+30+31);
qui pourrait être automatisées en utilisant une énumération des mois associée a une fonction getNombreJours() par exemple. Ton code pourrait être fait en quelques lignes avec une conception plus judicieuse
cs_fontome_123
Messages postés13Date d'inscriptionlundi 28 février 2005StatutMembreDernière intervention 1 mai 2008 28 nov. 2006 à 23:41
rien a dire mon ami juste du commentaire comme on te l'as signalé avant bonne continuation
JaVaCode
Messages postés3Date d'inscriptionmercredi 24 novembre 2004StatutMembreDernière intervention 6 janvier 2009 25 juin 2006 à 14:56
rien a dire il falait juste ajouter des petits comm pour rendre ton code claire
cs_iubito
Messages postés629Date d'inscriptionmercredi 3 juillet 2002StatutMembreDernière intervention 9 octobre 2006 10 avril 2006 à 07:18
Et la classe Calendar, ça ne marche pas ?
Calendar cal = Calendar.getInstance();
cal.set(Calendar.YEAR, 1234);
cal.set(Calendar.MONTH, Calendar.OCTOBER);
cal.set(Calendar.DAY_OF_MONTH, 23);
//et puis :
cal.get(Calendar.DAY_OF_WEEK);// un truc comme ça
//ou utiliser un DateFormat (voir la javadoc)
new DateFormat(...).format(cal.getTime())
JpHugo
Messages postés3Date d'inscriptionjeudi 5 janvier 2006StatutMembreDernière intervention 8 avril 2006 8 avril 2006 à 10:57
Ton programme tourne parfaitement, mais il n'est pas très lisible. Il n'y a pas assez de javaDocs et d'espaces.
13 mars 2011 à 15:50
Ton code n'est pas très bien écrit, je pense en particulier a tes interminables suites de if :
if(_mm==1)
_xM = 365;
if(_mm==2)
_xM = 365- 31;
if(_mm==3)
_xM = 365-(31+28);
if(_mm==4)
_xM = 365-(31+28+31);
if(_mm==5)
_xM = 365-(31+28+31+30);
if(_mm==6)
_xM = 365-(31+28+31+30+31);
if(_mm==7)
_xM = 365-(31+28+31+30+31+30);
if(_mm==8)
_xM = 365-(31+28+31+30+31+30+31);
if(_mm==9)
_xM = 365-(31+28+31+30+31+30+31+31);
if(_mm==10)
_xM = 365-(31+28+31+30+31+30+31+31+30);
if(_mm==11)
_xM = 365-(31+28+31+30+31+30+31+31+30+31);
qui pourrait être automatisées en utilisant une énumération des mois associée a une fonction getNombreJours() par exemple. Ton code pourrait être fait en quelques lignes avec une conception plus judicieuse
28 nov. 2006 à 23:41
25 juin 2006 à 14:56
10 avril 2006 à 07:18
Calendar cal = Calendar.getInstance();
cal.set(Calendar.YEAR, 1234);
cal.set(Calendar.MONTH, Calendar.OCTOBER);
cal.set(Calendar.DAY_OF_MONTH, 23);
//et puis :
cal.get(Calendar.DAY_OF_WEEK);// un truc comme ça
//ou utiliser un DateFormat (voir la javadoc)
new DateFormat(...).format(cal.getTime())
8 avril 2006 à 10:57