Почему перезаписывается массив?

Question

[rinatoptimus]

Есть цикл

arr = [];
......
......
this.httpService.getList().subscribe(data => 
      {
        this.arr = data;
        for(let i=0; i<this.arr.length; i++) {
          this.arr.push(this.arr[i]);
          // this.arr = [];
        }
      }
    );

Вопрос в заголовке.

Answer 1

[Антон Спирин]

А вы что сделать хотели?
Ваш цикл никогда не закончится, так как вы постоянно добавляете элементы в массив. Перед каждой итерацией идет проверка:
i < this.arr.length
А у вас и i и this.arr.length после каждой итерации на 1 увеличиваются и как следствие this.arr.length всегда больше.

Возможно вы хотели сделать, что-то вроде этого:

this.httpService.getList().subscribe(data => {
  this.arr = [...this.arr, ...data];
});

или:

this.httpService.getList().subscribe(data => {
  this.arr = this.arr.concat(data);
});

Если важно расширять старый массив, а не возвращать новый то:

this.httpService.getList().subscribe(data => {
  data.forEach(el => this.arr.push(el));
});

Comments

[rinatoptimus]

Пусть увелич-ся. Это код для подгрузки элементов при скролле. Типа загружать, пока не кончатся элементы. Или вы не об этом?

[Антон Спирин]

rinatoptimus, я не об этом, а о том что вы все неправильно сделали.
Строкой:
this.arr = data;
вы уничтожаете все старые данные. А дальше идет некорректный бесконечный цикл, который может завершиться только падением приложения, когда ваш массив займет всю память.
Посмотрите вариант решения из ответа. Он вам должен подойти.

[rinatoptimus]

Антон Спирин, вы имеете в виду

this.httpService.getList().subscribe(data => 
      {
        this.arr = [...this.arr, ...data];
      }
 );

Это спред?

[Антон Спирин]

rinatoptimus, именно. Добавил еще пару вариантов. В том числе, вариант с мутацией старого массива. Используйте его если не нужна иммутабельность. Выбирайте какой больше подходит. И изучите на досуге встроенные методы массивов. Многие из них очень удобны.

[rinatoptimus]

Антон Спирин, Спасибо большое! Спред офигенен.