Здравствуйте, LaptevVV, Вы писали:

LVV>https://tproger.ru/translations/stanford-cpp-style-guide/
LVV>Практически так же и пишу.
LVV>Только скобки блока — всегда на отдельных строках.

Я вместо

const int VOTING_AGE = 18;

обычно #define делаю.

Используйте ссылочные переменные, а не указатели. Одна из причин — это то, что ссылочные переменные, в отличие от указателей, не могут принимать значение NULL:

// Плохая практика
// Принимает указатель
void process(BankAccount* account) {
...
}

// Хорошая практика
// Принимает адресную ссылку
void process(BankAccount& account) {
...
}

Иногда NULL в качестве параметра может подразумевать отдельную ветвь логики работы функции