Opened 4 years ago

Closed 4 years ago

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

HW #2 itsenko.ekaterina

Reported by: Екатерина Иценко Owned by: Дмитрий Лапшин (lapshin)
Component: HW #2 (X0) Version: 2.0
Keywords: Cc:

Description


Change History (4)

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

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

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

  1. Формат вывода: пробелы после двоеточий.
  2. Определение конца игры оч часто лажает.
  3. silent в какой-то момент выключился.
  4. Ты приняла ход 0 0 0.

9.

Тесты: маловато будет. 0.

Стиль:

  1. StdioBoardView: много дублирования, и просто код сложный, можно проще.
  2. Board::field а. публичный б. зачем динамическая память?
  3. DO_CHECK не там.
  4. Магические константы.
  5. Зачем тебе енамы, если внутри доски у тебя char?
  6. move/can_move не проверяют игрока, модель его не контролирует.
  7. argc == 1 можно честнее.

7.

comment:2 Changed 4 years ago by Екатерина Иценко

Type: ожидаются исправления → ожидается проверка

Вроде должно было стать лучше) С 0 0 0 ничего разумного сделать не получилось, к сожалению:(

comment:3 Changed 4 years ago by Екатерина Иценко

Owner: changed from Екатерина Иценко to Дмитрий Лапшин (lapshin)

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

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

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

  1. Определение конца игры лажает в половине случаев (где-то посреди поля находит победу, но редко и не находит существующую).
  2. Ходы плохие всё так же принимаются.

9.

Тесты: 5.

Стиль:

  1. Непонятно зачем int32_tшки ну ладно.
  2. Код проверки состояния игры всё ещё очень сложный.
  3. DO_CHECK не там.
  4. Отступы в разных файлах разные.

9.

Note: See TracTickets for help on using tickets.