Discussion :

[SICarmy]

Bonjour, j'ai cette petite fonction qui me permet de verifier un <textarea id="autre"> et je regarde si il est vide ou non.

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
 
function verifAutre()
{	
             var content
	var FS=document.Autorisation.getElementsByTagName("fieldset");
 
	for(var i=0; i<FS.length; i++){
		content  = FS[i].getElementById("autre");
		if (content.value == "")
			{
			return false;
			}
	}
	return true;
}

Mon problème vient du fait qu'il ne veux pas affecter a content la valeur car "Erreur : Cet objet ne gère pas cette propriété ou cette méthode" au niveau du getElementById.

Si je fais document.getElementById("autre"), ca marche sauf que je veux pas dans tout le document mais seulement le limiter a mon fieldset. Merci

PS : Je dois dévellopper ca sous IE8 (entreprise :/)

[Bovino]

• Un id doit être unique dans la page.
• getElementById() est une méthode de l'objet document et rien d'autre.

[SICarmy]

Ahh ok, et si je veux récuperer le même champ "autre", il faut mieux utiliser des classes ou des noms?

[Kaamo]

Comme l'a mentionné Bovino, document.getElementById("autre") et s'assurer que l'id "autre" est unique dans la page.

[SICarmy]

Comme l'a mentionné Bovino, document.getElementById("autre") et s'assurer que l'id "autre" est unique dans la page.

Oui mais j'ai plusieurs champs autre dans plusieurs Fieldset. Et je veux la même fonction pour gerer tout les fieldsets et champs autres d'un coup

[Bovino]

Si tu veux récupérer plusieurs champs, il vaut mieux passer par une classe et getElementsByClassName() en faisant attention aux versions de IE non compatibles (8 et inférieures) et du fait que cette méthode renvoie une collection.

[Kaamo]

J'ajouterai que si tu as plusieurs fieldset qui contiennent, chacun, un textarea, les id de ces derniers doivent quand même être différents.
getElementsByTagName sinon ? Qui est supporté jusqu'à IE6 si je ne m'abuse.

[SICarmy]

J'ai essayé mais j'ai vite abandonné quand j'ai vu que ce n'était pas compatible IE8!

J'ai fait ca :

Code :

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

1
2
3
4
5
6
7
8
9
10
var content
content  = document.querySelectorAll(".autre");
	for (var i=0; i < content.length; i++)
	{
		if (content[i].value == "")
		{
			return false;
		}
	}
return true;

Et ca marche, merci des suggestions par rapport aux IDs etc. J'ai changé tout mes ID pour qu'ils soient uniques, et j'ai tout remplacé par des classes.