CALENDRIER SIMPLE
cs_hornetbzz
Messages postés
59
Date d'inscription
lundi 1 décembre 2008
Statut
Membre
Dernière intervention
3 janvier 2011
-
3 janv. 2011 à 02:56
007Julien Messages postés 276 Date d'inscription mercredi 22 septembre 2010 Statut Membre Dernière intervention 8 janvier 2014 - 8 févr. 2011 à 01:23
007Julien Messages postés 276 Date d'inscription mercredi 22 septembre 2010 Statut Membre Dernière intervention 8 janvier 2014 - 8 févr. 2011 à 01:23
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/52667-calendrier-simple
https://codes-sources.commentcamarche.net/source/52667-calendrier-simple
8 févr. 2011 à 01:23
4 janv. 2011 à 19:41
Il est sur que beaucoup de chose m'échappent encore !!
En tout cas, merci des conseils que je vais étudier précieusement!!
Bonne année à tous et toutes
Un débutant initié !!
3 janv. 2011 à 16:53
3 janv. 2011 à 15:25
84 Errors, 152 warning(s)
http://validator.w3.org
Bonne continuation.
Nico
3 janv. 2011 à 02:56
- quelques critiques à la volée :
1) codes css, html et php mélangés, un vrai bronx => séparer les codes, ça ne fera pas plus de lignes.
2) css pas compatible tous navigateurs, dommage pour un calendrier dans la section "initiés" => exemple du (a:hover..) qui ne marche pas sous IE6 à mon avis (pas testé)
3) conditions ternaires:
soit pas utilisées alors que ça vaudrait le coup de se poser la question sur ces lignes
if($premierjour==1)$coef=0; ...
ou sur celles ci
if(isset($_REQUEST['choixmois']))$choixmois =$_REQUEST['choixmois'];
soit "bizarrement" utilisées, détournées de leur logique: ($jour_increment<=$nbrejourmois)?$numsemaine= date('W', mktime(0,0,0,$choixmois,$jour_increment, $choixannee)):$numsemaine='';
4) Variable Globale $_REQUEST: cette superglobale serait utile si tu avais besoin de modifier indépendemment $_GET et $POST. Mais ici, je ne vois pas l'intéret d'utiliser cette variable, surtout avec aussi peu de contrôle, alors que ton formulaire utilise la méthode POST donc tu pourrais restreindre la recup des données avec $_POST sans ouvrir un trou béant aux $_GET que tu n'utilises pas.
5) Créer des tableau de mois à la mano, c'est juste pas beau. Un truc du genre serait plus sympa, si toutefois il n'y avait pas une méthode encore plus directe, ce qui ne me surprendrait pas : for ($i 0; $i < 12; $i++) { $month[] mktime(0,0,0,(date('m') + $i),1); }
Bon, meilleurs voeux, au moins tu as quelques bonnes résolutions à suivre :-)