Size: a a a

2020 May 03

V

Victor in Public | Kottans
Oleg
Всім салют.
Як можна зарефакторити щоб замінити тернарник в reduce?
thx
function countPoints(gameResults) {
gameResults.reduce((totalPoints, current) =>
totalPoints + (current[0] > current[2] ? 3 : current[0] === current[2] ? 1 : 0), 0);
}
countPoints(['3:1', '1:0', '0:0', '2:3']);
Я б не сказал что тут вложенный тернарник плохо (многие не согласятся))
Но как по мне Джон все правильно сказал, стоит этот тернарник вынестии назвать нормально все сразу становится читаемым
источник

V

Victor in Public | Kottans
вот вроде ж все читаемо с первого раза?
источник

IK

Illya Klymov in Public | Kottans
Victor
вот вроде ж все читаемо с первого раза?
Нет
источник

V

Victor in Public | Kottans
😟
источник

EE

Ev Er in Public | Kottans
не
источник

V

Victor in Public | Kottans
а можете тогда плз идеальный вариант скинуть? ну понятно что if заюзать, надо ли деструктуризацию массива?
источник

IK

Illya Klymov in Public | Kottans
Victor
а можете тогда плз идеальный вариант скинуть? ну понятно что if заюзать, надо ли деструктуризацию массива?
Для начала - кто сказал что в счёте одна цифра?
источник

O

Oleg in Public | Kottans
нюанс в тому, що прикручений eslint і він матюкається)
источник

V

Victor in Public | Kottans
Illya Klymov
Для начала - кто сказал что в счёте одна цифра?
ааа, так current это не массив, я не дочитал до примера использования)
источник

O

Oleg in Public | Kottans
>
источник

V

Victor in Public | Kottans
не думал что строчки так юзаться будут, тогда конечно не надо так)
источник

O

Oleg in Public | Kottans
Всім дякую
источник

V

Victor in Public | Kottans
вот это будет считаться идеальным вариантом?
источник

O

Oleg in Public | Kottans
Victor
вот это будет считаться идеальным вариантом?
источник

SL

Sergey Lysenko in Public | Kottans
Насколько я понимаю там ожидается строка, как аргумент
источник

IK

Illya Klymov in Public | Kottans
Там речь об отдельном элементе
источник

V

Victor in Public | Kottans
угу, юзаешь как-то так
gameResults.reduce(
   (totalPoints, current) => totalPoints + getPoints(current),
   0
 )
источник

O

Oleg in Public | Kottans
Illya Klymov
Там речь об отдельном элементе
з одним аргументом функції видає ту ж помилку
источник

IK

Illya Klymov in Public | Kottans
Oleg
з одним аргументом функції видає ту ж помилку
Вы почти наверняка массив передали
источник

IK

Illya Klymov in Public | Kottans
Из одного элемента
источник