Opened 4 years ago

Closed 4 years ago

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

HW #2 krasov.vladislav

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

Description


Change History (5)

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

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

Не собирается ничем.

comment:2 Changed 4 years ago by krasov.vladislav

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

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

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

Ну попытка к 2 дедлайну потрачена, к 3 засылай.

Я глянул, как-то работает, что-то на правду похоже, но формат не соблюдает и исход игры не всегда правильно определяет.

comment:4 Changed 4 years ago by krasov.vladislav

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

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

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

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

  1. Валгринд на каждом тесте.
  2. Не везде корректно определяет конец игры.

10.

Тесты: 5.

Стиль:

  1. #pragma once/include guards.
  2. Почему change_player публичный метод? Доска доверяет, как её крутят?
  3. Если есть get_state, зачем публичные check_?
  4. Почему Board что-то знает о silent?! Почему через публичное поле?!
  5. Код проверки состояния сложен, дублируется.
  6. Постоянно какие-то странные отрицания вижу. !is_silent, метод чтения возвращает бул если он неправильный...
  7. Относительные инклуды.

7.

Note: See TracTickets for help on using tickets.