Вопрос №19201 от пользователя Мария Ковшарова в уроке «Дестракчеринг», курс «JS: Коллекции»

Kirill Mokevnin

has(acc, year) ? acc[year]++ : acc[year] = 1;

Вот это страшная строчка, все худшие паттерны собрались в одном месте). Посмотрите в рекомендации линтера, он сильно ругался на этот код.

Vasiliy Usachev

Никак не пойму, почему ругают тернарные операции. Мое второе решение, по-моему, даже лучше учительского. :))) Такое маленькое и понятное.

Мария Ковшарова

Поправила, согласна, свежий взгляд помог, действительно, выглядело не очень :) Линтер ругался на операцию ++, к этому вопрос, почему этот оператор считается плохим паттерном? Просто довольно часто вижу именно в "рабочей" среде использование инкремента/декремента. Как и тернарники, но с ними раз на раз не приходится, иногда совершенно нечитабельные их делают, а уж тернарник в тернарнике вообще песня!

Kirill Mokevnin

. Мое второе решение, по-моему, даже лучше учительского. :))) Такое маленькое и понятное.

В задании просят реализовать итеративный процесс самостоятельно (то етсь без использования reduce). Цель этой задачи отработать деструктуризацию, а не написать самое короткое решение.

Kirill Mokevnin

к этому вопрос, почему этот оператор считается плохим паттерном?

Попробуйте погуглить сообщения линтера и вы попадете на документацию eslint где все объясняется с примерами.