cs_hornetbzz
Messages postés59Date d'inscriptionlundi 1 décembre 2008StatutMembreDernière intervention 3 janvier 2011 15 nov. 2010 à 16:05
félicitations, bel exercice :-)
D'abord les commentaires "pele-mêle" :
- "switch case" existe aussi en javascript, ça aurait peut-être été mieux adapté en terme de codage (je veux dire par là, mieux que la suite de "if"),
- innerHTML est "capricieux" selon les navigateurs, mieux vaut utiliser le DOM W3C.
Ensuite juste deux petits commentaires perso ou petites idées d'amélioration:
1) ça serait encore plus sympa de "détecter" les objets présents sur la page et d'y attacher un evenement.
En détachant bien les actions qui en découlent. Cela te permettrait d'attacher l'action "x" sur l'evenement "y", et rendrait le script plus universel, et surtout plus portable sur un site existant. Bon j'avoue, je fais un peu ma pub: http://hornetbzz.developpez.com/tutoriels/javascript/dom/
2) Comme seconde amélioration, il y a une manière moins intrusive d'introduire javascript, et que tu pourrais employer facilement. Cette méthode est très bien décrite ici : http://www.pompage.net/pompe/javascript-non-intruisif-chapitre-4/ Là aussi, ça va dans le sens d'un portage facilité dans une appli existante.
15 nov. 2010 à 16:05
D'abord les commentaires "pele-mêle" :
- "switch case" existe aussi en javascript, ça aurait peut-être été mieux adapté en terme de codage (je veux dire par là, mieux que la suite de "if"),
- innerHTML est "capricieux" selon les navigateurs, mieux vaut utiliser le DOM W3C.
Ensuite juste deux petits commentaires perso ou petites idées d'amélioration:
1) ça serait encore plus sympa de "détecter" les objets présents sur la page et d'y attacher un evenement.
En détachant bien les actions qui en découlent. Cela te permettrait d'attacher l'action "x" sur l'evenement "y", et rendrait le script plus universel, et surtout plus portable sur un site existant. Bon j'avoue, je fais un peu ma pub: http://hornetbzz.developpez.com/tutoriels/javascript/dom/
2) Comme seconde amélioration, il y a une manière moins intrusive d'introduire javascript, et que tu pourrais employer facilement. Cette méthode est très bien décrite ici : http://www.pompage.net/pompe/javascript-non-intruisif-chapitre-4/ Là aussi, ça va dans le sens d'un portage facilité dans une appli existante.
bonne continuation :-)