Opened 4 years ago

Closed 4 years ago

#750 closed ожидаются исправления (задача сдана)

HW #2

Reported by: Milyausha Sabirova Owned by: Артур Гулецкий (huletski)
Component: HW #2 (X0) Version: 1.0
Keywords: Cc:

Description

Ввод и вывод пока сделала через iostream, позже сделаю через cstdio. Тесты из условия должны работать, но иногда ввод может обрабатываться некорректно(ввод 4х интов воспринимается как 2 хода подряд,а не Bad move; если ввести что-то кроме интов, программа уходит в бесконечный цикл), постараюсь исправить.
Не получилось написать Makefile, собирающий тесты (возможно, успею разобраться с этим и исправить мейкфайл до проверки).

Change History (2)

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

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

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

  • ошибки в интерфейсe где-то не хватает пробелов, где-то переводов строк, иногда поле выводится лишний раз; сравнивайте с примером;
  • про проблемы с корректностью ввода вы знаете;
  • про stdio вместо cstdio тоже.

Стиль

Board.h

  • отсутствует const у методов;
  • размер поля нужно вынести в константу;

12: getCell

Board.cpp

3: списки инициализации;
6: memset;
61: в методе похожие блоки кода идут друг за другом, нужно от этого избавиться (выделить общую часть и переиспользовать ее);
25, 113: избыточный if, возвращайте непосредственно значение условия;

StdioBoardView?.h

  • поля класса должны быть приватными;
  • const у методов;
  • храните ссылку/указатель на Board, а не копию;

StdioBoardView?.cpp

6: списки инициализации;
54-64: со switch было бы лаконичнее;

main.cpp

7: избыточный if

Test.cpp

7: отступы

BoardTest?.cpp

  • тестов слишком мало.

Баллы

Корректность: 13.5;
Стиль: 5 (так как код будет дописываться).

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

Resolution: задача сдана
Status: assignedclosed

Дедлайн

Note: See TracTickets for help on using tickets.