HW #2 chistyakova.polina

[chistyakova.polina]

Извини, мне так стыдно >/ / /<
Я очень старалась, но что-то как-то не очень.....

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

Не страдай самобичеванием. Тут конечно хватает что чинить, но в целом-то почти всё готово.

CMakeLists.txt: версию, пожалуйста, не новее 3.10.

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

1. Лажает с проверкой состояния игры (говорит, что победа, когда вообще-то ещё нет, или наоборот не замечает).
2. На -1 -1 так грубо не надо, можно потихонечку выйти.
3. Невалидные ходы (в том числе вида 0 0 0) тебя роняют вместо Bad move!.

Очень много тестов не прошло, 4.

Тесты: непохоже на полный набор, и структура файлов нарушена. 1.

Стиль:

1. Что-то структурно пошло не так. Смотри:
   • В классическом MVC у нас есть Model, которая управляет состоянием, View, способная взаимодействовать с пользователем (в обе стороны), и Controller, управляющий порядок взаимодействия пользователя с моделью и управляющий отображением.
   • В этом задании Controller и View слишком сложно разделить: маленькие и плотно связанные, поэтому у нас это одна сущность.
   • А нарелаза ты не совсем правильно. Board почему-то значет, что писать на экран, Controller содержит ввод-вывод с пользователем (он не должен знать!), View умеет лишь печатать строчку (мило, но не его). Короче, надо переразбить.
2. (argv == 2) а давай почестнее.
3. Board::stop_game.
4. Интерфейс View огонь, правильно: ведь если делать с бонусом или настоящим контроллером, то он нужен, или хотя бы полезно. Другое дело что он не оч правильный, но это выше уже было.
5. Магические чиселки. Вообще говоря, если завести константу 10, то после этого можно было хранить не вектор векторов, а вполне конкретный State[SIZE][SIZE]. Тем более у вектора есть конструктор от значения и размера.
6. Код вида if (...) return true; else return false; слегка намекает.
7. Тудушки по делу ;)

В целом написано очень классно, примечаний на мелком уровне нет. 7.

Удачи чинить, ты справишься!