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

Всем привет!:thanks:

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

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

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

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

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

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

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

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

В классе Player можно и через конструктор, но как тогда ограничить количество возможных игроков( что было только 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'));

магический квадрат и крестики нолики на основе собственного велосипедостроения
:lol:
https://jsfiddle.net/vlasenkofedor/5ad26zc7/

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