Discussion :

[slayess]

Bonjour à tous,

Je reprend un site qui utilise une fonction JS(ci-dessous) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
Object.defineProperty(AssociativeArray.prototype, "length", {
	get: function () {
		var count = 0;
		for (var key in this) {
			if (this.hasOwnProperty(key))
			count++;
			}
		return count;
		}
});

Qui retourne le nombre d'entrée dans un tableau (exemple ci-dessous ) :

Code :

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

1
2
3
 
marqueurs
	Object { poi_LrztJQQrtFOJIAk=Ol, poi_XEnqiMflxMZpZZo=Ol}

Cela fonctionne correctement sous FF, mais d'après ce que j'ai pu lire cela n'est compatible pour IE qu'après la version >= 9, sinon cela retourne une erreur du type :
Cet objet ne gère pas cette propriété ou cette méthode

Quelqu'un serait-il si il est possible de mettre un patch pour IE ?
Je ne suis pas à l'origine du code, éplucher tout le code s’avérait plutôt fastidieux.

Merci par avance à celui qui pourra m’éclairer.

[Watilin]

Le but de ce bout de code est de mettre un propriété length en lecture seule aux objets AssociativeArray, de telle sorte qu'elle renvoie toujours le bon nombre d'éléments lorsqu'elle est consultée.

Sous IE<9, une telle façon de faire n'est pas possible. Le plus propre serait de passer par une fonction classique qui fera office de getter :

Code :

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

1
2
3
4
5
6
7
8
9
AssocitativeArray.prototype.getLength = function () {
	var count = 0;
	for (var key in this) {
		if (this.hasOwnProperty(key))
		count++;
		}
	return count;
	};
}

Mais cela implique de réécrire tous les passages du code qui utilisent directement length pour leur faire appeler getLength() à la place, et apparemment tu préfères éviter ça.

J'ai pensé à des rustines à base de setInterval ou try/catch, mais quand on réfléchit bien on se rend compte que ça ne peut pas marcher. Du coup je ne vois pas d'autre solution…

[Slayes]

Merci de pour votre réponse.
Je vais tout de même faire le tour du code pour faire les modifs, ça sera plus propre.

Bien que ce soit 2 manières de faire différentes, par curiosité, connaitrais tu les avantages de l'une ou l'autre de ces méthodes. A part que l'une n'est compatible IE que si >= 9

[SylvainPV]

Je pense que le principal avantage de defineProperty est le confort d'écriture et le fait de ne pas polluer le scope en devant y déclarer les fonctions getter et setter.

Pour les bibliothèques, c'est une aubaine. Ils peuvent fournir leurs propres propriétés plutôt que toujours des méthodes. Par exemple, plutôt que la méthode val() de jQuery qui est un setter ou un getter selon si l'on passe un argument ou pas, on pourrait envisager de définir une propriété spéciale comme $value.

Code :

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

1
2
3
 
var resultat = calculFoireux(); //oups, mon calculFoireux a renvoyé la valeur undefined
$("#monChamp").val(resultat); //le code ne renvoie aucune erreur, il est interprété comme un getter et donc le résultat ne change pas

Code :

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

1
2
3
 
var resultat = calculFoireux(); //oups, mon calculFoireux a renvoyé la valeur undefined
$("#monChamp").$value = resultat; //ici aucune ambiguité possible, c'est bien un setter

[Slayes]

Merci pour ces précisions
Sujet résolu.