ClickHouse/docs/zh/development/style.md
2020-01-30 13:34:55 +03:00

839 lines
24 KiB
Markdown
Raw Blame History

This file contains ambiguous Unicode characters

This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.

# 如何编写 C++ 代码
## 一般建议
**1.** 以下是建议,而不是要求。
**2.** 如果你在修改代码,遵守已有的风格是有意义的。
**3.** 代码的风格需保持一致。一致的风格有利于阅读代码,并且方便检索代码。
**4.** 许多规则没有逻辑原因; 它们是由既定的做法决定的。
## 格式化
**1.** 大多数格式化可以用 `clang-format` 自动完成。
**2.** 缩进是4个空格。 配置开发环境,使得 TAB 代表添加四个空格。
**3.** 左右花括号需在单独的行。
```cpp
inline void readBoolText(bool & x, ReadBuffer & buf)
{
char tmp = '0';
readChar(tmp, buf);
x = tmp != '0';
}
```
**4.** 若整个方法体仅有一行 `描述` 则可以放到单独的行上。 在花括号周围放置空格(除了行尾的空格)。
```cpp
inline size_t mask() const { return buf_size() - 1; }
inline size_t place(HashValue x) const { return x & mask(); }
```
**5.** 对于函数。 不要在括号周围放置空格。
```cpp
void reinsert(const Value & x)
```
```cpp
memcpy(&buf[place_value], &x, sizeof(x));
```
**6.** 在`if``for``while`和其他表达式中,在开括号前面插入一个空格(与函数声明相反)。
```cpp
for (size_t i = 0; i < rows; i += storage.index_granularity)
```
**7.** 在二元运算符(`+``-``*``/```...)和三元运算符 `?:` 周围添加空格。
```cpp
UInt16 year = (s[0] - '0') * 1000 + (s[1] - '0') * 100 + (s[2] - '0') * 10 + (s[3] - '0');
UInt8 month = (s[5] - '0') * 10 + (s[6] - '0');
UInt8 day = (s[8] - '0') * 10 + (s[9] - '0');
```
**8.** 若有换行,新行应该以运算符开头,并且增加对应的缩进。
```cpp
if (elapsed_ns)
message << " ("
<< rows_read_on_server * 1000000000 / elapsed_ns << " rows/s., "
<< bytes_read_on_server * 1000.0 / elapsed_ns << " MB/s.) ";
```
**9.** 如果需要,可以在一行内使用空格来对齐。
```cpp
dst.ClickLogID = click.LogID;
dst.ClickEventID = click.EventID;
dst.ClickGoodEvent = click.GoodEvent;
```
**10.** 不要在 `.``->` 周围加入空格
如有必要,运算符可以包裹到下一行。 在这种情况下,它前面的偏移量增加。
**11.** 不要使用空格来分开一元运算符 (`--`, `++`, `*`, `&`, ...) 和参数。
**12.** 在逗号后面加一个空格,而不是在之前。同样的规则也适合 `for` 循环中的分号。
**13.** 不要用空格分开 `[]` 运算符。
**14.**`template <...>` 表达式中,在 `template``<` 中加入一个空格,在 `<` 后面或在 `>` 前面都不要有空格。
```cpp
template <typename TKey, typename TValue>
struct AggregatedStatElement
{}
```
**15.** 在类和结构体中, `public` `private` 以及 `protected``class/struct` 无需缩进,其他代码须缩进。
```cpp
template <typename T>
class MultiVersion
{
public:
/// Version of object for usage. shared_ptr manage lifetime of version.
using Version = std::shared_ptr<const T>;
...
}
```
**16.** 如果对整个文件使用相同的 `namespace`,并且没有其他重要的东西,则 `namespace` 中不需要偏移量。
**17.**`if`, `for`, `while` 中包裹的代码块中,若代码是一个单行的 `statement`,那么大括号是可选的。 可以将 `statement` 放到一行中。这个规则同样适用于嵌套的 `if` `for` `while` ...
但是如果内部 `statement` 包含大括号或 `else`,则外部块应该用大括号括起来。
```cpp
/// Finish write.
for (auto & stream : streams)
stream.second->finalize();
```
**18.** 行的某尾不应该包含空格。
**19.** 源文件应该用 UTF-8 编码。
**20.** 非ASCII字符可用于字符串文字。
```cpp
<< ", " << (timer.elapsed() / chunks_stats.hits) << " μsec/hit.";
```
**21** 不要在一行中写入多个表达式。
**22.** 将函数内部的代码段分组,并将它们与不超过一行的空行分开。
**23.** 将 函数,类用一个或两个空行分开。
**24.** `const` 必须写在类型名称之前。
```cpp
//correct
const char * pos
const std::string & s
//incorrect
char const * pos
```
**25.** 声明指针或引用时,`*` 和 `` 符号两边应该都用空格分隔。
```cpp
//correct
const char * pos
//incorrect
const char* pos
const char *pos
```
**26.** 使用模板类型时,使用 `using` 关键字对它们进行别名(最简单的情况除外)。
换句话说,模板参数仅在 `using` 中指定,并且不在代码中重复。
`using`可以在本地声明,例如在函数内部。
```cpp
//correct
using FileStreams = std::map<std::string, std::shared_ptr<Stream>>;
FileStreams streams;
//incorrect
std::map<std::string, std::shared_ptr<Stream>> streams;
```
**27.** 不要在一个语句中声明不同类型的多个变量。
```cpp
//incorrect
int x, *y;
```
**28.** 不要使用C风格的类型转换。
```cpp
//incorrect
std::cerr << (int)c <<; std::endl;
//correct
std::cerr << static_cast<int>(c) << std::endl;
```
**29.** 在类和结构中,组成员和函数分别在每个可见范围内。
**30.** 对于小类和结构,没有必要将方法声明与实现分开。
对于任何类或结构中的小方法也是如此。
对于模板化类和结构,不要将方法声明与实现分开(因为否则它们必须在同一个转换单元中定义)
**31.** 您可以将换行规则定在140个字符而不是80个字符。
**32.** 如果不需要 postfix请始终使用前缀增量/减量运算符。
```cpp
for (Names::const_iterator it = column_names.begin(); it != column_names.end(); ++it)
```
## Comments
**1.** 请务必为所有非常重要的代码部分添加注释。
这是非常重要的。 编写注释可能会帮助您意识到代码不是必需的,或者设计错误。
```cpp
/** Part of piece of memory, that can be used.
* For example, if internal_buffer is 1MB, and there was only 10 bytes loaded to buffer from file for reading,
* then working_buffer will have size of only 10 bytes
* (working_buffer.end() will point to position right after those 10 bytes available for read).
*/
```
**2.** 注释可以尽可能详细。
**3.** 在他们描述的代码之前放置注释。 在极少数情况下,注释可以在代码之后,在同一行上。
```cpp
/** Parses and executes the query.
*/
void executeQuery(
ReadBuffer & istr, /// Where to read the query from (and data for INSERT, if applicable)
WriteBuffer & ostr, /// Where to write the result
Context & context, /// DB, tables, data types, engines, functions, aggregate functions...
BlockInputStreamPtr & query_plan, /// Here could be written the description on how query was executed
QueryProcessingStage::Enum stage = QueryProcessingStage::Complete /// Up to which stage process the SELECT query
)
```
**4.** 注释应该只用英文撰写。
**5.** 如果您正在编写库,请在主头文件中包含解释它的详细注释。
**6.** 请勿添加无效的注释。 特别是,不要留下像这样的空注释:
```cpp
/*
* Procedure Name:
* Original procedure name:
* Author:
* Date of creation:
* Dates of modification:
* Modification authors:
* Original file name:
* Purpose:
* Intent:
* Designation:
* Classes used:
* Constants:
* Local variables:
* Parameters:
* Date of creation:
* Purpose:
*/
```
这个示例来源于 [http://home.tamk.fi/~jaalto/course/coding-style/doc/unmaintainable-code/](http://home.tamk.fi/~jaalto/course/coding-style/doc/unmaintainable-code/)。
**7.** 不要在每个文件的开头写入垃圾注释(作者,创建日期...)。
**8.** 单行注释用三个斜杆: `///` ,多行注释以 `/**`开始。 这些注释会当做文档。
注意:您可以使用 Doxygen 从这些注释中生成文档。 但是通常不使用 Doxygen因为在 IDE 中导航代码更方便。
**9.** 多行注释的开头和结尾不得有空行(关闭多行注释的行除外)。
**10.** 要注释掉代码,请使用基本注释,而不是“记录”注释。
**11.** 在提交之前删除代码的无效注释部分。
**12.** 不要在注释或代码中使用亵渎语言。
**13.** 不要使用大写字母。 不要使用过多的标点符号。
```cpp
/// WHAT THE FAIL???
```
**14.** 不要使用注释来制作分隔符。
```cpp
///******************************************************
```
**15.** 不要在注释中开始讨论。
```cpp
/// Why did you do this stuff?
```
**16.** 没有必要在块的末尾写一条注释来描述它的含义。
```cpp
/// for
```
## Names
**1.** 在变量和类成员的名称中使用带下划线的小写字母。
```cpp
size_t max_block_size;
```
**2.** 对于函数(方法)的名称,请使用以小写字母开头的驼峰标识。
```cpp
std::string getName() const override { return "Memory"; }
```
**3.** 对于类结构的名称使用以大写字母开头的驼峰标识。接口名称用I前缀。
```cpp
class StorageMemory : public IStorage
```
**4.** `using` 的命名方式与类相同或者以__t`命名。
**5.** 模板类型参数的名称:在简单的情况下,使用`T`; `T``U`; `T1``T2`。
对于更复杂的情况,要么遵循类名规则,要么添加前缀`T`。
```cpp
template <typename TKey, typename TValue>
struct AggregatedStatElement
```
**6.** 模板常量参数的名称:遵循变量名称的规则,或者在简单的情况下使用 `N`
```cpp
template <bool without_www>
struct ExtractDomain
```
**7.** 对于抽象类型(接口),用 `I` 前缀。
```cpp
class IBlockInputStream
```
**8.** 如果在本地使用变量,则可以使用短名称。
在所有其他情况下,请使用能描述含义的名称。
```cpp
bool info_successfully_loaded = false;
```
**9.** `define` 和全局常量的名称使用带下划线的 `ALL_CAPS`
```cpp
#define MAX_SRC_TABLE_NAMES_TO_STORE 1000
```
**10.** 文件名应使用与其内容相同的样式。
如果文件包含单个类,则以与该类名称相同的方式命名该文件。
如果文件包含单个函数,则以与函数名称相同的方式命名文件。
**11.** 如果名称包含缩写,则:
- 对于变量名,缩写应使用小写字母 `mysql_connection`(不是 `mySQL_connection` )。
- 对于类和函数的名称,请将大写字母保留在缩写 `MySQLConnection`(不是 `MySqlConnection`
**12.** 仅用于初始化类成员的构造方法参数的命名方式应与类成员相同,但最后使用下划线。
```cpp
FileQueueProcessor(
const std::string & path_,
const std::string & prefix_,
std::shared_ptr<FileHandler> handler_)
: path(path_),
prefix(prefix_),
handler(handler_),
log(&Logger::get("FileQueueProcessor"))
{
}
```
如果构造函数体中未使用该参数,则可以省略下划线后缀。
**13.** 局部变量和类成员的名称没有区别(不需要前缀)。
```cpp
timer (not m_timer)
```
**14.** 对于 `enum` 中的常量请使用带大写字母的驼峰标识。ALL_CAPS 也可以接受。如果 `enum` 是非本地的,请使用 `enum class`
```cpp
enum class CompressionMethod
{
QuickLZ = 0,
LZ4 = 1,
};
```
**15.** 所有名字必须是英文。不允许音译俄语单词。
```
not Stroka
```
**16.** 缩写须是众所周知的(当您可以在维基百科或搜索引擎中轻松找到缩写的含义时)。
```
`AST`, `SQL`.
Not `NVDH` (some random letters)
```
如果缩短版本是常用的,则可以接受不完整的单词。
如果注释中旁边包含全名,您也可以使用缩写。
**17.** C++ 源码文件名称必须为 `.cpp` 拓展名。 头文件必须为 `.h` 拓展名。
## 如何编写代码
**1.** 内存管理。
手动内存释放 (`delete`) 只能在库代码中使用。
在库代码中, `delete` 运算符只能在析构函数中使用。
在应用程序代码中,内存必须由拥有它的对象释放。
示例:
- 最简单的方法是将对象放在堆栈上,或使其成为另一个类的成员。
- 对于大量小对象,请使用容器。
- 对于自动释放少量在堆中的对象,可以用 `shared_ptr/unique_ptr`
**2.** 资源管理。
使用 `RAII` 以及查看以上说明。
**3.** 错误处理。
在大多数情况下,您只需要抛出一个异常,而不需要捕获它(因为`RAII`)。
在离线数据处理应用程序中,通常可以接受不捕获异常。
在处理用户请求的服务器中,通常足以捕获连接处理程序顶层的异常。
在线程函数中,你应该在 `join` 之后捕获并保留所有异常以在主线程中重新抛出它们。
```cpp
/// If there weren't any calculations yet, calculate the first block synchronously
if (!started)
{
calculate();
started = true;
}
else /// If calculations are already in progress, wait for the result
pool.wait();
if (exception)
exception->rethrow();
```
不处理就不要隐藏异常。 永远不要盲目地把所有异常都记录到日志中。
```cpp
//Not correct
catch (...) {}
```
如果您需要忽略某些异常,请仅针对特定异常执行此操作并重新抛出其余异常。
```cpp
catch (const DB::Exception & e)
{
if (e.code() == ErrorCodes::UNKNOWN_AGGREGATE_FUNCTION)
return nullptr;
else
throw;
}
```
当使用具有返回码或 `errno` 的函数时,请始终检查结果并在出现错误时抛出异常。
```cpp
if (0 != close(fd))
throwFromErrno("Cannot close file " + file_name, ErrorCodes::CANNOT_CLOSE_FILE);
```
`不要使用断言`
**4.** 异常类型。
不需要在应用程序代码中使用复杂的异常层次结构。 系统管理员应该可以理解异常文本。
**5.** 从析构函数中抛出异常。
不建议这样做,但允许这样做。
按照以下选项:
- 创建一个函数( `done``finalize` ),它将提前完成所有可能导致异常的工作。 如果调用了该函数,则稍后在析构函数中应该没有异常。
- 过于复杂的任务(例如通过网络发送消息)可以放在单独的方法中,类用户必须在销毁之前调用它们。
- 如果析构函数中存在异常,则最好记录它而不是隐藏它(如果 logger 可用)。
- 在简单的应用程序中,依赖于`std::terminate`对于C++ 11中默认情况下为 `noexcept` 的情况)来处理异常是可以接受的。
**6.** 匿名代码块。
您可以在单个函数内创建单独的代码块,以使某些变量成为局部变量,以便在退出块时调用析构函数。
```cpp
Block block = data.in->read();
{
std::lock_guard<std::mutex> lock(mutex);
data.ready = true;
data.block = block;
}
ready_any.set();
```
**7.** 多线程。
在离线数据处理程序中:
- 尝试在单个CPU核心上获得最佳性能。 然后,您可以根据需要并行化代码。
在服务端应用中:
- 使用线程池来处理请求。 此时,我们还没有任何需要用户空间上下文切换的任务。
Fork不用于并行化。
**8.** 同步线程。
通常可以使不同的线程使用不同的存储单元(甚至更好:不同的缓存线),并且不使用任何线程同步(除了`joinAll`)。
如果需要同步,在大多数情况下,在 `lock_guard` 下使用互斥量就足够了。
在其他情况下,使用系统同步原语。不要使用忙等待。
仅在最简单的情况下才应使用原子操作。
除非是您的主要专业领域,否则不要尝试实施无锁数据结构。
**9.** 指针和引用。
大部分情况下,请用引用。
**10.** 常量。
使用 const 引用,指向常量的指针,`const_iterator`和 const 指针。
`const` 视为默认值,仅在必要时使用非 `const`
当按值传递变量时,使用 `const` 通常没有意义。
**11.** 无符号。
必要时使用`unsigned`。
**12.** 数值类型。
使用 `UInt8` `UInt16` `UInt32` `UInt64` `Int8` `Int16` `Int32` 以及 `Int64` `size_t` `ssize_t` 还有 `ptrdiff_t`
不要使用这些类型:`signed / unsigned long``long long``short``signed / unsigned char``char`。
**13.** 参数传递。
通过引用传递复杂类型 (包括 `std::string`)。
如果函数中传递堆中创建的对象,则使参数类型为 `shared_ptr` 或者 `unique_ptr`.
**14.** 返回值
大部分情况下使用 `return`。不要使用 `[return std::move(res)]{.strike}`
如果函数在堆上分配对象并返回它,请使用 `shared_ptr``unique_ptr`
在极少数情况下,您可能需要通过参数返回值。 在这种情况下,参数应该是引用传递的。
```cpp
using AggregateFunctionPtr = std::shared_ptr<IAggregateFunction>;
/** Allows creating an aggregate function by its name.
*/
class AggregateFunctionFactory
{
public:
AggregateFunctionFactory();
AggregateFunctionPtr get(const String & name, const DataTypes & argument_types) const;
```
**15.** 命名空间。
没有必要为应用程序代码使用单独的 `namespace`
小型库也不需要这个。
对于中大型库,须将所有代码放在 `namespace` 中。
在库的 `.h` 文件中,您可以使用 `namespace detail` 来隐藏应用程序代码不需要的实现细节。
`.cpp` 文件中,您可以使用 `static` 或匿名命名空间来隐藏符号。
同样 `namespace` 可用于 `enum` 以防止相应的名称落入外部 `namespace`(但最好使用`enum class`)。
**16.** 延迟初始化。
如果初始化需要参数,那么通常不应该编写默认构造函数。
如果稍后您需要延迟初始化,则可以添加将创建无效对象的默认构造函数。 或者,对于少量对象,您可以使用 `shared_ptr / unique_ptr`
```cpp
Loader(DB::Connection * connection_, const std::string & query, size_t max_block_size_);
/// For deferred initialization
Loader() {}
```
**17.** 虚函数。
如果该类不是用于多态使用,则不需要将函数设置为虚拟。这也适用于析构函数。
**18.** 编码。
在所有情况下使用 UTF-8 编码。使用 `std::string` and `char *`。不要使用 `std::wstring``wchar_t`
**19.** 日志。
请参阅代码中的示例。
在提交之前,删除所有无意义和调试日志记录,以及任何其他类型的调试输出。
应该避免循环记录日志,即使在 Trace 级别也是如此。
日志必须在任何日志记录级别都可读。
在大多数情况下,只应在应用程序代码中使用日志记录。
日志消息必须用英文写成。
对于系统管理员来说,日志最好是可以理解的。
不要在日志中使用亵渎语言。
在日志中使用UTF-8编码。 在极少数情况下您可以在日志中使用非ASCII字符。
**20.** 输入-输出。
不要使用 `iostreams` 在对应用程序性能至关重要的内部循环中(并且永远不要使用 `stringstream` )。
使用 `DB/IO` 库替代。
**21.** 日期和时间。
参考 `DateLUT` 库。
**22.** 引入头文件。
一直用 `#pragma once` 而不是其他宏。
**23.** using 语法
`using namespace` 不会被使用。 您可以使用特定的 `using`。 但是在类或函数中使它成为局部的。
**24.** 不要使用 `trailing return type` 为必要的功能。
```cpp
[auto f() -&gt; void;]{.strike}
```
**25.** 声明和初始化变量。
```cpp
//right way
std::string s = "Hello";
std::string s{"Hello"};
//wrong way
auto s = std::string{"Hello"};
```
**26.** 对于虚函数,在基类中编写 `virtual`,但在后代类中写 `override` 而不是`virtual`。
## 没有用到的 C++ 特性。
**1.** 不使用虚拟继承。
**2.** 不使用 C++03 中的异常标准。
## 平台
**1.** 我们为特定平台编写代码。
但在其他条件相同的情况下,首选跨平台或可移植代码。
**2.** 语言: C++17.
**3.** 编译器: `gcc`。 此时2017年12月代码使用7.2版编译。(它也可以使用`clang 4` 编译)
使用标准库 (`libstdc++` 或 `libc++`)。
**4.** 操作系统Linux Ubuntu不比 Precise 早。
**5.** 代码是为x86_64 CPU架构编写的。
CPU指令集是我们服务器中支持的最小集合。 目前它是SSE 4.2。
**6.** 使用 `-Wall -Wextra -Werror` 编译参数。
**7.** 对所有库使用静态链接,除了那些难以静态连接的库(参见 `ldd` 命令的输出).
**8.** 使用发布的设置来开发和调试代码。
## 工具
**1.** KDevelop 是一个好的 IDE.
**2.** 调试可以使用 `gdb` `valgrind` (`memcheck`) `strace` `-fsanitize=...``tcmalloc_minimal_debug`.
**3.** 对于性能分析,使用 `Linux Perf` `valgrind` (`callgrind`),或者 `strace -cf`
**4.** 源代码用 Git 作版本控制。
**5.** 使用 `CMake` 构建。
**6.** 程序的发布使用 `deb` 安装包。
**7.** 提交到 master 分支的代码不能破坏编译。
虽然只有选定的修订被认为是可行的。
**8.** 尽可能经常地进行提交,即使代码只是部分准备好了。
目的明确的功能,使用分支。
如果 `master` 分支中的代码尚不可构建,请在 `push` 之前将其从构建中排除。您需要在几天内完成或删除它。
**9.** 对于不重要的更改,请使用分支并在服务器上发布它们。
**10.** 未使用的代码将从 repo 中删除。
## 库
**1.** 使用C ++ 14标准库允许实验性功能以及 `boost``Poco` 框架。
**2.** 如有必要,您可以使用 OS 包中提供的任何已知库。
如果有一个好的解决方案已经可用,那就使用它,即使这意味着你必须安装另一个库。
(但要准备从代码中删除不好的库)
**3.** 如果软件包没有您需要的软件包或者有过时的版本或错误的编译类型,则可以安装不在软件包中的库。
**4.** 如果库很小并且没有自己的复杂构建系统,请将源文件放在 `contrib` 文件夹中。
**5.** 始终优先考虑已经使用的库。
## 一般建议
**1.** 尽可能精简代码。
**2.** 尝试用最简单的方式实现。
**3.** 在你知道代码是如何工作以及内部循环如何运作之前,不要编写代码。
**4.** 在最简单的情况下,使用 `using` 而不是类或结构。
**5.** 如果可能,不要编写复制构造函数,赋值运算符,析构函数(虚拟函数除外,如果类包含至少一个虚函数),移动构造函数或移动赋值运算符。 换句话说,编译器生成的函数必须正常工作。 您可以使用 `default`
**6.** 鼓励简化代码。 尽可能减小代码的大小。
## 其他建议
**1.**`stddef.h` 明确指定 `std ::` 的类型。
不推荐。 换句话说,我们建议写 `size_t` 而不是 `std::size_t`,因为它更短。
也接受添加 `std::`
**2.** 为标准C库中的函数明确指定 `std::`
不推荐。换句话说,写 `memcpy` 而不是`std::memcpy`。
原因是有类似的非标准功能,例如 `memmem`。我们偶尔会使用这些功能。`namespace std`中不存在这些函数。
如果你到处都写 `std::memcpy` 而不是 `memcpy`,那么没有 `std::``memmem` 会显得很奇怪。
不过,如果您愿意,仍然可以使用 `std::`
**3.** 当标准C++库中提供相同的函数时使用C中的函数。
如果它更高效,这是可以接受的。
例如,使用`memcpy`而不是`std::copy`来复制大块内存。
**4.** 函数的多行参数。
允许以下任何包装样式:
```cpp
function(
T1 x1,
T2 x2)
```
```cpp
function(
size_t left, size_t right,
const & RangesInDataParts ranges,
size_t limit)
```
```cpp
function(size_t left, size_t right,
const & RangesInDataParts ranges,
size_t limit)
```
```cpp
function(size_t left, size_t right,
const & RangesInDataParts ranges,
size_t limit)
```
```cpp
function(
size_t left,
size_t right,
const & RangesInDataParts ranges,
size_t limit)
```
[来源文章](https://clickhouse.tech/docs/en/development/style/) <!--hide-->