Discussion :

[Seeverine]

Bonjour a tous !

Je viens demander votre aide aujourd'hui parce que j'ai un petit problème avec un switch que j'ai placé dans une fonction js. Voila il n'exécute pas toutes les possibilités, je m'explique:
il y a 3 cas : ma valeur vaut 1, ma valeur vaut 2 et le "default". Le problème est que le "default" est le seul cas à etre pris en compte :s

Le but de ma fonction est de changer la classe de ma cellule selon cette fameuse valeur donc forcément je me retrouve toujours avec la classe par défaut ..

Est-ce que quelqu'un peut m'aider à trouver l'erreur svp ?? Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
<script>
function changeClasse(valeur,indicateur){
	switch(indicateur){
	case '1': document.getElementById('donnees'+valeur).className='rouge'; break;
	case '2': document.getElementById('donnees'+valeur).className='bleu'; break;
	default: document.getElementById('donnees'+valeur).className='blanc';
	}
}
</script>

Code php :

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

1
2
3
4
5
while($donnees = $qry->fetch()){
	$indicateur = $donnees[0];
	echo $indicateur;
	echo "<script>changeClasse(".$i.$j.",".$indicateur.")</script>";
}

[sekaijin]

l'erreur est de tout mélanger

php s'exécute sur le serveur javascript sur le poste du client

pourquoi le php ne produit-il pas le html correct ? Oui si tu préfère pourquoi php produit un html avec des cellules qui on la classe par défaut
pour ensuite générer un javascript
qui lui devra remettre la bonne classe
ne serait-il pas plus simple de produire directement la bonne classe dans les cellules ?

de plus ton code laisse supposer que tu parcours ton ensemble de données pour les mettre dans les cellules
et que tu le parcours encore une fois pour générer le javascript.

Je ne vais pas corriger les erreur de php de ton code
ni t'aider à corriger la multitude d'erreur dans ta solution

la conception est bien trop foireuse.
Je te conseil donc de simplement affecter en pjp la classe qui va bien au moment ou le php génère le source HTML pour remplir la table.

A+JYT

[Bovino]

Sans vouloir laisser penser que les conseils de sekaijin peuvent ne pas être appliqués, un petit rappel quant au switch.

Un switch va te permettre de comparer une variable (ici indicateur) avec différentes valeurs.
Mais les comparaisons faites sont des comparaisons strictes (équivalent de ===). Or ton script PHP laisse à penser que la fonction générée par ton PHP appelle la fonction JavaScript avec des paramètres numériques, donc que tu compares des numériques avec des chaines (1 === '1') donc tous les case renvoient faux et tu ne vas que dans le default.