Discussion :

[askrim]

Bonjour,

J'ai besoin d'aide pour un problème de sélecteur(je pense): mon code récupère les nom de label, les envoie via la fonction ajax qui me renvoie un liste json des contenu, à partir de ca je crée une image et affiche son title via la fonction tooltip....tout va bien jusqu'à la ligne souligné où Firebug m'affiche

TypeError: listField[j].after is not a function

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
$(document).ready(function() {
	var listReq = $("form label").filter(".required");
	var listOpt = $("form label").filter(".optional");
	var listField = $.merge(listReq,listOpt);
 
	var param = "sessionid" + document.location.href.split("sessionid")[1];
 
	var list=new Array();
	for(var i=0;i<listField.length;i++ ){
		list.push(listField[i].getAttribute("for"));
	}
 
	$.ajax({
		type: "POST",
		url: "/@@appid@@/App-Saas-Commun_Aide/Form?"+param,
		data: "list="+list,
		success: function(results){
			var fields = JSON.parse(results);
			for(var i=0;i<fields.length;i++ ){
				for(var j=0;j<listField.length;j++ ){
					if(listField[j].getAttribute("for").split('__')[1] == fields[i].FIELD){
						listField[j].after('<img title="' + fields[i].CONTENT + '" src="/@@appid@@/Style/IMG/bt_help_gris.gif" height="17" width="17">');
						listField[j].next('img').tooltip();
					}
				}
			}
		}
	});
});

Merci d'avance

[Bovino]

Ton if

Code :

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

if(listField[j].getAttribute("for").split('__')[1] == fields[i].FIELD)

indique que listField[j] est un élément DOM, pas un objet jQuery et dans ce cas, il ne possède effectivement pas de méthode .after().

[askrim]

Merci de ta réponse bovino!

La solution si ca peut aider:

Code :

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

1
2
3
4
5
6
7
8
for(var j=0;j<listField.length;j++ ){
	if(listField[j].getAttribute("for").split('__')[1] == fields[i].FIELD){
	        var jListField = $(listField.get(j));
		jListField.after('<img title="' + fields[i].CONTENT + '" src="/@@appid@@/Style/IMG/bt_help_gris.gif" height="17" width="17">');
		jListField.next('img').tooltip();
	}
}

[Bovino]

Code :

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

var jListField = $(listField[j]);

me semble plus simple et approprié...

[askrim]

Oui c'est vrai, c'est dû à un copier-coller, mais il ya une grande différence en terme de vitesse ou de coût???

[Bovino]

Sur des scripts aussi légers, les différences de performances existent, mais sont insignifiantes, seulement c'est maladroit.
D'ailleurs, ma solution n'est pas optimale non plus puisque listField est déjà une collection d'objets jQuery, donc

Code :

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

listField.eq(j);

est suffisant.
Ce qu'il faut comprendre, c'est qu'à chaque fois que tu passes par $(), tu forces ton script à réexécuter toute la moulinette jQuery pour transformer le paramètre en objet jQuery, ce qui est particulièrement inutile si tu pars déjà d'un objet jQuery.