1) getCurrentPlace по факту возвращает nextPlace относительно currentPlace 2) getCurrentPlace/getFirstPlace можно упростить используя Array.find 3) всегда лучше явно возвращать результат (см. getFirstPlace) 4) в этом примере нет смысла вкладывать функции в друг друга 5) артикли не используются в именах функций :)
Имхо. Не думаю что объявление функций в функции это хорошая практика. const не везде где можно. Функции можно сделать более "чистыми", передавая в них аргументы а не работая с внешними переменными.
1) getCurrentPlace по факту возвращает nextPlace относительно currentPlace 2) getCurrentPlace/getFirstPlace можно упростить используя Array.find 3) всегда лучше явно возвращать результат (см. getFirstPlace) 4) в этом примере нет смысла вкладывать функции в друг друга 5) артикли не используются в именах функций :)
Имхо. Не думаю что объявление функций в функции это хорошая практика. const не везде где можно. Функции можно сделать более "чистыми", передавая в них аргументы а не работая с внешними переменными.
хз. наоборот так кажется более интуитивно понятным (с функциями в функциях). Если они нигде больше не будут использоваться, то зачем их выносить. Благодаря тому, что они внутри, ты сразу понимаешь, что они нужны исключительно для этой функции.