#2 JS04(base2): 9

Открыто
1 год назад открыта gitgod · комментариев: 1
gitgod прокомментировал 1 год назад

http://gitlab.a-level.com.ua/MRovenskyi/js4online_homework/src/master/03/main.js#L123-L129 - в сложных случаях лучше в одну строку if не писать.

не ясно зачем спрашивать пароль если пользователь не ок;

http://gitlab.a-level.com.ua/MRovenskyi/js4online_homework/src/master/03/main.js#L129 - нет смысла писать else if с противоположным условием, else подразумевает противоположность условия;

http://gitlab.a-level.com.ua/MRovenskyi/js4online_homework/src/master/03/main.js#L135 - "" && null всегда дает результат null. Эту логику лучше реализовать через default

http://gitlab.a-level.com.ua/MRovenskyi/js4online_homework/src/master/03/main.js#L123-L129 - в сложных случаях лучше в одну строку `if` не писать. не ясно зачем спрашивать пароль если пользователь не ок; http://gitlab.a-level.com.ua/MRovenskyi/js4online_homework/src/master/03/main.js#L129 - нет смысла писать `else if` с противоположным условием, `else` подразумевает противоположность условия; http://gitlab.a-level.com.ua/MRovenskyi/js4online_homework/src/master/03/main.js#L135 - `"" && null` всегда дает результат `null`. Эту логику лучше реализовать через `default`
MRovenskyi прокомментировал 1 год назад
Владелец
http://gitlab.a-level.com.ua/MRovenskyi/js4online_homework/src/master/03/main.js#L123-L129 http://gitlab.a-level.com.ua/MRovenskyi/js4online_homework/src/master/03/main.js#L129 http://gitlab.a-level.com.ua/MRovenskyi/js4online_homework/src/master/03/main.js#L135 исправил
Войдите, чтобы присоединиться к обсуждению.
Нет меток
Нет этапа
Нет ответственного
2 участников
Загрузка...
Отмена
Сохранить
Пока нет содержимого.