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

IrinaZagaynova / oop

0 stars 0 forks source link

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

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

https://github.com/IrinaZagaynova/oop/blob/c47a4d74b32100d8f69b4ff323e06570508346ed/lab4/Shapes/Shapes/ShapeHandler.cpp#L47-L61

[ ] В лямбде shared_ptr-ы лучше принимать по константной ссылке, чтобы избежать относительного дорогостоящего увеличения и уменьшения счетчика ссылок

alexey-malov commented 4 years ago

https://github.com/IrinaZagaynova/oop/blob/c47a4d74b32100d8f69b4ff323e06570508346ed/lab4/Shapes/Shapes/ShapeHandler.cpp#L79-L87

[ ] Эта функция нигде кроме тестов не используется. Ее следует удалить, так как она не несет ценности