Opened 4 years ago

Closed 4 years ago

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

HW#2 chaykova.anastasiya

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

Description


Change History (4)

comment:1 Changed 4 years ago by chaykova.anastasiya

Component: HW #1 (BMP) → HW #2 (X0)

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

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

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

  1. Формат вывода: пустая строчка перед доской.
  2. Ты приняла ввод 0 0 0.
  3. Один раз неправильно определила победителя (где не было).
  4. Валгринд на каждом тесте.

10.

Тесты:

  1. Неверное имя исполняемого файла.

4.

Стиль:

  1. Магические константы.
  2. Пробелы и переносы.
  3. Дублирование кода в проверке состояния игры.
  4. bool mode очень ясно. Кстати argc == 2 это самонадеянно, давай честнее.
  5. StdioBoardView::sendField можно getCell в переменную и/или сделать switch.
  6. Определение Test::failedNum не там.

7.

comment:3 Changed 4 years ago by chaykova.anastasiya

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

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

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

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

  1. Один раз не нашла конец игры.
  2. Принимаешь плохие ходы, не принимаешь нормальные с пробелами.

13.

Тесты: 5.

Стиль:

  1. Пробелы и переносы.
  2. Определение Test::xxxNum не там. DO_CHECK тоже.

9.

Note: See TracTickets for help on using tickets.