Discussion :

[rozierromain]

Bonjour,

j'ai un fichier php qui génère des divs dynamiquement, et qui ressemble à cela

Code php :

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

1
2
3
4
5
6
7
<?php
do { ?>
<div id = "<?php echo $numero ?>">
<a onclick="colornote(<?php echo $numero ?>)">colorer</a>
<?php
} while (condition...)
?>

la fonction javascript utilisée :

Code :

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

1
2
3
4
5
function colornote(ladiv)
{	
alert('Coloration de la div '+ladiv);
document.getElementById(ladiv).style.backgroundColor='#808080'; 
}

La première partie de la fonction fonctionne bien (alert), et me donne bien le bon ID de la div, en revanche getElementById(ladiv) ne fonctionne pas.


Une idée pour m'aider ??

Merci d'avance

[Bovino]

J'espère que la partie PHP du code donné n'est pas du copier/coller car il y a des erreurs de syntaxe...

Sinon :

Code :

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

1
2
<div id = "<?php echo $num ?>">
<a on onclick="colornote(<?php echo $numero ?>)>colorer</a>

Essaye en utilisant les mêmes variables

[rozierromain]

j'ai recopié quelque chose de simplifié dans le post, mais ce n'est pas l'erreur car dans ma source, les variables concordent !

merci quand même
code a priori corrigé

[Watilin]

Ça me choque moi aussi mais apparemment cette syntaxe marche.

Je propose plus propre :

Code :

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

1
2
3
4
5
6
7
<?php
do {
        echo "<div id=\"$num\">";
        echo "<a onclick=\"colornote($numero)\">colorer</a>";
        echo "</div>";
} while (condition…);
?>

Fermer les balises php au milieu d’un bloc, y’a rien de pire pour la maintenance

[rozierromain]

merci pour la syntaxe, je ne demande qu'à apprendre, mais ça ne résout pas mon problème de getElementById

[Watilin]

Ah, je crois que j’ai trouvé. C’est facile de s’emmêler les pinceaux entre PHP et JS.

getElementById demande une chaîne. Si tu lui passes un truc sans guillemets, il pense que c’est une variable…

Avec ma syntaxe, ça donnera ceci :

Code PHP :

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

1
2
3
4
5
6
7
<?php
do {
	echo "<div id=\"$numero\">";
	echo "<a onclick=\"colornote('$numero')\">colorer</a>";
	echo "</div>";
} while (condition…);
?>

J’ai rajouté des simple quotes dans colornote().

[SpaceFrog]

Apriori c'est un string que tu passes non ?

Code :

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

<a onclick="colornote('<?php echo $numero ?>')">colorer</a>