Почему не отсеиваются нули при сравнении 0 < 0?

Question

[Михаил Оловянников]

Делал задачку на codewars. Надо было объединить два массива, упорядочить их и отсеять повторяющиеся элементы. Я сделал так:

function mergeArrays(arr1, arr2) {
	let result = []
	return result
		.concat(arr1, arr2)
		.sort((a, b) => a - b)
		.filter((item, index, array) => {
			return array[index + 1] ? item < array[index + 1] : true
			// Если предыдущий элемент меньше последующего - оставляем его
		})
}

let arr1 = [5, 4, 3, 2, 1]
let arr2 = [-5, -4, -3, -2, 5, 4, 3, 2, 1, 0, 0]

console.log(mergeArrays(arr1, arr2)) // [-5, -4, -3, -2, 0, 0, 1, 2, 3, 4, 5]
console.log(5 < 5) // false
console.log(0 < 0) // false

Но у меня из-за чего-то не отсеиваются нули, хотя 0 < 0 дает false и должен работать аналогично сравнению всех других чисел. В чем тут проблема?

Comments

[WbICHA]

return 0 ? item < array[index + 1] : true

Answer 1

[Wataru]

array[index + 1] ? item < array[index + 1] : true Что эта конструкция делает, можете объяснить? Ошибка в ней.

Comments

[Михаил Оловянников]

Ну, она работает, как мне кажется, так: если индекс массива + 1 не выходит за диапазон массива, то она начинает сравнивать текущий элемент массива с последующим. Если же индекс массива + 1 вышел за диапазон массива — значит все элементы уже прошли свои сравнения (а с undefined нам сравнивать не нужно) и мы просто возвращаем последний элемент чтобы наш результирующий массив был полным.

Я переделывал эту функцию таким образом (он работает и нули удаляет):

function mergeArrays(arr1, arr2) {
	let result = []
	result = result.concat(arr1, arr2).sort((a, b) => a - b)

	for (let i = result.length; i > 0; i--) {
		if (result[i - 1] != undefined) {
			if(result[i] - result[i - 1] === 0) {
				result.splice(i, 1)
			}
		}
	}
	return result
}

Но мне интересно почему не работает то, о чем я спрашивал. Я не могу понять.

[Alexandroppolus]

Михаил Оловянников,

если индекс массива + 1 не выходит за диапазон массива

это самый мерзкий способ проверять выход за пределы диапазона.
не делай так на собеседовании.

есть же index и array.length

[Михаил Оловянников]

Alexandroppolus, я переписал это вот так:

return index < array.length - 1  ? item < array[index + 1] : true

Понял в чем была ошибка. Это тоже плохой способ?

[Alexandroppolus]

Михаил Оловянников, этот единственно верный

[Михаил Оловянников]

Alexandroppolus, спасибо вам. Тяжело конечно это все анализировать.)

Answer 2

[Wexter]

А что по вашему должен вернуть этот код?
0 ? .. : true

Спойлер

он всегда вернёт true

Comments

[Михаил Оловянников]

Точно. Выходит у меня неправильно первое условие написано... Я неправильно написал то, что хотел.

Answer 3

[mayton2019]

Автор ты конечно читер еще тот. Codewars - это для самостоятельной работы.

И после того как ты ее заканчиваешь - там открывается форум где можно обсуждать и сравнивать
наилучшие решения.

Вобщем зря ты сюда пришел.

Comments

[Михаил Оловянников]

Почему читер? Я сам решил задачу вторым способом. Потом я захотел его переписать на .filter и столкнулся с проблемой. Не смог ее решить - написал сюда. Теперь вижу где была ошибка а главное я ее понял.

[mayton2019]

Михаил Оловянников, в следующий раз - не пались. Просто не говори что задача с codewars.