What is the best way to work with a list in Angular 2(4)?

Question

JavaScript

There is a small application in angular 4. There is a service for working with a specific entity. It stores an array of objects and allows you to add / change / remove elements.
Here is an example of deletion:

deleteItem(itemId) {
    this.items.forEach((item, index) => {
      if (itemId === item.id) {
        this.items.splice(index, 1);
      }
    });
    this.localStorageService.cacheData('items', this.items);
  }

But as I was told that this option is very bad and it is better not to do it. And how, in your opinion, will the removal of elements be most efficiently and optimally implemented?

Answer 1

deleteItem(itemId) {
  this.items = this.items.filter(({id}) => itemId !== id);
  this.localStorageService.cacheData('items', this.items);
}