gitluup
commented
5 years ago https://github.com/ShubinGeorge/cplusplus-tasks/blob/8db790cd8821c1f09d845fb315fff209157df788/ParticleSystem/ParticleSystem/partical_system.h#L42 Первый const не нужен. Пишите просто size_t GetSize() const;
https://stackoverflow.com/questions/8716330/purpose-of-returning-by-const-value
Гораздо другой разговор, когда что-то по ссылке возвращается. Тогда const в самом начале очень-очень полезен, т.к. он означает, что нельзя менять то, на что указывает возвращаемая ссылка.
https://github.com/ShubinGeorge/cplusplus-tasks/blob/5426fdc6a930c072b4c246c9402cddb2cc47ee57/ParticleSystem/ParticleSystem/game.cpp#L75 Зачем же мы писали метод ParticleSystem::Push?) Здесь как раз проще и понятнее, если его вызвать. И так будет правильнее, потому что вы извне класса ParticleSystem пользуетесь знанием о том, что id частицы - это её номер в векторе. Так быть не должно - вся подобная логика должна быть только внутри класса ParticleSystem.
https://github.com/ShubinGeorge/cplusplus-tasks/blob/5426fdc6a930c072b4c246c9402cddb2cc47ee57/ParticleSystem/ParticleSystem/game.cpp#L84 То же самое здесь - надо вызвать ParticleSystem::Update
https://github.com/ShubinGeorge/cplusplus-tasks/blob/5426fdc6a930c072b4c246c9402cddb2cc47ee57/ParticleSystem/ParticleSystem/game.cpp#L95 А здесь, соответсвтенно, ParticleSystem::Render
https://github.com/ShubinGeorge/cplusplus-tasks/blob/5426fdc6a930c072b4c246c9402cddb2cc47ee57/ParticleSystem/ParticleSystem/partical_system.cpp#L73 Пробельчика после if не хватает
https://github.com/ShubinGeorge/cplusplus-tasks/blob/5426fdc6a930c072b4c246c9402cddb2cc47ee57/ParticleSystem/ParticleSystem/partical_system.h#L42 Для наших текущих целей этот метод вовсе не обязателен. А если уж он и есть, то должен быть константным (а сейчас он не константный, а возвращает константу, смысл чего не совсем ясен)
Обратите внимание, что между методами то одна, то две строки. Это непоследовательность) Сделайте везде 2.
Я бы ещё гравитацию побольше сделал, потому что сейчас она еле заметна.