Piège JavaScript avec undefined et les closures

Vous avez fort probablement déjà utilisé la propriété undefined en JavaScript pour vérifier si une variable a été assignée ou pas ? Considérez simplement l'exemple suivant qui déclare ce genre de fonction :

var app = new Object();
var x = '';
var y;

app.isDefined = function(value){
   return value !== undefined;
}

console.log( app.isDefined(x) ? 'defined' : 'undefined' );
console.log( app.isDefined(y) ? 'defined' : 'undefined' );

Logiquement, vous vous attendez à ce que la variable x soit définie alors que y ne le soit pas. Si oui, vous avez raison, c'est tout à fait le comportement attendu. Mais qu'arriverait-t-il si on définissait volontairement une variable undefined dans ce script ou involontairement lors de l'inclusion d'une librairie externe ? Le concept de closure viendrait introduire une vulnérabilité :

var app = new Object();
var x = '';
var y;

var undefined = 18; // oups...

app.isDefined = function(value){
   return value !== undefined;
}

console.log( app.isDefined(x) ? 'defined' : 'undefined' );
console.log( app.isDefined(y) ? 'defined' : 'undefined' );

Dans ce cas-ci, les deux appels retourneraient 'defined'.

L'idéal serait de ne jamais se fier à la propriété undefined, à moins de la redéclarer à l'intérieur de la fonction qui compte l'utiliser :

var undefined = 18; // ok

app.isDefined = function(value){
   // sans danger pour le undefined dans la portée globale
   var undefined;
   return value !== undefined;
}

Ainsi, on évitera les mauvaises surprises reliées à ce genre de pratique.