La variable invisible

Version imprimable

Re-bonjour ,me re-voilo� :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
function changeConges(formulaire)
{
	var mois=formulaire.mois.value;// récupérer les valeurs des champs Mois et Jour
	var jour=formulaire.jour.value;
	var type=formulaire.type.value;
	// récupérer le noeud du jour ciblé
	var TR=document.getElementById(mois);
	var TDs=TR.childNodes;
	for(var i=0;i<TDs.length;i++)
	{
		if((TDs[i]['id'])==jour)
		{
			TDs[i].className=type;
			TDs[i].innerHTML=type.toUpperCase();
		}
	}
}

la variable "type" semble �tre une variable fant�me : rien ne change.
Si je code en dur par exemple :

Code:

1
2
TDs[i].className='ma_classe';
			TDs[i].innerHTML='blablabla';

tout fonctionne parfaitement

12/10/2012, 18h28
Watilin

L� comme �a je vois pas. �a pourrait �tre un conflit avec les noms de tes champs de formulaire, mais comme tu es dans une fonction et tu utilises bien var, ce n'est pas �a.

En revanche, je vois une possible simplification :

Code:

if((TDs[i]['id'])==jour) …

Tu s�lectionnes un �l�ment par� Son id. �a ne te rappelle pas une fonction connue ?
12/10/2012, 18h45
exe2bin

Citation:

Tu s�lectionnes un �l�ment par� Son id. �a ne te rappelle pas une fonction connue ?

tu penses probablement � la fonction getElementById() ;
100% d'accord avec toi , j'am�liorerai cela en temps utiles ; je vais essayer de changer le nom du champ ...
12/10/2012, 19h06
exe2bin

j'ai chang� les noms de champs et m�me le nom de la variable mais toujours rien !
12/10/2012, 19h32
NoSmoking

Bonjour,
je ne vois pas l�int�r�t de parcourir les enfants de la TR pour r�cup�rer un seul �l�ment via son ID, autant le faire tout de suite.
12/10/2012, 19h39
exe2bin

Citation:

je ne vois pas l�int�r�t de parcourir les enfants de la TR pour r�cup�rer un seul �l�ment via son ID, autant le faire tout de suite.

Enti�rement d'accord mais ce n'est pas le probl�me.
Chaque chose en son temps
12/10/2012, 20h57
exe2bin

Citation:
je ne vois pas l�int�r�t de parcourir les enfants de la TR pour r�cup�rer un seul �l�ment via son ID, autant le faire tout de suite.

c'est parce qu'il existe plusieurs <td> avec le m�me ID
13/10/2012, 04h08
Watilin

C'est mal !
Un id doit �tre unique dans la page. Si ta page contient plusieurs id identiques, les navigateurs vont se comporter de mani�re ind�termin�e (bien que pr�visible) pour essayer de satisfaire ton script. Ce n'est pas une mani�re stable de programmer.

Un id doit �tre unique !
13/10/2012, 08h04
NoSmoking

v�rifies quand m�me qu'il n'y a pas un espace ou autre qui tra�ne, car comme l'a �crit Watilin les navigateurs vont quand m�me r�ussir l� o� ils nedevraient m�me pas essayer.

:idea: On a le droit d'avoir plusieurs id�es mais qu'un seul ID.

