109
правок
Изменения
Нет описания правки
* Не надо называть одинаковые вещи разными именами и наоборот. Если хочется иметь функцию-геттер size() и private поле size, назовите поле size_.
* Не стоит копировать чужой код, особенно, не думая.
* Лучше подключать сначала внешние хидерызаголовочные файлы, и только потом свои. Желательно каждую группу (boost, stl, свои, ...) выделять пустыми строками. Если внутренние хидеры заголовочные файлы (которые часто добавляют все в глобальное пространство имен) будут объявлены до внешних (в которых все обычно в каком-нибудь неймспейсе), то может случится конфликт имен.
* Стоит делать конструктор от одного аргумента explicit, если его не предполагается использовать для неявного приведения. Неявного приведения разных сущностей быть не должно, int → big_int — OK, string → big_int — плохо.
* В заголовочных файлах у сигнатур функций необходимо писать имена аргументов, чтобы можно было понять, который что значит. Возможным исключением являются операторы, для которых это и так достаточно очевидно.
* Конструктор копирования и оператор присваивания умеют создаваться автоматически. Если для класса нет необходимости писать специальную версию этих функций и нет каких-то других причин для этого, то лучше и не делать.
* Если какой-то из private-членов заканчивается на подчеркивание, то лучше чтобы это выполнялось для всех private-членов.
* Вокруг бинарных операторов ставятся пробелы (при использовании).
