Opened 4 years ago

Closed 4 years ago

#721 closed ожидаются исправления (задача сдана)

HW #2 shirokov.kirill

Reported by: shirokov.kirill Owned by: shirokov.kirill
Component: HW #2 (X0) Version: 2.0
Keywords: Cc:

Description


Change History (2)

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

Owner: changed from Дмитрий Лапшин (lapshin) to shirokov.kirill
Summary: HW #2 → HW #2 shirokov.kirill
Type: ожидается проверка → ожидаются исправления
Version: 1.0 → 2.0

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

  1. Неправильный Makefile.
  2. Формат вывода чёрт возьми. Много где.
  3. На очень многих тестах валгринд говорит об неинициализированных переменных.
  4. Не все финалы игры определяются корректно.
  5. Ты принимаешь ход 0 0 0.

8.

Тесты: явно не проверяют все публичные методы, явно не ловят все баги. 2.

Стиль:

  1. Магические числа.
  2. Пробелы почти везде.
  3. this-> избыточно.
  4. Дикое дублирование кода в check_winner.
  5. GameController::start_game(int) я ж про перечисления рассказывал. Ну или bool silent…
  6. argc == 1 а может лучше честно?
  7. В тестах нарушена структура файлов.
  8. В тестах: ну и зачем std::string?
  9. Board: а чо всё публичное? Ещё по названиям не ясно, что происходит.
    • Тем более смущают методы change_current_player и check_winner (тем что требует подсказки снаружи).

5.

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

Resolution: → задача сдана
Status: assigned → closed
Note: See TracTickets for help on using tickets.