Как можно оптимизировать код решения этой задачи?

Question

[Aristotel Paskalov]

Для практики решил выполнить задачи отсюда

Задача:

/**
 * Известны результаты каждой из 4х четвертей баскетбольной встречи.
 * Нужно определить победителя матча. Побеждает команда,
 * набравшая больше очков в течение всего матча.
 *
 * Напишите функцию getWinner(points) возвращающую номер победившей команды,
 * либо undefined в случае ничьей.
 *
 * Пример:
 * getWinner(['23-26', '24-30', '30-27', '35-31']) === 2
 * getWinner(['36-32', '32-24', '20-28', '30-26']) === 1
 * getWinner(['36-18', '22-31', '27-21', '19-34']) === undefined
 */

Моё решение:

function getWinner(points) {
    let firstTeamPoints = [];
    let secondTeamPoints = [];

    points.map(item => {
        firstTeamPoints.push(item.slice(0, 2));
        secondTeamPoints.push(item.slice(3, 5));
    })

    let firstTeamSummary = 0;
    firstTeamPoints.forEach(item => {
       firstTeamSummary += +item;
    });

    let secondTeamSummary = 0;
    secondTeamPoints.forEach(item => {
        secondTeamSummary += +item;
    });

    if(firstTeamSummary === secondTeamSummary) {
        return undefined;
    }

    return firstTeamSummary > secondTeamSummary ? 1 : 2;
}

Костыль ли это? Как можно улучшить данный код?

Comments

[Somewhere Intech]

const getWinner = array => (result = array.reduce((r,v) => r + v.split('-').reduce((a,b) => Number(a) - Number(b)), 0)) === 0 ? undefined : result > 0 ? 1 : 2

Answer 1

[0xD34F]

function getWinner(points) {
  const [ a, b ] = points.reduce((acc, n) => (
    n.split('-').forEach((m, i) => acc[i] += +m),
    acc
  ), [ 0, 0 ]);

  return a === b
    ? undefined
    : a < b ? 2 : 1;
}

UPD. Всё равно слишком длинно. Вот так хорошо будет:

const getWinner = points => [ 2, , 1 ][Math.sign(eval(points.join('+'))) + 1];

Всё, "оптимизировал".

Comments

[Aristotel Paskalov]

Ох, мне до такого уровня ещё далеко, но спасибо