HW #2

[Дмитрий Свиридкин]

Нолики начинают первыми

При неверном формате может произойти зацикливание

std::cin >> x >> y;

При таком чтении два числа, разделенные переводом строки, будут считаться корректным ходом.
Используйте getline и istringstream

#include "../include/BoardTest.h"
Что-то у вас пути относительные вылезли

Программа, запускающая тесты, должна возвращать ненулевой код, если что-то не прошло. У вас всегда 0.

Не используйте нижнее подчеркивание в начале имен. Используйте в конце, если нужно.

enum game_status {
    draw,
    player_wins,
    not_ended
};

Возможно, стоит player_wins разделить на X_wins, O_wins (возможно, в прошедшем времени)

От дополнительного поля name в структуре Player можно избавиться.

---

15 + 7 + 0

[Дмитрий Свиридкин]

bool readCell(int &x, int &y, Player cur_player); Лучше подойдет название readMove. или requestMove

Для констант размеров стоит завести константы именованные.

От структуры Player совсем толку не осталось. Ее можно убрать, а методы сделать свободными функциями.

В коде чтения двух чисел делается что-то очень запутанное. Попробуйте отделить чтение с валидацией формата от проверки корректности хода.

---

20 + 8.7 + 0