IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

JavaScript Discussion :

La méthode getElementById() est devenue inutile ?


Sujet :

JavaScript

Vue hybride

Message précédent Message précédent   Message suivant Message suivant
  1. #1
    Membre extrêmement actif Avatar de psychadelic
    Profil pro
    Inscrit en
    Mai 2010
    Messages
    2 532
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Mai 2010
    Messages : 2 532
    Par défaut La méthode getElementById() est devenue inutile ?
    exemple
    Code html : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    <!DOCTYPE html>
    <html lang="fr">
    <head><meta charset="UTF-8" /></head>
    <body>
      <span id="truc">texte initial...</span>
      <script>
        truc.innerText = 'bla bla bla';
      </script>
    </body></html>
    et ça marche!
    J'ai "rapidement et mal" cherché , je n'ai trouvé aucune new ou info sur le sujet.
    l'impression d'avoir raté un train..

    Est-ce LA bonne pratique et on doit abandonner la méthode getElementById ??

  2. #2
    Membre Expert
    Homme Profil pro
    Inscrit en
    Octobre 2011
    Messages
    2 910
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations forums :
    Inscription : Octobre 2011
    Messages : 2 910
    Par défaut
    Salut,

    Ce point a déjà été discuté dans ce forum mais je ne sais plus où...
    Cette pratique peut poser problème*, NoSmoking en avait parlé mais je ne sais plus où...

    *Que se passe-t-il si l'ID correspond au nom d'une variable ? Ou si L'ID vaut "document" ou "windows" ou... ?

    PS : Ailleurs on a ceci : Directly reference HTML elements... Dans lequel il y a les liens de deux autres fils...

  3. #3
    Invité
    Invité(e)
    Par défaut
    Bonjour,

    1- il ne faut pas confondre "ce qui est possible" et "ce qui est recommandé".
    Ce n'est pas parce qu'on PEUT traverser en dehors des clous qu'on DOIT le faire...


    2- J'ajoute qu'à partir du moment où on laisse le programme "décider" ce qu'il doit faire, on en perd la maitrise.
    D'où l'importance d'utiliser "use strict", et de toujours déclarer les variables.


    N.B. En ne lisant que le titre ("La méthode getElementById() est devenue inutile ?"), j'avais plutôt pensé :
    Oui, dans le sens où on peut désormais utiliser à la place querySelelector().
    Dernière modification par Invité ; 19/11/2018 à 10h35.

  4. #4
    Membre extrêmement actif Avatar de psychadelic
    Profil pro
    Inscrit en
    Mai 2010
    Messages
    2 532
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Mai 2010
    Messages : 2 532
    Par défaut
    Perso, je trouve que tout ce qui peut alléger le code est bienvenu, et permet d'éviter des erreurs de codage.

    Donner des noms explicites au variables est vital pour la survie d'un code; voit trop souvent des choses du genre
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    Prix_1 =  document.getElementById("Prix_1");
    Prix_2 =  document.getElementById("Prix_2");
    Prix_3 =  document.getElementById("Prix_3");
    alors qu'il s'agit tout simplement de Prix_Hors_Taxe, montant_TVA et Prix_net...

    et franchement quel est l'intérêt d'écrire en 2 lignes ce qui peut se faire en une seule, et qui divise par 2 le risque de faute de frappe au clavier
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    Prix_Net =  document.getElementById("Prix_Net");
    Prix_Net.innerText =...
    en revanche, pour la minification optimisée de code, c'est pas forcément un cadeau, sauf à minifier aussi le code HTML et le code CSS...
    d'ailleurs je me demande si ça existe (un tripe minifieur [html,css,js] qui puisse y réaffecter les noms de variables longues) ??

  5. #5
    Invité
    Invité(e)
    Par défaut
    1- Personne t'empêche d'écrire :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    document.getElementById("Prix_Net").innerText .....
    Après... tu fais comme tu veux.

    2- Je ne vois pas bien le rapport entre "Donner des noms explicites" et TA question initiale...
    Si tu trolles tes propres discussions...

    3- Reprenons ton 1er exemple.
    Si, en amont de ton code, il existe un script, un autre bout de code, perdu dans la masse de lignes de code de multiples scripts importés ou rédigés par toi, où il est écrit :
    Alors ton bout de code se prend le mur...
    ET t'as l'air malin, avec ton code "allégé" !

  6. #6
    Membre extrêmement actif Avatar de psychadelic
    Profil pro
    Inscrit en
    Mai 2010
    Messages
    2 532
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Mai 2010
    Messages : 2 532
    Par défaut

    Justement c'est pour cela que j'ai parlé de noms explicites.
    (et bien sur appeler ses variables truc machin ou bidule est aussi peu explicite que Prix_1, _2.. ou pire id="window" or id="document" comme le suggère Beginner dans son premier post et auquel ma réponse des "noms explicites" faisait référence.

    quand à utiliser document.getElementById("Prix_Net").innerText ok, mais quand on doit se retrouver à écrire un peu partout dans son code ces 35caratères, surtout pour utiliser au final le même nom de variable (explicite) que celui de l'Id qui lui est attribué, alors si, on allège le code de 27caracteres à chaque fois.

  7. #7
    Expert confirmé Avatar de CosmoKnacki
    Homme Profil pro
    Justicier interdimensionnel
    Inscrit en
    Mars 2009
    Messages
    2 986
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Charente Maritime (Poitou Charente)

    Informations professionnelles :
    Activité : Justicier interdimensionnel

    Informations forums :
    Inscription : Mars 2009
    Messages : 2 986
    Par défaut
    Citation Envoyé par jreaux62 Voir le message
    N.B. En ne lisant que le titre ("La méthode getElementById() est devenue inutile ?"), j'avais plutôt pensé :
    Oui, dans le sens où on peut désormais utiliser à la place querySelelector().
    D'un point de vue purement fonctionnel oui, mais ce n'est pas le cas du point de vue du style d'écriture, qui consiste à afficher une intention claire en utilisant la méthode la mieux adaptée au besoin (ici je veux un élément à partir de son id => j'utilise getElementById, y'a pas d'erreur de casting), ni du point de vue du travail nécessaire: getElementById renvoie un élément en passant son paramètre à un hash alors que querySelector en est encore à parser son expression pour savoir ce qu'on lui demande.
    Plus généralement, si les sélecteurs css peuvent exprimer des chemins simples d'une manière qui est familière au développeurs css, on reste très loin de l'efficacité et du pouvoir expressif de libxml.

    Maintenant pour en revenir à truc.innerText = 'bla bla bla', je ne l'écrirai pas pour deux raisons, la première étant celle que les autres ont déjà évoquée (le risque de télescopage avec une variable), l'autre étant qu'une propriété comme innerText ne devrait être autorisée qu'en lecture et pas en écriture. Là encore, il ne faut pas forcément succomber à toutes les facilités qu'offre un langage.

  8. #8
    Rédacteur/Modérateur

    Avatar de SpaceFrog
    Homme Profil pro
    Développeur Web Php Mysql Html Javascript CSS Apache - Intégrateur - Bidouilleur SharePoint
    Inscrit en
    Mars 2002
    Messages
    39 659
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 75
    Localisation : Royaume-Uni

    Informations professionnelles :
    Activité : Développeur Web Php Mysql Html Javascript CSS Apache - Intégrateur - Bidouilleur SharePoint
    Secteur : Industrie

    Informations forums :
    Inscription : Mars 2002
    Messages : 39 659
    Billets dans le blog
    1
    Par défaut
    J'imagine ...

    Ma page Developpez - Mon Blog Developpez
    Président du CCMPTP (Comité Contre le Mot "Problème" dans les Titres de Posts)
    Deux règles du succès: 1) Ne communiquez jamais à quelqu'un tout votre savoir...
    Votre post est résolu ? Alors n'oubliez pas le Tag

    Venez sur le Chat de Développez !

  9. #9
    Rédacteur

    Avatar de danielhagnoul
    Homme Profil pro
    Étudiant perpétuel
    Inscrit en
    Février 2009
    Messages
    6 389
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 74
    Localisation : Belgique

    Informations professionnelles :
    Activité : Étudiant perpétuel
    Secteur : Enseignement

    Informations forums :
    Inscription : Février 2009
    Messages : 6 389
    Billets dans le blog
    125
    Par défaut


    Cette technique me semble hasardeuse pour des raisons déjà dites dans cette discussion.

    On peut mettre un peut de discipline en commençant systématiquement un ID par "id", exemple : "idCamelCase".

    Dans ce cas, le risque de collision est réduit, mais non nul avec d'autres variables et la réponse "un ID doit être unique dans la page web" est une vedette du forum.

    C'est la raison pour laquelle j'utilise toujours la fonction kIDUnique();, elle est incluse dans mon fichier "https://danielhagnoul.developpez.com/lib/dvjh/dvjhUtilities-1.8.4.js", voir mon blog

    J'ai fait quelques tests, la technique fonctionne en mode strict et dans un script externe, même dans un module externe.

    Code HTML : Sélectionner tout - Visualiser dans une fenêtre à part
    <span id="idBody">texte initial...</span>

    module:

    Code JavaScript : 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
    window.addEventListener('load', ev => {
        // le DOM est construit et la page web est visible
        console.log(`Window load : ${new kDvjhDate()}`);
     
        // début code du test
     
        console.log(typeof idBody); // object
     
        idBody.innerText = 'bla bla bla';
     
        // fin code du test
     
        kIDUnique();
    }, {
        capture: false,
        passive: true,
        once: false
    });

    Blog

    Sans l'analyse et la conception, la programmation est l'art d'ajouter des bogues à un fichier texte vide.
    (Louis Srygley : Without requirements or design, programming is the art of adding bugs to an empty text file.)

+ Répondre à la discussion
Cette discussion est résolue.

Discussions similaires

  1. [méthodes]Qu'est ce que les specs d'un projet ?
    Par débutant_en_C dans le forum Méthodes
    Réponses: 18
    Dernier message: 08/01/2010, 17h52
  2. IE est devenu fou :(
    Par kagura dans le forum IE
    Réponses: 13
    Dernier message: 06/08/2006, 15h19
  3. Réponses: 5
    Dernier message: 04/07/2006, 10h17
  4. Réponses: 2
    Dernier message: 31/05/2006, 18h49
  5. problème avec la méthode getElementById() dans Firefox
    Par matrouba dans le forum Général JavaScript
    Réponses: 5
    Dernier message: 19/12/2005, 08h55

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo