#2 JS04(base2): 9

開啟中
gitgod1 年之前創建 · 1 條評論
gitgod commented 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 commented 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 исправил
Sign in to join this conversation.
未選擇標籤
未選擇里程碑
未指派成員
2 參與者
正在加載...
取消
保存
尚未有任何內容