function call on object with delay

Question

I am trying to make this code return each employees name.

var company = {
    employees: [
        {
            name: "doug"
        },
        {
            name: "AJ"
        }
    ],
    getName: function(employee){
        return employee.name
    },
    getNames: function(){
        return this.employees.map(this.getName)
    },
    delayedGetNames: function(){
        setTimeout(this.getNames,500)
    }
}

console.log(company.delayedGetNames());

However, when I run the code I get "TypeError: Cannot read property 'map' of undefined"

I have tried doing

setTimeout(this.getNames.bind(this),500)

I just get undefined returned to me.

Can anyone help me out?

setTimeout has no return value. Your console.log will always return undefined. You need to use Promises. — lolbas
– lolbas, Commented Feb 28, 2018 at 8:47

Giannis Mp · Accepted Answer · 2018-02-28 09:13:55Z

3

You need to add a callback to your function in order to get the names.

var company = {
    employees: [
        {
            name: "doug"
        },
        {
            name: "AJ"
        }
    ],
    getName: function(employee){
        return employee.name
    },
    getNames: function(){
        return this.employees.map(this.getName)
    },
    delayedGetNames: function(cb){
        setTimeout(()=>cb(this.getNames()),500)
    }
}
company.delayedGetNames(names => console.log(names))

edited Feb 28, 2018 at 9:13

answered Feb 28, 2018 at 9:00

Giannis Mp

1,3197 silver badges13 bronze badges

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

Comments

Kirill Simonov · Accepted Answer · 2018-02-28 09:02:09Z

1

Or, using Promise, you could write something like this:

var company = {
    employees: [
        {
            name: "doug"
        },
        {
            name: "AJ"
        }
    ],
    getName: function(employee){
        return employee.name
    },
    getNames: function(){
        return this.employees.map(this.getName)
    },
    delayedGetNames: function() {
        return new Promise(resolve => setTimeout(() => resolve(this.getNames()), 1000));
    }
}

company.delayedGetNames().then(console.log);

answered Feb 28, 2018 at 9:02

Kirill Simonov

8,4913 gold badges23 silver badges47 bronze badges

Comments

izengod · Accepted Answer · 2018-02-28 09:24:42Z

1

With few tricks and getters

var company = {
    employees: [
        {
            name: "doug"
        },
        {
            name: "AJ"
        }
    ],
    getName: function(employee){
        return employee.name
    },
    get getNames(){
        console.log(this.employees.map(x => this.getName(x)));
    },
    get delayedGetNames(){
        setTimeout(this.getNames,500)
    }
}

console.log(company.delayedGetNames);

edited Feb 28, 2018 at 9:24

answered Feb 28, 2018 at 9:03

izengod

1,1565 gold badges17 silver badges44 bronze badges

Collectives™ on Stack Overflow

function call on object with delay

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related