Івент лише для дівчат?

Яка гарна дівчина s.dou.ua/...​sor/techmagic/2/s2/11.jpg

І маршрутки також (або взагалі їх прибрати)

Мне сегодня пришло.

Для рев’ю кода є сайти codereview.stackexchange.com

Так цих code style є 100500 в C++. Як прийнято в команді або проекті, так і краще писати.

Трішки переглянув код. Зараз до джунів дуже великі вимоги. На трейні спокійно взяв би з таким проектом. А на джуна потрібно було ще пройти співбесіду.

Напишу свої зауваження.

1. Замість складної конструкції:
std::vector<Direction> checkedDirections; checkedDirections.reserve(directionCount); checkedDirections.push_back(direction_);
Можна простіше
std::vector<Direction> checkedDirections = { direction_ };

2. В одному і тому ж файлі то є this-> для доступу до локальних членів, то ні. Вибрати один стиль.

3. Definition пустих конструкторів і деструкторів в cpp файлах не потрібні. Краще використати = default;. Навіть більше, якщо деструктор не віртуальний, то навіть не треба його оголошувати.

4. Потрібно користуватись статичними аналізаторами коду: Cppcheck, ReSharper C++, clang тощо. Наприклад, змінна bool directionFound = false; в EnemyTank.cpp не використовується і її можна спокійно видалити.

5. Не потрібно перевіряти змінну на nullptr перед її видаленням.
Замість
if (input_ != nullptr) { delete input_; input_ = nullptr; }
можна спокійно писати
delete input_; input_ = nullptr;
А краще взагалі не використовувати голі вказівники. Лише smart.

6. Англійська неідеальна. Можна поставити spell checker плагіни і перевіряти код. «foundedObject» — «foundObject».

7. Для релізів exe файлів в GitHub є окрема фіча «Releases».

8. Назви комітів у Git повинні бути осмисленими. Коміти — це найкраща історія та документація проекту.

Можливо, геймдев-проекти не писав.
Скажу, що в clang і кількох відомих open source проектах, які читав, такого коду не зустрічав.

Так хіба в C прийнято, тому що там немає bool.

Пічалька тоді, якщо так прийнято на проекті.

Відразу кидається в очі if (isReadyToFire() == true) { ... }. Пишуть же простіше if (isReadyToFire()) { ... }.

dou.ua/forums/topic/22879

Як можна без настільного тенісу в IT?

NTUU KPI

легко гуглиться

Not found

HTML і CSS — це не мови програмування.
Зайве речення «Knowledge the basics of methodologies:»
Досвід нерелевантний, не потрібно на ньому загострювати увагу і виділяти так багато місця в резюме.
Резюме англійською написане, чому назви компаній кирилицею?

Ну почему? Почему нельзя было вместить все на одну страницу?
Формат даты разный «Sep 2017», «Sept 2017», «09.2017».
Interests и References убрать.
Очень много слов «Knowledge» и «Experience».
А кем хотите работать?

Тестером кажете?
olxua-ring03.akamaized.net/...​r-dt-830b-tester-kiev.jpg

Age не указывать. Кстати, по почте можно узнать ваш возраст :)
Вместить все на одну страницу.
Информацию по English вынести выше, так как это важный скилл.
Убрать «Leading a healthy lifestyle», это расскажете на собеседовании.
Правильно писать «Git».
Это не скилл: «The development actively used the interaction with the server (POST, GET)»

Краще ці знаки в свої попередні повідомлення повставляйте.

Можете расставить знаки препинания? Не могу понять, что вы хотели сказать.