Ошибка в коде JS (задача на codewars)?

Question

[Святослав Смирнов]

Условие задачи:

In this Kata, you will remove the left-most duplicates from a list of integers and return the result.

Моё решение:

function solve(arr){
    let mas = [];
   arr = arr.map((item,index,array) => {
      if(mas.includes(item) == false){
        mas.push(item);
      }
      else{
        array.splice(index,1);
      }
    });
  return arr;
}

Выдаёт ошибку: Expected: [4, 6, 3], instead got: [undefined, undefined, undefined, undefined, undefined, ] . В чём проблема?

Comments

[Sun_Day]

Ну давай начнем с того, что ты используешь map для итерации массива(все-таки почитай для чего конкретно он используется) и при этом ничего не возвращаешь. Ничего не смущает?
И то что ты хотел сделать - все равно будет оставлять дубликаты.

[Святослав Смирнов]

Спасибо!

Answer 1

[0xD34F]

Да без понятия, в чём там у вас проблема. Что за ошибка - об этом вы почему-то предпочли умолчать. Так что, какой вопрос - такой ответ. Вот вам решения рабочие, сравнивайте со своим, думайте:

Можно собирать новый массив:

function solve(arr) {
  const last = arr.reduce((acc, n, i) => (acc[n] = i, acc), {});
  return arr.filter((n, i) => last[n] === i);
}

Или удалять элементы из существующего:

function solve(arr) {
  let numDeleted = 0;

  arr.forEach(function(n, i, a) {
    a[i - numDeleted] = n;
    numDeleted += i !== this.get(n);
  }, new Map(arr.map((n, i) => [ n, i ])));

  arr.length -= numDeleted;

  return arr;
}

Или и то и другое сразу:

const solve = arr => (
  arr.splice(0, arr.length, ...[...new Set(arr.reverse())].reverse()),
  arr
);

Comments

[Святослав Смирнов]

Expected: [4, 6, 3], instead got: [undefined, undefined, undefined, undefined, undefined, ]

[0xD34F]

А о входных данных надо догадаться самостоятельно? Лучше бы просто ссылку на задачу добавили в вопрос.

Глянул ваш код - мрак какой-то. Может, всё-таки стоит читать инструкцию, прежде чем пользоваться инструментом? - что делает map, вы явно не знаете. Откройте документацию, и разберитесь.

Обходите массив от начала к концу и при этом удаляете из него элементы - так те, что расположены после удалённых, не будут обработаны.

Вырезать пытаетесь дубликаты, расположенные после первого элемента, а не до последнего.

Как бы мог выглядеть исправленный вариант вашего кода (выкидываем map, меняем направление обхода массива):

function solve(arr) {
  const unique = [];

  for (let i = arr.length; i--;) {
    if (unique.includes(arr[i])) {
      arr.splice(i, 1);
    } else {
      unique.push(arr[i]);
    }
  }

  return arr;
}

[Святослав Смирнов]

Спасибо, просто не знаю как правильно изучать javascript. Читаю документацию на learn.js, выполняю задачи. Кажется, что понимаю, но на деле оказывается, что нет.

Answer 2

[Rsa97]

В том, что алгоритм, придуманный вами, нерабочий.

solve ([3, 4, 4, 3, 6, 3]);
// Array(6) [ undefined, undefined, undefined, undefined, undefined, <1 empty slot> ]