#2 JS04(base 2): 8

Open
opened 1 year ago by gitgod · 0 comments
gitgod commented 1 year ago

http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L67-L80 - имя по умолчанию должно срабатывать когда имени нет, а не когда оно есть (!!)

http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L201-L205 - это можно засунуть в else, вместо написания условия;

http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L190-L194 - это можно сделать одним сравнением, если вместо числа хранить слово. Все равно это слово нужно для этих if-ов - http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L178-L186

http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L187-L189 - этот else никогда не сработает

http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L214-L222 - при понимании квадратных скобок данных switch не нужен

http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L232 - как бы тут выбрать любую валюту, а не только UAH

http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L67-L80 - имя по умолчанию должно срабатывать когда имени **нет**, а не когда оно есть (`!!`) http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L201-L205 - это можно засунуть в `else`, вместо написания условия; http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L190-L194 - это можно сделать одним сравнением, если вместо числа хранить слово. Все равно это слово нужно для этих `if`-ов - http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L178-L186 http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L187-L189 - этот `else` никогда не сработает http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L214-L222 - при понимании квадратных скобок данных `switch` не нужен http://gitlab.a-level.com.ua/Plutisko/HW/src/master/HW2_2/work.js#L232 - как бы тут выбрать _любую валюту_, а не только `UAH`
Sign in to join this conversation.
No Label
No Milestone
No assignee
1 Participants
Loading...
Cancel
Save
There is no content yet.