HW#2 subbotina.olesya

[Дмитрий Лапшин (lapshin)]

Корректность:

1. Неправильно определяешь конец игры (иногда рано, на полной доске тоже).
2. Много плохих ошибок от Valgrind на каждый тест.
3. Ты принимаешь ход вида 0 0 0.

9.

Тесты: не выводят статистику. 4.

Стиль:

1. Перечисления модели знают о текстовом представлении, что немножко странно.
2. Магические числа.
3. Размер доски одновременно и константа времени компиляции, и динамическое поле класса. Это плохо.
4. Доска публична.
5. can_move/move не проверяют игрока, который совершает ход.
6. Для геттеров наверное норм делать их тело inline, описав их в классе, но для runAllTests это уже менее уместно.
7. Не ясно, зачем Board &board публично.
8. Стиль:

   if () {
   } else {
   }
   

9. Писать if (...) return true; else return false; несколько нелогично.
10. Жестокое дублирование кода в проверке состояния игры.

7.

[Дмитрий Лапшин (lapshin)]

Корректность:

1. Валгринд орёт по делу. Очень по делу.
2. silent режим формат вывода.
3. Плохие ходы принимаешь.

10

Тесты: 5.

Стиль:

1. Размер доски одновременно и константа времени компиляции, и динамическое поле класса. Это плохо.
2. Ты используешь геттеры даже внутри класса.
   • И при этом #define SIZE.
3. Код проверки состояния дублирует очень много кода, и очень сложен.
4. if (player == EMPTY) { ...; continue; } if (x == player && x != EMPTY) а как может x == EMPTY?

8.