Ivanova475 / cplusplus-tasks

0 stars 0 forks source link

homework9-particle-system #9

Open gitluup opened 5 years ago

gitluup commented 5 years ago

https://github.com/Ivanova475/cplusplus-tasks/blob/a6747a911e860ed85c9d0f04ddef9397f2e16c29/ParticleSystem/Seminar9/game.cpp#L46 Что я вижу? Неявные преобразования?


https://github.com/Ivanova475/cplusplus-tasks/blob/a6747a911e860ed85c9d0f04ddef9397f2e16c29/ParticleSystem/Seminar9/window.cpp#L101 Вот этот метод теперь можно и нужно убрать, т.к. Game теперь общается с частицами только с помощью класса ParticleSystem. У Game'а не должно возникнуть соблазна самому отрисовывать частицы.


https://github.com/Ivanova475/cplusplus-tasks/blob/a6747a911e860ed85c9d0f04ddef9397f2e16c29/ParticleSystem/Seminar9/particle_system.cpp#L14 Некрасиво это смотрится! Попробуй после const float radius перенести закрывающую курглую скобку на новую отдельную строку, а запятые в этих строках https://github.com/Ivanova475/cplusplus-tasks/blob/a6747a911e860ed85c9d0f04ddef9397f2e16c29/ParticleSystem/Seminar9/particle_system.cpp#L15 https://github.com/Ivanova475/cplusplus-tasks/blob/a6747a911e860ed85c9d0f04ddef9397f2e16c29/ParticleSystem/Seminar9/particle_system.cpp#L16 https://github.com/Ivanova475/cplusplus-tasks/blob/a6747a911e860ed85c9d0f04ddef9397f2e16c29/ParticleSystem/Seminar9/particle_system.cpp#L17 Перенести в начало, а не в конце ставить


И цвет шариков поменяй! :)