WW #10

[Артур Гулецкий (huletski)]

Замечания:

• неверный формат печати фигур: отсутствует ":" после %id% -> -0.25;
• Scheme::is_inside_figure зачем-то печатает сообщение, если фигура не найдена. Реакция на такую ситуацию может быть разная, выбор конкретной лучше оставить коду-пользователю класса Scheme -> -0.25;
• Figure.hpp:12. В общем случае, лучше использовать “отглагольный” имена методов: s/id_getter/get_id. Вариант “id()” тоже ок; неясно почему метод неконстантный;
• Circle.hpp. Код Circle c1(1, 0, 0, 1, "c1"), c2 = c1; компилируется, но падает при запуске. Надo-то определение Circle доделывать (hint: rule of 3), same for Scheme -> -1;
• Scheme.hpp. Отсутствует const у методов, которые не изменяют состояние объекта -> -0.25;
• Scheme.hpp:20, 21. Используется разный стиль именования членов-данных класса (с нижним подчеркиванием в конце и без), придерживайтесь одного;
• Circle.cpp:5. Используйте списки инициализации, для инициализации полей класса -> -0.5;
• Circle.cpp:10. лучше strncpy, тем более, что strlen считается строкой выше;
• Circle.cpp:15. К полям можно обращаться без this->, если нет конфликтов с локальными переменными, так кода получается более компактным;
• Circle.cpp:21. При переносе выражения, перед второй и последующими строками обычно ставятся дополнительные отступы (как в Rectangle::print);
• Rectangle.cpp:21. if лишний: return dx <= width / 2 && dy <= height / 2;
• Scheme.cpp:49. похоже на std::memmove;
• Scheme.cpp. Дублируется код поиска элемента по id. Лучше его вынести в метод.

Баллы: 7.75, нужно доделывать.