Discussion :

[seriux55]

Bonsoir,

Voici mon code,

Du code HTML :

Code html :

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

1
2
<h3 id="miss"><a href="">Messages</a></h3>
<div id="lesmessages"></div>

Le code JS :

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
$("#miss").on('click',function(){
 
	if(timermsg !='') clearTimeout(timermsg);
	refreshlistmsg();
});
 
function refreshlistmsg(){
 
	jQuery.ajax({
		type: 'POST', 
		url:  'controllers/accueil/messages.php', 
		dataType : 'json',
		success: function(jsonn) {
 
			var zzz     = JSON.stringify(jsonn);											
			var objj    = eval('(' + zzz + ')');
			var msgajax = '';
			var iddd    = '';
			for( var z in objj ) {
 
				var ligne = objj[z];
				var idm   = 'msg' + ligne.id;
 
				msgajax  +='<a href="" id="' + idm + '" return false;">';
 
				iddd      = "#msg" + ligne.id;				
 
				$(' " ' + iddd + ' " ').on('click',function(){
 
					alert('oki');
					refreshmsg(titi);
				});
				msgajax += 'Hello';															
			}
			document.getElementById('lesmessages').innerHTML = msgajax;											
		} 
	})
	timermsg = setTimeout("refreshlistmsg()", 20000);		
}
 
function refreshmsg(titi){														
	jQuery.ajax({
		type: 'POST', 
		url:  'controllers/accueil/msg.php', 
		dataType : 'json',
		data: { id: titi },
		success: function(json) {
 
			var xxx  = JSON.stringify(json);														
			var obj    = eval('(' + xxx + ')');
			var msgajx = '<ul>';
			for( var x in obj ) {
				var ligne = obj[x];
				msgajx += '<li><div class="amessage"><img src="images/man.jpg" alt="" /><div class="amgcont"><div class="amesshead"><h3>Zinedine Zidane</h3><a>'+ ligne.depot +'</a><a>En ligne</a></div><p class="inpros">'+ ligne.msg +'</p></div></div></li>';
			}
			msgajx += '</lu>';
			document.getElementById( "msgajax" ).innerHTML = msgajx;															
		} 
	});
	timer = setTimeout("refreshmsg(titi)", 20000);
}

Mon soucis est dans la ligne 28:

Code :

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

$(' " ' + iddd + ' " ').on('click',function(){

Ce que je veux en fait, est exécuter une fonction au moment ou je clique sur le lien qui a comme id : iddd= "#msg" + ligne.id;

Et comme vous l'avez constaté iddd varie,

Le code comme il est m'affiche l'erreur : Error: Syntax error, unrecognized expression: " #msg16 "

sachant que 16 est l'id récupérer depuis la base.

Merci :-)

[thelvin]

Et... Ils sont censés servir à quoi, les guillemets autour de iddd ?

[Invité]

Bonjour,
Mets une class commune au lieu des ids.
Et id dans un attribut (data par ex.) si tu as besoin de le récupérer :

Code :

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

1
2
 '<a href="" class="msg" data="'+ligne.id+'">';
$(".msg").on(......

N.B. Tu mixes aussi du JavaScript pur avec du jQuery ?

Code js :

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

document.getElementById('lesmessages').innerHTML = msgajax;

S'écrit en jQuery :

Code js :

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

$('#lesmessages').html(msgajax);

[Bovino]

Code :

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

$(' " ' + iddd + ' " ')

Là, tu demandes de récupérer les éléments dont l'id est (si iddd vaut foo) : " foo " et non ceux dont l'id est foo...

[SpaceFrog]

si c'est un id ... => #

Code :

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

$(' # ' + iddd ).on('click',function(){

$ attend juste un string de selecteur

en temps normal tu aurais ecrit:

Code :

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

$(' #monid').on('click',function(){

il ne faut donc pas aller chercher midi à 14 heures ...

Code :

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

1
2
mavar="monid";
$(' # ' + mavar ).on('click',function(){

[Invité]

@SpaceFrog
Je pense que tu n'as pas bien lu son script : le # s'y trouve bien

Code js :

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

iddd      = "#msg" + ligne.id;

Comme l'a dit Bovino, la syntaxe correcte est :

Code js :

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

$( iddd ).on('click',function(){

Il me semble néanmoins que passer par une class est plus judicieux (cf mon message précédent).

[SpaceFrog]

si le # y est ...

Code :

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

1
2
mavar="monid";
$( mavar ).on('click',function(){