conditions pour fonction alert

Version imprimable

Bonjour,

Je cherche à réaliser un contrôle de formulaire en javascript. Mon formulaire est composé des champs de type " radio " et textarea.
Les différents champs ont tous le même nom ...
J'aimerai donc réaliser un contrôle différents suivant le type du champs
Le nom de mon champs est un numérique qui me renvoi donc :
Code:

1 2 champs radio :<input type='radio' name='2' value=3 id='detail2' '> textarea :<textarea name='3' rows='10' cols='70'>
La fonction suivant contient un compteur qui incremente donc la valeur et me renvoi alors le nom du champs a contrôler.
Je transmet à la variable " champs " le nom du champs à chaque incrementation.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 function champsok(){ // Ici debut de ma boucle pour incrementation champs = document.getElementById(1); if ( champs.tagName == "input" && champs.getAttribute("type") == "radio" ) { if ( (document.getElementsByName(1)[0].checked == false) && (document.getElementsByName(1)[1].checked == false) && (document.getElementsByName(1)[2].checked == false) && (document.getElementsByName(1)[3].checked == false) || (document.getElementsByName(1)[4].checked == false) ) {alert("Veuillez tout remplir");return false;} } if ( champs.tagName == "textarea" && champs.value == "" ){ alert("Veuillez tout remplir");return false;} }//ici fin de ma boucle d'incrementation }//fin fonction
Je ne comprends psa pourquoi je n'arrive pas à verifier que mes champs soient bien saisit :cry:

30/05/2008, 10h27
tusssss

getElementsByName ne fonctionne pas à priori. Je ne comprends pas ton test sur les boutons radios(des && et des ||), une solution facile avec les boutons radios est d'en cocher un par défaut, car pour le decocher il faudra en cocher un autre et cela évite une vérification.(C'est la que tous les anciens du forum s'insurgent contre ma stratégie :D )

Enfin bref c'est pas très clair pour moi, tout ce petit pavé pour dire que je peux pas aider...A part mes remarques du début.
30/05/2008, 10h34
SpaceFrog

Code:

getElementsByName(1)

le name doit être du string

Code:

getElementsByName('1')

Déja merci de vos réponses aussi rapide !
Je suis pas un pro en javascript, c'est sur que la fonction est perfectible :oops:.
Comme j'ai plusieur bouton radio pour la meme question, ( soit 4 ou soit 5) donc je verifie si les 4 premiers ne sont pas cochés ( le 5eme étant pas obligatoire ... )

Ce qui serait bien c'est que pour n'importe qu'elle question avec réponses de type radio avec 4 ou 5 bouton ou bien réponses avec type textarea, j'ai un contrôle qui verifie que la question soit répondu.

ps: je viens de trouver une erreur qui ne corrige pas mon problème .. le nom de l'id est detail+incrementation et non pas l'incrementation
Code:

1 2 3 4 5 6 7 8 9 10 11 function champsok(){ champs = document.getElementById('detail1'); if ( champs.tagName == "input" && champs.getAttribute("type") == "radio" ){ if((document.getElementsByName('1')[0].checked == false) && (document.getElementsByName('1')[1].checked == false) && (document.getElementsByName('1')[2].checked == false) && (document.getElementsByName('1')[3].checked == false) || (document.getElementsByName('1')[4].checked == false)) alert("Veuillez tout remplir");return false; } }
J'ai fais un test avec simplement un alert() dans la condition qui verifie mon type de champs ... mais une fois que je rajoute ma condition sur les boutons radio, mais impossible de valider le formulaire si un textarea est présent. " Et oai une textarea peut pas être checked :aie:"

30/05/2008, 11h04
tusssss

Ta vérification a donc un problème, si j'ai bien compris, tu controles soit 4 soit 5 boutons radios avec la même formule. Donc si tu te retrouves dans un cas avec 5 boutons radios, et que tu coches le 4ème ou le 5ème, ton code est sensé renvoyer que cette question n'a pas reçu de réponse.

Oui je suis d'accord avec toi, c'est pourquoi j'ai mis pour le 5eme bouton un " OU " comme ca si un des 4 premier bouton ou le 5eme n'est pas coché .. je retourne le alert().
Cela ne fonctionnerai pas si j'avais mis un " && " pour le dernier ... Dans se cas la condition me renverrais à chaque fois l'alerte puisque " un des 4 premier bouton et le 5eme n'est pas coché "

Je pense pas me tromper dans ma logique, si ? :roll:

Ce code fonctionne bien
Code:

1 2 3 4 5 6 7 function champsok(){ champs = document.getElementById('<?php echo "detail".$ic; ?>'); if ( champs.tagName == "INPUT" && champs.getAttribute("type") == "radio" ) alert("Veuillez tout remplir");return false; }

30/05/2008, 11h24
tusssss

JE me suis mal exprimé je pense :D

Pas facile à expliquer.

Ce que je veux dire c'est que ton code équivaut à :

(si 1,2,3,4 non cochés ou 5 non coché) message d'alerte.

Donc d'après ma logique quoiqu'il arrive quand 5 est non coché, tu recois le message d'alerte(quelque soit la valeur de 1,2,3 et 4) ou alors quand 1 et 2 et 3 et 4 sont non cochés tu recois le message quelque soit la valeur de 5.

Bon après relecture de ma premiere explication, ca ne voulait rien dire. J'espere que celle-ci est compréhensible.

Ha ouai mince c'est vrai .. a trop réfléchir j'en perds mon raisonnement...
Je vais retirer le 5eme bouton pour voir tout ca.

Même ce code ne fonctionne pas. En faite des que je met la seconde condition, sa ne fonctionne pas, en cliquant sur mon submit sans cocher une réponse je valide.

Code:

1
2
3
4
5
6
7
8
9
10
     champs = document.getElementById('<?php echo "detail".$ic; ?>');
         if ( champs.tagName == "INPUT" && champs.getAttribute("type") == "radio" ){
                  if ((document.getElementsByName('<?php echo $ic; ?>')[0].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[1].checked == false) && 
                     (document.getElementsByName('<?php echo $ic; ?>')[2].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[3].checked == false))             
                      alert("Veuillez tout remplir");return false;    
                 }  
 
        }

