如何将元素推送到数组而不重复它们？

Question

我有this issue here recorded on a video 让你更容易理解。如您所见，我正在从一个数组中选择一些元素并将这些元素推送到另一个名为$scope.favoriteLeague = []; 的数组中，以便制作最喜欢的联赛列表，在该视频中，我将元素推送任意多次，并且我不希望这样，我希望每个元素只能在列表中选择一次，如果用户尝试选择已经在收藏夹数组中的元素，则显示一条消息。

我正在使用lodash 和angular，这是代码：

$scope.favoriteLeague = []; 

$scope.addToFavorites = function(league) {

   $scope.favoriteLeague.push(league);

};

html

<ion-item ng-repeat="league in favoriteLeague">

    {{league.name}}

</ion-item>

Answer 1

您应该检查它是否不在数组中。 @itcouldevenbeaboat 很接近，但应该是 === -1 而不是 !== -1

$scope.addToFavorites = function(league) {

    if ($scope.favoriteLeague.indexOf(league) === -1){
        $scope.favoriteLeague.push(league);
    }

};

Answer 2

.indexOf() 方法是您可能正在寻找的。将其包裹在调用 addToFavorites 的代码中（这样，如果值已经存在，您甚至不会调用该函数）：

if ($scope.favoriteLeague.indexOf(league) === -1) {

    . . . your existing call to addToFavorites . . .

}

或者，如果你使用 jQuery，你可以使用$.inArray() 方法来做同样的事情。

Answer 3

实际上我根据@itcouldevenbeaboat 的答案得到了自己的解决方案

$scope.addToFavorites = function(league) {
  if ($scope.favoriteLeague.indexOf(league) === -1) {
    $scope.favoriteLeague.push(league);
  }else {
    console.log('already exists!!!!!');
  }
};

Answer 4

在推送之前检查数组是否包含联赛。

$scope.addToFavorites = function(league) {

    $scope.favoriteLeague.indexOf(league) === -1 ? $scope.favoriteLeague.push(league) : void 0;

};