Почему изменяется arr2, в чем моя ошибка?

Question

[szQocks]

https://codepen.io/kapysta-xxl/pen/jOZQRGv?editors=1112

Решил написать метод для массива, вроде делаю всё логично но почему то arr2 изменяется, то есть каким-то образом у меня в итоге работает глубокое копирование, и я так и не понял где оно или ошибка в чем-то другом ?

Суть метода в том что бы найти дубли в массивах, где дублей больше двух и вернуть результаты в новом массиве.

Comments

[szQocks]

вывод , я туплю !

Answer 1

[Александр]

Поэтому arr.splice(index, 1)

Comments

[szQocks]

Я знаю что поэтому, почему ссылка сохраняется на arr1 ?

[WbICHA]

szQocks, а почему она должна пропасть?

[szQocks]

WbICHA, да чёт я переработал походу уже) не должна, туплю, просто мне нужно было сделать так

https://codepen.io/kapysta-xxl/pen/jOZQRGv?editors=1111

[WbICHA]

szQocks, ну, у тебя ещё и метод неправильно работает. Финальный фильтр должен на равенство индекса проверять, а не наоборот.
Да и код сильно усложнён, всё это делается в одну строку и намного читабельнее:

return this.filter((v, i) => arr.includes(v) && this.indexOf(v) === i)

[szQocks]

WbICHA, не не, там всё ок, просто можно было по короче сделать не спорю, как смог так смог, специально проверяю на не равенство, для того что бы узнать если ли ещё похожие элемент в этом же массиве но с другим индексом

в итоге результат должен быть [1,17]

единственное исправил название массива в аргументах, на _arr

Answer 2

[Alexandroppolus]

arr (который аргумент функции findClone) - это не копия массива arr2, а ссылка на него. Потому, меняя arr методом splice, ты фактически меняешь arr2