Раскройте свою мысль про "один инпут". Например, инпуту нужен label, в который как раз пойдёт "Ккал". Вы хотите зачем-то сломать семантику или использование label допустимо?
Чтобы оценивать тестовое задание нужно знать тестовое задание.
"Не принять" могли не задание, а ваше резюме, например.
Тестовое задание призвано показать ваш уровень, а вы там сразу пишете "писал я его на скорую руку левой пяткой".
Если ваша цель — за разумное время получить качественный результат, то стоит обратиться к тому, кто этим профессионально занимается. За дополнительную оплату этот человек может согласиться вдобавок всё вам объяснить.
Если цель — набивать шишки и учиться таким образом, то продолжайте изучать самостоятельно, только сразу готовьтесь к тому, что будете делать ошибки, а их последствия отразятся на клиентах.
Вам уже пять раз сказали - сделайте вывод строки в консоль и сами убедитесь, что я прав.
Иного тут просто быть не может - index out of bounds означает, что элемента 1 в массиве нет. Нет его там потому, что в какой-то строке нет дефиса. Это может быть пустая строка в конце файла, например.
