#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 учесника
Учитавање...
Откажи
Сачувај
Још нема садржаја.