Javascript : return object?

Question

I am trying to return object from function but it returns nothing any idea?

var todo = [{
    id: 'a01',
    titre: 'Citation',
    message: 'Vous êtes de ceux-là ? Ca tombe bien, je lai été moi aussi !',
    completed: false
}];

getById: function(todoId) {
    for (var i = 0; i < todo.length; i++) {
        if (todo[i].id == todoId) {
            return todo;
        }
    }
}

you are returning the full array

Danmoreng
– Danmoreng

2016-05-02 11:46:05 +00:00
Commented May 2, 2016 at 11:46 — Danmoreng
– Danmoreng, Commented May 2, 2016 at 11:46
probably you want to return todo[i]

Danilo
– Danilo

2016-05-02 11:46:56 +00:00
Commented May 2, 2016 at 11:46 — Danilo
– Danilo, Commented May 2, 2016 at 11:46
use return todo[i];

thepio
– thepio

2016-05-02 11:47:06 +00:00
Commented May 2, 2016 at 11:47 — thepio
– thepio, Commented May 2, 2016 at 11:47
what value did you passed to getById method?

gurvinder372
– gurvinder372

2016-05-02 11:47:51 +00:00
Commented May 2, 2016 at 11:47 — gurvinder372
– gurvinder372, Commented May 2, 2016 at 11:47

R. Oosterholt · Accepted Answer · 2016-05-02 11:47:18Z

2

return the array element instead of the complete array:

getById: function(todoId){
    for(var i=0;i<todo.length;i++){
        //console.log(todo[i].id == todoId);
        if(todo[i].id == todoId){
            return todo[i];
        }
    }
}

answered May 2, 2016 at 11:47

R. Oosterholt

8,2003 gold badges56 silver badges80 bronze badges

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

2 Comments

letseasy Over a year ago

Can you explain me why u use todo[i] with indice ?

or hor Over a year ago

maybe the result will be clearer for you if you use console.debug(todo);console.debug(todo[i]); and check out the console output (hit F12 and find Console)

Mr. X · Accepted Answer · 2016-05-02 11:48:58Z

-1

Try this code:

var todo=[{
    id:'a01',
    titre:'Citation',
    message:'Vous êtes de ceux-là ? Ca tombe bien, je lai été moi aussi !',
    completed:false
}];

var getById = function(todoId){
    for(var i=0;i<todo.length;i++){
        //console.log(todo[i].id == todoId);
        if(todo[i].id == todoId){
            return todo[i];
        }
    }
}

answered May 2, 2016 at 11:48

Mr. X

1,0371 gold badge16 silver badges29 bronze badges

3 Comments

Siraj ul Haq Over a year ago

It will be return todo[i]; instead of return todo[0];

Mr. X Over a year ago

Yes, I changed it. @SirajulHaq

Mr. X Over a year ago

Edited to fix the index. You can run this code in your console followed by getById('a01').

Collectives™ on Stack Overflow

Javascript : return object?

2 Answers 2

2 Comments

3 Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

3 Comments

Related