ShubinGeorge / cplusplus-tasks

Represetory for C++ homework
2 stars 0 forks source link

homework10-final-particle-system #12

Closed gitluup closed 5 years ago

gitluup commented 5 years ago

https://github.com/ShubinGeorge/cplusplus-tasks/blob/db9719524f1dd85dc61c05f9affaed4ad6e7576f/SystemWithInteraction/SystemWithInteraction/partical_system.cpp#L7 Это должно быть внутри класса ParticleSystem, потому что этот псевдоним типа относится именно к ParticleSystem и ни к чему более (это называется принципом локальности, т.е. всё должно быть как можно более локальным и, соответственно, как можно менее глобальным).

Этот using уже есть в классе ParticleSystem, возможно, вы просто забыли его отсюда убрать.


https://github.com/ShubinGeorge/cplusplus-tasks/blob/db9719524f1dd85dc61c05f9affaed4ad6e7576f/SystemWithInteraction/SystemWithInteraction/partical_system.h#L53 Метод GetSize не нужен


Ещё можно сказать, что у вас между реализациями методов то 2 строки, то одна) Хоть это и мелочь, но я считаю, что даже за такими вещами лучше следить


https://github.com/ShubinGeorge/cplusplus-tasks/blob/db9719524f1dd85dc61c05f9affaed4ad6e7576f/SystemWithInteraction/SystemWithInteraction/partical_system.cpp#L210 Здесь, по-моему, не должно быть двойки


gitluup commented 5 years ago

ОК. Потом на досуге можете подумать, как реализовать все или некоторые фичи для системы частиц :)