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

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/625f4dc7f7babbf9140241f6a8fb1ee193baf22f/lab4/Shapes/Shapes/CPoint.h#L4-L14

[ ] Состояние объекта CPoint после создания не определено, так как у класса нет ни конструктора, ни явно заданных значений для полей x и y

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/625f4dc7f7babbf9140241f6a8fb1ee193baf22f/lab4/Shapes/Shapes/IShape.h#L4-L11

[ ] C.35: A base class destructor should be either public and virtual, or protected and nonvirtual. Исправить, написать в комментариях к данному Issue, какая с вышеупомянутым кодом проблема и как ее вы решили

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/625f4dc7f7babbf9140241f6a8fb1ee193baf22f/lab4/Shapes/Shapes/CRectangle.h#L5-L26

[ ] В этом классе некоторые данных хранятся избыточно (размеры и координаты правого нижнего угла могут быть выражены друг через друга). Без крайней нужды лучше так не делать, чтобы не приходилось следить за тем, чтобы хранить их в согласованном виде.

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/625f4dc7f7babbf9140241f6a8fb1ee193baf22f/lab4/Shapes/Shapes/ShapesProcessor.h#L2-L26

[ ] Удалить избыточные #include-ы из заголовочного файла
[ ] Метод GetShapeByIndex небезопасен. Мы не знаем, какой диапазон индексов является безопасным. Если вам нужен доступ к коллекции, то лучше передать в ShapesProcessor ссылку на коллекцию фигур снаружи.
[ ] Этот класс сейчас решает несколько задач: считывает фигуры, хранит фигур, ищет минимальную и максимальную фигуры. Выглядит так, что его стоило бы разделить на несколько более простых

alexey-malov commented 4 years ago

https://github.com/RomanovAleksandr/oop/blob/625f4dc7f7babbf9140241f6a8fb1ee193baf22f/lab4/Shapes/Shapes/main.cpp#L6-L13

[ ] Программа пытается открыть файл shapes.txt, который с ней не поставляется. Этот файл не мог бы послужить примером использования, но не надо его открывать в самой программе

alexey-malov commented 4 years ago

[ ] Пользователь программы не знает, в каком формате надо вводить данные о фигурах, а программа ему никакой инструкции не выдает.

RomanovAleksandr commented 4 years ago

IShape* triangle = new CTriangle; delete triangle; В таком случае деструктор CTriangle не вызывается Добавил виртуальные деструкторы в IShape и ISolidShape

RomanovAleksandr / oop

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