Java Script array.indexOf is not working

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, Jun 21 '16 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., Jun 21 '16 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, Jun 21 '16 at 13:34

score 2 · Accepted Answer · edited Jun 21 '16 at 14:10

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 '16 at 14:10

Sam Marland

552
3
24

answered Jun 21 '16 at 13:32

asdf_enel_hak

7,474
5
42
84

1

`map` would be more suitable as OP needs the index. For example: `var index = product.map(function(p) { return p.Name; }).indexOf(item.Name);` – Oscar Barrett Jun 21 '16 at 13:38
@OscarBarrett, yes, that's exactly what I need. Please add comment as answer. – andrey.shedko Jun 21 '16 at 14:10
@asdf_enel_hak, yes, your solution is working but that's not what I need. – andrey.shedko Jun 21 '16 at 14:12
@andrey.shedko yep I see – asdf_enel_hak Jun 21 '16 at 14:38
@andrey.shedko I would have added my comment as an answer, however this question has been marked as a duplicate and is therefore locked. – Oscar Barrett Jun 22 '16 at 02:22

Java Script array.indexOf is not working

1 Answers1