Activer/désactiver bouton submit en fin du questionnaire

Citation:

---

Envoyé par Billy KiT

Pour les accolades, ça condense l'écriture, ça marche (pareil je pense), mais 1 fois sur 1 environ, si on ajoute un else, on se mange.

---

On se "mange" seulement quand on ne sait pas ce qu'on fait. Un compilateur ne fait pas de la ratatouille avec le code qu'on écrit... Si on écrit deux fois la même chose, il fera deux fois la même chose.

Déjà une bonne habitude à prendre est d'indenter correctement son code (en tout cas c'est une très mauvaise habitude que de ne pas le faire). Ca aide celui qui écrit à relire son code et, dans le cas précis, ça aiderait ceux qui essaye de t'aider de comprendre ce que tu crois que ton code fait.

En l'état ton code fait ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

function affiche_bouton() { var nbQuestion = 2; var nbChecked = 0;   for (i=0; i<document.getElementsByTagName("input").length; i++) if (document.getElementsByTagName("input")[i].type=="radio") if (document.getElementsByTagName("input")[i].checked) nbChecked++; if (nbChecked == nbQuestion) { document.getElementById('submite').disabled=''; } else { document.getElementById('submite').disabled='disabled'; } }

---

Et dans le cas présent tu peux totalement remplacer tes 2 ifs par un seul avec un &&.

J'ai indenté justement après avoir mis mon source tout à l'heure :)

C'est vrai que j'ai fait ça un peu rapidement, ça arrive d'oublier certaines choses essentielles quand on est pressé ^^.

Citation:

---

Envoyé par Loceka

On se "mange" seulement quand on ne sait pas ce qu'on fait. Un compilateur ne fait pas de la ratatouille avec le code qu'on écrit... Si on écrit deux fois la même chose, il fera deux fois la même chose.

Déjà une bonne habitude à prendre est d'indenter correctement son code (en tout cas c'est une très mauvaise habitude que de ne pas le faire). Ca aide celui qui écrit à relire son code et, dans le cas précis, ça aiderait ceux qui essaye de t'aider de comprendre ce que tu crois que ton code fait.

En l'état ton code fait ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

function affiche_bouton() { var nbQuestion = 2; var nbChecked = 0;   for (i=0; i<document.getElementsByTagName("input").length; i++) if (document.getElementsByTagName("input")[i].type=="radio") if (document.getElementsByTagName("input")[i].checked) nbChecked++; if (nbChecked == nbQuestion) { document.getElementById('submite').disabled=''; } else { document.getElementById('submite').disabled='disabled'; } }

---

Et dans le cas présent tu peux totalement remplacer tes 2 ifs par un seul avec un &&.

---

tu répètes exactement tout ce que j'ai dis 11 messages plus tôt

Enfin, c'est plutôt bon signe, au moins nous sommes d'accord. ;)

:cry: ne vous fâchez pas ! Je suis d'accord avec vous.
Qui va me dire qu'à première vue, dans le code suivant, on va avoir l'alerte a!=1 alors que a=1

Code:

---

1 2 3 4 5 6 7

a = 1; b = 2;   if (a==1) if (b==1) alert ('b=1'); else alert ('a!=1');

---

C'était un simple conseil sans prétention d'un membre non chevronné. Je vois que l'image vous a fait sursauter, désolé. J'ai vu des erreurs bien plus c.. sournoises.

si encore les accolades {}étaient payantes, je comprendrais qu'on en fasse l'economie ...
mais elle sont gratuites alors n'hésitez pas utilisez les pareil pour les ;

Citation:

---

Envoyé par SpaceFrog

si encore les accolades {}étaient payantes, je comprendrais qu'on en fasse l'economie ...
mais elle sont gratuites alors n'hésitez pas utilisez les pareil pour les ;

---

parfois, le code est tellement "simple et lisible" que je ne les mets pas mais dès qu'il y a un doute sur la lisibilité, il est évident de les mettre vu leurs gratuité^^

par exemple dans des cas comme celui-là, je ne les mets pas tjs :

Code:

---

1 2 3 4 5 6 7 8

FUNCTION(){ BOUCLE{ IF(CONDITION) RETURN; EXECUTION_DE_CODE; } RETURN; }

---