ревью крестики- нолики

[arthas]

Всем привет!

Недавно начал изучать js и программирование в целом.
Написал свою первую программу, крестики и нолики в стиле ООП
Если у кого то есть желание или возможность, просьба указать на ошибки

Ссылка на код на Гитхабе

Заранее спасибо за помощь

[рони]

arthas,
просто для примера ...
крестики нолики на поле любой величины

https://javascript.ru/forum/misc/812...tml#post530163

[voraa]

Что бы уж было совсем ООП panel тоже надо бы как объект со своим классом.
Какой смысл циклов в функции checkDiagonals?
Зачем в player нужны makeX() и makeO()? Почему нельзя через конструктор?

Ну и лучше сначала заполнить board ячейками, а потом уже засовывать board в dody.

[arthas]

Что бы уж было совсем ООП panel тоже надо бы как объект со своим классом.
Какой смысл циклов в функции checkDiagonals?
Зачем в player нужны makeX() и makeO()? Почему нельзя через конструктор?

Ну и лучше сначала заполнить board ячейками, а потом уже засовывать board в dody.

Про панель согласен вынесу в отдельный класс, циклы уберу

В классе Player можно и через конструктор, но как тогда ограничить количество возможных игроков( что было только X И O)?

Последнее поправил, спасибо за советы!

[voraa]

но как тогда ограничить количество возможных игроков( что было только X И O)?

Да так же, как и сейчас.
Только вместо

let onePlayer = new player();
onePlayer.makeX();
let twoPlayer = new player();
twoPlayer.makeO();
let b = new board(onePlayer, twoPlayer);

Писать что то вроде

let b = new board(new Player('X'), new Player('O'));

[Vlasenko Fedor]

магический квадрат и крестики нолики на основе собственного велосипедостроения

https://jsfiddle.net/vlasenkofedor/5ad26zc7/

[arthas]

Всем спасибо за советы!
Код поправил.