Почему при суммировании элементов массива получается NaN?

Question

[OLEGatorRB]

Решаю задачу https://www.codewars.com/kata/whose-bicycle

Как я рассуждаю?

1. Создаю массив из объектов, которые входят в параметры функции whoseBicycle
let arr = Array.from(arguments);

2. Затем, используя map, делаю из объектов массивы:
arrMap = arr.map(item => Object.values(item));

3. Затем, нахожу сумму каждого из подмассивов, используя reduce, и вот здесь я уже не могу разобраться,
мне кажется надо использовать один reduce в другом - первый для того, чтобы выделить подмассивы, а второй (внутри первого) - считать сумму элементов входящих в него:

arrReduce = arrMap.reduce( function ( acc, subArr ) {
    subArr.reduce( function ( total, elem ) {
      total = total + elem
      console.log(total)
    }, 0)
  }, [])

А когда выводишь в консоль total - отображается NaN, откуда он берется?

Comments

[Elvis]

первое что приходит в голову - ты складываешь не числа, а строки.

[OLEGatorRB]

да, тоже так думал, но проверял typeof-ом вроде везде числа, и нет никаких строк там(((

[Synbulat Biishev]

А какие входные данные?

[VoidVolker]

OLEGatorRB Правила, пункт 3.8

[OLEGatorRB]

VoidVolker, поправил, спасибо, буду знать

[OLEGatorRB]

Synbulat Biishev,

I am the father of three wonderful sons. before the beginning of the school year, I promised them that I would buy a bicycle for someone who would bring the best marks at the end of the school year. it's time to keep promises and I count on you.

You have 3 input objects(school diaries) with school subjects and marks (1-10). For example:

{
  'algebra': 6,
  'history': 8,
  'physics': 9,
  'geography': 2,
  'chemistry': 9
}

[OLEGatorRB]

Synbulat Biishev,

I need to buy a bicycle for my first son.' // the sum of the marks is the highest  in the first diary.

'I need to buy a bicycle for my second son.' // the sum of the marks is the highest in the second diary.

'I need to buy a bicycle for my third son.' //  the sum of the marks is the highest in the third diary.,

[OLEGatorRB]

function whoseBicycle(diary1, diary2, diary3) {
  // good luck
}

[0xD34F]

мне кажется надо использовать один reduce в другом

не надо, достаточно одного

function whoseBicycle(...diaries) {
  const [ [ son ] ] = [ 'first', 'second', 'third' ]
    .map((n, i) => [
      n,
      Object.values(diaries[i]).reduce((acc, m) => acc + m, 0),
      ageTable[`${n}SonAge`],
    ])
    .sort((a, b) => (b[1] - a[1]) || (a[2] - b[2]));

  return `I need to buy a bicycle for my ${son} son.`;
}

arrReduce = arrMap.reduce( function ( acc, subArr ) {
    subArr.reduce( function ( total, elem ) {
      total = total + elem
      console.log(total)
    }, 0)
  }, [])

Просто блестяще, я считаю.

Вы не знаете, что делает метод reduce.
Вы не знаете, что из функций можно, а иногда и нужно возвращать значения.
Вы не знаете, каким будет результат выполнения функции, если ничего не вернуть.
Вы не знаете, что происходит при попытке сложить значения разных типов.

Ну, вы хотя бы знаете (теперь), что вам стоит погуглить.

когда выводишь в консоль total - отображается NaN, откуда он берется?

Ничего не вернули из коллбека reduce'а => на следующей итерации total принял значение undefined.
Сложение undefined и числа даёт NaN.
Всё.

[Дмитрий Беляев]

0xD34F, а откуда в коде из спойлера берется ageTable? Ну и поиск минимума сортировкой, это явно не то что стоит подсказывать новичкам, O(n log n) против O(n), даже при n=3 это разница в 1.5+ раза

[0xD34F]

откуда в коде из спойлера берется ageTable?

Дмитрий Беляев, из условия решаемой задачи (ссылка в начале вопроса).

поиск минимума сортировкой, это явно не то что стоит подсказывать новичкам

ещё могу предложить reduce

function whoseBicycle(...diaries) {
  const son = [ 'first', 'second', 'third' ]
    .map((n, i) => [
      Object.values(diaries[i]).reduce((acc, m) => acc + m, 0),
      ageTable[`${n}SonAge`],
      n,
    ])
    .reduce((son, n) => (son[0] > n[0] || (son[0] === n[0] && son[1] < n[1])) ? son : n, [ -1, 1 / 0 ])
    .pop();

  return `I need to buy a bicycle for my ${son} son.`;
}

[VolgaVolga]

Elvis,

ты складываешь не числа, а строки.

При сложении строк, как и строк с числами, результатом никогда не будет NaN.

[OLEGatorRB]

0xD34F, спасибо большое за помощь, разбираюсь

[OLEGatorRB]

спасибо всем, кто отликнулся и написал свои комментарии

[OLEGatorRB]

0xD34F, кое-что мне до сих пор непонятно из Вашего ответа, но надеюсь разберусь, Я вставил Ваше решение в codewars, не для того, чтобы галочку себе поставить, а для того, чтобы посмотреть как другие люди решили задачу, я нашел одно, в котором я нашел ответы на все те вопросы, которые меня мучили.
Через час я написал свой вариант.

function whoseBicycle(dairy1, dairy2, dairy3) {
  const son = ['first', 'second', 'third']

  let diaries = Array.from(arguments);

  const diariesMap = diaries.map(function(diary){
    const values = Object.values(diary)
    
    return values.reduce((acc, cur) => acc + cur, 0);     
  })
  
  let max = Math.max(...diariesMap);

  return `I need to buy a bicycle for my ${son[diariesMap.indexOf(max)]} son.`;
}