Calling function from another in same object

Question

In the following code, I want pass a reference to the print_season function with parameter "winter" into the function inner_function.

tony = {
    print_season: function (season) {
        console.log(">>season is" + season);
    },

    report: function () {
        console.log(">>report");
        this.inner_object.inner_function(this.print_season("winter"));
    }, 

    inner_object: {
        inner_function: function(callback) {
            console.log(">>inner_function=" + callback());
        }
    }
}

tony.report();

However, when I do the function is invoked rather than passed and I end up with:

TypeError: callback is not a function
    console.log(">>inner_function=" + callback());

How do I pass the function with specific parameters in this case but to ensure it is not invoked?

Thanks.

qwertynl · Answer 1 · 2013-12-11T19:11:16.063

1

You are not passing a function.

You are actually just passing undefined

You might want print_season to return a callback function:

...

print_season: function (season) {
    // return a callback function
    return function() {
        console.log(">>season is" + season);
    };
},

...

edited Dec 11 '13 at 19:11

answered Dec 11 '13 at 19:04

qwertynl

3,912
1
21
43

I think the OP is aware of this: *"However, when I do the function is invoked rather than passed and I end up with: [...]"*. The question is *"How do I pass the function with specific parameters in this case but to ensure it is not invoked?"* – Felix Kling Dec 11 '13 at 19:09
Ahh ok will make update then :-) – qwertynl Dec 11 '13 at 19:09
@FelixKling I made an update. – qwertynl Dec 11 '13 at 19:11

score 0 · Answer 2 · answered Dec 11 '13 at 19:03

0

You are invoking print_season.

Try something like this.

this.inner_object.inner_function(function() { this.print_season("winter") });

answered Dec 11 '13 at 19:03

Daniel A. White

187,200
47
362
445

score 0 · Answer 3 · answered Dec 11 '13 at 19:03

Try this:

tony = {
print_season: function (season) {
    console.log(">>season is" + season);
},

report: function () {
    console.log(">>report");
    this.inner_object.inner_function(function(){this.print_season("winter")});
}, 

inner_object: {
    inner_function: function(callback) {
        console.log(">>inner_function=" + callback());
    }
}
}
tony.report();

Or this:

tony = {
print_season: function (season) {
 return function(){
     console.log(">>season is" + season);
 }
},

report: function () {
    console.log(">>report");
    this.inner_object.inner_function(this.print_season("winter"));
}, 

inner_object: {
    inner_function: function(callback) {
        console.log(">>inner_function=" + callback());
    }
}
}
tony.report();

The goal is to have a function (callback) not something else.

Calling function from another in same object

3 Answers3