malalam
Messages postés10839Date d'inscriptionlundi 24 février 2003StatutMembreDernière intervention 2 mars 201025 23 sept. 2005 à 13:47
Hello,
ta fonction est marrante.
Néanmoins, un reproche : si la date passée est valide, ta fonction ne renvoie que false, à priori. Tu devrais renvoyer la date.
Ensuite, un truc qui...bon...est un choix : elle inverse systématiquement, si c'est possible, jour et mois. S on met
1998/05/06 on obtient 06/05/1998.
Idem pour ça : echo dateValidReturnTry ('05/1998/06');
Et ça : echo dateValidReturnTry ('05199806');
retourne un truc bizarre ;-): 19/05/9806 . Je doute que ta fonction soit encore utilisée en 9806...tu peux mettre une borne, je pense.
Enfin je trouve ton code peu clair : surtout au niveau des variables.
Je ne vois pas l'intérêt de déclarer une fonction DANS une autre fonction.
Ceci :
$date = str_replace('-', '/', $date);
$date = str_replace(' ', '/', $date);
peut-être raccourci, éventuellement.
J'ai plus le temps de continuer, lol. Bref, je trouve ta fonction sympa:-)
elinep
Messages postés56Date d'inscriptionjeudi 4 décembre 2003StatutMembreDernière intervention 3 décembre 2009 22 sept. 2005 à 21:06
Je pige pas trop ce que fait ton code...
Ya un truc qiu me parait byzarre dans ton if avec tes 12 millions de conditions : un nombre entier est forcemment > 4, =3 ou <2 non? donc ta condition est forcement vrai et ta fonction retourne systematiquement false ?
23 sept. 2005 à 13:47
ta fonction est marrante.
Néanmoins, un reproche : si la date passée est valide, ta fonction ne renvoie que false, à priori. Tu devrais renvoyer la date.
Ensuite, un truc qui...bon...est un choix : elle inverse systématiquement, si c'est possible, jour et mois. S on met
1998/05/06 on obtient 06/05/1998.
Idem pour ça : echo dateValidReturnTry ('05/1998/06');
Et ça : echo dateValidReturnTry ('05199806');
retourne un truc bizarre ;-): 19/05/9806 . Je doute que ta fonction soit encore utilisée en 9806...tu peux mettre une borne, je pense.
Enfin je trouve ton code peu clair : surtout au niveau des variables.
Je ne vois pas l'intérêt de déclarer une fonction DANS une autre fonction.
Ceci :
$date = str_replace('-', '/', $date);
$date = str_replace(' ', '/', $date);
peut-être raccourci, éventuellement.
J'ai plus le temps de continuer, lol. Bref, je trouve ta fonction sympa:-)
22 sept. 2005 à 21:06
Ya un truc qiu me parait byzarre dans ton if avec tes 12 millions de conditions : un nombre entier est forcemment > 4, =3 ou <2 non? donc ta condition est forcement vrai et ta fonction retourne systematiquement false ?