Discussion :

[bastian06n]

Bonjour,
J'ai une fonction pour savoir si l'on a bien remplit tout les inputs type 'radio' d'un formulaire, il fonctionne bien sous FF et Chrome mais pas IE.
Est ce que quelqu'un peut y jeter un coup d'oiel s'il vous plait

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
function valid_form(form){
	var questions = getRadios(form);
	for(var i in questions ){
		//objet radio
		var radio = form[questions[i]];
		if(isRadioChecked(radio) == false ){
			return false;
		}
	}
	return true;
 }
 
function getRadios(form){
	var is = form.getElementsByTagName("input");
	var tab = new Array();
	for(var i = 0;i<is.length;i++){
		var t = is[i].getAttribute("type");
		if(t=="radio"){
			var name = is[i].getAttribute("name");
			if(!tab.contains(name)){
				tab.push(name);
			}
		}
	}
	return tab;
}
 
 function isRadioChecked(radio){
	var c = radio.length;
	while(c>0){
		c--;
		if(radio[c].checked) {
			return true;
		}
	}
	alert('Toutes les questions doivent être Remplit');
	return false;
}

Pour vérifier mon formulaire je l'appel de la façon suivante:

Code :

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

onSubmit='return valid_form(this);'

Une idée?

Merci

[javatwister]

tu as ta fonction getRadios() quelque part?

sinon, ton nommage de paramètres est un brun dangereux (form, radio...)

[bastian06n]

tu as ta fonction getRadios() quelque part?

sinon, ton nommage de paramètres est un brun dangereux (form, radio...)

Ah, comment ca dangereux?

[SpaceFrog]

Code :

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

var radio = form[questions[i]];

une syntaxe plus rigoureuse serait moins dangereuse et plus compatible ...

on peut voir le html des elements du form ?

[javatwister]

il fonctionne bien sous FF et Chrome mais pas IE

qu'est-ce qui te le prouve?

pourquoi places-tu un return dans une boucle while?

peut-on voir la fonction getRadios()?

[bastian06n]

on peut voir le html des elements du form ?

Oui les voila (c'est une boucle php qui les affiches les id sont des entier:

Code :

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

<input  type="radio" name="question_<?php echo $row['id'];?>" value="5"/>

PS: j'ai rajouté la fonction getradio() j'avais omit de la mettre

[bastian06n]

qu'est-ce qui te le prouve?

pourquoi places-tu un return dans une boucle while?

peut-on voir la fonction getRadios()?

1) Ce qui me le prouve c'est que IE m'envoie sur la page d'enregistrement du formulaire au lieu de me faire l'alerte lorsque je ne remplit pas tout les boutons radio

2) le return sert à savoir si les boutons sont checked, il return true si checked sinon il fait un false et une alerte

[javatwister]

à l'arrache, getAttribute va poser pbm sous IE7;

à part ça, je trouve abusif d'utiliser 3 fonctions pour tester une série de radio!

[bastian06n]

à l'arrache, getAttribute va poser pbm sous IE7;

à part ça, je trouve abusif d'utiliser 3 fonctions pour tester une série de radio!

Si tu as autre chose je veux bien que tu me montre, ce n'est pas mon code je l'ai adapter.

[javatwister]

je ne comprends pas non plus ton radio.length

[SpaceFrog]

Code :

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

<input  type="radio" name="question_<?php echo $row['id'];?>" value="5"/>

=>

Code :

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

1
2
3
4
 
document.forms["name_du_form"].elements["question_1"]
//ou si il y en a plusieurs portant le même name
document.forms["name_du_form"].elements["question_1"][indice]

[bastian06n]

je ne comprends pas non plus ton radio.length

ca sert a recuperer la liste des inputs radio pour chaque name

[javatwister]

stp, simplement le html généré avec ta série de boutons!

[bastian06n]

stp, simplement le html généré avec ta série de boutons!

<input type="radio" name="question_1" value="0"/>
<input type="radio" name="question_1" value="1.5"/>
<input type="radio" name="question_1" value="3"/>
<input type="radio" name="question_1" value="5"/>
.........

<input type="radio" name="question_34" value="0"/>
<input type="radio" name="question_34" value="1.5"/>
<input type="radio" name="question_34" value="3"/>
<input type="radio" name="question_34" value="5"/>

etc... autant de fois qu'il y a de question dans ma table sql; (le name est composé de 'question_'+Id de la question comme le dis le code php...)

[javatwister]

pourquoi tu ne mets pas un checked="checked" par défaut sur l'un des éléments de chaque série?; le client ne pourra pas tout décocher!

[bastian06n]

Il pourrait penser avoir déjà remplit cette question

[Watilin]

Salut,
au vu de la façon dont tu t’en sers, je suppose que ta variable questions est un tableau contenant les attributs name de tes checkboxes. Vérifie que ces attributs name sont bien tous correctement définis.

Sinon le problème peut venir du cache, qui est un peu plus tenace chez IE que chez FF… On ne sait jamais

En pièce jointe, avec une fonction getRadios extrapolée, j’ai fait un document qui marche sous FF et IE8.
Attention toutefois, la fonction isRadioChecked n’est pas capable en l’état de traiter les checkboxes toutes seules (voir ligne 54).



Edit: Ouh mazette, une foule de réponses en si peu de temps ^^ Hum, bon, il ne me semble pas que ma réonse fasse doublon avec une précédente…

[SpaceFrog]

Code :

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

document.forms["name_du_form"].elements["question_1"].length

[bastian06n]

Code :

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

document.forms["name_du_form"].elements["question_1"].length

Oui et? Je comprends pas...

[bastian06n]

Vérifie que ces attributs name sont bien tous correctement définis.

Salut,
les names sont bien tous définit :/