0

У меня есть функция, убирающая идущие друг за другом одинаковые элементы, после того как метод reduce доходит до тройки в данном примере(конкретно на строчке - return previous=item), значению previous не присваивается значение item и метод заканчивает работу, не убирая тройки.

В чем может быть проблема?

function uniqueInOrder(iterable){
  let arr
//Если элемент строка - преобразовывает ее в массив
  if (typeof(iterable) == "string") {
    arr = iterable.split('')
  }else {
    arr = iterable
  }
//После каждого элемента создается аккумулирующее значение, равное прошлому элементу
  arr.reduce(function(previous, item, index) {
    if (previous!==item) {
      return previous=item;
    } else if (previous==item) {
      arr.splice(index,1)
      return previous=item;
    }
  },0)
  return arr
}
console.log( uniqueInOrder([1,2,2,3,3]) )
8
  • Зачем вам тут вообще reduce?
    – HTO HOT
    17 июл 2022 в 14:06
  • Мне показалось, что это будет самое удобное и правильное решение, также я хочу потренироваться с методом reduce
    – Heropaster
    17 июл 2022 в 14:08
  • Самый простой вариант это [1,2,2,3,3].filter((i, ind, arr) => arr.indexOf(i) == ind). Конкретно в вашем коде проблема в том, что вы пытаетесь изменять массив через splice внутри reduce. index указывает не на тот индекс, который вы хотите, он не учитывает сделаный splice, т.к. reduce "копирует" массив перед тем как начать по нему идти.
    – Ein
    17 июл 2022 в 14:16
  • Вам нужно, чтобы массив, который приходит как параметр, мутировал? Или вам достаточно чтобы функция возвращала новый массив с неповторяющимися значениями?
    – HTO HOT
    17 июл 2022 в 14:17
  • Ein Спасибо за способ решения, я забыл уточнить, что значения могут повторяться, но главное чтобы повторяющиеся значения не шли подряд друг за другом.
    – Heropaster
    17 июл 2022 в 14:22

1 ответ 1

0

Решение через reduce:

function uniqueInOrder(iterable){
  let arr

  if (typeof(iterable) == "string") {
    arr = iterable.split('')
  }else {
    arr = iterable
  }
  ///Начальное значеение у reduce будет пустой массив, далее с помощью previous.slice(-1)[0] будем получать последний элемент массива и сравнивать его со следующим элементом из arr. Если они не равны то добавляем его в конец, если они равны то ничего не делаем с аккумулирующим массивом
  return arr.reduce(function(previous, item) {
    if (previous.slice(-1)[0]!==item) {
      return previous.concat(item)
    }
    return previous  
  },[])
}
console.log(uniqueInOrder([1,2,2,2,3,3]))

Возможно такая реализация алгоритма методом reduce вас испугает, но за-то поняв этот код, поймёте как работает метод reduce.

Вот вам в помощь изучения: https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Global_Objects/Array/Reduce

1
  • Спасибо за ответ!
    – Heropaster
    17 июл 2022 в 14:37

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.