Практические занятия по курсу ОиМП/C++ check — различия между версиями
Материал из Wiki - Факультет компьютерных наук
(→Именование) |
|||
Строка 55: | Строка 55: | ||
== Именование == | == Именование == | ||
+ | <ol> | ||
<li>Не используйте транслит в именах. | <li>Не используйте транслит в именах. | ||
− | |||
− | |||
<source lang="cpp"> | <source lang="cpp"> | ||
+ | // плохо | ||
size_t dlina_massiva; | size_t dlina_massiva; | ||
</source> | </source> | ||
− | |||
− | |||
<source lang="cpp"> | <source lang="cpp"> | ||
+ | // хорошо | ||
size_t length; | size_t length; | ||
</source></li> | </source></li> | ||
<li>Давайте переменным осмысленные имена. В частности, не используйте однобуквенных имён за исключением имён итераторов (i, j, k, ...), координат (x, y, z). | <li>Давайте переменным осмысленные имена. В частности, не используйте однобуквенных имён за исключением имён итераторов (i, j, k, ...), координат (x, y, z). | ||
− | |||
− | |||
<source lang="cpp"> | <source lang="cpp"> | ||
+ | // плохо | ||
char alf(int j) { | char alf(int j) { | ||
... | ... | ||
Строка 79: | Строка 77: | ||
} | } | ||
</source> | </source> | ||
− | |||
− | |||
<source lang="cpp"> | <source lang="cpp"> | ||
+ | // хорошо | ||
char get_char(int j) { | char get_char(int j) { | ||
... | ... | ||
Строка 87: | Строка 84: | ||
</source> | </source> | ||
− | |||
<source lang="cpp"> | <source lang="cpp"> | ||
+ | // плохо | ||
int ln; | int ln; | ||
</source> | </source> | ||
− | |||
− | |||
<source lang="cpp"> | <source lang="cpp"> | ||
+ | // хорошо | ||
int length; | int length; | ||
+ | </source></li> | ||
+ | |||
+ | <li>Функциям, которые возвращают bool, лучше давать имена, начинающиеся на is или has. | ||
+ | <source lang="cpp"> | ||
+ | // плохо | ||
+ | bool graph_connected() { ... } | ||
+ | </source> | ||
+ | <source lang="cpp"> | ||
+ | // хорошо | ||
+ | bool is_connected_graph() { ... } | ||
+ | </source> | ||
+ | <source lang="cpp"> | ||
+ | // плохо | ||
+ | bool element(int n) { ... } | ||
+ | </source> | ||
+ | <source lang="cpp"> | ||
+ | // хорошо | ||
+ | bool has_element(int n) { ... } | ||
</source></li> | </source></li> | ||
</ol> | </ol> |
Версия 00:31, 28 февраля 2015
Оформление
- Отступ - 4 пробела. Символы табуляции использовать для отступов запрещено.
- Отступами выделяются: тела функций, структур/классов, вложенных блоков.
- Бинарные операторы отбиваются пробелами с двух сторон, после унарных пробелы не ставятся: a += b + -c
- Пробелы после открывающей скобки и перед закрывающей скобкой не ставятся: f(1, (2 + 3)). Закрывающая скобка должна идти на той же строке, что и последнее выражение.
- Максимальная длина строки - 100 символов.
- Перед ; пробел не ставится. После ; в for ставится пробел.
- Пустые блоки записываются как {} (а не ;).
- Открывающая { пишется на той же строке, что и начало блока (if, while, for, объявление функции).
- else пишется на той же строке, что и закрывающая } от if: if (...) {...\n} else {
- Однострочные комментарии отделяются от кода двумя пробелами и начинаются с пробела.
- Пробелы в конце строки запрещены.
- Файл должен заканчиваться переводом строки.
- В range-base for двоеточие обрамляется пробелами.
- В начале/конце блока, после public/private/protected пустые строки не ставятся.
- Перед объявлением функции/структуры/класса - пустая строка обязательна.
- Секции include-ов и using-ов были упорядочены по алфавиту.
Плохо:
#include <vector> #include <iostream>
Хорошо:
#include <iostream> #include <vector>
Плохо:
using std::vector; using std::cin; using std::cout;
Хорошо:
using std::cin; using std::cout; using std::vector;
- Имя шаблона и параметр шаблона НЕ должны разделяться пробелом.
Плохо:
vector <int> v;
Хорошо:
vector<int> v
- Не используйте транслит в именах.
// плохо size_t dlina_massiva;
// хорошо size_t length;
- Давайте переменным осмысленные имена. В частности, не используйте однобуквенных имён за исключением имён итераторов (i, j, k, ...), координат (x, y, z).
// плохо char alf(int j) { ... } // или char alphabet(int j) { ... }
// хорошо char get_char(int j) { ... }
// плохо int ln;
// хорошо int length;
- Функциям, которые возвращают bool, лучше давать имена, начинающиеся на is или has.
// плохо bool graph_connected() { ... }
// хорошо bool is_connected_graph() { ... }
// плохо bool element(int n) { ... }
// хорошо bool has_element(int n) { ... }
- Если требуется перебрать элементы коллекции, предпочитайте range-base for. Он лаконичней и легче читается.
// плохо for (size_t i = 0; i < neighbors_list[vertex].size(); ++i) { if (!(visited[neighbors_list[vertex][i]])) { dfs(neighbors_list[vertex][i], visited); } }
// хорошо for (const int& neighbor_vertex : neighbors_list[vertex]) { if (!(visited[neighbor_vertex])) { dfs(neighbor_vertex, visited); } }
- Необходимо явно подключать заголовочные файлы, в которых объявляются используемые функции/классы/... Запрещено явно подключать один и тот же заголовочный файл дважды.
- В качестве логических операторов следует использовать &&, ||, ... Их аналоги and, or, ... запрещены.
- Запрещено использовать приведение типов в стиле C - следует использовать *_cast.
- Конструктор от одного аргумента должен быть объявлен explicit.
- Порядок аргументов функции: сначала входные параметры (по значению либо константой ссылке), затем выходные (по указателю).
- При объявлении виртуальной функции следует использовать один и только один из спецификаторов virtual, final, override.
- При объявлении переменной спецификаторы static/extern/... идут перед именем типа.
- Запрещено использовать директиву using namespace. Нужно либо каждый раз указывать namespace явно, либо импортировать конкретные типы/переменные: using std::vector.
- Передавайте аргументы в функции по константной ссылке везде, где это уместно.
Плохо:
void print_vector(vector<int> v) { ... }
Хорошо:
void print_vector(const vector<int>& v) { ... }
- Везде, где это возможно, используйте префиксный инкремент и декремент.
Плохо:
i++; it--;
Хорошо:
++i; --it;
- Не создавайте функции с избыточным числом аргументов. Например, в функцию, печатающую вектор, не нужно передавать размер вектора.
Плохо:
void print_vector(const vector<int>& v, int n) { for (size_t i = 0; i < n; ++i) { cout << v[i] << " "; } }
Хорошо:
void print_vector(const vector<int>& v) { // Используйте v.size(), чтобы получить длину вектора for (size_t i = 0; i < v.size(); ++i) { cout << v[i] << " "; } }
- Пишите код так, чтобы не было предупреждений (warnings) компилятора. В частности, избегайте сравнений знаковых (int) и беззнаковых (size_t) переменных.
Плохо:
for (int i = 0; i < v.size(); ++i) { ... }
Хорошо:
for (size_t i = 0; i < v.size(); ++i) { ... }