How to remove user from list in AngularJS?

Question

I have implemented a confirm button where the confirm button will remove a user from the list. But for some reason, it does not remove the user from the list. Can anyone check my js code to see what I have done wrong.

Here is my code

$scope.doDelete = function(user) {
    var index = $scope.userInfo.users.indexOf(user);
    $scope.userInfo.users.splice(index, 1);
    $window.location.href = '#/user';
}

html

<button class="delete" ng-click="doDelete(person)">Confirm</button>

What does index log out before you splice? What does your array look like? — Sterling Archer
– Sterling Archer, Commented Jun 6, 2017 at 22:31
@SterlingArcher Sorry, I can't understand what you've just said. You want me to console.log(indexOf); ? — user7960664
– user7960664, Commented Jun 6, 2017 at 22:32
After index is assigned (the line below) do console.log(index) and make sure you're passing a valid index to splice. It will also help to see what $scope.userInfo.users logs out as well — Sterling Archer
– Sterling Archer, Commented Jun 6, 2017 at 22:34
It doesn't "work" because you reload the page with $windows.location.href. But you're doing it right. Just you're reloading your page, so the data is reinitialised — Zooly
– Zooly, Commented Jun 6, 2017 at 22:39

Chris Halcrow · Accepted Answer · 2017-06-06 23:50:37Z

2

You remove the item using:

$scope.userInfo.users.splice(index, 1);

This only removes it in memory (this change isn't persisted). You then reload the page using:

$window.location.href = '#/user';

So your array will be reset back to what it was before you removed the item.

answered Jun 6, 2017 at 23:50

Chris Halcrow

32.5k19 gold badges202 silver badges235 bronze badges

Sign up to request clarification or add additional context in comments.

11 Comments

user7960664 Over a year ago

should I erase what I have in doDelete Function and just paste the one you wrote?

user7960664 Over a year ago

if yes, it still does not answered my question

user7960664 Over a year ago

I want the changes to take place

ippi Over a year ago

Nowhere are you showing how you get $scope.userInfo.users in the first place or even what it contains or where it's located. Maybe it's in a different scope altogether, but we can't know that because we are not mind readers.

ippi Over a year ago

The obvious thing to do is

console.log($scope.userInfo.users); var index = $scope.userInfo.users.indexOf(user); $scope.userInfo.users.splice(index, 1); console.log($scope.userInfo.users);

or set a breakpoint, check the value, step forward, check the value. There is no apparent to why splice wouldn't work in here. So the problem lies with something you haven't shown us.

|

Collectives™ on Stack Overflow

How to remove user from list in AngularJS?

1 Answer 1

11 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

11 Comments

Your Answer

Sign up or log in

Post as a guest

Related