Замечания по Factory

Coguar / OOD

0 stars 0 forks source link

Closed alexey-malov closed 7 years ago

alexey-malov commented 7 years ago

[x] Предоставить примеры текстовых файлов, содержащих инструкции для рисования какого-нибудь осмысленного изображения

alexey-malov commented 7 years ago

[x] В методах фабрики, создающих конкретные фигуры лучше принимать istream&. Зависимость от абстрактных классов лучше, чем от конкретных

alexey-malov commented 7 years ago

[x] Вместо приема указателя на ICanvas в фигурах лучше принимать ссылку, тогда не нужно будет проверять на null. Еще можно воспользоваться gsl::not_null http://stackoverflow.com/questions/33306553/gslnot-nullt-vs-stdreference-wrappert-vs-t https://github.com/Microsoft/GSL/tree/master/include/gsl

То же самое касается других мест