Функция фильтрации массива в диапазоне?

Question

[sashadobrota]

Есть вот такая функция фильтрации массива в диапазоне.

myArr = [5, 3, 8, 1, 12, 4, 5, 6, 1, 54, 7];

function filterRangeInPlace(arr, min, max) {
	for (var i = 0; i < arr.length; i += 1) {
			if (arr[i] < min || arr[i] > max) {
					arr.splice(i--, 1);
			}
	}
}

filterRangeInPlace(myArr, 2, 5); //[5, 3, 4, 5]

Вопрос: почему arr.splice(i--, 1)? Зачем i-- ? Не могу сообразить, спасибо.

Answer 1

[Сергей Соколов]

i-- затем, чтобы, удалив только что какой-то элемент вне диапазона, «текущий» элемент указывал на следующий.

Например, i=2, текущий элемент arr[2] это 8, вне диапазона. arr.splice(2,1) удалит этот элемент. Теперь на месте arr[2] оказывается уже 1.

Если не сделать i--, следующее значение i в итерации цикла будет уже 3, а arr[3] это 12 – выходит, "1" пропустили и даже не рассматривали.

Comments

[sashadobrota]

Спасибо, уже разобрался, в итоге пришел к тому, что вот такая запись хоть и длиннее на одну строчку, зато мне, новичку, сразу понятно.

function filterRangeInPlace(arr, min, max) {
  for (var i = 0; i < arr.length; i += 1) {
      if (arr[i] < min || arr[i] > max) {
          arr.splice(i, 1);
          i = i - 1;
      }
  }
}

Всё таки не зря в книге Дугласа Крокфорда "JavaScript The Good Parts" операторы инкремента и декремента отмечены как не очень хорошая практика. Опять же - моё субьективное мнение.
В любом случае, спасибо за уделённое мне внимание.

[Сергей Соколов]

sashadobrota: вот вы уже и не новичок : ) i-- это использует в вычислении текущее значение i, а затем сделает i = i - 1

В то время, как --i сначала сделает i = i - 1 а потом обновлённое значение i использует в вычислении. По-моему это удобнее, чем лишняя строчка и вполне читабельно.