Hola !! je suis enti�rement ok avec les pratiques de bonne programmation concernant les id's uniques ou autre mais il faut avouer que �a m'arrange bougrement !
Consid�rons ce bout de code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 function changeConges(formulaire) { var mois=formulaire.mois.value;// récupérer les valeurs des champs Mois et Jour var jour=formulaire.jour.value; var g=formulaire.genre.value; // récupérer le noeud du jour ciblé var TR=document.getElementById(mois); var TDs=TR.childNodes; TDs.className=g; for(var i=0;i<TDs.length;i++) { if((TDs[i]['id'])==jour) { TDs[i].className=g; TDs[i].innerHTML=g.toUpperCase(); } } }
l'id recherch� dans les enfants de TR est unique ;m�me s'il existe plusieurs id semblables dans le document.
Sinon ,est il possible de remplacer les id's par autre chose ,comme un attribut quelconque ?
Cela dit je ne voudrais pas para�tre insolent mais je ne vois toujours pas le rapport avec la variable g et le fait que si je code en dur l'affectation de l'attribut et du texte de TDs tout fonctionne au poil !?

13/10/2012, 11h19
Bovino

Tout d'abord, sans voir le code HTML associ�, il y a des choses que l'on ne peut pas voir.
Par exemple, si le champ genre est de type radio, alors il y en a plusieurs et var g=formulaire.genre.value; n'est pas correct.

Ensuite, tu commets de grosses erreurs.

Code:

var TDs=TR.childNodes;

Non, les childNodes d'un tr, ce ne sont pas tous les td ! Ce sont les td ainsi que les diff�rents noeuds texte qui peuvent exister dans le DOM (par exemple les sauts de lignes).
Ensuite

Code:

TDs.className=g;

Tu ne peux pas affecter un nom de classe � une collection : donc erreur de script, il ne va pas plus loin...

Code:

for(var i=0;i<TDs.length;i++)

L� encore, dans la boucle, tu traites tous les �l�ments comme des �l�ments HTML alors que tous ne le sont pas.
13/10/2012, 12h08
exe2bin

Citation:

Non, les childNodes d'un tr, ce ne sont pas tous les td !

c'est vrai mais les noeuds texte ne poss�dent pas d'Id.

Citation:

Tu ne peux pas affecter un nom de classe � une collection

vrai aussi mais ce n'est pas :

Code:

TDs.className=g;

mais bien :

Code:

TDs[i].className=g;

Citation:

L� encore, dans la boucle, tu traites tous les �l�ments comme des �l�ments HTML alors que tous ne le sont pas.

Ah bon ! de quels types d'�l�ments parles-tu ?
13/10/2012, 17h06
NoSmoking

Tu peux parcourir la collection cells de la TR et faire un test sur la className.

Apr�s test des valeurs re�ues par l'argument de ma fonction il se trouve que les variables mois et jour sont ok mais que g est "undefined" ; pas de bol ! c'est celle qu'il me faut ! Qui peut me dire pourquoi cette valeur de formulaire n'est pas initialis�e ?
le formulaire :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<form name="formulaire" action="http://127.0.0.1/HTML/recap.php">
	CA<input type="radio" name="genre" value="ca"/>
	RH<input type="radio" name="genre" value="rh"/>
	RT<input type="radio" name="genre" value="rt"/>
	HP<input type="radio" name="genre" value="hp"/>
	JS<input type="radio" name="genre" value="js"/>
	FR<input type="radio" name="genre" value="fr"/>
	FP<input type="radio" name="genre" value="fp"/>
	RR<input type="radio" name="genre" value="rr"/>
	Otaf<input type="radio" name="genre" value="otaf"/><br/>
	Mois :<input type="text" name="mois" maxlength="10" size="10"/>
	Jour :<input type="text" name="jour" maxlength="2" size="2"/>
	Duree :<input type="text" name="duree" maxlength="2" size="2"/><br/>
	<button onclick="changeConges(document.formulaire);return false;">OK</button>
</form>

La fonction :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
function changeConges(formulaire)
{
	var mois=formulaire.mois.value;// récupérer les valeurs des champs Mois et Jour
	var jour=formulaire.jour.value;
	var g=formulaire.genre.value;
	window.alert('mois : '+mois+'\njour : '+jour+'type : '+g);
	// récupérer le noeud du jour ciblé
	var TR=document.getElementById(mois);
	var TDs=TR.childNodes;
	for(var i=0;i<TDs.length;i++)
	{
		if((TDs[i]['id'])==jour)
		{
			TDs[i].className=g;
			//TDs[i].innerHTML=g.toUpperCase();
		}
	}
}

13/10/2012, 20h02
Watilin
Parce que formulaire.genre est un tableau ;)
Code:

1 2 >>> formulaire.genre [input ca, input rh, input rt, input hp, input js, input fr, input fp, input rr, input otaf]
Il faut que tu parcoures ce tableau pour savoir lequel de ces input est checked.
14/10/2012, 00h54
NoSmoking

Citation:

Envoy� par Bovino

Tout d'abord, sans voir le code HTML associ�, il y a des choses que l'on ne peut pas voir.
Par exemple, si le champ genre est de type radio, alors il y en a plusieurs et var g=formulaire.genre.value; n'est pas correct.

:applo:
14/10/2012, 11h08
exe2bin

Merci � Watilin qui a per�u le probl�me pour sa clart� !
:ccool: