Discussion :

[myst28]

Bonjour,

en javascript je dois récupérer le nombre de boutons radio sélectionnés et afficher un message si il est inférieur au nombre de radio existant (normalement au nombre de famille différente de radio mais je n'en suis pas là) le problème est que la fonction ne fonctionne pas et aucun message ne s'affiche voila le code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
function checkNumberCheckedRadio(){
	var nbcoche=0;
	var nbradio=0;
	var inputs=document.getElementsByTagName("input");
	for (var i=0;i<=inputs.length;i++){
		if (inputs[i].type=="radio"){
			nbradio++;
			if (inputs[i].checked=="true"){
				nbcoche++;
			}
		}
	}
	if (nbcoche<nbradio){
		window.alert("vous n'avez pas cocher "+(nbradio - nbcoche)+" boutton");
	}
}

[ProgElecT]

Salut

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

			if (inputs[i].checked===true){

me semble t il.

[Watilin]

Salut,

Si ce n’est pas déjà fait, ouvre ta consoque avec F12 et exécute ces lignes une par une, en essayant de comprendre pourquoi tu obtiens ces résultats. Ce sont des indices qui t’aideront à comprendre ton problème et à le résoudre.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
typeof true
typeof "true"
true == "true"
typeof document.getElementsByTagName("input")[0].checked

[myst28]

Si tu pense que le probleme vient des guillemet du true sache que j ai deja essayer de les enlever et sa marche pas plus

[ProgElecT]

Salut

Ton code if (inputs[i].checked=="true"){.
Mon codeif (inputs[i].checked===true){. trois caractères =, pour autant je ne prétend pas que c'est la solution

[myst28]

Fonctionne pas non plus

[Beginner.]

Salut,

En plus de ce qui a déjà été dit (true et non "true") il y a un problème dans la condition de la boucle for : for (var i=0;i<=inputs.length;i++)...

Il ne faut pas mettre "<=" mais "<" comme ceci : for (var i = 0; i < inputs.length; i++)...

PS : On peut se passer de la condition if (inputs[i].type == "radio") et de nbradio++; si dés le départ on cherche les input de type "radio" avec : var inputs = document.querySelectorAll("input[type=radio]");...

Le nombre de bouton radio s'obtient alors avec : var nbradio = inputs.length ;

[Beginner.]

On peut raccourcir encore : le nombre de bouton coché peut s’obtenir avec var nbcoche = document.querySelectorAll("input[type=radio]:checked").length;...

La fonction peut être raccourcie comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
function checkNumberCheckedRadio() {
    var nbradio = document.querySelectorAll("input[type=radio]").length;
    var nbcoche = document.querySelectorAll("input[type=radio]:checked").length;
    if (nbcoche < nbradio) {
        window.alert("vous n'avez pas cocher " + (nbradio - nbcoche) + " boutton");
    }
}

[Watilin]

Fonctionne pas non plus

Ce genre de réponse est inutile. As-tu un message d’erreur dans la console ?

[myst28]

Un message d erreur non c est juste que la fenetre d alerte ne s ouvre pas

[Beginner.]

J'ai bien un message d'erreur qui se produit dans mes tests et c'est dû à ce que j'ai expliqué au début de mon message 7... Une fois que j'ai réglé ce problème le message (alert) s'affichait bien alors que ce n'était pas le cas avant...

Mais bon apparemment vous n'avez pas tenu compte de cette correction et du coup votre problème ne risque pas de se régler...

[myst28]

Merci c bon sa fonctionne (entre les erreurs et l ordinateur qui fonctionne mal sa vraiment etait galere)