Calling function from another in same object [duplicate]

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 · Accepted Answer · 2013-12-11 19:11:16Z

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, 2013 at 19:11

answered Dec 11, 2013 at 19:04

qwertynl

3,9331 gold badge24 silver badges43 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Felix Kling Over a year ago

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?"

qwertynl Over a year ago

Ahh ok will make update then :-)

qwertynl Over a year ago

@FelixKling I made an update.

Daniel A. White · Accepted Answer · 2013-12-11 19:03:40Z

0

You are invoking print_season.

Try something like this.

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

answered Dec 11, 2013 at 19:03

Daniel A. White

192k49 gold badges389 silver badges473 bronze badges

Comments

Charlie Affumigato · Accepted Answer · 2013-12-11 19:03:50Z

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.

Collectives™ on Stack Overflow

Calling function from another in same object [duplicate]

3 Answers 3

3 Comments

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

Comments

Linked

Related