Alors je suis a moitié heureux !!! :lol:

Ce code fonctionne correctement ! Mais j'aimerais qu'on m'aide pour le cas ou j'ai une question à 5 boutons. Car si je selectionne le 5eme bouton, le code me renvoi l'alert... Et je ne sais pas trop comment faire la.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 function champsok(){ 
 <?php 
 for ($ic=1; $ic<=$nbr; $ic++) { ?>
     champs = document.getElementById('<?php echo "detail".$ic; ?>');
         if ( champs.tagName == "INPUT" && champs.getAttribute("type") == "radio" ){
                  if ((document.getElementsByName('<?php echo $ic; ?>')[0].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[1].checked == false) && 
                     (document.getElementsByName('<?php echo $ic; ?>')[2].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[3].checked == false))             
                      {alert("Veuillez tout remplir");return false;}
        }
<?php } ?> // fin boucle condition for
}

Tu peux remplacer ca :

Code:

1
2
3
4
5
6
 
 if ((document.getElementsByName('<?php echo $ic; ?>')[0].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[1].checked == false) && 
                     (document.getElementsByName('<?php echo $ic; ?>')[2].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[3].checked == false))             
                      {alert("Veuillez tout remplir");return false;}

Par :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
var total=0;
for (i=0;i<document.getElementsByName('<?php echo $ic; ?>').length;i++)
{
     if (document.getElementsByName('<?php echo $ic; ?>')[i].checked == false)
     {
           total=total++;
     }else{
            break;
     }
}
if (total!=document.getElementsByName('<?php echo $ic; ?>').length)
{alert("Veuillez tout remplir");return false;}

Ca pourrait bien fonctionner, aucune certitude au niveau de la condition length sur le document.getElementsByName...

J'ai mis en place ton code, mais il me valide quand même le formulaire apres avoir cliquer sur le submit.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 function champsok(){ 
 <?php 
 for ($ic=1; $ic<=$nbr; $ic++) { ?>
     champs = document.getElementById('<?php echo "detail".$ic; ?>');
         if ( champs.tagName == "INPUT" && champs.getAttribute("type") == "radio" ){
                var total=0;
                for (i=0;i<document.getElementsByName('<?php echo $ic; ?>').length;i++){
                     if (document.getElementsByName('<?php echo $ic; ?>')[i].checked == false){
                        total=total++;
                     }else{
                           break;
                     }
                }
                if (total!=document.getElementsByName('<?php echo $ic; ?>').length){
                alert("Veuillez tout remplir");return false;}
 
<?php } ?> // fin boucle condition for
     }

Mais j'apprécis le compteur, je n'y avais pas pensé.

30/05/2008, 12h28
Dam1en
Je peux pas simplement verifier que mon element global est à un moment checked ?

par exemple :
Code:

1 2 document.getElementsById('<?php echo $ic; ?>')checked == true
Au lieu de verifier à chaque fois sur chaque bouton si il à été validé ou non ?
30/05/2008, 13h39
tusssss

Je ne crois pas non, pour moi la meilleure solution consiste à cocher par défaut un bouton radio, de cette manière, tu auras toujours une des propositions cochées, plus besoin de les vérifier. Solution de facilité en fait.
30/05/2008, 14h38
Dam1en

