Opened 4 years ago

Closed 4 years ago

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

HW #2

Reported by: gordeeva.tatyana Owned by: Артур Гулецкий (huletski)
Component: HW #2 (X0) Version: 2.0
Keywords: Cc:

Description

Извините за опоздание :(

Change History (3)

comment:1 Changed 4 years ago by Артур Гулецкий (huletski)

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

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

Недочеты в логике обработки ходов. Например:

  • ход, состоящий из не двух (1, 3 и тд) чисел, должен считаться некорректным;
  • некорректный ход 1t2 приводит к зависанию приложения.

Стиль

  • код StdioBoardView? нужно вынести в отдельные файлы;

Board.cpp

  • код поиска победных линий дублируется, подумайте, как можно от этого избавиться;

135: имя запутывает. Лучше has_empty_cells, игра, если речь о ней, может быть finished и по другой причине;
168, 183: дублирование логики вывода доски, нужно реорганизовать цикл так, чтобы дублирования не было (one and a half loop).

BoardTest?.cpp

77: выглядит более логичным заменить тесты на testChangePlayerManyOdd, testChangePlayerManyEven;

Баллы:

Корректность: 16
Стиль: 7

comment:2 Changed 4 years ago by gordeeva.tatyana

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

comment:3 Changed 4 years ago by Артур Гулецкий (huletski)

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

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

Ок.

Стиль

Board.h

6: константы лучше сделать статическими константами класса;
7: направления - статической константой в методе;
26: не хватает запрещения op=.

Board.cpp

35: лучше std::size_t для типа индекса массива.

Баллы

Корректность: 20
Стиль: 9.75

Note: See TracTickets for help on using tickets.