ClickHouse/docs/zh/development/style.md

827 lines
24 KiB
Markdown
Raw Normal View History

# 如何编写 C++ 代码 {#ru-he-bian-xie-c-dai-ma}
## 一般建议 {#yi-ban-jian-yi}
**1.** 以下是建议,而不是要求。
**2.** 如果你在修改代码,遵守已有的风格是有意义的。
**3.** 代码的风格需保持一致。一致的风格有利于阅读代码,并且方便检索代码。
**4.** 许多规则没有逻辑原因; 它们是由既定的做法决定的。
## 格式化 {#ge-shi-hua}
**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();
```
2021-01-28 08:59:42 +00:00
**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/。
**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
2019-05-17 00:12:19 +00:00
std::string getName() const override { return "Memory"; }
```
**3.** 对于类结构的名称使用以大写字母开头的驼峰标识。接口名称用I前缀。
``` cpp
2019-05-17 00:12:19 +00:00
class StorageMemory : public IStorage
```
2020-10-13 17:23:29 +00:00
**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.** 文件名应使用与其内容相同的样式。
如果文件包含单个类则以与该类名称相同的方式命名该文件CamelCase
如果文件包含单个函数则以与函数名称相同的方式命名文件camelCase
**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)
```
2020-10-13 17:23:29 +00:00
**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` 拓展名。
## 如何编写代码 {#ru-he-bian-xie-dai-ma}
**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.** 返回值
2020-12-12 01:20:43 +00:00
大部分情况下使用 `return`。不要使用 `return std::move(res)`
如果函数在堆上分配对象并返回它,请使用 `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``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
2020-12-12 01:20:43 +00:00
auto f() -> void
```
**25.** 声明和初始化变量。
``` cpp
//right way
std::string s = "Hello";
std::string s{"Hello"};
//wrong way
auto s = std::string{"Hello"};
```
**26.** 对于虚函数,在基类中编写 `virtual`,但在后代类中写 `override` 而不是`virtual`。
## 没有用到的 C++ 特性。 {#mei-you-yong-dao-de-c-te-xing}
**1.** 不使用虚拟继承。
**2.** 不使用 C++03 中的异常标准。
## 平台 {#ping-tai}
**1.** 我们为特定平台编写代码。
但在其他条件相同的情况下,首选跨平台或可移植代码。
**2.** 语言: C++20.
2021-04-22 01:20:03 +00:00
**3.** 编译器: `clang`。 此时2021年03月代码使用11版编译。它也可以使用`gcc` 编译 but it is not suitable for production
使用标准库 (`libc++`)。
**4.** 操作系统Linux Ubuntu不比 Precise 早。
2020-10-13 17:23:29 +00:00
**5.** 代码是为x86_64 CPU架构编写的。
CPU指令集是我们服务器中支持的最小集合。 目前它是SSE 4.2。
**6.** 使用 `-Wall -Wextra -Werror` 编译参数。
**7.** 对所有库使用静态链接,除了那些难以静态连接的库(参见 `ldd` 命令的输出).
**8.** 使用发布的设置来开发和调试代码。
## 工具 {#gong-ju}
**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 中删除。
## 库 {#ku}
2021-07-06 00:31:39 +00:00
**1.** The C++20 standard library is used (experimental extensions are allowed), as well as `boost` and `Poco` frameworks.
2021-07-06 00:31:39 +00:00
**2.** It is not allowed to use libraries from OS packages. It is also not allowed to use pre-installed libraries. All libraries should be placed in form of source code in `contrib` directory and built with ClickHouse.
2021-07-06 00:31:39 +00:00
**3.** Preference is always given to libraries that are already in use.
## 一般建议 {#yi-ban-jian-yi-1}
**1.** 尽可能精简代码。
**2.** 尝试用最简单的方式实现。
**3.** 在你知道代码是如何工作以及内部循环如何运作之前,不要编写代码。
**4.** 在最简单的情况下,使用 `using` 而不是类或结构。
**5.** 如果可能,不要编写复制构造函数,赋值运算符,析构函数(虚拟函数除外,如果类包含至少一个虚函数),移动构造函数或移动赋值运算符。 换句话说,编译器生成的函数必须正常工作。 您可以使用 `default`
**6.** 鼓励简化代码。 尽可能减小代码的大小。
## 其他建议 {#qi-ta-jian-yi}
**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)
```
2020-01-30 10:34:55 +00:00
[来源文章](https://clickhouse.tech/docs/en/development/style/) <!--hide-->