Opened 4 years ago

Closed 4 years ago

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

HW #2 kirillova.evgeniya

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

Description


Change History (3)

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

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

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

  1. Определение конца игры иногда рано/неправильно завершает.
  2. Ты допускаешь ход 0 0 0.

13.

Тесты: 5.

Стиль:

  1. Board::field ну давай приличнее, без динамической памяти.
  2. Магические константы!
  3. Дублирование кода в проверке игры.
  4. argc == 2 а давай честнее.
  5. Дублирование кода в StdioBoardView, кажется можно что-то вынести, что-то объединить.
  6. Местами пробелы.

7.

Очень приятное впечатление от посылки)

comment:2 Changed 4 years ago by kirillova.evgeniya

Owner: changed from kirillova.evgeniya to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка
Version: 2.0 → 3.0

Я вроде бы постаралась сделать корректный ввод, но проверка получилась какой-то очень страшной, наверное, это не очень правильно...
Но ход 0 0 0 не допускается, ход 0-0 тоже

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

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

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

  1. Пару раз определение конца игры слажало.

13.

Тесты: 5.

Стиль:

  1. Доска публична зачем-то.
  2. Размер магический.
  3. DO_CHECK не там.
  4. StdioBoardView зачем поля x, y, input?
  5. Код хода игры дублируется, код проверки состояния игры дублируется.
  6. Местами пробелы.

7.

Note: See TracTickets for help on using tickets.