Вопрос №59549 от пользователя Ярослав Михайлов в уроке «Удаление элементов массива», курс «JS: Массивы»

Ярослав Михайлов

Добрый день. У меня в решении нет проверки на пустой массив, поскольку по логике цикл for...of не запускается на пустом массиве и я возвращаю просто пустой, созданный внутри функции массив result. Это нормальная практика (в плане читабельности и уязвимости) или лучше явно прописать условие, как у учителя?

3 0

Максим Литвинов

Добрый день, Ярослав! Мне будет проще подсказать вам, если я увижу ваш код. Для этого отправьте ваше решение на ревью и приложите ссылку на него в комментарии. Подробнее о ревью можете прочитать здесь: https://help.hexlet.io/ru/articles/111135-kod-revyu

0

Ярослав Михайлов

Максим Литвинов, пардон, конечно: https://ru.hexlet.io/code_reviews/474771

0

Максим Литвинов

Вы правы, в случае пустого массива, цикл не запустится и просто вернётся исходное значение result. Но проверку пограничного случая всё таки лучше использовать, так как сейчас в коде происходит проверка четности первого элемента массива const fst = Math.abs(arr[0]); Если массив пустой, произойдет выход за пределы массива. JS в таком случае просто вернёт undefined, но в некоторых других языках это вызовет ошибку. Проверка пограничного случая как раз поможет избежать этого

1

Используйте Хекслет по максимуму!

  • Задавайте вопросы по уроку
  • Проверяйте знания в квизах
  • Проходите практику прямо в браузере
  • Отслеживайте свой прогресс

Зарегистрируйтесь или войдите в свой аккаунт

Отправляя форму, вы соглашаетесь c «Политикой конфиденциальности» и «Условиями оказания услуг»

Рекомендуемые программы

С нуля до разработчика. Возвращаем деньги, если не удалось найти работу.

Иконка программы Фронтенд-разработчик
Профессия
Разработка фронтенд-компонентов веб-приложений
19 января 8 месяцев
Иконка программы Python-разработчик
Профессия
Разработка веб-приложений на Django
19 января 8 месяцев
Иконка программы PHP-разработчик
Профессия
Разработка веб-приложений на Laravel
19 января 8 месяцев
Иконка программы Node.js-разработчик
Профессия
Разработка бэкенд-компонентов веб-приложений
19 января 8 месяцев
Иконка программы Верстальщик
Профессия
Вёрстка с использованием последних стандартов CSS
в любое время 5 месяцев
Иконка программы Java-разработчик
Профессия
Разработка приложений на языке Java
19 января 10 месяцев
Иконка программы Разработчик на Ruby on Rails
Профессия
Новый
Создает веб-приложения со скоростью света
19 января 5 месяцев