cs_jeca
Messages postés341Date d'inscriptionmercredi 17 juillet 2002StatutMembreDernière intervention14 juillet 201114 20 févr. 2009 à 10:02
Bonjour,
Le code est propre et bien documenté.
Mais il vaut mieux utiliser :
$cssFin = 'jour';
plutôt que :
$cssFin = "jour";
car le parseur php analyse la chaine pour voir s'il y a une expression à évaluer, ce qui n'est pas le cas ; donc il travaille pour rien.
D'autre part, je doute fort que cette ligne puisse fonctionner :
if (($date >= 22.09) && ($date < 21.12))
Enfin, les date des saisons et de Pa^ques sont mobiles, ce qui veut dire qu'il faut modifier le programme à la mimine tous les ans.
darkpisto
Messages postés1Date d'inscriptionlundi 9 février 2009StatutMembreDernière intervention16 février 2009 16 févr. 2009 à 19:54
Code source utilisé en production sur un site : c'est très joli, et surtout ça MARCHE !! ;)
Fidji56
Messages postés58Date d'inscriptionvendredi 30 janvier 2009StatutMembreDernière intervention26 mars 2010 16 févr. 2009 à 11:16
Moi je pense que pour un débutant c'est déjà pas si mal
20 févr. 2009 à 10:02
Le code est propre et bien documenté.
Mais il vaut mieux utiliser :
$cssFin = 'jour';
plutôt que :
$cssFin = "jour";
car le parseur php analyse la chaine pour voir s'il y a une expression à évaluer, ce qui n'est pas le cas ; donc il travaille pour rien.
D'autre part, je doute fort que cette ligne puisse fonctionner :
if (($date >= 22.09) && ($date < 21.12))
Enfin, les date des saisons et de Pa^ques sont mobiles, ce qui veut dire qu'il faut modifier le programme à la mimine tous les ans.
16 févr. 2009 à 19:54
16 févr. 2009 à 11:16