#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 участника
Зареждане...
Отказ
Запис
Все още няма съдържание.