#1 JS04(base 2): 7

Öppen
öppnade 2 år sedan av gitgod · 0 kommentarer
gitgod kommenterad 2 år sedan

http://gitlab.a-level.com.ua/VAshurov/HWforJS/src/master/js_hw3_p2/main.js#L5-L8 - если продублировать case, но не дублировать alert, будет лучше;

http://gitlab.a-level.com.ua/VAshurov/HWforJS/src/master/js_hw3_p2/main.js#L33-L50 - тут надо было написать аналог switch с пропущенными break;

http://gitlab.a-level.com.ua/VAshurov/HWforJS/src/master/js_hw3_p2/main.js#L68 - нет смысла сравнивать с true или false, так как то выражение, которое вы сравниваете уже может быть интерпретировано как true или false;

http://gitlab.a-level.com.ua/VAshurov/HWforJS/src/master/js_hw3_p2/main.js#L79 - такое лучше переносить на несколько строк;

http://gitlab.a-level.com.ua/VAshurov/HWforJS/src/master/js_hw3_p2/main.js#L226-L288 - это лучше написать через 2 if + else:

  • один if - если у тебя тоже самое что у пользователя, то ничья;
  • если ты выиграл
  • иначе - ты проиграл
http://gitlab.a-level.com.ua/VAshurov/HWforJS/src/master/js_hw3_p2/main.js#L5-L8 - если продублировать `case`, но не дублировать `alert`, будет лучше; http://gitlab.a-level.com.ua/VAshurov/HWforJS/src/master/js_hw3_p2/main.js#L33-L50 - тут надо было написать аналог `switch` с пропущенными `break`; http://gitlab.a-level.com.ua/VAshurov/HWforJS/src/master/js_hw3_p2/main.js#L68 - нет смысла сравнивать с `true` или `false`, так как то выражение, которое вы сравниваете _уже_ может быть интерпретировано как `true` или `false`; http://gitlab.a-level.com.ua/VAshurov/HWforJS/src/master/js_hw3_p2/main.js#L79 - такое лучше переносить на несколько строк; http://gitlab.a-level.com.ua/VAshurov/HWforJS/src/master/js_hw3_p2/main.js#L226-L288 - это лучше написать через 2 `if` + else: - один `if` - если у тебя тоже самое что у пользователя, то ничья; - если ты выиграл - иначе - ты проиграл
Logga in för att delta i denna konversation.
Ingen Etikett
Ingen Milsten
Ingen förvärvare
1 Deltagare
Laddar...
Avbryt
Spara
Det finns inget innehåll än.