Как найти, есть ли элемент в массиве?

Question

[microf]

Основы основ. Делаю добавление в "корзину"

var     ProductsList = [],
                Cart = function (id, name) {
                    this.id = id;
                    this.name = name;
                };

  function addProduct(id, name) {

            var newItem = new Cart(id, name);
               ProductList.push(newItem);
                                   }

Нужно, чтобы если элемент с id уже есть в Cart - то не добавлять (ну return сделать). Как понимаю, можно для этого использовать

angular.forEach(Cart, function(id) {
// и вот здесь понять не могу как сравнить приходящий id с Cart.id
}

Answer 1

[Виталий Инчин ☢]

var ProductsList = [],
      Cart = function (id, name) {
           this.id = id;
           this.name = name;
       }
;

function addProduct(id, name) {
      ProductsList.some(function(e){
         return e.id == id;
      })||ProductsList.push(new Cart(id, name));
}

Comments

[microf]

Спасибо. Оно работает. Но, если можно, объясните начинающему, а что это? some(function(e) {...}

[Виталий Инчин ☢]

microf: learn.javascript.ru/array-iteration#every-some

[microf]

Спасибо. "Я буду учить, я буду учить, я буду учить"

Answer 2

[Stopy]

Вставляй не в массив, а в другой объект, типо вот так { id: {name: name} }, а проверка на существование в корзине сама собой решится

Answer 3

[profesor08]

Согласно документации angular.forEach, перебор элементов должен выглядеть так для вашего примера.

angular.forEach(ProductsList, function(value, key) {

}

Answer 4

[Евгений Петров]

Underscore. Js underscorejs.ru/#arrays