Java Script array.indexOf is not working [duplicate]

Question

That's was surprise, but simple function array.IndexOf is not working.

$scope.nextProduct = function (pos, item) {
    switch (pos) {
        case 0: product = $scope.Menu[0].Breakfast
            break
        case 1: product = $scope.Menu[0].Lunch
            break
        case 2: product = $scope.Menu[0].BeforTraining
            break
        case 3: product = $scope.Menu[0].AfterTraining
            break
        case 4: product = $scope.Menu[0].Dinner
            break
        default: product = $scope.Menu[0].Breakfast
            break
    }
    var index = product.indexOf(item.Name);
    product[index - 1].IsSelect = false;
    product[index + 1].IsSelected = true;
}

indexOf return -1 but I'm completely sure that the item exist in array. What can be wrong here?

No, the object doesn't exist: an object whose "name" is the same as item.Name exists, that's very different. — Denys Séguret
– Denys Séguret, Commented Jun 21, 2016 at 13:31
Your searching an array of objects for a string, how does it know you want to match the .Name property? — Alex K.
– Alex K., Commented Jun 21, 2016 at 13:32
Have you tried to debug this in the console? If you set a breakpoint on line 211. You can then execute JS in the console. This will be the quickest way to debug your problem. — Sam Marland
– Sam Marland, Commented Jun 21, 2016 at 13:34

Sam Marland · Accepted Answer · 2016-06-21 14:10:21Z

2

By this expression you are searching for a string in object array.

product.indexOf(item.Name);

Instead you should run:

var res = product.filter(function(elem){
    return elem.Name == someValue
})

this returns array matching your value

edited Jun 21, 2016 at 14:10

Sam Marland

5823 silver badges24 bronze badges

answered Jun 21, 2016 at 13:32

asdf_enel_hak

7,6585 gold badges47 silver badges91 bronze badges

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

5 Comments

Oscar Barrett Over a year ago

map would be more suitable as OP needs the index. For example: var index = product.map(function(p) { return p.Name; }).indexOf(item.Name);

andrey.shedko Over a year ago

@OscarBarrett, yes, that's exactly what I need. Please add comment as answer.

andrey.shedko Over a year ago

@asdf_enel_hak, yes, your solution is working but that's not what I need.

asdf_enel_hak Over a year ago

@andrey.shedko yep I see

Oscar Barrett Over a year ago

@andrey.shedko I would have added my comment as an answer, however this question has been marked as a duplicate and is therefore locked.

Collectives™ on Stack Overflow

Java Script array.indexOf is not working [duplicate]

1 Answer 1

5 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Linked

Related