CALCUL DATES & JOURS FÉRIÉS

Commentaire de sebmafate le 30/11/2008 10:00:52 administrateur CS

Plusieurs remarques :
L'idée de faire une classe pour retrouver les jours fériés est pas mal... mais malheureusement, la réalisation ne me plait pas !
- Pourquoi retourner la date de Paques (et autres) sous la forme d'une chaine de caractères ? Pourquoi pas un DateTime ?
- Pourquoi créer les DateTimes à partir de DateTime.Parse(), il suffit d'utiliser le constructeur de DateTime : DateTime(year, month, day)

Finalement, je n'ai pas tout lu...

Commentaire de sebmafate le 30/11/2008 10:45:31 administrateur CS

Ce n'est pas parce qu'on est vieux qu'on est dispensé de penser.
Mon premier commentaire n'a rien de méchant... je te donne juste des conseils.
On commence tous un jour... toi c'est l'informatique, moi, c'est la photo !

Commentaire de petifa le 04/12/2008 15:49:56

slt
l'age ne change pas au fait que quelqu'un qui débute est débutant. Mais bon j'ai quelques remarques à premier abord.

- Dans ta classe : J_Paques
# public string Jour_Pacques(int Annee)
# {
ton traitement est assez complexe, il serait plus judicieux d'utiliser des variable plus compréhensibles que a, b ...e

- ta fonction mod est inutille tu peux mettre directement
# int a = Mod( Annee, 19);
int a = Annee%19;

- Dans la classe Jours_Feries
# public DateTime[] J_Feries = new DateTime[13];
Cette variable devrait etre en private


-sinon, je suis d'accord avec sebmafate, utiliser un PARSE prend du temps car il y a des traitements derrirèe pour convertir une chaine de caractère en Date. Utilise le constructeur de DateTime, ca sera plus efficace et evitera les erreurs de conversion (En anglais et francais le jour et mois sont inversés)
# private void JourFerie(int Annee)
# {
# J_Feries[0]= DateTime.Parse("01/01/" + Annee.ToString());
J_Feries[0]= new DateTime(Annee,1,1)


- Ensuite cette structure n'est pas tr_s valide : utilise un mutateur (c'est à dire une fonction pour modifier directement une variable de classe
#  public int Annee // Année
# {
#   set
#   {
#    AN = value ;
#    JourFerie(AN) ;
#   }
# }
par exemple :
public setAN(int Annee)
{
    this.AN = Annee;
this.JourFerie()
}
et dans ce cas la variable Annee dans JourFerie() , remplace la par this.AN
Tu devrais faire de meme pour toutes les autres fonctions que tu as mis comme ca, cela allègera le code


- Idem pour les autres fonction :
#  public string Pacques
# {
#   get
#   {
#     return J_Feries[1].ToLongDateString().Substring(0, J_Feries[1].ToLongDateString().Length - 4) ;
#   }
# }
Remplace par
public string Pacques()
{ return J_Feries[1].toString(); }


-Dernière chose, tu devrais mettre des commentaires,
#  private void Calcul()
# {
On ne sais pas en lisant le titre ou les commentaires ce que fait cette fonction.


Voila je ne peux dire que ca pour le moment
j'ai pas regardé le code pour l'affichage ...