Javascript & Html demande de correction - Javascript/Node.js - Programmation
Marsh Posté le 11-03-2015 à 23:19:47
Corriger quoi ? Quel est le soucis ? Il s'agirait pas de faire le boulot à ta place des fois ?
En tout cas, concernant ton code JS :
1) mets tes fonctions ensemble, là ca va dans tout les sens
2) en JS, si tu ne mets pas le mot clé "var" devant une variable, elle est automatiquement globale, même si elle est définie dans une fonction
3) replace tes "new Array" par la version litérale, càd []
new Array('Dimanche', 'Lundi', 'Mardi', 'Mercredi', 'Jeudi', 'Vendredi', 'Samedi');
devient
['Dimanche', 'Lundi', 'Mardi', 'Mercredi', 'Jeudi', 'Vendredi', 'Samedi'];
4) les if en pagaille se remplacent facilement par un switch
5) ne fait pas window.setTimeout("rerejouer();",500);
mais window.setTimeout (rerejouer, 500);
6) méfies toi des variables qui portent presque le même nom et qui representent presque la même chose, genre moi/mois, jour/jours
7) conseil : même si dans un if tu n'as qu'une seule ligne derriere, mets quand même les accolades, tu gagneras en lisibilité et en temps de debug
au lieu de
if (notes[i]==0) message='Do';
if (notes[i]==1) message='Dodiese';
if (notes[i]==2) message='Re';
remplaces par
if (notes[i]==0) { message='Do'; }
if (notes[i]==1) { message='Dodiese'; }
if (notes[i]==2) { message='Re'; }
Dailleurs, ici un switch est bien meilleur, là ton code va passer tout tes if les uns apres les autres, alors que ce n'est pas necessaire
switch (notes[i])
{
case 0:
message='Do';
break;
case 1:
message='DoDiese';
break;
//...
}
8) evites de mélanger les type, si tu utilises une variable pour stocker un nombre, ne la transforme pas en string plus loin, sauf si tu veux que ton code soit tout lent
h = date.getHours();
if (h < 10)
{
h = "0"+h; // nope ! crée une autre variable !
}
9) relis ton html, ca va dans tout les sens
et "font" est déprecié depuis 1 siecle environ
10) mets tout le code ensemble, le js avec le js (pas besoin de plusieurs balises script), et le html avec le html. Les onclick dépannent, mais ne devraient pas être la base du taf
bonne chance
Marsh Posté le 12-03-2015 à 12:44:38
J'ajouterais qu'il est mieux de sortir le CSS et le JS du fichier HTML, et donc d'avoir trois fichiers différents (il faut ensuite importer les fichiers CSS et JS dans le HTML).
Marsh Posté le 12-03-2015 à 19:36:03
Merci de votre réponse enfaite lorsque je lance cet page je ne voit ni les image et je n'entend pas le son associer aux images.. En tout cas merci et je vais essayer de tout corriger
Marsh Posté le 11-03-2015 à 22:46:18
Bonsoir, voila je travail actuellement sur mon projet de fin d'années en ISN, je suis un élève de terminale S et je vous demande un peu d'aide pour m'aider a corrigé mon code source, je programme donc en Javascript et en Html sous le logiciel Fraise qui est sur Mac OS X.
Voici le code source :