Discussion :

[lvr]

Bonjour,

J'essaye d'accéder depuis des méthodes privées des méthodes publiques et je reçois toujours un "Uncaught ReferenceError: resize is not defined" au niveau de l'appel de ma fonction autoinit.

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
var Pages = function() {
    var autoinit = function() {
        resize(true);
    }(); // auto-init
 
    return {
        resize: function(force) {
            console.log("resize requested");
        }
    };
};
 
pages=new Pages();

Quelle est la solution ?
J'ai bien pensé à

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
var Pages = function() {
    var autoinit = function() {
        _resize(true);
    }(); // auto-init
 
    _resize: function(force) {
        console.log("resize requested");
    }
    return {
        resize: _resize;
    }
    };
};

mais bon si je dois faire ça pour tes méthodes publiques potentiellement appelables depuis une méthode privées

[lvr]

J'ai aussi essayé avec un "me." masi sans résultat

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
11
12
var Pages = function() {
    var me=this;
    var autoinit = function() {
        me.resize(true);
    }(); // auto-init
 
    return {
        resize: function(force) {
            console.log("resize requested");
        }
    };
};

D'ailleurs j'ai vraiement du mal à voir quand utiliser un "me" (pointant vers "this")...

[Bovino]

Il s'agit avant tout d'une mauvaise compréhension du modèle objet de JavaScript qui est pas prototype et non par classes.

Ce que tu essayes de créer est un constructeur et un constructeur sert à définir des propriétés et méthodes en les attachant à this, il n'est donc pas supposé renvoyer de valeur.

D'autre part, tu écris bien que Pages() est une fonction, la notation resize: function(){...} n'a donc pas lieu d'être puisqu'elle est réservée à la définition d'un objet.

Code :

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

1
2
3
4
5
6
7
8
var Pages = function() {
    this.resize = function(force) {
        console.log("resize requested");
    };
    this.resize();
};
 
pages=new Pages();

[lvr]

Le problème si j'écris ça comme ça, c'est que mon function "resize" n'est plus publique et que donc un

Code :

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

pages.resize()

foire maintenant !

Il s'agit avant tout d'une mauvaise compréhension du modèle objet de JavaScript qui est pas prototype et non par classes.

Effectivement, j'arrive pas à capter vraiment le principe et j'essaye de me remettre au maximum sur un terrain connu: le java.

[Bovino]

Oui, mais vouloir calquer le modèle objet de JavaScript sur celui de Java n'est pas une bonne idée.
D'autre part, il n'y a pas, en JavaScript, de notion de visibilité, même s'il est possible de l'émuler.

Ceci dit, dans l'exemple que je t'ai donné, la méthode resize() est disponible pour toute instance de Pages :

Code :

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

1
2
3
4
5
6
7
8
9
var Pages = function() {
    this.resize = function(arg) {
        console.log("resize requested", arg);
    };
    this.resize('Dans constructeur');
};
 
var pages=new Pages();
pages.resize('Hors constructeur');

affiche bien les deux messages.
Si ça ne fonctionne pas dans ton implémentation, c'est qu'il y a d'autres problèmes non reproduits dans ton exemple.

[sekaijin]

Quant à avoir un membre privé

Code :

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

1
2
3
4
5
6
7
8
9
var Pages = function() {
   var a = function() {
     alert("a");
   };
   this.resize = function(arg) {
     a();
   };
   this.resize('Dans constructeur');
};

a est une méthode privée utilisée dans la méthode publique resize

A+JYT

[MaitreKaio]

La réponse que t'a donné Bovino est parfaitement valide.

Code :

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

1
2
3
4
5
6
7
8
9
var Pages = function() {
   var a = function() {
     alert("a");
   };
   this.resize = function(arg) {
     a();
   };
   this.resize('Dans constructeur');
};

De cette manière la méthode resize est accessible depuis l'extérieur (donc "publique"), et le page.resize() ne "foire" absolument pas. En tous cas pas chez moi...

[lvr]

Ca doit être cette histoire de "this". Je vais essayer mot-pour-mot votre proposition. Merci.

[lvr]

Ca marche...

Ce que j'avais écrit était basé sur le principe "closure" tel que décrit dans l'article Trois fondamentaux de Javascript. Dans ce cas de figure, les nouvelles instances ne se créent pas en faisant un "new", mais juste en faisant

Code :

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

var pages = Pages();

Votre solution est plus du type "prototype" car vous faites un

Code :

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

var pages = new Pages();

? En même temps, cette manière de définir les fonctions publiques via "this", n'est pas décrite dans le cadre du mode "prototype"....

Ca marche, c'est le principal. J'ai transformé mes autres classes dans la même optique, sans vraiement piger pourquoi je n'arrivais pas à appeller des méthodes publiques depuis les méthodes privées dans le mode "closure".

[sekaijin]

tout membre défini dans un constructeur de la forme this.XXX est un membre public.

en javascript une méthode n'est qu'un membre qui est une référence à une fonction.

A+JYT