HW #2

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

Не работает победа при размещении по диагонали -- программа зацикливается, ожидая новые ходы.

O move:
(какой-то ход, заполняющий диагональ) 
..........
.....X....
...X......
..O...X...
...O...X..
....O.....
.....O.X..
......O...
.......O..
..........

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

Board(int size = 3, int number = 3);
Второй параметр лучше переименовать, чтоб было понятно, что это длина победной комбинации.

Player Board::winner() {
    static bool principalDiagonal;
    static bool secondaryDiagonal;
    static bool vertical;
    static bool horizontal;

Неконстантные статические локальные переменные лучше никогда не делать.

std::cin >> x >> y;
if (std::cin.good())
    return true;

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

---

5 + 7 + 0

[sunko.elena]

В описании правил игры написано, что целью является выставить 5 своих знаков в ряд на любой из двух диагоналей, разве не подразумеваются главная и побочная? Просто в примере эта диагональ главной не является, то есть он же и не должен выводить победу 0

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

Ход 0-0 воспринимается как валидный

Довольно сложную логику с определением текущего состояния поля можно успростить, сделав функцию, проверяющую N клеток в заданном направлении относительно начальной точки.

Не очень понятно, почему функция, которая должна распарсить int из стррки, называется incorrectArgument.
Попробуйте сделать чтение проще: getline + istringstream + чтение двух строк.

---

17 + 7.5 + 0