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

Yaroslav Mikhaylov

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

3 0

Maksim Litvinov

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

0

Yaroslav Mikhaylov

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

0

Maksim Litvinov

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

1

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

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

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

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

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

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

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