#2 JS04(base 2): 5

Открыто
1 год назад открыта gitgod · комментариев: 0
gitgod прокомментировал 1 год назад

http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L52-L67 - надо было написать аналог switch с пропущенными break;

http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L79 - "или" дает одно значение из двух. Так что не ясно, с чем происходит сопоставление в case - с возрастом или с годом;

http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L80 - year никогда не равен строго false, как сработает || в этом случае?

http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L100 - не используется или (||)

http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L106-L109 - что бы не писать пустые if лучше инвертировать условие, в таком случае код из else переносится в if, а else удаляется;

http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L123-L125 - тут не хватает ||;

http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L240-L242 - проще ничью определить по равенству ходов компа и человека, а в else засунуть "ты проиграл"

http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L52-L67 - надо было написать аналог `switch` с пропущенными `break`; http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L79 - "или" дает **одно значение** из двух. Так что не ясно, с чем происходит сопоставление в `case` - с возрастом или с годом; http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L80 - `year` никогда не равен строго `false`, как сработает `||` в этом случае? http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L100 - не используется **или** (`||`) http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L106-L109 - что бы не писать пустые `if` лучше инвертировать условие, в таком случае код из `else` переносится в `if`, а `else` удаляется; http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L123-L125 - тут не хватает `||`; http://gitlab.a-level.com.ua/seregi4/homework/src/master/03/index.js#L240-L242 - проще ничью определить по равенству ходов компа и человека, а в `else` засунуть "ты проиграл"
Войдите, чтобы присоединиться к обсуждению.
Нет меток
Нет этапа
Нет ответственного
1 участников
Загрузка...
Отмена
Сохранить
Пока нет содержимого.