anatolykorablyov / alien_game Goto Github PK

C++ 98.50% C 1.50%

alien_game's People

в классах поля лучше называть с префиксом m_, так меньше будет путаницы между полями и параметрами методов тех же классов
в конструктор Player вместо 6 чисел X, Y, ... можно передать sf::FloatRect и sf::Vector2f. Вообще не стоит писать функции с числом параметров более 5-7, в таких функциях легко запутаться
в конструкторе Player вместо такого присвоения ``health = 100; armor = 100; weapon = 1;` следует использовать списки инициализации конструктора
перегруженные виртуальные методы базового класса Entity следует помечать словом override. Вместо void update(float time, float coordX, float coordY); в классе Player должно быть void update(float time, float coordX, float coordY) override, а лучше и вовсе void update(float time, sf::Vector2f pos) override.
в level.h следует всё, что ниже ///////////////, перенести в новый файл level.cpp. Тогда не будет проблем с включением level.h в разные файлы.

в файле ActionPlayer.h надо оставить только объявления функций, а их определения перенести в новый файл ActionPlayer.cpp
Есть много дублирований вида one->getRect().intersects(two->getRect()), лучше добавить функцию bool Intersects(Entity &one, Entity &two) или даже метод bool Intersects(Entity &other) внутри класса Entity.
В функцию void damage_player_to_enemys(Entity* &one, Entity* &two) не надо передавать ссылки на указатели, ведь функция не проверяет указатели на nullptr. Поскольку передача нулевых указателей тут не имеет смысла, то следует использовать просто ссылки.
предыдущее замечание касается также pick_up_bonuses_and_exit

Функция main слишком велика. Следует её упростить, предлагаю следующие этапы (после каждого этапа рекомендую делать commit)

упростить for (it = entities.begin(); it != entities.end(); it++), используя range-based for и получая элементы массива entities по изменяемой ссылке. Тогда не будет множества разыменований итератора, код станет намного более читаемым. Про range-based for: http://en.cppreference.com/w/cpp/language/range-for
создать структуру Application, в которой поместить отдельные методы для инициализации, для запуска основного цикла, для обработки событий в пределах цикла, для обработки коллизий, для рисования объектов. Тогда функция main уменьшится в объёме, возможно вообще останется 3 строки: создание Application, вызов метода app.run() и return 0;