Каждый раз, когда я нажимаю кнопку «Добавить элемент», он повторно добавляет item в массив items. Например, если я нажимаю «Добавить элемент» в пять раз массив:Добавить элемент в массив, если он не находится в массиве

items: [item, item, item, item, item]

Как я могу изменить мой код, чтобы добавить первый item, но не добавлять дополнительные item S в массив, например, так:

items: [item]

Я попытался заменить var i = -1 на var i = 0, но это никогда не добавляет item в массив items.

Вот мое мнение:

<body ng-controller="MainCtrl"> 
    <button ng-click="addItem()">Add {{item}}</button> 
    <p>items: {{items | json}}</p> 
</body>

... и контроллер:

var app = angular.module('plunker', []); 

app.controller('MainCtrl', function($scope) { 
    $scope.item = 'item'; 
    $scope.items = []; 
    $scope.addItem = function() { 
    for (var i = -1; i < $scope.items.length; i++) { 
     if ($scope.items[i] != $scope.item) { 
      $scope.items.push($scope.item); 
     } 
    } 
    }; 
});

Вот мой Plunker: http://plnkr.co/edit/OT900pXEcxGgkpJZna2x?p=preview

источник

2015-07-09 Ryan

Вы пробовали, если $ scope.items.indexOf ($ scope.item)! = -1 чем push item в массиве? – immirza

Вы хотите только «нажимать» один раз (в первый раз)? – Amit

Каково ваше обоснование использования '$ scope.items [i]', когда 'i' равно -1? Что это должно достичь? – doldt

Добавить простую проверку

В addItem(), добавить простая проверка с использованием indexOf проверить, если элемент в массиве:

$scope.addItem = function() { 

    if ($scope.items.indexOf($scope.item) != -1){ 
     // The item is already in the array, abort! 
     return; 
    } 

    $scope.items.push($scope.item); 
}

http://plnkr.co/edit/GKtREKBXUD7J54oxTaDq?p=preview

источник

2015-07-09 19:32:57

Perfect - спасибо! – Ryan

Here's a forked version of your plunkr.

Он использует indexOf, которая является встроенной функцией, чтобы сказать, содержит ли массив в elment или нет (он возвращает -1, когда элемент не найден):

$scope.addItem = function() { 
    if($scope.items.indexOf($scope.item) === -1){ 
     $scope.items.push($scope.item); 
    } 
};

Я не уверен, что вы пытались достичь ieve путем ссылки $scope.items[-1], заботиться о том, чтобы разработать?

источник

2015-07-09 19:33:42 doldt

Добавить элемент в массив, если он не находится в массиве

ответ

Добавить простую проверку

http://plnkr.co/edit/GKtREKBXUD7J54oxTaDq?p=preview

Смежные вопросы