Discussion :

[exe2bin]

Re-bonjour ,me re-voilo� :

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

S�lectionner tout - Visualiser dans une fen�tre � part

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

tout fonctionne parfaitement

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

S�lectionner tout - Visualiser dans une fen�tre � part

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

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

[exe2bin]

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 ...

[exe2bin]

j'ai chang� les noms de champs et m�me le nom de la variable mais toujours rien !

[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.

[exe2bin]

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

[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

[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 !

[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.

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

[exe2bin]

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 :

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
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 !?

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

S�lectionner tout - Visualiser dans une fen�tre � part

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 :

S�lectionner tout - Visualiser dans une fen�tre � part

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 :

S�lectionner tout - Visualiser dans une fen�tre � part

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.

[exe2bin]

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.

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

vrai aussi mais ce n'est pas :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

TDs.className=g;

mais bien :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

TDs[i].className=g;

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 ?

[NoSmoking]

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

[exe2bin]

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 :

S�lectionner tout - Visualiser dans une fen�tre � part

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 :

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
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();
		}
	}
}

[Watilin]

Parce que formulaire.genre est un tableau

Code Firebug :

S�lectionner tout - Visualiser dans une fen�tre � part

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.

[NoSmoking]

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.

[exe2bin]

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