Script ne fonctionne pas sous IE check de bouton radio

Version imprimable

Voir 40 message(s) de cette discussion en une page

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:

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:

onSubmit='return valid_form(this);'

Une idée?

Merci

03/08/2010, 16h02
javatwister

tu as ta fonction getRadios() quelque part?

sinon, ton nommage de paramètres est un brun dangereux (form, radio...)
03/08/2010, 16h05
bastian06n

Citation:

Envoyé par javatwister

tu as ta fonction getRadios() quelque part?

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

Ah, comment ca dangereux?
03/08/2010, 16h09
SpaceFrog

Code:

var radio = form[questions[i]];

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

on peut voir le html des elements du form ?
03/08/2010, 16h13
javatwister

Citation:

Envoyé par bastian06n

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()?
03/08/2010, 16h15
bastian06n

Citation:

Envoyé par SpaceFrog

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:

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

PS: j'ai rajouté la fonction getradio() j'avais omit de la mettre
03/08/2010, 16h19
bastian06n

Citation:

Envoyé par javatwister

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
03/08/2010, 16h19
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!
03/08/2010, 16h23
bastian06n

Citation:

Envoyé par 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!

Si tu as autre chose je veux bien que tu me montre, ce n'est pas mon code je l'ai adapter.
03/08/2010, 16h23
javatwister

je ne comprends pas non plus ton radio.length

Code:

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

=>
Code:

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]

03/08/2010, 16h25
bastian06n

Citation:

Envoyé par javatwister

je ne comprends pas non plus ton radio.length

ca sert a recuperer la liste des inputs radio pour chaque name
03/08/2010, 16h27
javatwister

stp, simplement le html généré avec ta série de boutons!
03/08/2010, 16h31
bastian06n

Citation:

Envoyé par javatwister

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...)
03/08/2010, 16h32
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!
03/08/2010, 16h35
bastian06n

Il pourrait penser avoir déjà remplit cette question
03/08/2010, 16h40
Watilin

1 pièce(s) jointe(s)

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 :P

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… 8-)
03/08/2010, 16h42
SpaceFrog

Code:

document.forms["name_du_form"].elements["question_1"].length
03/08/2010, 16h45
bastian06n

Citation:

Envoyé par SpaceFrog

Code:

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

Oui et? Je comprends pas...
03/08/2010, 16h47
bastian06n

Citation:

Envoyé par Watilin

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

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

Voir 40 message(s) de cette discussion en une page