concatener id et getElementById

**rozierromain** · 07/10/2010, 10h59

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** · 07/10/2010, 11h10

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** · 07/10/2010, 11h23

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** · 07/10/2010, 11h26

Ç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** · 07/10/2010, 11h30

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

**Watilin** · 07/10/2010, 11h46

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** · 07/10/2010, 12h08

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>

**rozierromain** · 07/10/2010, 12h30

oui, c'est une string.

J'ai essayé de modifier la fonction, mais c'est pas mieux, je suis toujours bloqué :

Code :

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

1
2
3
4
5
6
function colornote(ladiv)
{
	var ladiv2 = " ' "+ladiv+" ' ";
	alert(ladiv2);
	document.getElementById(ladiv2).style.backgroundColor='#808080'; 
}

**andry.aime** · 07/10/2010, 12h34

Bonjour,

Code :

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

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

un point virgule après l'echo.

En élément possède-t-il l'id passé en paramètre?

A+.

**rozierromain** · 07/10/2010, 12h37

Envoyé par andry.aime

Bonjour,

Code :

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

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

un point virgule après l'echo.

En élément possède-t-il l'id passé en paramètre?

A+.

oui oui !

**andry.aime** · 07/10/2010, 12h39

Peut-on voir le code HTML généré ?

**rozierromain** · 07/10/2010, 12h51

le code source réel :

Code php :

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
19
20
21
22
23
24
25
26
27
<?php
	do {
		$nomdiv = $row_NOTE['NUMNOTECLIENT'];?>
        <a onclick="montrerlanote('<?php echo $nomdiv ?>')" style="cursor:pointer">
        <?php
		echo date("d-m-Y",$row_NOTE['DATE'])." - ".$row_NOTE['TITRE'];
		?>
        </a>
        <br />
        <div id="note_<?php echo $row_NOTE['NUMNOTECLIENT'];?>">
        contenu de la div <?php echo $row_NOTE['NUMNOTECLIENT'];?>
        </div>
        <?php
		}
	while ($row_NOTE = mysql_fetch_assoc($NOTE));};
 
?>
 
<script>
 
function montrerlanote(ladiv)
{
	var ladiv2 = " 'div_"+ladiv+" ' ";
	alert(ladiv2);
	document.getElementById(ladiv2).style.backgroundColor='#808080'; 
}
</script>

Le code HTML généré est :

Code html :

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

1
2
3
4
5
6
7
8
9
10
11
<a onclick="montrerlanote('152')" style="cursor:pointer">
07-10-2010 - 2
</a>
<br />       
<div id="note_152">contenu de la div 152</div>
 
<a onclick="montrerlanote('153')" style="cursor:pointer">
07-10-2010 - 3
</a>
<br />       
<div id="note_153">contenu de la div 153</div>