Почему arr.reduce отработало з ошибкой?

Question

[Николай Антонов]

Мне надо собрать уникальные ключи с коллекции и для сохранения промежуточного результата решил воспользоваться reduce. Но при переборе массива у меня на 3 шаге ошибка:
Линк на песочницу.

function getUniuqeCarriers(data) {
  return data.reduce((sum, current) => {
      console.log(current.carrier)
      // ошибка на 3 шаге
      return sum.push(current.carrier)
    
  }, [])
}

P.S. Мap не предлагать, так как мне надо уникальные ключи а не перебрать все. Я показал в тестовом примере что простой перебор не работает.
Update. Мой ответ Нашол решение - линк
Помогла статья - хабр

Comments

[Lynn «Кофеман»]

Потому что push возвращает число. И вообще, тут проще map импользовать

[Николай Антонов]

Алексей Тен: мне надо массив уникальных елементов. С map надо дополнительная внешняя переменная.

[OVK2015]

Николай Антонов: Зачем здесь reduce?
К data с вашего jsbin.com

var carriers = [];
data.forEach(function(item)
{    
    carriers[item.carrier] !== undefined ? carriers[item.carrier]++ : carriers[item.carrier] = 0;              
});
console.log(carriers);

Answer 1

[romy4]

Запустите свой код просто в консоли хрома и увидите ошибку. ваша песочница малость глючная

Comments

[Николай Антонов]

А ваша консоль все правильно отработала?

[romy4]

Николай Антонов: не. sum.push - функция не найдена

Answer 2

[Златослав Десятников]

Решение тут: JSBin

Comments

[Николай Антонов]

Ну и как вы сделаете функцию которая возвратит уникальные ключи? Map не позволяет сохранить промежуточный результат, а reduce позволяет.

[Златослав Десятников]

Николай Антонов: обновил ответ.

Answer 3

[Слава]

push возвращает количество элементов.
return sum.concat([current.carrier]);
или
return [...sum, current.carrier];
или

sum.push(current.carrier);
return sum;