Remove last element from list in angularjs

Question

  <div class="col-md-2">
    <md-button class="md-raised md-primary" ng-click="deleteDiv()">Remove</md-button>
  </div>
scope.deleteDiv = function() {
        alert(scope.itemsToAdd.length);
        if(scope.itemsToAdd.length > 1) {
            scope.itemsToAdd.splice(scope.itemsToAdd.length,1)
        }
    };

Hi.I am new to AngularJs.I have a doubt on removing a dynamically created div. itemsToAdd is an array which contains four fields.I add those fields dynamically using push and ng-repeat.When Remove button is clicked I get the length of the array and only delete if it is greater than 1. The removing procedure I posted,Is is correct? where am i doing wrong?.Thanks

So you always want to delete the last one? Or you have a delete button per entity? — devqon
– devqon, Commented Feb 9, 2017 at 10:14
@Mohammed so you are telling me to try scope.itemsToAdd_splice() ? — Anusha Bayya
– Anusha Bayya, Commented Feb 9, 2017 at 10:14

Avantika Saini · Accepted Answer · 2017-02-09 10:22:55Z

2

You need to pass the index of the element to be deleted as the parameter of the splice function:

if(scope.itemsToAdd.length > 1) {
     scope.itemsToAdd.splice(scope.itemsToAdd.length-1,1); //index of the last element is the length of the array minus 1
}

answered Feb 9, 2017 at 10:22

Avantika Saini

7924 silver badges9 bronze badges

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

Comments

Mohammed · Accepted Answer · 2017-02-09 10:16:00Z

0

GO through this, Its one of the type you want

<html>
<script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.4.8/angular.min.js"></script>
<body ng-app="myApp" ng-controller="todoCtrl">

<h2>My Todo List</h2>

<form ng-submit="todoAdd()">
    <input type="text" ng-model="todoInput" size="50" placeholder="Add New">
    <input type="submit" value="Add New">
</form>

<br>

<div ng-repeat="x in todoList">
    <input type="checkbox" ng-model="x.done"> <span ng-bind="x.todoText"></span>
</div>

<p><button ng-click="remove()">Remove marked</button></p>

<script>
var app = angular.module('myApp', []); 
app.controller('todoCtrl', function($scope) {
    $scope.todoList = [{todoText:'Clean House', done:false}];

    $scope.todoAdd = function() {
        $scope.todoList.push({todoText:$scope.todoInput, done:false});
        $scope.todoInput = "";
    };

    $scope.remove = function() {
        var oldList = $scope.todoList;
        $scope.todoList = [];
        angular.forEach(oldList, function(x) {
            if (!x.done) $scope.todoList.push(x);
        });
    };
});
</script>

answered Feb 9, 2017 at 10:16

Mohammed

6582 gold badges12 silver badges33 bronze badges

1 Comment

LionC Over a year ago

Consider explaining how you solved OPs problem instead of just posting code to make your answer more useful for OP and future readers.

Utku Apaydin · Accepted Answer · 2017-02-09 10:18:09Z

0

<tr ng-repeat="person in persons track by $index">
        <td>{{person.name}} - # {{person.id}}</td>
        <td>{{person.description}}</td>
        <td nowrap=nowrap>
            <a href="#!/edit"><i class="icon-edit"></i></a>
            <button ng-click="delete($index)"><i class="icon-minus-sign"></i></button>
        </td>
</tr>

and in the controlelr

$scope.delete=function(index){
    $scope.persons.splice(index,1);
}

answered Feb 9, 2017 at 10:18

Utku Apaydin

1621 silver badge4 bronze badges

2 Comments

Anusha Bayya Over a year ago

my button is outside the ng-repeat so i cannot use index variable

Utku Apaydin Over a year ago

can you share complate table i can find a workaround for that

Collectives™ on Stack Overflow

Remove last element from list in angularjs

3 Answers 3

Comments

1 Comment

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related