Change History (3)

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

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

В решении присутствуют артефакты сборки (папка obj с содержимым, исполняемый файл, служебные файлы идеи (.idea)), нужно их удалить:

{lab_10}[2037]$ pwd && svn up && svn status
/home/hfx/dvl/cpp19/kazakov.nikita/lab_10
Updating '.':
At revision 2434.
{lab_10}[2038]$ ls
include  lab_10  Makefile  obj  README.md  src

Scheme::push_back_figure должна сохранять (и печатать) переменные в порядке добавления, решение сохраняет в обратном.

Прочие замечания:

  • Figure.hpp:11. Почему поле id стало публичным?
  • Circle.hpp. Текущая реализация компилирует код Circle c1(1, 0, 0, 0, “label”), c2 = c1;, но при запуске программа падает. Нужно доделывать (hint: правило трех);
  • Scheme.hpp:19. Храните количество элементов в массиве/свободных элементов на данный момент в std::size_t, зачем нужен вектор неясно;
  • Scheme.hpp:19, 20. разный стиль названия членов-данных класса (с подчеркиванием в конце и без);
  • Figure.cpp:3. Используйте списки инициализации;
  • Rectangle.cpp:4, 13, 22. Строки по 100+ символов. Нужно разбить на несколько;
  • Circle.cpp:11, 12. strncpy;
  • Scheme.cpp:38, 57. Дублируется код поиска по id.

Замечание о списках инициализации актуально для всех классов.

Баллы: 4, доделывайте.

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

На всякий случай: в папке с вашими решениями лежит lab_09, вы собираетесь его сдавать (соотвествующий тикет я не нашел)?

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

Resolution: задача сдана
Status: assignedclosed
Note: See TracTickets for help on using tickets.