HW #2 chizhova.mariya

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

В include_directories пишутся папки)

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

1. Формат вывода. Много всякого, починить сам не смог.
2. Неправильно начинается игра.
3. Неправильно читаются ходы.
   • Включая неправильные — ход 1 1 1 ты примешь.
4. Неправильно определяешь некоторые исходы игры, большую часть тестов.

5.

Тесты: не обнаружен исполняемый файл.

Стиль:

1. Board: магические константы! Особенно весело то, что есть отдельное поле с доской фиксированного размера, а отдельно поля с динамическим размером (которые даже не const).
   • Для внешних пользователей сделала геттеры — это похвально, но не решает проблемы.
2. StdioBoardView должен держать *ссылку*.
3. Board::move верит, что вызывающий указал правильную фигуру, и не проверяет.
4. Дублирование кода в проверке состояния игры с дублированиями кода и магическими числами.
5. Зачем хранить символы в качестве доски, когда есть enum?
6. main: зачем new?
7. Структура циклов в коде самой игры несколько сложна.

6.

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

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

1. Всё ещё формат вывода не до конца соблюдаешь: перевод строки перед первой доской, пробел после двоеточия.
2. На некоторых тестах неправильно определяет состояние игры. На них же Valgrind говорит об неинициализированных переменных.
3. Неправильно разбираешь ввод, считая ход 0 0 0 корректным.

12.

Тесты: 5.

Стиль:

1. В include_directories пишутся папки). Инклуды пишем без относительных путей.
2. Кстати лишние инклуды.
3. Имена значений перечеслений не в одном стиле.
4. Не очень хорошо, что значения в перечисления в модели что-то знают об отображении.
5. Board: магические константы! Особенно весело то, что есть отдельное поле с доской фиксированного размера, а отдельно поля с динамическим размером (которые даже не const).
   • Для внешних пользователей сделала геттеры — это похвально, но не решает проблемы.
   • Рекомендую вместо полей и геттеров завести статическую константу. static constexpr int FIELD_SIZE = 10;, и лучше сразу 17 стандарт включить, иначе ещё проблемы.
6. StdioBoardView должен держать ссылку. Ты держишь поле.
7. StdioBoardView: список инициализации?
8. DO_CHECK: много объявлений, все не там.
9. Тоже не там:

   int Test::failedNum = 0;
   int Test::totalNum = 0;
   

10. Board::move/сan_move верит, что вызывающий указал правильную фигуру, и не проверяет.

6.

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

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

1. Объявляешь ничью на неполной доске иногда.
2. В режиме silent нарушился формат.
3. Принимаешь неверный ход.

12.

Тесты: 5.

Стиль:

1. Почему Board::check публичный? Зачем Board::row,col?
2. FIVE это-то понятно, что 5, а имя так себе.
3. DO_CHECK: не там. Логичнее в Test.h.

9.