Opened 4 years ago

Closed 4 years ago

#755 closed ожидается проверка (задача сдана)

HW #2

Reported by: kozubaev.nurmukhammad Owned by: Дмитрий Лапшин (lapshin)
Component: HW #2 (X0) Version: 1.0
Keywords: Cc:

Description


Change History (1)

comment:1 Changed 4 years ago by Дмитрий Лапшин (lapshin)

Resolution: → задача сдана
Status: assigned → closed

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

  1. Ну координаты не в том порядке.
  2. И формат вывода по мелкому.

Кроме этого проблем нет, хорошо!
11.

Тесты:

  1. Неправильное имя бинарного файла.
  2. Тестов явно не хватает. И покрывают не все методы.

0.

Стиль:

  1. GameState: непонятно, зачем аббревиатура в значениях.
  2. Board::getCell непонятно зачем константная ссылка на енам.
  3. Имена надо бы давать параметрам в объявлениях функций.
  4. Проблемы с пробелами, отступами и переносами. Местами такие большие, что я читать в исходном виде забил, очень-очень-очень плохо.
  5. Зачем friend class Test;?
  6. runSilentMode не очень хорош, лучше было бы параметр конструктора.
  7. Магические константы. Везде.
  8. Код проверки состояния игры дублируется.
  9. DO_CHECK не там (ожидался в Test.h)
  10. По части структуры файлов между Test и BoardTest нарушено.
  11. Не очень ясно, зачем StdioBoardView поля needToPrint и gameIsFinished.

5.

Note: See TracTickets for help on using tickets.