Notation par points JavaScript

Question

La ligne suivante est apparemment écrite au mieux en notation point. J'essaie de nettoyer mon code JavaScript pour le rendre strict. Qu'est-ce que ça veut dire? J'ai ajouté un peu de contexte à ma ligne de code, au cas où cela aiderait? Je ne sais rien sur DOM. Je n'essaie pas de supporter Internet Explorer 4, ce n'est pas mon code et je ne serais pas capable d'écrire moi-même JavaScript. Je n'essaie de l'obtenir conforme et l'outil JSLint dit à ce sujet:

problème à la ligne 17 caractère 43: [ « dhtmltooltip »] est mieux écrit en notation par points.

Answer 1

Il existe two ways to access properties of an object in JavaScript.

notation Dot

foo.bar.baz 

notation CROCHET

foo['bar']['baz'] 

Vous utilisez cette dernière partie de votre code. Douglas Crockford, qui a écrit JSLint (un outil qui donne ce message d'erreur), est d'avis qu'il est préférable d'utiliser la notation par points lorsque cela est possible.

Answer 2

Ce qui suit semble être plus convivial.

var tipobj; 
if (document.all) 
    tipobj = document.all["dhtmltooltip"]; 
else if (document.getElementById) 
    tipobj = document.getElementById("dhtmltooltip"); 
else 
    tipobj = ""; 

Answer 3

Il utilise la capacité de vérification pour récupérer un élément avec l'id dhtmltooltip et retombant à un String vide s'il n'y a pas une capacité de faire la récupération.

MISE À JOUR: Comme d'autres l'ont souligné, le chèque de getElementById devrait être le premier, et pourrait probablement être omis puisque tout navigateur qui pourrait être appelé « moderne » avec un visage impassible a eu pendant longtemps.

MISE À JOUR 2: Avec le nouveau contexte, JSLint se plaint qu'il n'est pas document.all.dhtmltooltip. Vous devriez probablement juste réécrire le tout comme:

var tipobj = document.getElementById("dhtmltooltip"); 

et être fait avec.

Answer 4

Une recherche rapide sur Google indique que document.all n'est utilisé que pour prendre en charge IE4. Il s'agit d'un tableau qui permet au navigateur d'accéder à différentes parties du DOM (voir here.)

Le code que vous avez publié vérifie d'abord si document.all existe. Sinon, il définit tipobj à "". Maintenant, au-delà de ça, ça ne vaut pas vraiment la peine de déchiffrer la ligne que vous avez publiée, à moins que vous n'ayez vraiment besoin du support IE4. Étant donné que très peu de personnes utilisent encore IE4 et que ce morceau de code n'est conforme à aucune norme moderne, je laisserais tomber cette ligne et réglerais tipobj à "".

Answer 5

JSLint veut que ce soit:

var tipobj= document.all ? document.all.dhtmltooltip 
         : document.getElementById 
          ? document.getElementById("dhtmltooltip") 
          : ""; 

Mais de nos jours est tout à fait raisonnable de supposer que document.getElementById existe, il a été introduit sur le DOM Level Core 2 dès l'an 2000.

document.all est mort, à moins que vous essayez de soutenir vraiment vieux navigateurs comme IE4 (12 ans!!):

var tipobj = document.getElementById("dhtmltooltip"); 

Les deux extraits ci-dessus sont un bon exemple ab le coût de la complexité de soutien très old browser versions:

alt text http://ejohn.org/images/cost-benefit.png

Answer 6

Il semble que les seuls vrais problèmes sont la mise en forme/syntaxe. Cela devrait fonctionner exactement de la même manière et se conformer aux meilleures pratiques de javascript. La différence principale est l'utilisation de la notation par points javascript au lieu de la notation de parenthèse.

if (ie || ns6) { 
    var tipobj = document.all ? document.all.dhtmltooltip : document.getElementById ? document.getElementById("dhtmltooltip") : ""; 
} 

Answer 7

pourquoi ne pas simplement utiliser:

var tipobj = dhtmltooltip.id 

Je ne sais pas pourquoi la version longue est nécessaire à moins que la notation des points ne fonctionne pas dans tous les navigateurs?

Answer 8

Si la notation par points est un problème, vous pouvez toujours définir l'option/* jslint sub: true */pour la remplacer.

Answer 9

Comme il a été répondu par Quentin both ways are valid.

Une des raisons pour lesquelles je préfère utiliser elem.bar au lieu de elem['bar'] est qu'il enregistre 3 caractères. Certes, ce n'est pas une grande amélioration, mais un 3 points gratuits par affectation n'est pas mauvais.