CALENDRIER SIMPLE

Signaler
Messages postés
60
Date d'inscription
lundi 1 décembre 2008
Statut
Membre
Dernière intervention
3 janvier 2011
-
007Julien
Messages postés
279
Date d'inscription
mercredi 22 septembre 2010
Statut
Membre
Dernière intervention
8 janvier 2014
-
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

007Julien
Messages postés
279
Date d'inscription
mercredi 22 septembre 2010
Statut
Membre
Dernière intervention
8 janvier 2014
2
Un détail mais $coef=($premierjour+6)%7; éviterait les lignes 63 à 69 qui ne constituent pas vraiment de la programmation...
cs_manuche
Messages postés
2
Date d'inscription
lundi 27 septembre 2004
Statut
Membre
Dernière intervention
26 février 2011

Ben !! qu'est ce que cela doit être, expert !!
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é !!
Profil bloqué
Bizarre pas tout vu mais fait des essais sécurité c pas le top mais bon ??? Variable ???? navigateur'S qui veut po Merci mais si tu suis les conseils de hornetbzz normalement cela devrait le faire Bonne continuation
tatactic
Messages postés
22
Date d'inscription
mardi 15 mars 2011
Statut
Membre
Dernière intervention
14 janvier 2012

Peut-être aussi juste à penser à valider la source obtenue dans ton browser...
84 Errors, 152 warning(s)
http://validator.w3.org
Bonne continuation.
Nico
cs_hornetbzz
Messages postés
60
Date d'inscription
lundi 1 décembre 2008
Statut
Membre
Dernière intervention
3 janvier 2011

hum "initié", ça me parait un brin optimiste.

- 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 :-)