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

jQuery Discussion :

Compléxité inutile du code ?


Sujet :

jQuery

  1. #1
    Membre à l'essai
    Homme Profil pro
    Inscrit en
    Mars 2013
    Messages
    30
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Mars 2013
    Messages : 30
    Points : 24
    Points
    24
    Par défaut Compléxité inutile du code ?
    Dans une méthode d'objet, pourquoi utiliser
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
     
    $("#id").on("mouseover",(function(self) {
            return function() {
              self.unemethode();
            };
          })(this));
    et pas directement
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
     
    var self=this;
    $("#id").on("mouseover",function(){ self.unemethode(); });
    qui marche très bien et est quand même plus lisible ?
    Est-ce que dans le second cas, la variable self est référencée différemment dans le ramasse-miette ?
    Merci pour vos lumières...

  2. #2
    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 640
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 74
    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 640
    Points : 66 665
    Points
    66 665
    Billets dans le blog
    1
    Par défaut
    pourquoi pas encore plus directement

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    $("#id").on("mouseover",function(){ $(this).unemethode(); });

  3. #3
    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
    Points : 22 933
    Points
    22 933
    Billets dans le blog
    125
    Par défaut


    Dans un gestionnaire d'événement, la fonction de rappel reçoit automatiquement l'objet "event" en premier argument.

    Donc, "self" est "event" sauf s'il y a une déclaration préalable du style : var self = this;

  4. #4
    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
    Points : 22 933
    Points
    22 933
    Billets dans le blog
    125
    Par défaut
    Il nous manque le contexte d'emploi, pour comprendre l'utilité du return. Mais ici, self c'est window.document !

    Code HTML : Sélectionner tout - Visualiser dans une fenêtre à part
    <div id="id">Hello !</div>

    Code JS : 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
    $( function(){ // forme abrégée de $(document).ready(function(){
     
        $( "#id" ).on( "mouseover", 
            ( function( self ){
     
                // self est this et this est document !
     
                return function(){
     
                    console.log( self, self.URL );
                };
     
            })( this ) // pas de ; c'est la fonction de rappel de on()
        );
     
    });

  5. #5
    Membre à l'essai
    Homme Profil pro
    Inscrit en
    Mars 2013
    Messages
    30
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Mars 2013
    Messages : 30
    Points : 24
    Points
    24
    Par défaut
    Citation Envoyé par SpaceFrog Voir le message
    pourquoi pas encore plus directement

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    $("#id").on("mouseover",function(){ $(this).unemethode(); });
    non, ce n'est pas ce que je veux faire (voir précisions + loin dans le fil)...

  6. #6
    Membre à l'essai
    Homme Profil pro
    Inscrit en
    Mars 2013
    Messages
    30
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Mars 2013
    Messages : 30
    Points : 24
    Points
    24
    Par défaut
    Citation Envoyé par danielhagnoul Voir le message


    Dans un gestionnaire d'événement, la fonction de rappel reçoit automatiquement l'objet "event" en premier argument.

    Donc, "self" est "event" sauf s'il y a une déclaration préalable du style : var self = this;
    Merci pour ce rappel. Cependant, je n'ai pas passé self comme paramètre de la fonction callback transmise.

  7. #7
    Membre à l'essai
    Homme Profil pro
    Inscrit en
    Mars 2013
    Messages
    30
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Mars 2013
    Messages : 30
    Points : 24
    Points
    24
    Par défaut
    Le contexte d'emploi, c'est une méthode d'objet.
    Voici un exemple tout bête.
    La méthode construire attache les gestionnaires à la div dont l'id est référencée.

    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
    <div id="idtest1">idtest1</div>
    <script>
     
     var maclasse=function(id){
       this.id_=id;
     };
     maclasse.prototype={
       onmouseover:function() {
         $('#'+this.id_).html("onmouseover appelé => "+this.id_);
       },
       onmouseout:function() {
         $('#'+this.id_).html("onmouseout appelé => "+this.id_);
       },
       construire:function() {
         var self=this;
         $('#'+this.id_).on("mouseover", function() { self.onmouseover(); });
         $('#'+this.id_).on("mouseout", function() { self.onmouseout(); });
       }  
     };
     var x=new maclasse("idtest1");
     x.construire();
     
    </script>

  8. #8
    Membre expérimenté
    Profil pro
    Inscrit en
    Mars 2002
    Messages
    1 132
    Détails du profil
    Informations personnelles :
    Âge : 52
    Localisation : France

    Informations forums :
    Inscription : Mars 2002
    Messages : 1 132
    Points : 1 418
    Points
    1 418
    Par défaut
    Bonjour,

    Citation Envoyé par loubigou Voir le message
    Merci pour ce rappel. Cependant, je n'ai pas passé self comme paramètre de la fonction callback transmise.
    Tout à fait et c'est bien pour ça que tes 2 codes ne sont pas identiques.

    La callback de l'évènement "mouseover" en jQuery reçoit un objet EVENT et "this" correspond à l'élément DOM survolé (dans ton exemple l'élément ayant "id" pour id)

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    var self=this;
    $("#id").on("mouseover",function(){ 
        console.log(self == this); // renvoie "false"
    });

  9. #9
    Membre à l'essai
    Homme Profil pro
    Inscrit en
    Mars 2013
    Messages
    30
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Mars 2013
    Messages : 30
    Points : 24
    Points
    24
    Par défaut
    Oui, la fonction callback est exécuté via un apply dans le contexte de l'élément du DOM auquel elle est rattachée (c'est bien pour cela que je ne mets pas this mais self). Mais je n'ai toujours pas la réponse à la première question de mon post.
    Juste je me demandais si la première syntaxe présentait un intérêt du point de vue de la gestion de la mémoire.
    Je ne crois pas en fait...

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

Discussions similaires

  1. Libération mémoire + variable inutile en fin de code
    Par kenshi240683 dans le forum GTK+ avec C & C++
    Réponses: 2
    Dernier message: 12/06/2008, 17h44
  2. [CKEditor] Code inutile (et gênant) généré lorsque le texte est vide sous Firefox
    Par Hervé Saladin dans le forum Bibliothèques & Frameworks
    Réponses: 3
    Dernier message: 05/05/2008, 12h02
  3. [outil] Code inutile
    Par BigNic dans le forum C#
    Réponses: 5
    Dernier message: 23/07/2007, 16h58
  4. [ Code ] Analyse de code - Attribut Inutile
    Par geegee dans le forum Eclipse Java
    Réponses: 2
    Dernier message: 19/05/2004, 09h07

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