FORMULAIRE DE CONGÉS AVEC CALCUL DES JOURS OUVRÉS

Commentaire de kiki2sirom le 10/04/2006 11:48:34

ET BIM !
y a du taf , fou yaya, je commence..... :
* déjà ta page formulaire pourrait s'appeler form.html, non ? (où se trouve le php là-dedans)
* ton HTML est CATASTROPHIQUE : où sont les balises DOCTYPE, body, html, head......
essaye de regarder comment se construit un document HTML, damn !!
* un tableau en HTML c'est : <table><tr><td></td></tr></table> au minimum, et toi là tu as des <tr> qui s'ouvrent et qui ne se ferment pas...
*une seule <form> globale utile, parce qu'autant de formes que de types de congés, c'est de l'abus quand même là....
* sur le validateur HTML du W3C : 32 erreurs juste sur la page form.php
* <br /> et non <br/>
/////////////////////////////////////////////////
* je passe au côté javascript (il est où celui-là ??) : il te faut tester si une date1 et une date2 a été remplie et si date1<=date2 quand tu cliques sur "Valider", d'où le besoin du Javascript, tester 2 dates, c'est pas bien compliqué en plus, parce que sinon sur ta page php, je te raconte même pas le binz
////////////////////////////////////////////////
passons côté PHP, je ne pense pas être déçu non plus :
entres autres parce que c'est vraiment trop bordélique :
* <?php en lieu et place de <? pour une histoire de compatibilité de ton code
* pourquoi tu calcules à chaque type de congés le nombre de jours fériés ??? ils vont changer selon que tu vas prendre congés payés ou RTT ??? de plus l'idée de faire une fonction pour récupérer les jours fériés, pourquoi pas, une fonction est réutilisable, tu la mets en dehors de ton 1er if, par exemple...
*
par exemple pour optimiser ça :
// Calcul du jour de pâques
$date_paques = easter_date($annee);
$jour_paques = date("d", $date_paques);
$mois_paques = date("m", $date_paques);
if($jour_paques == $jour && $mois_paques == $mois) $nbFerie++;
tu peux faire ça:
if (($mois==date("n", easter_date($annee)+1*$jour))&& ($jour==date("j", easter_date($annee)+1*$jour)))

Tout est optimisable, surtout, tout est à refaire, désolé de te l'apprendre.

Je m'arrête là parce que bon, j'ai pas le temps de tout voir, VRAIMENT !!!

Cordialement

kiki2sirom