Замечания по программе Shapes

[alexey-malov]

https://github.com/voronindim/OOP/blob/75b6af6a75668f3db7eba7fbf092b14909018fa4/lab_4/Shapes/ICanvasDrawable.h#L4-L8

• [ ] C.35: A base class destructor should be either public and virtual, or protected and nonvirtual. Разобраться и уметь объяснить (написать в комментариях к этому issue)

• [ ] Вряд ли ожидается, что метод draw изменит состояние объекта. Метод лучше сделать константным

[alexey-malov]

• [ ] Статический метод конвертирования цвета в строку дублируется между классами фигур.

[alexey-malov]

• [ ] В cmake-файле не должны использоваться абсолютные пути. Особенно, привязанные к конкретной платформе

[alexey-malov]

https://github.com/voronindim/OOP/blob/75b6af6a75668f3db7eba7fbf092b14909018fa4/lab_4/Shapes/CCanvas.h#L5-L16

• [ ] Вместо зависимости от RenderWindow лучше использовать зависимость от RenderTarget. Тогда холст можно будет связать еще и с текстурой

[alexey-malov]

https://github.com/voronindim/OOP/blob/75b6af6a75668f3db7eba7fbf092b14909018fa4/lab_4/Shapes/CShapeProcessor.cpp#L5-L15

• [ ] Лучше в качестве значений map использовать regex-ы. Тогда не будет для каждой фигуры заново из строки создаваться объект регулярного выражения.

• [ ] Почитайте про non-capturing groups в регулярных выражениях. Тогда вы избавитесь от избыточных групп.

-[ ] Еще лучше будет хранить в качестве значений - функции для парсинга, а не регулярки. Сами регулярки - в виде статических констант внутри функций. Тогда регулярное выражение будет объявлено близко к месту его использования, что упростит анализ кода.