Какой метод работы со списком оптимален в Angular 2(4)?

Question

[gallantalex]

Есть небольшое приложение на angular 4. Есть сервис, для работы с определенной сущностью. Он хранит в себе массив объектов и позволяет добавлять/изменять/удалять элементы.
Вот пример удаления:

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

Но как мне сказали, что такой вариант очень плох и так лучше не делать. А как по вашему мнению будет наиболее эффективно и оптимально реализовано удаление элементов?

Answer 1

[Negwereth]

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

Comments

[gallantalex]

Да, мне хотелось бы делать через filter, но при использовании этой функции, похоже перебивается ссылка, и в дальнейшем в шаблоне представления элементы не обновляются, не удаляются и не добавляются на экране вывода. Хотя в самом массиве все норм, и если перезагрузить страницу, то из local storage все подгрузится правильно.
Вот я в шаблоне, в контроллере так получаю список:
this.items= this.itemService.getItemList();
А в представлении так вывожу:

<ul class="list-group items-list">
    <li class="list-group-item d-flex justify-content-between" *ngFor="let item of items">
     {{item.title}}
    </li>
</ul>