Forum d'entraide PHPFrance

J'ai cette fonction :
qui fonctionne très bien toute seule mais dès que j'ajoute celle-ci :

elle ne fonctionne plus !

Est-ce normal?
Je ne connais quaziment rien en JS j'essaie d'apprendre en observant donc j'ai surement fait une erreur quelque part...

salut

  function enlever()
   {
      if ((window.document.forms['my_form'].elements['jde_1'].checked==false) || (window.document.forms['my_form'].elements['jde_2'].checked==false) || (window.document.forms['my_form'].elements['jde_3'].checked == false) || (window.document.forms['my_form'].elements['jde_4'].checked == false) || (window.document.forms['my_form'].elements['jde_5'].checked == false))
      {
         window.document.forms['my_form'].elements['jde'].checked = false) // <---------- pq cette paranthèse ???? elle ne sert à rien, mets un ";" à la limte.
      }
   } 

Une petite remarque sur ton écriture de : qui est totalement juste puisque ça marche.
Le window devant est inutile; en fait on précise la fenetre avec un document lorsque l'on va chercher des données sur une autre fenetre que celle d'origine.

mais celle-ci fonctionne aussi et est un peu moins longue et t'évite d'appeler le formulaire à chaque fois. Mais bien sur il faut que chacun de tes objets ait un id différent.
Et tu as aussi la même possibilté mais avec les names avec getElementsByName()

salut

  function enlever()
   {
      if ((window.document.forms['my_form'].elements['jde_1'].checked==false) || (window.document.forms['my_form'].elements['jde_2'].checked==false) || (window.document.forms['my_form'].elements['jde_3'].checked == false) || (window.document.forms['my_form'].elements['jde_4'].checked == false) || (window.document.forms['my_form'].elements['jde_5'].checked == false))
      {
         window.document.forms['my_form'].elements['jde'].checked = false) // <---------- pq cette paranthèse ???? elle ne sert à rien, mets un ";" à la limte.
      }
   } 

Une petite remarque sur ton écriture de :

Code : Tout sélectionner

window.document.forms['my_form'].elements['jde_1'].checked

qui est totalement juste puisque ça marche.
Le window devant est inutile; en fait on précise la fenetre avec un document lorsque l'on va chercher des données sur une autre fenetre que celle d'origine.

mais celle-ci fonctionne aussi et est un peu moins longue et t'évite d'appeler le formulaire à chaque fois. Mais bien sur il faut que chacun de tes objets ait un id différent.
Et tu as aussi la même possibilté mais avec les names avec getElementsByName()

Code : Tout sélectionner

document.getElementById('jde_2').checked = true;

Moi et mes copier coller ... je pensais avoir une erreur mais tout de meme pas aussi grossiere que cette ) !!!
Je suis désolé du dérangement, cela fonctionne tres bien a présent !
Et merci une fois de plus a Axel pour m'avoir enseigné une astuce pour javascript !

A bientot pour de nouvelles erreurs de ma part ... (nan promis j'essayrais de faire plus attention la prochaine fois ! )