Oui, mais le problème c'est que si la personne oubli de répondre. La réponse sera prise en compte pour une valeure par défaut... JE dois obliger les gens à répondre.
30/05/2008, 14h41
tusssss

Citation:

Envoyé par Dam1en

Oui, mais le problème c'est que si la personne oublient de répondre. La réponse sera prise en compte pour une valeure par défaut.

Exact, elle n'a qu'a pas oublier :lol:. En général ce que je déteste plus que les formulaires ou il faut répondre aux endroits accompagnés d'une étoile ce sont les formulaires où l'on doit donner une réponse à chaque question sinon on se voit recharger la même page encore et encore...
30/05/2008, 14h46
Dam1en

Le but de le faire en javascript c'est justement que la page ne se raffraichit pas et donc les données restes selectionnées.
Il y a pas une fonction permettant de verifier que "document.getElementsByName(1)[4].checked "existe ?
ça me faciliterais la vie ...
30/05/2008, 14h47
SpaceFrog

on peut voir le code html de la balise form ?
le onsubmit ?
30/05/2008, 14h50
tusssss

Citation:

Envoyé par Dam1en

Le but de le faire en javascript c'est justement que la page ne se raffraichit pas et donc les données restes selectionnées.
Il y a pas une fonction permettant de verifier que "document.getElementsByName(1)[4].checked "existe ?
ça me faciliterais la vie ...

Code:

alert (document.getElementsByName(1)[4].checked);

?

J'ai remodifié mon code ... J'ai rajouté dans mon formulaire un champs hidden qui me récupere le nombre de boutons par questions...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
 function champsok(){ 
 <?php 
 for ($ic=1; $ic<=$nbr; $ic++) { ?>
     champs = document.getElementById('<?php echo "detail".$ic; ?>');
     var valeur=document.getElementsByName('cnt<?php echo $ic;?>;');
 
         if ( champs.tagName == "INPUT" && champs.getAttribute("type") == "radio" && valeur == 4){
                  if ((document.getElementsByName('<?php echo $ic; ?>')[0].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[1].checked == false) && 
                     (document.getElementsByName('<?php echo $ic; ?>')[2].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[3].checked == false))             
                      {alert("Veuillez remplir tout les champs");return false;}
        }
         if( champs.tagName == "INPUT" && champs.getAttribute("type") == "radio" && valeur == 5){
                  if ((document.getElementsByName('<?php echo $ic; ?>')[0].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[1].checked == false) && 
                     (document.getElementsByName('<?php echo $ic; ?>')[2].checked == false) && 
                     (document.getElementsByName('<?php echo $ic; ?>')[4].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[3].checked == false))             
                      {alert("Veuillez remplir tout les champs");return false;}
        }  
<?php } ?> // fin boucle condition for
     }

Mais je ne sais pas ce qui peut foirer dans mon script la :(

30/05/2008, 15h39
SpaceFrog

y'a pas de return true ?

Je vous remercis pour votre aide, j'ai enfin résolu mon problème.
Voici le code

valeur recupere le nombre de boutton radio que j'ai par question ... j'ai donc fais 2 boucles .. une boucle pour 4 boutons et une autre pour 5 ...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 
<script language="javascript"> 
 function champsok(){ 
 <?php 
 for ($ic=1; $ic<=$nbr; $ic++) { ?>
     champs = document.getElementById('<?php echo "detail".$ic; ?>');
     valeur = document.getElementsByName("cnt<?php echo $ic;?>")[0].value;
 
         if ( champs.tagName == "INPUT" && champs.getAttribute("type") == "radio" && valeur == 4){
                  if ((document.getElementsByName('<?php echo $ic; ?>')[0].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[1].checked == false) && 
                     (document.getElementsByName('<?php echo $ic; ?>')[2].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[3].checked == false))             
                      {alert("Veuillez remplir la question N° <?php echo $ic; ?>");return false;}
        }
         if( champs.tagName == "INPUT" && champs.getAttribute("type") == "radio" && valeur == 5){
                  if ((document.getElementsByName('<?php echo $ic; ?>')[0].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[1].checked == false) && 
                     (document.getElementsByName('<?php echo $ic; ?>')[2].checked == false) && 
                     (document.getElementsByName('<?php echo $ic; ?>')[3].checked == false) && 
                      (document.getElementsByName('<?php echo $ic; ?>')[4].checked == false))             
                      {alert("Veuillez remplir la question N° <?php echo $ic; ?>");return false;}
        }  
<?php } ?> // fin boucle condition for
     }
</script>

Pour le return true; .... il ne m'est pas nécessaire puisque je test que quand ca va pas ... Si les conditions sont bonnes sa renvoi true par défaut