Почему не изменяется массив?

Question

[Ruslan]

Добрый день. Подскажите пожалуйста почему в таком коде не изменяется исходный массив

'use strict';

function filterRange(arr, a, b){
  for (let i= 0; i<arr.length; i++){
      if (arr[i] < a || arr[i] >b){
          arr.splice(arr[i], 1);
        
      }
  }
}
let arr = [5, 3, 8, 1];
filterRange(arr, 1, 4);
console.log(arr);

Answer 1

[James026]

Просто у тебя нечего вырезать из массива)
arr.splice(arr[i], 1); у тебя тут получается
1) arr.splice(5, 1);
2) arr.splice(8, 1);

Что выходит за границы массива)

Замени в коде arr.splice(arr[i], 1); на arr.splice(i, 1); и ты увидишь что все ок

Answer 2

[Александр Лелека]

Попробуйте так

function filterRange(arr, a, b){
for (let i= 0; i if (arr[i] < a || arr[i] >b){
arr.splice(arr[i], 1);

}
}

return arr;
}
let arr = [5, 3, 8, 1];
arr = filterRange(arr, 1, 4);
console.log(arr);

Comments

[James026]

в js массив передается по ссылке

[Ruslan]

James026, Да, увидел. Спасибо)))