Какие недочеты вы скажете по коду?

Question

[Артем Громаков]

https://github.com/AntimatteryCreator/seawarship
Это мое тестовое задание, хочу получить критику, хочу устроится junior на js. В компании сказали, что слабо. На мой вопрос "где слабо?" и" что подтянуть?", я не получил ответа. Срок задания был 2-3 дня. Что успел написать за 3 дня - отправил. Писал все сам. Уважаемые люди, кто понимает больше, подскажите, пожалуйста, в какую сторону надо учить?

Comments

[developer007]

а можно саму задачу?

[Артем Громаков]

hwnd32,

Я понимаю, что задача не доделана, но в данном случае меня интересует не вопрос законченности продукта, а слабых сторон кода. Потому что сейчас я бы написал, что то подобное уже быстрее и передо мной стоит вопрос архитектурных решений

[Roman]

Артем Громаков,
https://proglib.io/p/js-design-patterns/
https://jwinters.ru/книги/shablony-igrovogo-progra...

[Артем Громаков]

Roman, Полезно) Спасибо

Answer 1

[xfg]

Высокая цикломатическая сложность подобного кода. Бизнес-логика смешивается с DOM. Практически весь код дублируется.

Такой код сложно поддерживать. Сложно читать. Сложно тестировать. Это действительно слабо. Всё же мы пишем для людей, а не для компьютера. Написать сложный код, очень просто. Написать простой код, очень сложно.

Comments

[Артем Громаков]

Спасибо за критику. Буду работать над этим.

[xfg]

Артем Громаков, попробуйте начать с дублирования кода. Размер кода уменьшиться в двое. Ну а для компании скорее дело было даже не в коде, а в том, что задача фактически не решена. Думаю там желали видеть нечто конечное на Angular/React/Vue.

Answer 2

[developer007]

Меня смущает это

let ships = {
'OneShip': 4,
'TwoShip': 3,
'ThreeShip': 2,
'FourShip': 1
};

Я бы декомпозировал метод

function init(startgame) {

на мелкие т.к не понятно что происходит в таких конструкциях

computer.getBattlegroundArray().forEach(element => {

из нейминга мелких методов стало бы очевиднее .

вообщем над неймингом работать и не писать всё в кучу.

возможно вам ЧСВ ревьювер попался и всё.

Comments

[WapSter]

Или просто нашли кого друго и ввели в заблуждение человека.

[Артем Громаков]

Тимур Турсунбаев, или я просто не доделал до ТЗ, а они смотрят на умение находить решение, которое бы удовлетворяла ТЗ за отведенное время.