HW #2

[Артур Гулецкий (huletski)]

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

• формат: не выводится перевод строки перед полем; в silent режиме отсутствует вывод поля в конце партии;
• ошибки, связанные с определением корректности ввода (e.g. при вводе одного числа ход не считается неверным, при вводе буквы программа зависает);
• io реализован на потоках, тогда как нужно было реализовать на stdio.

Стиль

Board.h

• отсутствует const у константных методов;

Board.cpp

6: списки инициализации;
34: зачем хранить в локальной переменной? можно возвращать значение _false_ сразу;
51: тернарный оператор вместо if сделал бы код компактнее;
62: дублирование блоков кода (циклы), нужно подумать как от этого избавиться;

StdioBoardView?.cpp

39: если есть отдельный контроллер, странно видеть логику игрового цикла во view;
47: тернарный оператор;

main.cpp

10: избыточный if инициализируйте значение silence (лучшее silent_mode[_enabled]) непосредственно условием из if;

---

Баллы

Корректность: 14;
Стиль: 6 (т.к. код еще будет меняться, io реализован не на базе stdio).

[Артур Гулецкий (huletski)]

Дедлайн