#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 учасників
Завантажується...
Скасувати
Зберегти
Тут ще немає жодного вмісту.