Angularjs splice in loop

Question

Я создаю это приложение angularjs, его первое. Так огромная кривая обучения. Я не могу удалить элементы, так же как мой поиск не работает. Мне удалось получить сортировку столбцов и разбиение на страницы, чтобы работать.

У меня нет ошибок, поэтому я не уверен, почему поиск и удаление строки элементов в таблице не работает.

Я попытался с помощью этого:

 $scope.remove = function(client){ 
      for(var i = $scope.clients.length - 1; i >= 0; i--){ 
       if($scope.clients[i].name == client.name){ 
         $scope.clients.splice(i,1); 
       } 
      } 
     } 

Рабочий пример: http://plnkr.co/edit/f2ozbP4JxrRSvRGJAmQi

ОТВЕТ, РАБОЧАЯ:

Благодарности @DTing

$scope.remove = function(client){ 
     $scope.filteredItems = $scope.filteredItems.filter(function(item) { 
     return item.name !== client.name; 
     }); 
     $scope.groupToPages(); 
    } 

Answer 1

Это должно быть вашей удалить функции:

 $scope.remove = function(client){ 
      for(var i = $scope.pagedItems[$scope.currentPage].length - 1; i >= 0; i--){ 
       if($scope.pagedItems[$scope.currentPage][i].name == client.name){ 
        $scope.pagedItems[$scope.currentPage].splice(i,1); 
        break; 
       } 
      } 
     } 

Это должно быть вашей функции поиска (создание нота $ scope.clientsCopy):

 $scope.clientsCopy = angular.copy(data); 

     $scope.searchMatch = function (haystack, needle) { 
      if (!needle) { 
       return true; 
      } 
      return haystack.toLowerCase().indexOf(needle.toLowerCase()) !== -1; 
     }; 

     // init the filtered items 
     $scope.search = function() { 
      $scope.filteredItems = $filter('filter')($scope.clientsCopy, function (item) { 
       for(var attr in item) { 
        if ($scope.searchMatch(item.name, $scope.query)) 
         return true; 
        } 
        return false; 
       } 
      ); 
      // take care of the sorting order 
      if ($scope.sortField !== '') { 
       $scope.filteredItems = $filter('orderBy')($scope.filteredItems, $scope.sortField, $scope.reverse); 
      } 
      $scope.currentPage = 0; 
      // now group by pages 
      $scope.groupToPages(); 
     }; 

Plunker

Answer 2

Вы пытаетесь изменить свой $scope.clients который на самом деле пуст, потому что в вашем groupToPages Вы можете установить его в пустой массив и повторно заполнить или использовать.

$scope.groupToPages = function() { 
     $scope.clients = []; 

Вы можете заставить его работать, удалив элементы из массива filtersItems.

$scope.remove = function(client){ 
    $scope.filteredItems.splice($scope.filteredItems.indexOf(client),1); 
    $scope.groupToPages(); 
} 

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