Which is the best way to reference a private method from returned object in javascript? I leave you some sample code:
var HelloWorld = (function(){
var hello = function(){
console.log("hello")
},
return{
addToList: function(){
//how can i reference hello from here dynamically like:
//this["hello"] or this["hell" + "o"]
}
}
})()
Because the return is still inside the closure, you can just call hello directly. So:
hello();
To make it "dynamic", as the other answers recommend, you need to save hello to something. If you want to attach it to this instead of another object, you just need to store off a reference to this so you can access it later.
var HelloWorld = (function(){
var self = this;
this.hello = function(){
console.log("hello")
};
return {
addToList: function(){
self["hello"]();
}
}
})();
hello() is not a method. It's just a function in a closure. So, you can just call it from within your addToList() method.
If you want the hello function to behave like a method with this set to the object instance, you'd have to pass the instance to it as in this example.
var HelloWorld = (function(){
var hello = function(){
console.log("hello")
},
return{
addToList: function(){
//how can i reference hello from here dynamically like:
hello.call(this);
}
}
})()
If what you're really trying to do is to access the hello function by a string reference, you cannot easily access local variables by string name. If you wanted to do that, you could have to put the hello function into a local object like this:
var HelloWorld = (function(){
var locals = {
hello: function(){
console.log("hello")
}
};
return{
addToList: function(){
//how can i reference hello from here dynamically like:
locals["hello"].call(this);
}
}
})()
As is, you cannot reference the function hello "dynamically" as mentioned in the comment of your sample code. (I am assuming your definition of "dynamically" is to be given a string containing the word "hello" and somehow use it to reference the function hello.)
You could, however, move hello into an object and reference it off that object using square bracket notation:
var HelloWorld = (function () {
var privateCode = {
hello: function () {
console.log('hello');
}
};
return {
addToList: function () {
// access via `privateCode['hello']()`
}
};
}());
You can't use this, because you haven't made it a part of the object.
var HelloWorld = (function () {
var hello = function () { console.log("hello"); };
return {
addToList : function () { hello(); }
};
}());
That will work fine.
If you need to access it by using a string, then you need to make hello, then you have two options:
1) make a public function, which you CAN call with a string, which calls hello
return {
command : function (string) { if (this[string]) { this[string](); },
sayHello : function () { hello(); }
};
2) make a private object, which stores your methods (then you can call it with a string):
var private_actions = {
hello : function () { console.log("hello"); }
};
return {
command : function (string) {
if (private_actions[string]) { private_actions[string](); }
}
};
Since hello only exists in the scope of an anonymous IIFE, you'd need to store it some intermediate object in order to be able to dynamically access hello from a public method:
var HelloWorld = (function(){
var privates = {
hello: function (){
console.log("hello")
}
};
return {
addToList: function (){
privates['hello']();
}
}
})();
HelloWorld.addToList();
this['hello']();seems fine to me...hellofunction declaration, not a comma.,before the return/after the hello function should be a;.