For loop on javascript object

Question

I have this javascript object:

var input = [
    {
        id: 1,
        title: 'michael',
        favorite: 'bmw'
    },
    {
        id: 2,
        title: 'steve',
        favorite: 'bentley'
    },
    {
        id: 3,
        title: 'denise',
        favorite: 'ford'
    },
]

And this is my function, it is supposed to loop through input and when called with console.log print a specific favorite, depending on which element on the list has been read by the loop.

findYourFavorite = function() {
    for(let x=0; x<3; x++) {
        if(input[x].title) {
            var response = input[x].favorite
        }
    }
    return response
}

If I run console.log(findYourFavorite('michael')) it will always print the same title, which is the last one in the javascript object (denise).

Ex:

console.log(findYourFavorite('michael'))
ford

It is not working randomly, as it should, I hope I'm explaining myself, if You need further clarification please do let me know.

Right.... because you are overlaying the value of response ever iteration. stackoverflow.com/questions/500431/… — Taplar
– Taplar, Commented Sep 30, 2020 at 22:08
Your findYourFavorite function does not take any parameters, which is already a flaw — UnholySheep
– UnholySheep, Commented Sep 30, 2020 at 22:10

CertainPerformance · Accepted Answer · 2020-09-30 22:10:03Z

Your condition if(input[x].title) { makes no sense - all the titles are truthy. Only assign to response if the title is equal to the parameter (which you also need to define).

var input = [
    {
        id: 1,
        title: 'michael',
        favorite: 'bmw'
    },
    {
        id: 2,
        title: 'steve',
        favorite: 'bentley'
    },
    {
        id: 3,
        title: 'denise',
        favorite: 'ford'
    },
];

const findYourFavorite = function(titleToFind) {
    for(let x=0; x<3; x++) {
        if(input[x].title === titleToFind) {
            var response = input[x].favorite
            return response;
        }
    }
}
console.log(findYourFavorite('denise'))

Or, much more concisely, use .find:

var input = [
    {
        id: 1,
        title: 'michael',
        favorite: 'bmw'
    },
    {
        id: 2,
        title: 'steve',
        favorite: 'bentley'
    },
    {
        id: 3,
        title: 'denise',
        favorite: 'ford'
    },
];

const findYourFavorite = (titleToFind) => input
  .find(({ title }) => title === titleToFind)
  .favorite;
console.log(findYourFavorite('denise'))

Collectives™ on Stack Overflow

For loop on javascript object

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related