赞
踩
已剪辑自: https://www.bookstack.cn/read/openharmony-1.0-zh-cn/contribute-OpenHarmony-cpp-coding-style-guide.md
C++语言编程规范
2 命名
通用命名
文件命名
- 建议2.2.1 C++文件以.cpp结尾,头文件以.h结尾
- 建议2.2.2 C++文件名和类名保持一致
函数命名
类型命名
- 建议2.4.1 避免滥用 typedef或者#define 对基本类型起别名
变量命名
- 规则2.5.1 全局变量应增加 ‘g_’ 前缀,静态变量命名不需要加特殊前缀
- 规则2.5.2 类的成员变量命名以小驼峰加后下划线组成
宏、常量、枚举命名
3 格式
行宽
- 建议3.1.1 行宽不超过 120 个字符
缩进
- 规则3.2.1 使用空格进行缩进,每次缩进4个空格
大括号
- 规则3.3.1 使用 K&R 缩进风格
函数声明和定义
- 规则3.4.1 函数声明和定义的返回类型和函数名在同一行;函数参数列表超出行宽时要换行并合理对齐
函数调用
- 规则3.5.1 函数调用入参列表应放在一行,超出行宽换行时,保持参数进行合理对齐
if语句
- 规则3.6.1 if语句必须要使用大括号
- 规则3.6.2 禁止 if/else/else if 写在同一行
循环语句
- 规则3.7.1 循环语句必须使用大括号
switch语句
- 规则3.8.1 switch 语句的 case/default 要缩进一层
表达式
- 建议3.9.1 表达式换行要保持换行的一致性,运算符放行末
变量赋值
- 规则3.10.1 多个变量定义和赋值语句不允许写在一行
初始化
- 规则3.11.1 初始化换行时要有缩进,并进行合理对齐
指针与引用
- 建议3.12.1 指针类型”`*`“跟随变量名或者类型,不要两边都留有或者都没有空格
- 建议3.12.2 引用类型”`&`“跟随变量名或者类型,不要两边都留有或者都没有空格
编译预处理
- 规则3.13.1 编译预处理的”#”统一放在行首,嵌套编译预处理语句时,”#”可以进行缩进
空格和空行
- 规则3.14.1 水平空格应该突出关键字和重要信息,避免不必要的留白
- 建议3.14.1 合理安排空行,保持代码紧凑
类
- 规则3.15.1 类访问控制块的声明依次序是 public:, protected:, private:,缩进和 class 关键字对齐
- 规则3.15.2 构造函数初始化列表放在同一行或按四格缩进并排多行
4 注释
注释风格
文件头注释
- 规则3.1 文件头注释必须包含版权许可
函数头注释
- 规则4.3.1 禁止空有格式的函数头注释
代码注释
- 规则4.4.1 代码注释放于对应代码的上方或右边
- 规则4.4.2 注释符与注释内容间要有1空格;右置注释与前面代码至少1空格
- 规则4.4.3 不用的代码段直接删除,不要注释掉
- 建议4.4.1 正式交付给客户的代码不能包含 TODO/TBD/FIXME 注释
5 头文件
头文件职责
- 建议5.1.1 每一个.cpp文件应有一个对应的.h文件,用于声明需要对外公开的类与接口
头文件依赖
- 规则5.2.1 禁止头文件循环依赖
- 规则5.2.2 头文件必须编写`#define`保护,防止重复包含
- 规则5.2.3 禁止通过声明的方式引用外部函数接口、变量
- 规则5.2.4 禁止在extern “C”中包含头文件
- 建议5.2.1尽量避免使用前置声明,而是通过`#include`来包含头文件
6 作用域
命名空间
- 建议6.1.1 对于cpp文件中不需要导出的变量,常量或者函数,请使用匿名namespace封装或者用static修饰
- 规则6.1.1 不要在头文件中或者#include之前使用using导入命名空间
全局函数和静态成员函数
- 建议6.2.1 优先使用命名空间来管理全局函数,如果和某个class有直接关系的,可以使用静态成员函数
全局常量和静态成员常量
- 建议6.3.1 优先使用命名空间来管理全局常量,如果和某个class有直接关系的,可以使用静态成员常量
全局变量
- 建议6.4.1 尽量避免使用全局变量,考虑使用单例模式
7 类
构造,拷贝构造,赋值和析构函数
- 规则7.1.1 类的成员变量必须显式初始化
- 建议7.1.1 成员变量优先使用声明时初始化(C++11)和构造函数初始化列表初始化
- 规则7.1.2 为避免隐式转换,将单参数构造函数声明为explicit
- 规则7.1.3 如果不需要拷贝构造函数、赋值操作符 / 移动构造函数、赋值操作符,请明确禁止
- 规则7.1.4 拷贝构造和拷贝赋值操作符应该是成对出现或者禁止
- 规则7.1.5 移动构造和移动赋值操作符应该是成对出现或者禁止
- 规则7.1.6 禁止在构造函数和析构函数中调用虚函数
继承
- 规则7.2.1 基类的析构函数应该声明为virtual
- 规则7.2.2 禁止虚函数使用缺省参数值
- 规则7.2.3 禁止重新定义继承而来的非虚函数
多重继承
- 建议7.3.1 使用多重继承来实现接口分离与多角色组合
重载
8 函数
函数设计
- 规则8.1.1 避免函数过长,函数不超过50行(非空非注释)
内联函数
- 建议8.2.1 内联函数不超过10行(非空非注释)
函数参数
- 建议8.3.1 函数参数使用引用取代指针
- 建议8.3.2 使用强类型参数,避免使用void*
- 建议8.3.3 函数的参数个数不超过5个
9 C++其他特性
常量与初始化
- 规则9.1.1 不允许使用宏来表示常量
- 建议9.1.1 一组相关的整型常量应定义为枚举
- 规则9.1.2 不允许使用魔鬼数字
- 规则9.1.3 常量应该保证单一职责
- 规则9.1.4 禁止用memcpy_s、memset_s初始化非POD对象
- 建议9.1.2 变量使用时才声明并初始化
表达式
- 规则9.2.1 含有变量自增或自减运算的表达式中禁止再次引用该变量
- 规则9.2.2 switch语句要有default分支
- 建议9.2.1 表达式的比较,应当遵循左侧倾向于变化、右侧倾向于不变的原则
- 建议9.2.2 使用括号明确操作符的优先级
类型转换
- 规则9.3.1 如果确定要使用类型转换,请使用由C++提供的类型转换,而不是C风格的类型转换
- 建议9.3.1 避免使用`dynamic_cast`
- 建议9.3.2 避免使用`reinterpret_cast`
- 建议9.3.3 避免使用`const_cast`
资源分配和释放
- 规则9.4.1 单个对象释放使用delete,数组对象释放使用delete []
- 建议9.4.1 使用 RAII 特性来帮助追踪动态分配
标准库
- 规则9.5.1 不要保存std::string的c_str()返回的指针
- 建议9.5.1 使用std::string代替char*
- 规则9.5.2 禁止使用auto_ptr
- 建议9.5.2 使用新的标准头文件
const的用法
- 规则9.6.1 对于指针和引用类型的形参,如果是不需要修改的,请使用const
- 规则9.6.2 对于不会修改成员变量的成员函数请使用const修饰
- 建议9.6.1 初始化后不会再修改的成员变量定义为const
异常
- 建议9.7.1 C++11中,如果函数不会抛出异常,声明为`noexcept`
模板
宏
10 现代C++特性
代码简洁性和安全性提升
- 建议10.1.1 合理使用`auto`
- 规则10.1.1 在重写虚函数时请使用`override`或`final`关键字
- 规则10.1.2 使用`delete`关键字删除函数
- 规则10.1.3 使用`nullptr`,而不是`NULL`或`0`
- 规则10.1.4 使用`using`而非`typedef`
- 规则10.1.5 禁止使用std::move操作const对象
智能指针
- 规则10.2.1 优先使用智能指针而不是原始指针管理资源
- 规则10.2.2 优先使用`unique_ptr`而不是`shared_ptr`
- 规则10.2.3 使用`std::make_unique`而不是`new`创建`unique_ptr`
- 规则10.2.4 使用`std::make_shared`而不是`new`创建`shared_ptr`
Lambda
- 建议10.3.1 当函数不能工作时选择使用`lambda`(捕获局部变量,或编写局部函数)
- 规则10.3.1 非局部范围使用`lambdas`,避免使用按引用捕获
- 建议10.3.2 如果捕获`this`,则显式捕获所有变量
- 建议10.3.3 避免使用默认捕获模式
接口 - 建议10.4.1 不涉及所有权的场景,使用 ``` T* ``` 或 ``` T& ``` 作为参数,而不是智能指针
规则并不是完美的,通过禁止在特定情况下有用的特性,可能会对代码实现造成影响。但是我们制定规则的目的“为了大多数程序员可以得到更多的好处”, 如果在团队运作中认为某个规则无法遵循,希望可以共同改进该规则。 参考该规范之前,希望您具有相应的C++语言基础能力,而不是通过该文档来学习C++语言。
代码需要在保证功能正确的前提下,满足可读、可维护、安全、可靠、可测试、高效、可移植的特征要求。
规则:编程时必须遵守的约定(must)
建议:编程时应该遵守的约定(should)
本规范适用通用C++标准, 如果没有特定的标准版本,适用所有的版本(C++03/11/14/17)。
无论是’规则’还是’建议’,都必须理解该条目这么规定的原因,并努力遵守。 但是,有些规则和建议可能会有例外。
在不违背总体原则,经过充分考虑,有充足的理由的前提下,可以适当违背规范中约定。 例外破坏了代码的一致性,请尽量避免。’规则’的例外应该是极少的。
下列情况,应风格一致性原则优先:
修改外部开源代码、第三方代码时,应该遵守开源代码、第三方代码已有规范,保持风格统一。
驼峰风格(CamelCase) 大小写字母混用,单词连在一起,不同单词间通过单词首字母大写来分开。 按连接后的首字母是否大写,又分: 大驼峰(UperCamelCase)和小驼峰(lowerCamelCase)
类型 命名风格
类类型,结构体类型,枚举类型,联合体类型等类型定义, 作用域名称 | 大驼峰 |
函数(包括全局函数,作用域函数,成员函数) | 大驼峰 |
全局变量(包括全局和命名空间域下的变量,类静态变量),局部变量,函数参数,类、结构体和联合体中的成员变量 | 小驼峰 |
宏,常量(const),枚举值,goto 标签 | 全大写,下划线分割 |
注意: 上表中常量是指全局作用域、namespace域、类的静态成员域下,以 const或constexpr 修饰的基本数据类型、枚举、字符串类型的变量,不包括数组和其他类型变量。 上表中变量是指除常量定义以外的其他变量,均使用小驼峰风格。
我们推荐使用.h作为头文件的后缀,这样头文件可以直接兼容C和C++。 我们推荐使用.cpp作为实现文件的后缀,这样可以直接区分C++代码,而不是C代码。
目前业界还有一些其他的后缀的表示方法:
如果当前项目组使用了某种特定的后缀,那么可以继续使用,但是请保持风格统一。 但是对于本文档,我们默认使用.h和.cpp作为后缀。
C++的头文件和cpp文件名和类名保持一致,使用下划线小写风格。
如果有一个类叫DatabaseConnection,那么对应的文件名:
结构体,命名空间,枚举等定义的文件名类似。
函数命名统一使用大驼峰风格,一般采用动词或者动宾结构。
class List {public: void AddElement(const Element& element); Element GetElement(const unsigned int index) const; bool IsEmpty() const;};namespace Utils { void DeleteUser();}
类型命名采用大驼峰命名风格。 所有类型命名——类、结构体、联合体、类型定义(typedef)、枚举——使用相同约定,例如:
// classes, structs and unionsclass UrlTable { ...class UrlTableTester { ...struct UrlTableProperties { ...union Packet { ...// typedefstypedef std::map<std::string, UrlTableProperties*> PropertiesMap;// enumsenum UrlTableErrors { ...
对于命名空间的命名,建议使用大驼峰:
// namespacenamespace OsUtils {namespace FileUtils {}}
除有明确的必要性,否则不要用 typedef/#define 对基本数据类型进行重定义。 优先使用<cstdint>
头文件中的基本类型:
有符号类型 无符号类型 描述
int8_t | uint8_t | 宽度恰为8的有/无符号整数类型 |
int16_t | uint16_t | 宽度恰为16的有/无符号整数类型 |
int32_t | uint32_t | 宽度恰为32的有/无符号整数类型 |
int64_t | uint64_t | 宽度恰为64的有/无符号整数类型 |
intptr_t | uintptr_t | 足以保存指针的有/无符号整数类型 |
通用变量命名采用小驼峰,包括全局变量,函数形参,局部变量,成员变量。
std::string tableName; // Good: 推荐此风格std::string tablename; // Bad: 禁止此风格std::string path; // Good: 只有一个单词时,小驼峰为全小写
全局变量是应当尽量少使用的,使用时应特别注意,所以加上前缀用于视觉上的突出,促使开发人员对这些变量的使用更加小心。
int g_activeConnectCount;void Func(){ static int packetCount = 0; ...}
class Foo {private: std::string fileName_; // 添加_后缀,类似于K&R命名风格};
对于struct/union的成员变量,仍采用小驼峰不加后缀的命名方式,与局部变量命名风格一致。
宏、枚举值采用全大写,下划线连接的格式。 全局作用域内,有名和匿名namespace内的 const 常量,类的静态成员常量,全大写,下划线连接;函数局部 const 常量和类的普通const成员变量,使用小驼峰命名风格。
#define MAX(a, b) (((a) < (b)) ? (b) : (a)) // 仅对宏命名举例,并不推荐用宏实现此类功能enum TintColor { // 注意,枚举类型名用大驼峰,其下面的取值是全大写,下划线相连 RED, DARK_RED, GREEN, LIGHT_GREEN};int Func(...){ const unsigned int bufferSize = 100; // 函数局部常量 char *p = new char[bufferSize]; ...}namespace Utils { const unsigned int DEFAULT_FILE_SIZE_KB = 200; // 全局常量}
建议每行字符数不要超过 120 个。如果超过120个字符,请选择合理的方式进行换行。
例外:
如果一行注释包含了超过120 个字符的命令或URL,则可以保持一行,以方便复制、粘贴和通过grep查找;
包含长路径的 #include 语句可以超出120 个字符,但是也需要尽量避免;
编译预处理中的error信息可以超出一行。 预处理的 error 信息在一行便于阅读和理解,即使超过 120 个字符。
#ifndef XXX_YYY_ZZZ#error Header aaaa/bbbb/cccc/abc.h must only be included after xxxx/yyyy/zzzz/xyz.h, because xxxxxxxxxxxxxxxxxxxxxxxxxxxxx#endif
只允许使用空格(space)进行缩进,每次缩进为 4 个空格。不允许使用Tab符进行缩进。 当前几乎所有的集成开发环境(IDE)都支持配置将Tab符自动扩展为4空格输入;请配置你的IDE支持使用空格进行缩进。
K&R风格 换行时,函数(不包括lambda表达式)左大括号另起一行放行首,并独占一行;其他左大括号跟随语句放行末。 右大括号独占一行,除非后面跟着同一语句的剩余部分,如 do 语句中的 while,或者 if 语句的 else/else if,或者逗号、分号。
如:
struct MyType { // 跟随语句放行末,前置1空格 ...};int Foo(int a){ // 函数左大括号独占一行,放行首 if (...) { ... } else { ... }}
推荐这种风格的理由:
对于空函数体,可以将大括号放在同一行:
class MyClass {public: MyClass() : value_(0) {}private: int value_;};
在声明和定义函数的时候,函数的返回值类型应该和函数名在同一行;如果行宽度允许,函数参数也应该放在一行;否则,函数参数应该换行,并进行合理对齐。 参数列表的左圆括号总是和函数名在同一行,不要单独一行;右圆括号总是跟随最后一个参数。
换行举例:
ReturnType FunctionName(ArgType paramName1, ArgType paramName2) // Good:全在同一行{ ...}ReturnType VeryVeryVeryLongFunctionName(ArgType paramName1, // 行宽不满足所有参数,进行换行 ArgType paramName2, // Good:和上一行参数对齐 ArgType paramName3){ ...}ReturnType LongFunctionName(ArgType paramName1, ArgType paramName2, // 行宽限制,进行换行 ArgType paramName3, ArgType paramName4, ArgType paramName5) // Good: 换行后 4 空格缩进{ ...}ReturnType ReallyReallyReallyReallyLongFunctionName( // 行宽不满足第1个参数,直接换行 ArgType paramName1, ArgType paramName2, ArgType paramName3) // Good: 换行后 4 空格缩进{ ...}
函数调用时,函数参数列表放在一行。参数列表如果超过行宽,需要换行并进行合理的参数对齐。 左圆括号总是跟函数名,右圆括号总是跟最后一个参数。
换行举例:
ReturnType result = FunctionName(paramName1, paramName2); // Good:函数参数放在一行ReturnType result = FunctionName(paramName1, paramName2, // Good:保持与上方参数对齐 paramName3);ReturnType result = FunctionName(paramName1, paramName2, paramName3, paramName4, paramName5); // Good:参数换行,4 空格缩进ReturnType result = VeryVeryVeryLongFunctionName( // 行宽不满足第1个参数,直接换行 paramName1, paramName2, paramName3); // 换行后,4 空格缩进
如果函数调用的参数存在内在关联性,按照可理解性优先于格式排版要求,对参数进行合理分组换行。
// Good:每行的参数代表一组相关性较强的数据结构,放在一行便于理解int result = DealWithStructureLikeParams(left.x, left.y, // 表示一组相关参数 right.x, right.y); // 表示另外一组相关参数
我们要求if语句都需要使用大括号,即便只有一条语句。
理由:
if (objectIsNotExist) { // Good:单行条件语句也加大括号 return CreateNewObject();}
条件语句中,若有多个分支,应该写在不同行。
如下是正确的写法:
if (someConditions) { DoSomething(); ...} else { // Good: else 与 if 在不同行 ...}
下面是不符合规范的案例:
if (someConditions) { ... } else { ... } // Bad: else 与 if 在同一行
和条件表达式类似,我们要求for/while循环语句必须加上大括号,即便循环体是空的,或循环语句只有一条。
for (int i = 0; i < someRange; i++) { // Good: 使用了大括号 DoSomething();}
while (condition) { } // Good:循环体是空,使用大括号
while (condition) { continue; // Good:continue 表示空逻辑,使用大括号}
坏的例子:
for (int i = 0; i < someRange; i++) DoSomething(); // Bad: 应该加上括号
while (condition); // Bad:使用分号容易让人误解是while语句中的一部分
switch 语句的缩进风格如下:
switch (var) { case 0: // Good: 缩进 DoSomething1(); // Good: 缩进 break; case 1: { // Good: 带大括号格式 DoSomething2(); break; } default: break;}
switch (var) {case 0: // Bad: case 未缩进 DoSomething(); break;default: // Bad: default 未缩进 break;}
较长的表达式,不满足行宽要求的时候,需要在适当的地方换行。一般在较低优先级运算符或连接符后面截断,运算符或连接符放在行末。 运算符、连接符放在行末,表示“未结束,后续还有”。 例:
// 假设下面第一行已经不满足行宽要求
if (currentValue > threshold && // Good:换行后,逻辑操作符放在行尾 someConditionsion) { DoSomething(); ...}int result = reallyReallyLongVariableName1 + // Good reallyReallyLongVariableName2;
表达式换行后,注意保持合理对齐,或者4空格缩进。参考下面例子
int sum = longVaribleName1 + longVaribleName2 + longVaribleName3 + longVaribleName4 + longVaribleName5 + longVaribleName6; // Good: 4空格缩进int sum = longVaribleName1 + longVaribleName2 + longVaribleName3 + longVaribleName4 + longVaribleName5 + longVaribleName6; // Good: 保持对齐
每行只有一个变量初始化的语句,更容易阅读和理解。
int maxCount = 10;bool isCompleted = false;
下面是不符合规范的示例:
int maxCount = 10; bool isCompleted = false; // Bad:多个变量初始化需要分开放在多行,每行一个变量初始化int x, y = 0; // Bad:多个变量定义需要分行,每行一个int pointX;int pointY;...pointX = 1; pointY = 2; // Bad:多个变量赋值语句放同一行
例外:for 循环头、if 初始化语句(C++17)、结构化绑定语句(C++17)中可以声明和初始化多个变量。这些语句中的多个变量声明有较强关联,如果强行分成多行会带来作用域不一致,声明和初始化割裂等问题。
初始化包括结构体、联合体、及数组的初始化
结构体或数组初始化时,如果换行应保持4空格缩进。 从可读性角度出发,选择换行点和对齐位置。
const int rank[] = { 16, 16, 16, 16, 32, 32, 32, 32, 64, 64, 64, 64, 32, 32, 32, 32};
*
“跟随变量名或者类型,不要两边都留有或者都没有空格指针命名: *
靠左靠右都可以,但是不要两边都有或者都没有空格。
int* p = NULL; // Goodint *p = NULL; // Goodint*p = NULL; // Badint * p = NULL; // Bad
例外:当变量被 const 修饰时,”*
“ 无法跟随变量,此时也不要跟随类型。
const char * const VERSION = "V100";
&
“跟随变量名或者类型,不要两边都留有或者都没有空格引用命名:&
靠左靠右都可以,但是不要两边都有或者都没有空格。
int i = 8;int& p = i; // Goodint &p = i; // Goodint*& rp = pi; // Good,指针的引用,*& 一起跟随类型int *&rp = pi; // Good,指针的引用,*& 一起跟随变量名int* &rp = pi; // Good,指针的引用,* 跟随类型,& 跟随变量名int & p = i; // Badint&p = i; // Bad
编译预处理的”#”统一放在行首,即使编译预处理的代码是嵌入在函数体中的,”#”也应该放在行首。
#if defined(__x86_64__) && defined(__GCC_HAVE_SYNC_COMPARE_AND_SWAP_16) // Good:"#"放在行首#define ATOMIC_X86_HAS_CMPXCHG16B 1 // Good:"#"放在行首#else#define ATOMIC_X86_HAS_CMPXCHG16B 0#endif int FunctionName() { if (someThingError) { ...#ifdef HAS_SYSLOG // Good:即便在函数内部,"#"也放在行首 WriteToSysLog();#else WriteToFileLog();#endif }}
内嵌的预处理语句”#”可以按照缩进要求进行缩进对齐,区分层次。
#if defined(__x86_64__) && defined(__GCC_HAVE_SYNC_COMPARE_AND_SWAP_16) #define ATOMIC_X86_HAS_CMPXCHG16B 1 // Good:区分层次,便于阅读#else #define ATOMIC_X86_HAS_CMPXCHG16B 0#endif
水平空格应该突出关键字和重要信息,每行代码尾部不要加空格。总体规则如下:
常规情况:
void Foo(int b) { // Good:大括号前应该留空格int i = 0; // Good:变量初始化时,=前后应该有空格,分号前面不要留空格int buf[BUF_SIZE] = {0}; // Good:大括号内两侧都无空格
函数定义和函数调用:
int result = Foo(arg1,arg2); ^ // Bad: 逗号后面需要增加空格int result = Foo( arg1, arg2 ); ^ ^ // Bad: 函数参数列表的左括号后面不应该有空格,右括号前面不应该有空格
指针和取地址
x = *p; // Good:*操作符和指针p之间不加空格p = &x; // Good:&操作符和变量x之间不加空格x = r.y; // Good:通过.访问成员变量时不加空格x = r->y; // Good:通过->访问成员变量时不加空格
操作符:
x = 0; // Good:赋值操作的=前后都要加空格x = -5; // Good:负数的符号和数值之前不要加空格++x; // Good:前置和后置的++/--和变量之间不要加空格x--;if (x && !y) // Good:布尔操作符前后要加上空格,!操作和变量之间不要空格v = w * x + y / z; // Good:二元操作符前后要加空格v = w * (x + z); // Good:括号内的表达式前后不需要加空格int a = (x < y) ? x : y; // Good: 三目运算符, ?和:前后需要添加空格
循环和条件语句:
if (condition) { // Good:if关键字和括号之间加空格,括号内条件语句前后不加空格 ...} else { // Good:else关键字和大括号之间加空格 ...}while (condition) {} // Good:while关键字和括号之间加空格,括号内条件语句前后不加空格for (int i = 0; i < someRange; ++i) { // Good:for关键字和括号之间加空格,分号之后加空格 ...}switch (condition) { // Good: switch 关键字后面有1空格 case 0: // Good:case语句条件和冒号之间不加空格 ... break; ... default: ... break;}
模板和转换
// 尖括号(< and >) 不与空格紧邻, < 前没有空格, > 和 ( 之间也没有.vector<string> x;y = static_cast<char*>(x);// 在类型与指针操作符之间留空格也可以, 但要保持一致.vector<char *> x;
域操作符
std::cout; // Good: 命名空间访问,不要留空格int MyClass::GetValue() const {} // Good: 对于成员函数定义,不要留空格
冒号
// 添加空格的场景// Good: 类的派生需要留有空格class Sub : public Base {};// 构造函数初始化列表需要留有空格MyClass::MyClass(int var) : someVar_(var) { DoSomething();}// 位域表示也留有空格struct XX { char a : 4; char b : 5; char c : 4;};
// 不添加空格的场景// Good: 对于public:, private:这种类访问权限的冒号不用添加空格class MyClass {public: MyClass(int var);private: int someVar_;};// 对于switch-case的case和default后面的冒号不用添加空格switch (value){ case 1: DoSomething(); break; default: break;}
注意:当前的集成开发环境(IDE)可以设置删除行尾的空格,请正确配置。
减少不必要的空行,可以显示更多的代码,方便代码阅读。下面有一些建议遵守的规则:
int Foo(){ ...}int Bar() // Bad:最多使用连续2个空行。{ ...}if (...) { // Bad:大括号内的代码块行首不要加入空行 ... // Bad:大括号内的代码块行尾不要加入空行}int Foo(...){ // Bad:函数体内行首不要加空行 ...}
class MyClass : public BaseClass {public: // 注意没有缩进 MyClass(); // 标准的4空格缩进 explicit MyClass(int var); ~MyClass() {} void SomeFunction(); void SomeFunctionThatDoesNothing() { } void SetVar(int var) { someVar_ = var; } int GetVar() const { return someVar_; }private: bool SomeInternalFunction(); int someVar_; int someOtherVar_;};
在各个部分中,建议将类似的声明放在一起, 并且建议以如下的顺序: 类型 (包括 typedef, using 和嵌套的结构体与类), 常量, 工厂函数, 构造函数, 赋值运算符, 析构函数, 其它成员函数, 数据成员。
// 如果所有变量能放在同一行:MyClass::MyClass(int var) : someVar_(var) { DoSomething();}// 如果不能放在同一行,// 必须置于冒号后, 并缩进4个空格MyClass::MyClass(int var) : someVar_(var), someOtherVar_(var + 1) // Good: 逗号后面留有空格{ DoSomething();}// 如果初始化列表需要置于多行, 需要逐行对齐MyClass::MyClass(int var) : someVar_(var), // 缩进4个空格 someOtherVar_(var + 1) { DoSomething();}
一般的,尽量通过清晰的架构逻辑,好的符号命名来提高代码可读性;需要的时候,才辅以注释说明。
注释是为了帮助阅读者快速读懂代码,所以要从读者的角度出发,按需注释。
注释内容要简洁、明了、无二义性,信息全面且不冗余。
注释跟代码一样重要。
写注释时要换位思考,用注释去表达此时读者真正需要的信息。在代码的功能、意图层次上进行注释,即注释解释代码难以表达的意图,不要重复代码信息。
修改代码时,也要保证其相关注释的一致性。只改代码,不改注释是一种不文明行为,破坏了代码与注释的一致性,让阅读者迷惑、费解,甚至误解。
使用英文进行注释。
在 C++ 代码中,使用 /*
*/
和 //
都是可以的。
按注释的目的和位置,注释可分为不同的类型,如文件头注释、函数头注释、代码注释等等;
同一类型的注释应该保持统一的风格。
注意:本文示例代码中,大量使用 ‘//‘ 后置注释只是为了更精确的描述问题,并不代表这种注释风格更好。
/*
并不是所有的函数都需要函数头注释; 函数签名无法表达的信息,加函数头注释辅助说明;
函数头注释统一放在函数声明或定义上方,使用如下风格之一: 使用//
写函数头
// 单行函数头int Func1(void);// 多行函数头// 第二行int Func2(void);
使用/* */
写函数头
/* 单行函数头 */int Func1(void);/* * 另一种单行函数头 */int Func2(void);/* * 多行函数头 * 第二行 */int Func3(void);
函数尽量通过函数名自注释,按需写函数头注释。 不要写无用、信息冗余的函数头;不要写空有格式的函数头。
函数头注释内容可选,但不限于:功能说明、返回值,性能约束、用法、内存约定、算法实现、可重入的要求等等。 模块对外头文件中的函数接口声明,其函数头注释,应当将重要、有用的信息表达清楚。
例:
/* * 返回实际写入的字节数,-1表示写入失败 * 注意,内存 buf 由调用者负责释放 */int WriteString(const char *buf, int len);
坏的例子:
/* * 函数名:WriteString * 功能:写入字符串 * 参数: * 返回值: */int WriteString(const char *buf, int len);
上面例子中的问题:
代码上方的注释,应该保持对应代码一样的缩进。 选择并统一使用如下风格之一: 使用//
// 这是单行注释DoSomething();// 这是多行注释// 第二行DoSomething();
使用/*' '*/
/* 这是单行注释 */DoSomething();/* * 另一种方式的多行注释 * 第二行 */DoSomething();
代码右边的注释,与代码之间,至少留1空格,建议不超过4空格。 通常使用扩展后的 TAB 键即可实现 1-4 空格的缩进。
选择并统一使用如下风格之一:
int foo = 100; // 放右边的注释int bar = 200; /* 放右边的注释 */
右置格式在适当的时候,上下对齐会更美观。 对齐后的注释,离左边代码最近的那一行,保证1-4空格的间隔。 例:
const int A_CONST = 100; /* 相关的同类注释,可以考虑上下对齐 */const int ANOTHER_CONST = 200; /* 上下对齐时,与左侧代码保持间隔 */
当右置的注释超过行宽时,请考虑将注释置于代码上方。
被注释掉的代码,无法被正常维护;当企图恢复使用这段代码时,极有可能引入易被忽略的缺陷。 正确的做法是,不需要的代码直接删除掉。若再需要时,考虑移植或重写这段代码。
这里说的注释掉代码,包括用 / / 和 //,还包括 #if 0, #ifdef NEVER_DEFINED 等等。
TODO/TBD 注释一般用来描述已知待改进、待补充的修改点 FIXME 注释一般用来描述已知缺陷 它们都应该有统一风格,方便文本搜索统一处理。如:
// TODO(<author-name>): 补充XX处理// FIXME: XX缺陷
头文件是模块或文件的对外接口,头文件的设计体现了大部分的系统设计。 头文件中适合放置接口的声明,不适合放置实现(内联函数除外)。对于cpp文件中内部才需要使用的函数、宏、枚举、结构定义等不要放在头文件中。 头文件应当职责单一。头文件过于复杂,依赖过于复杂还是导致编译时间过长的主要原因。
通常情况下,每个.cpp文件都有一个相应的.h,用于放置对外提供的函数声明、宏定义、类型定义等。 如果一个.cpp文件不需要对外公布任何接口,则其就不应当存在。 例外:程序的入口(如main函数所在的文件),单元测试代码,动态库代码。
示例:
// Foo.h #ifndef FOO_H#define FOO_Hclass Foo {public: Foo(); void Fun();private: int value_;};#endif
// Foo.cpp#include "Foo.h"namespace { // Good: 对内函数的声明放在.cpp文件的头部,并声明为匿名namespace或者static限制其作用域 void Bar() { }}...void Foo::Fun(){ Bar();}
头文件循环依赖,指 a.h 包含 b.h,b.h 包含 c.h,c.h 包含 a.h, 导致任何一个头文件修改,都导致所有包含了a.h/b.h/c.h的代码全部重新编译一遍。
而如果是单向依赖,如a.h包含b.h,b.h包含c.h,而c.h不包含任何头文件,则修改a.h不会导致包含了b.h/c.h的源代码重新编译。
头文件循环依赖直接体现了架构设计上的不合理,可通过优化架构去避免。
#define
保护,防止重复包含为防止头文件被重复包含,所有头文件都应当使用 #define 保护;不要使用 #pragma once
定义包含保护符时,应该遵守如下规则: 1)保护符使用唯一名称; 2)不要在受保护部分的前后放置代码或者注释,文件头注释除外。
示例:假定timer模块的timer.h,其目录为timer/include/timer.h,应按如下方式保护:
#ifndef TIMER_INCLUDE_TIMER_H#define TIMER_INCLUDE_TIMER_H...#endif
只能通过包含头文件的方式使用其他模块或文件提供的接口。 通过 extern 声明的方式使用外部函数接口、变量,容易在外部接口改变时可能导致声明和定义不一致。 同时这种隐式依赖,容易导致架构腐化。
不符合规范的案例:
// a.cpp内容
extern int Fun(); // Bad: 通过extern的方式使用外部函数void Bar() { int i = Fun(); ...}
// b.cpp内容
int Fun() { // Do something}
应该改为:
// a.cpp内容
#include "b.h" // Good: 通过包含头文件的方式使用其他.cpp提供的接口void Bar() { int i = Fun(); ...}
// b.h内容
int Fun();
// b.cpp内容
int Fun() { // Do something}
例外,有些场景需要引用其内部函数,但并不想侵入代码时,可以 extern 声明方式引用。 如: 针对某一内部函数进行单元测试时,可以通过 extern 声明来引用被测函数; 当需要对某一函数进行打桩、打补丁处理时,允许 extern 声明该函数。
在 extern “C” 中包含头文件,有可能会导致 extern “C” 嵌套,部分编译器对 extern “C” 嵌套层次有限制,嵌套层次太多会编译错误。
在C,C++混合编程的情况下,在extern “C”中包含头文件,可能会导致被包含头文件的原有意图遭到破坏,比如链接规范被不正确地更改。
示例,存在a.h和b.h两个头文件:
// a.h内容
...#ifdef __cplusplusvoid Foo(int);#define A(value) Foo(value)#elsevoid A(int)#endif
// b.h内容
...#ifdef __cplusplusextern "C" {#endif#include "a.h"void B();#ifdef __cplusplus}#endif
使用C++预处理器展开b.h,将会得到
extern "C" { void Foo(int); void B();}
按照 a.h 作者的本意,函数 Foo 是一个 C++ 自由函数,其链接规范为 “C++”。 但在 b.h 中,由于 #include "a.h"
被放到了 extern "C"
的内部,函数 Foo 的链接规范被不正确地更改了。
例外: 如果在 C++ 编译环境中,想引用纯C的头文件,这些C头文件并没有extern "C"
修饰。非侵入式的做法是,在 extern "C"
中去包含C头文件。
#include
来包含头文件前置声明(forward declaration)通常指类、模板的纯粹声明,没伴随着其定义。
std::
的 symbol 时,其行为未定义(在C++11标准规范中明确说明)。#include
,某些场景下面前置声明和#include
互换以后会导致意想不到的结果。所以我们尽可能避免使用前置声明,而是使用#include头文件来保证依赖关系。
在C++ 2003标准规范中,使用static修饰文件作用域的变量,函数等被标记为deprecated特性,所以更推荐使用匿名namespace。
主要原因如下:
但是不要在 .h 中使用中使用匿名namespace或者static。
// Foo.cppnamespace { const int MAX_COUNT = 20; void InternalFun(){};}void Foo::Fun(){ int i = MAX_COUNT; InternalFun();}
说明:使用using导入命名空间会影响后续代码,易造成符号冲突,所以不要在头文件以及源文件中的#include之前使用using导入命名空间。 示例:
// 头文件a.hnamespace NamespaceA { int Fun(int);}
// 头文件b.hnamespace NamespaceB { int Fun(int);}using namespace NamespaceB;void G(){ Fun(1); }
// 源代码a.cpp#include "a.h"using namespace NamespaceA; #include "b.h"void main(){ G(); // using namespace NamespaceA在#include “b.h”之前,引发歧义:NamespaceA::Fun,NamespaceB::Fun调用不明确}
对于在头文件中使用using导入单个符号或定义别名,允许在模块自定义名字空间中使用,但禁止在全局名字空间中使用。
// foo.h#include <fancy/string>using fancy::string; // Bad,禁止向全局名字空间导入符号namespace Foo { using fancy::string; // Good,可以在模块自定义名字空间中导入符号 using MyVector = fancy::vector<int>; // Good,C++11可在自定义名字空间中定义别名}
说明:非成员函数放在名字空间内可避免污染全局作用域, 也不要用类+静态成员方法来简单管理全局函数。 如果某个全局函数和某个类有紧密联系, 那么可以作为类的静态成员函数。
如果你需要定义一些全局函数,给某个cpp文件使用,那么请使用匿名namespace来管理。
namespace MyNamespace { int Add(int a, int b);}class File {public: static File CreateTempFile(const std::string& fileName);};
说明:全局常量放在命名空间内可避免污染全局作用域, 也不要用类+静态成员常量来简单管理全局常量。 如果某个全局常量和某个类有紧密联系, 那么可以作为类的静态成员常量。
如果你需要定义一些全局常量,只给某个cpp文件使用,那么请使用匿名namespace来管理。
namespace MyNamespace { const int MAX_SIZE = 100;}class File {public: static const std::string SEPARATOR;};
说明:全局变量是可以修改和读取的,那么这样会导致业务代码和这个全局变量产生数据耦合。
int g_counter = 0;// a.cppg_counter++;// b.cppg_counter++;// c.cppcout << g_counter << endl;
使用单实例模式
class Counter {public: static Counter& GetInstance() { static Counter counter; return counter; } // 单实例实现简单举例 void Increase() { value_++; } void Print() const { std::cout << value_ << std::endl; }private: Counter() : value_(0) {}private: int value_;};// a.cppCounter::GetInstance().Increase();// b.cppCounter::GetInstance().Increase();// c.cppCounter::GetInstance().Print();
实现单例模式以后,实现了全局唯一一个实例,和全局变量同样的效果,并且单实例提供了更好的封装性。
例外:有的时候全局变量的作用域仅仅是模块内部,这样进程空间里面就会有多个全局变量实例,每个模块持有一份,这种场景下是无法使用单例模式解决的。
构造,拷贝,移动和析构函数提供了对象的生命周期管理方法:
X()
X(const X&)
operator=(const X&)
X(X&&)
C++11以后提供operator=(X&&)
C++11以后提供~X()
说明:如果类有成员变量,没有定义构造函数,又没有定义默认构造函数,编译器将自动生成一个构造函数,但编译器生成的构造函数并不会对成员变量进行初始化,对象状态处于一种不确定性。
例外:
示例:如下代码没有构造函数,私有数据成员无法初始化:
class Message {public: void ProcessOutMsg() { //… }private: unsigned int msgID_; unsigned int msgLength_; unsigned char* msgBuffer_; std::string someIdentifier_;};Message message; // message成员变量没有初始化message.ProcessOutMsg(); // 后续使用存在隐患// 因此,有必要定义默认构造函数,如下:class Message {public: Message() : msgID_(0), msgLength_(0), msgBuffer_(NULL) { } void ProcessOutMsg() { // … }private: unsigned int msgID_; unsigned int msgLength_; unsigned char* msgBuffer_; std::string someIdentifier_; // 具有默认构造函数,不需要显式初始化};
说明:C++11的声明时初始化可以一目了然的看出成员初始值,应当优先使用。如果成员初始化值和构造函数相关,或者不支持C++11,则应当优先使用构造函数初始化列表来初始化成员。相比起在构造函数体中对成员赋值,初始化列表的代码更简洁,执行性能更好,而且可以对const成员和引用成员初始化。
class Message {public: Message() : msgLength_(0) // Good,优先使用初始化列表 { msgBuffer_ = NULL; // Bad,不推荐在构造函数中赋值 }private: unsigned int msgID_{0}; // Good,C++11中使用 unsigned int msgLength_; unsigned char* msgBuffer_;};
说明:单参数构造函数如果没有用explicit声明,则会成为隐式转换函数。 示例:
class Foo {public: explicit Foo(const string& name): name_(name) { }private: string name_;};void ProcessFoo(const Foo& foo){}int main(void){ std::string test = "test"; ProcessFoo(test); // 编译不通过 return 0;}
上面的代码编译不通过,因为ProcessFoo
需要的参数是Foo类型,传入的string类型不匹配。
如果将Foo构造函数的explicit关键字移除,那么调用ProcessFoo
传入的string就会触发隐式转换,生成一个临时的Foo对象。往往这种隐式转换是让人迷惑的,并且容易隐藏Bug,得到了一个不期望的类型转换。所以对于单参数的构造函数是要求explicit声明。
说明:如果用户不定义,编译器默认会生成拷贝构造函数和拷贝赋值操作符, 移动构造和移动赋值操作符(移动语义的函数C++11以后才有)。 如果我们不要使用拷贝构造函数,或者赋值操作符,请明确拒绝:
将拷贝构造函数或者赋值操作符设置为private,并且不实现:
class Foo {private: Foo(const Foo&); Foo& operator=(const Foo&);};
使用C++11提供的delete, 请参见后面现代C++的相关章节。
拷贝构造函数和拷贝赋值操作符都是具有拷贝语义的,应该同时出现或者禁止。
// 同时出现class Foo {public: ... Foo(const Foo&); Foo& operator=(const Foo&); ...};// 同时default, C++11支持class Foo {public: Foo(const Foo&) = default; Foo& operator=(const Foo&) = default;};// 同时禁止, C++11可以使用deleteclass Foo {private: Foo(const Foo&); Foo& operator=(const Foo&);};
在C++11中增加了move操作,如果需要某个类支持移动操作,那么需要实现移动构造和移动赋值操作符。
移动构造函数和移动赋值操作符都是具有移动语义的,应该同时出现或者禁?止。
// 同时出现class Foo {public: ... Foo(Foo&&); Foo& operator=(Foo&&); ...};// 同时default, C++11支持class Foo {public: Foo(Foo&&) = default; Foo& operator=(Foo&&) = default;};// 同时禁止, 使用C++11的deleteclass Foo {public: Foo(Foo&&) = delete; Foo& operator=(Foo&&) = delete;};
说明:在构造函数和析构函数中调用当前对象的虚函数,会导致未实现多态的行为。 在C++中,一个基类一次只构造一个完整的对象。
示例:类Base是基类,Sub是派生类
class Base { public: Base(); virtual void Log() = 0; // 不同的派生类调用不同的日志文件};Base::Base() // 基类构造函数{ Log(); // 调用虚函数Log} class Sub : public Base { public: virtual void Log(); };
当执行如下语句: Sub sub;
会先执行Sub的构造函数,但首先调用Base的构造函数,由于Base的构造函数调用虚函数Log,此时Log还是基类的版本,只有基类构造完成后,才会完成派生类的构造,从而导致未实现多态的行为。 同样的道理也适用于析构函数。
说明:只有基类析构函数是virtual,通过多态调用的时候才能保证派生类的析构函数被调用。
示例:基类的析构函数没有声明为virtual导致了内存泄漏。
class Base {public: virtual std::string getVersion() = 0; ~Base() { std::cout << "~Base" << std::endl; }};
class Sub : public Base {public: Sub() : numbers_(NULL) { } ~Sub() { delete[] numbers_; std::cout << "~Sub" << std::endl; } int Init() { const size_t numberCount = 100; numbers_ = new (std::nothrow) int[numberCount]; if (numbers_ == NULL) { return -1; } ... } std::string getVersion() { return std::string("hello!"); }private: int* numbers_;};
int main(int argc, char* args[]){ Base* b = new Sub(); delete b; return 0;}
由于基类Base的析构函数没有声明为virtual,当对象被销毁时,只会调用基类的析构函数,不会调用派生类Sub的析构函数,导致内存泄漏。
说明:在C++中,虚函数是动态绑定的,但函数的缺省参数却是在编译时就静态绑定的。这意味着你最终执行的函数是一个定义在派生类,但使用了基类中的缺省参数值的虚函数。为了避免虚函数重载时,因参数声明不一致给使用者带来的困惑和由此导致的问题,规定所有虚函数均不允许声明缺省参数值。 示例:虚函数display缺省参数值text是由编译时刻决定的,而非运行时刻,没有达到多态的目的:
class Base {public: virtual void Display(const std::string& text = "Base!") { std::cout << text << std::endl; } virtual ~Base(){}};class Sub : public Base {public: virtual void Display(const std::string& text = "Sub!") { std::cout << text << std::endl; } virtual ~Sub(){}};int main(){ Base* base = new Sub(); Sub* sub = new Sub(); ... base->Display(); // 程序输出结果: Base! 而期望输出:Sub! sub->Display(); // 程序输出结果: Sub! delete base; delete sub; return 0;};
说明:因为非虚函数无法实现动态绑定,只有虚函数才能实现动态绑定:只要操作基类的指针,即可获得正确的结果。
示例:
class Base {public: void Fun();};class Sub : public Base { public: void Fun();};Sub* sub = new Sub(); Base* base = sub;sub->Fun(); // 调用子类的Fun base->Fun(); // 调用父类的Fun//...
在实际开发过程中使用多重继承的场景是比较少的,因为多重继承使用过程中有下面的典型问题:
多重继承具有下面的优点: 多重继承提供了一种更简单的组合来实现多种接口或者类的组装与复用。
所以,对于多重继承的只有下面几种情况下面才允许使用多重继承。
如果某个类需要实现多重接口,可以通过多重继承把多个分离的接口组合起来,类似 scala 语言的 traits 混入。
class Role1 {};class Role2 {};class Role3 {};class Object1 : public Role1, public Role2 { // ...};class Object2 : public Role2, public Role3 { // ...};
在C++标准库中也有类似的实现样例:
class basic_istream {};class basic_ostream {};class basic_iostream : public basic_istream, public basic_ostream {};
重载操作符要有充分理由,而且不要改变操作符原有语义,例如不要使用 ‘+’ 操作符来做减运算。 操作符重载令代码更加直观,但也有一些不足:
函数应该可以一屏显示完 (50行以内),只做一件事情,而且把它做好。
过长的函数往往意味着函数功能不单一,过于复杂,或过分呈现细节,未进行进一步抽象。
例外:某些实现算法的函数,由于算法的聚合性与功能的全面性,可能会超过50行。
即使一个长函数现在工作的非常好, 一旦有人对其修改, 有可能出现新的问题, 甚至导致难以发现的bug。 建议将其拆分为更加简短并易于管理的若干函数,以便于他人阅读和修改代码。
说明:内联函数具有一般函数的特性,它与一般函数不同之处只在于函数调用的处理。一般函数进行调用时,要将程序执行权转到被调用函数中,然后再返回到调用它的函数中;而内联函数在调用时,是将调用表达式用内联函数体来替换。
内联函数只适合于只有 1~10 行的小函数。对一个含有许多语句的大函数,函数调用和返回的开销相对来说微不足道,也没有必要用内联函数实现,一般的编译器会放弃内联方式,而采用普通的方式调用函数。
如果内联函数包含复杂的控制结构,如循环、分支(switch)、try-catch 等语句,一般编译器将该函数视同普通函数。 虚函数、递归函数不能被用来做内联函数。
说明:引用比指针更安全,因为它一定非空,且一定不会再指向其他目标;引用不需要检查非法的NULL指针。
如果是基于老平台开发的产品,则优先顺从原有平台的处理方式。 选择 const 避免参数被修改,让代码阅读者清晰地知道该参数不被修改,可大大增强代码可读性。
例外:当传入参数为编译期长度未知的数组时,可以使用指针而不是引用。
尽管不同的语言对待强类型和弱类型有自己的观点,但是一般认为c/c++是强类型语言,既然我们使用的语言是强类型的,就应该保持这样的风格。 好处是尽量让编译器在编译阶段就检查出类型不匹配的问题。
使用强类型便于编译器帮我们发现错误,如下代码中注意函数 FooListAddNode 的使用:
struct FooNode { struct List link; int foo;};struct BarNode { struct List link; int bar;}void FooListAddNode(void *node) // Bad: 这里用 void * 类型传递参数{ FooNode *foo = (FooNode *)node; ListAppend(&g_FooList, &foo->link);}void MakeTheList(){ FooNode *foo = NULL; BarNode *bar = NULL; ... FooListAddNode(bar); // Wrong: 这里本意是想传递参数 foo,但错传了 bar,却没有报错}
函数的参数过多,会使得该函数易于受外部变化的影响,从而影响维护工作。函数的参数过多同时也会增大测试的工作量。
如果超过可以考虑:
不变的值更易于理解、跟踪和分析,所以应该尽可能地使用常量代替变量,定义值的时候,应该把const作为默认的选项。
说明:宏是简单的文本替换,在预处理阶段时完成,运行报错时直接报相应的值;跟踪调试时也是显示值,而不是宏名;宏没有类型检查,不安全;宏没有作用域。
#define MAX_MSISDN_LEN 20 // 不好// C++请使用const常量const int MAX_MSISDN_LEN = 20; // 好// 对于C++11以上版本,可以使用constexprconstexpr int MAX_MSISDN_LEN = 20;
说明:枚举比#define
或const int
更安全。编译器会检查参数值是否位于枚举取值范围内,避免错误发生。
// 好的例子:enum Week { SUNDAY, MONDAY, TUESDAY, WEDNESDAY, THURSDAY, FRIDAY, SATURDAY};enum Color { RED, BLACK, BLUE};void ColorizeCalendar(Week today, Color color);ColorizeCalendar(BLUE, SUNDAY); // 编译报错,参数类型错误// 不好的例子:const int SUNDAY = 0;const int MONDAY = 1;const int BLACK = 0;const int BLUE = 1;bool ColorizeCalendar(int today, int color);ColorizeCalendar(BLUE, SUNDAY); // 不会报错
当枚举值需要对应到具体数值时,须在声明时显式赋值。否则不需要显式赋值,以避免重复赋值,降低维护(增加、删除成员)工作量。
// 好的例子:S协议里定义的设备ID值,用于标识设备类型enum DeviceType { DEV_UNKNOWN = -1, DEV_DSMP = 0, DEV_ISMG = 1, DEV_WAPPORTAL = 2};
程序内部使用,仅用于分类的情况,不应该进行显式的赋值。
// 好的例子:程序中用来标识会话状态的枚举定义enum SessionState { INIT, CLOSED, WAITING_FOR_RESPONSE};
应当尽量避免枚举值重复,如必须重复也要用已定义的枚举来修饰
enum RTCPType { RTCP_SR = 200, RTCP_MIN_TYPE = RTCP_SR, RTCP_RR = 201, RTCP_SDES = 202, RTCP_BYE = 203, RTCP_APP = 204, RTCP_RTPFB = 205, RTCP_PSFB = 206, RTCP_XR = 207, RTCP_RSI = 208, RTCP_PUBPORTS = 209, RTCP_MAX_TYPE = RTCP_PUBPORTS };
所谓魔鬼数字即看不懂、难以理解的数字。
魔鬼数字并非一个非黑即白的概念,看不懂也有程度,需要自行判断。 例如数字 12,在不同的上下文中情况是不一样的: type = 12; 就看不懂,但 monthsCount = yearsCount * 12
; 就能看懂。 数字 0 有时候也是魔鬼数字,比如 status = 0
; 并不能表达是什么状态。
解决途径: 对于局部使用的数字,可以增加注释说明 对于多处使用的数字,必须定义 const 常量,并通过符号命名自注释。
禁止出现下列情况: 没有通过符号来解释数字含义,如const int ZERO = 0
符号命名限制了其取值,如 const int XX_TIMER_INTERVAL_300MS = 300
,直接使用XX_TIMER_INTERVAL_MS
来表示该常量是定时器的时间间隔。
说明:一个常量只用来表示一个特定功能,即一个常量不能有多种用途。
// 好的例子:协议A和协议B,手机号(MSISDN)的长度都是20。const unsigned int A_MAX_MSISDN_LEN = 20;const unsigned int B_MAX_MSISDN_LEN = 20;// 或者使用不同的名字空间:namespace Namespace1 { const unsigned int MAX_MSISDN_LEN = 20;}namespace Namespace2 { const unsigned int MAX_MSISDN_LEN = 20;}
说明:POD
全称是Plain Old Data
,是C++ 98标准(ISO/IEC 14882, first edition, 1998-09-01)中引入的一个概念,POD
类型主要包括int
, char
, float
,double
,enumeration
,void
,指针等原始类型以及聚合类型,不能使用封装和面向对象特性(如用户定义的构造/赋值/析构函数、基类、虚函数等)。
由于非POD类型比如非聚合类型的class对象,可能存在虚函数,内存布局不确定,跟编译器有关,滥用内存拷贝可能会导致严重的问题。
即使对聚合类型的class,使用直接的内存拷贝和比较,破坏了信息隐蔽和数据保护的作用,也不提倡memcpy_s
、memset_s
操作。
对于POD类型的详细说明请参见附录。
说明:变量在使用前未赋初值,是常见的低级编程错误。使用前才声明变量并同时初始化,非常方便地避免了此类低级错误。
在函数开始位置声明所有变量,后面才使用变量,作用域覆盖整个函数实现,容易导致如下问题:
遵循变量作用域最小化原则与就近声明原则, 使得代码更容易阅读,方便了解变量的类型和初始值。特别是,应使用初始化的方式替代声明再赋值。
// 不好的例子:声明与初始化分离string name; // 声明时未初始化:调用缺省构造函数name = "zhangsan"; // 再次调用赋值操作符函数;声明与定义在不同的地方,理解相对困难// 好的例子:声明与初始化一体,理解相对容易string name("zhangsan"); // 调用构造函数
含有变量自增或自减运算的表达式中,如果再引用该变量,其结果在C++标准中未明确定义。各个编译器或者同一个编译器不同版本实现可能会不一致。 为了更好的可移植性,不应该对标准未定义的运算次序做任何假设。
注意,运算次序的问题不能使用括号来解决,因为这不是优先级的问题。
示例:
x = b[i] + i++; // Bad: b[i]运算跟 i++,先后顺序并不明确。
正确的写法是将自增或自减运算单独放一行:
x = b[i] + i;i++; // Good: 单独一行
函数参数
Func(i++, i); // Bad: 传递第2个参数时,不确定自增运算有没有发生
正确的写法
i++; // Good: 单独一行x = Func(i, i);
大部分情况下,switch语句中要有default分支,保证在遗漏case标签处理时能够有一个缺省的处理行为。
特例: 如果switch条件变量是枚举类型,并且 case 分支覆盖了所有取值,则加上default分支处理有些多余。 现代编译器都具备检查是否在switch语句中遗漏了某些枚举值的case分支的能力,会有相应的warning提示。
enum Color { RED = 0, BLUE};// 因为switch条件变量是枚举值,这里可以不用加default处理分支switch (color) { case RED: DoRedThing(); break; case BLUE: DoBlueThing(); ... break;}
当变量与常量比较时,如果常量放左边,如 if (MAX == v) 不符合阅读习惯,而 if (MAX > v) 更是难于理解。 应当按人的正常阅读、表达习惯,将常量放右边。写成如下方式:
if (value == MAX) {}if (value < MAX) {}
也有特殊情况,如:if (MIN < value && value < MAX)
用来描述区间时,前半段是常量在左的。
不用担心将 ‘==’ 误写成 ‘=’,因为if (value = MAX)
会有编译告警,其他静态检查工具也会报错。让工具去解决笔误问题,代码要符合可读性第一。
使用括号明确操作符的优先级,防止因默认的优先级与设计思想不符而导致程序出错;同时使得代码更为清晰可读,然而过多的括号会分散代码使其降低了可读性。下面是如何使用括号的建议。
二元及以上操作符, 如果涉及多种操作符,则应该使用括号
x = a + b + c; /* 操作符相同,可以不加括号 */x = Foo(a + b, c); /* 逗号两边的表达式,不需要括号 */x = 1 << (2 + 3); /* 操作符不同,需要括号 */x = a + (b / 5); /* 操作符不同,需要括号 */x = (a == b) ? a : (a – b); /* 操作符不同,需要括号 */
避免使用类型分支来定制行为:类型分支来定制行为容易出错,是企图用C++编写C代码的明显标志。这是一种很不灵活的技术,要添加新类型时,如果忘记修改所有分支,编译器也不会告知。使用模板和虚函数,让类型自己而不是调用它们的代码来决定行为。
建议避免类型转换,我们在代码的类型设计上应该考虑到每种数据的数据类型是什么,而不是应该过度使用类型转换来解决问题。在设计某个基本类型的时候,请考虑:
但是我们无法禁止使用类型转换,因为C++语言是一门面向机器编程的语言,涉及到指针地址,并且我们会与各种第三方或者底层API交互,他们的类型设计不一定是合理的,在这个适配的过程中很容易出现类型转换。
例外:在调用某个函数的时候,如果我们不想处理函数结果,首先要考虑这个是否是你的最好的选择。如果确实不想处理函数的返回值,那么可以使用(void)转换来解决。
说明:
C++提供的类型转换操作比C风格更有针对性,更易读,也更加安全,C++提供的转换有:
dynamic_cast
:主要用于继承体系下行转换,dynamic_cast
具有类型检查的功能,请做好基类和派生类的设计,避免使用dynamic_cast来进行转换。static_cast
:和C风格转换相似可做值的强制转换,或上行转换(把派生类的指针或引用转换成基类的指针或引用)。该转换经常用于消除多重继承带来的类型歧义,是相对安全的。如果是纯粹的算数转换,那么请使用后面的大括号转换方式。reinterpret_cast
:用于转换不相关的类型。reinterpret_cast
强制编译器将某个类型对象的内存重新解释成另一种类型,这是一种不安全的转换,建议尽可能少用reinterpret_cast
。const_cast
:用于移除对象的const
属性,使对象变得可修改,这样会破坏数据的不变性,建议尽可能少用。算数转换: (C++11开始支持) 对于那种算数转换,并且类型信息没有丢失的,比如float到double, int32到int64的转换,推荐使用大括号的初始方式。
double d{ someFloat };int64_t i{ someInt32 };
dynamic_cast
dynamic_cast
依赖于C++的RTTI, 让程序员在运行时识别C++类对象的类型。dynamic_cast
的出现一般说明我们的基类和派生类设计出现了问题,派生类破坏了基类的契约,不得不通过dynamic_cast
转换到子类进行特殊处理,这个时候更希望来改善类的设计,而不是通过dynamic_cast
来解决问题。reinterpret_cast
说明:reinterpret_cast
用于转换不相关类型。尝试用reinterpret_cast
将一种类型强制转换另一种类型,这破坏了类型的安全性与可靠性,是一种不安全的转换。不同类型之间尽量避免转换。
const_cast
说明:const_cast
用于移除对象的const
和volatile
性质。
使用const_cast转换后的指针或者引用来修改const对象,行为是未定义的。
// 不好的例子const int i = 1024; int* p = const_cast<int*>(&i);*p = 2048; // 未定义行为
// 不好的例子class Foo {public: Foo() : i(3) {} void Fun(int v) { i = v; }private: int i;};int main(void){ const Foo f; Foo* p = const_cast<Foo*>(&f); p->Fun(8); // 未定义行为}
说明:单个对象删除使用delete, 数组对象删除使用delete [],原因:
如果new和delete的格式不匹配,结果是未知的。对于非class类型, new和delete不会调用构造与析构函数。
错误写法:
const int MAX_ARRAY_SIZE = 100;int* numberArray = new int[MAX_ARRAY_SIZE];...delete numberArray;numberArray = NULL;
正确写法:
const int MAX_ARRAY_SIZE = 100;int* numberArray = new int[MAX_ARRAY_SIZE];...delete[] numberArray;numberArray = NULL;
说明:RAII是“资源获取就是初始化”的缩语(Resource Acquisition Is Initialization),是一种利用对象生命周期来控制程序资源(如内存、文件句柄、网络连接、互斥量等等)的简单技术。
RAII 的一般做法是这样的:在对象构造时获取资源,接着控制对资源的访问使之在对象的生命周期内始终保持有效,最后在对象析构的时候释放资源。这种做法有两大好处:
示例:使用RAII不需要显式地释放互斥资源。
class LockGuard {public: LockGuard(const LockType& lockType): lock_(lockType) { lock_.Aquire(); } ~LockGuard() { lock_.Relase(); }private: LockType lock_;};bool Update(){ LockGuard lockGuard(mutex); if (...) { return false; } else { // 操作数据 } return true;}
STL标准模板库在不同产品使用程度不同,这里列出一些基本规则和建议,供各团队参考。
说明:在C++标准中并未规定string::c_str()指针持久有效,因此特定STL实现完全可以在调用string::c_str()时返回一个临时存储区并很快释放。所以为了保证程序的可移植性,不要保存string::c_str()的结果,而是在每次需要时直接调用。
示例:
void Fun1(){ std::string name = "demo"; const char* text = name.c_str(); // 表达式结束以后,name的生命周期还在,指针有效 // 如果中间调用了string的非const成员函数,导致string被修改,比如operator[], begin()等 // 可能会导致text的内容不可用,或者不是原来的字符串 name = "test"; name[1] = '2'; // 后续使用text指针,其字符串内容不再是"demo"}void Fun2(){ std::string name = "demo"; std::string test = "test"; const char* text = (name + test).c_str(); // 表达式结束以后,+号产生的临时对象被销毁,指针无效 // 后续使用text指针,其已不再指向合法内存空间}
例外:在少数对性能要求非常高的代码中,为了适配已有的只接受const char*类型入参的函数,可以临时保存string::c_str()返回的指针。但是必须严格保证string对象的生命周期长于所保存指针的生命周期,并且保证在所保存指针的生命周期内,string对象不会被修改。
说明:使用string代替char*
有很多优势,比如:
需要注意的是某些stl实现中string是基于写时复制策略的,这会带来2个问题,一是某些版本的写时复制策略没有实现线程安全,在多线程环境下会引起程序崩溃;二是当与动态链接库相互传递基于写时复制策略的string时,由于引用计数在动态链接库被卸载时无法减少可能导致悬挂指针。因此,慎重选择一个可靠的stl实现对于保证程序稳定是很重要的。
例外: 当调用系统或者其它第三方库的API时,针对已经定义好的接口,只能使用char*
。但是在调用接口之前都可以使用string,在调用接口时使用string::c_str()获得字符指针。 当在栈上分配字符数组当作缓冲区使用时,可以直接定义字符数组,不要使用string,也没有必要使用类似vector<char>
等容器。
说明:在stl库中的std::auto_ptr具有一个隐式的所有权转移行为,如下代码:
auto_ptr<T> p1(new T);auto_ptr<T> p2 = p1;
当执行完第2行语句后,p1已经不再指向第1行中分配的对象,而是变为NULL。正因为如此,auto_ptr不能被置于各种标准容器中。 转移所有权的行为通常不是期望的结果。对于必须转移所有权的场景,也不应该使用隐式转移的方式。这往往需要程序员对使用auto_ptr的代码保持额外的谨慎,否则出现对空指针的访问。 使用auto_ptr常见的有两种场景,一是作为智能指针传递到产生auto_ptr的函数外部,二是使用auto_ptr作为RAII管理类,在超出auto_ptr的生命周期时自动释放资源。 对于第1种场景,可以使用std::shared_ptr来代替。 对于第2种场景,可以使用C++11标准中的std::unique_ptr来代替。其中std::unique_ptr是std::auto_ptr的代替品,支持显式的所有权转移。
例外: 在C++11标准得到普遍使用之前,在一定需要对所有权进行转移的场景下,可以使用std::auto_ptr,但是建议对std::auto_ptr进行封装,并禁用封装类的拷贝构造函数和赋值运算符,以使该封装类无法用于标准容器。
说明: 使用C++的标准头文件时,请使用<cstdlib>
这样的,而不是<stdlib.h>
这种的。
在声明的变量或参数前加上关键字 const 用于指明变量值不可被篡改 (如 const int foo
). 为类中的函数加上 const 限定符表明该函数不会修改类成员变量的状态 (如 class Foo { int Bar(char c) const; };
)。 const 变量, 数据成员, 函数和参数为编译时类型检测增加了一层保障, 便于尽早发现错误。因此, 我们强烈建议在任何可能的情况下使用 const。 有时候,使用C++11的constexpr来定义真正的常量可能更好。
不变的值更易于理解/跟踪和分析,把const作为默认选项,在编译时会对其进行检查,使代码更牢固/更安全。
class Foo;void PrintFoo(const Foo& foo);
尽可能将成员函数声明为 const。 访问函数应该总是 const。只要不修改数据成员的成员函数,都声明为const。 对于虚函数,应当从设计意图上考虑继承链上的所有类是否需要在此虚函数中修改数据成员,而不是仅关注单个类的实现。
class Foo {public: // ... int PrintValue() const // const修饰成员函数,不会修改成员变量 { std::cout << value_ << std::endl; } int GetValue() const // const修饰成员函数,不会修改成员变量 { return value_; }private: int value_;};
class Foo {public: Foo(int length) : dataLength_(length) {}private: const int dataLength_; };
noexcept
理由
noexcept
可以让编译器最大程度的优化函数,如减少执行路径,提高错误退出的效率。vector
等STL容器,为了保证接口的健壮性,如果保存元素的move运算符
没有声明为noexcept
,则在容器扩张搬移元素时不会使用move机制
,而使用copy机制
,带来性能损失的风险。如果一个函数不能抛出异常,或者一个程序并没有截获某个函数所抛出的异常并进行处理,那么这个函数可以用新的noexcept
关键字对其进行修饰,表示这个函数不会抛出异常或者抛出的异常不会被截获并处理。例如:extern "C" double sqrt(double) noexcept; // 永远不会抛出异常// 即使可能抛出异常,也可以使用 noexcept// 这里不准备处理内存耗尽的异常,简单地将函数声明为noexceptstd::vector<int> MyComputation(const std::vector<int>& v) noexcept{ std::vector<int> res = v; // 可能会抛出异常 // do something return res;}
示例
RetType Function(Type params) noexcept; // 最大的优化RetType Function(Type params); // 更少的优化// std::vector 的 move 操作需要声明 noexceptclass Foo1 {public: Foo1(Foo1&& other); // no noexcept};std::vector<Foo1> a1;a1.push_back(Foo1());a1.push_back(Foo1()); // 触发容器扩张,搬移已有元素时调用copy constructorclass Foo2 {public: Foo2(Foo2&& other) noexcept;};std::vector<Foo2> a2;a2.push_back(Foo2());a2.push_back(Foo2()); // 触发容器扩张,搬移已有元素时调用move constructor
注意 默认构造函数、析构函数、swap
函数,move操作符
都不应该抛出异常。
模板能够实现非常灵活简洁的类型安全的接口,实现类型不同但是行为相同的代码复用。
模板编程的缺点:
所以, 建议模板编程最好只用在少量的基础组件,基础数据结构上面。并且使用模板编程的时候尽可能把复杂度最小化,尽量不要让模板对外暴露。最好只在实现里面使用模板, 然后给用户暴露的接口里面并不使用模板, 这样能提高你的接口的可读性。 并且你应该在这些使用模板的代码上写尽可能详细的注释。
在C++语言中,我们强烈建议尽可能少使用复杂的宏
// 不推荐使用宏函数#define SQUARE(a, b) ((a) * (b)) // 请使用模板函数,内联函数等来替换。template<typename T> T Square(T a, T b) { return a * b; }
如果需要使用宏,请参考C语言规范的相关章节。 例外:一些通用且成熟的应用,如:对 new, delete 的封装处理,可以保留对宏的使用。
随着 ISO 在2011年发布 C++11 语言标准,以及2017年3月发布 C++17 ,现代C++(C++11/14/17等)增加了大量提高编程效率、代码质量的新语言特性和标准库。 本章节描述了一些可以帮助团队更有效率的使用现代C++,规避语言陷阱的指导意见。
auto
理由
auto
可以避免编写冗长、重复的类型名,也可以保证定义变量时初始化。auto
类型推导规则复杂,需要仔细理解。auto
。示例
// 避免冗长的类型名std::map<string, int>::iterator iter = m.find(val);auto iter = m.find(val);// 避免重复类型名class Foo {...};Foo* p = new Foo;auto p = new Foo;// 保证初始化int x; // 编译正确,没有初始化auto x; // 编译失败,必须初始化
auto 的类型推导可能导致困惑:
auto a = 3; // intconst auto ca = a; // const intconst auto& ra = a; // const int&auto aa = ca; // int, 忽略 const 和 referenceauto ila1 = { 10 }; // std::initializer_list<int>auto ila2{ 10 }; // std::initializer_list<int>auto&& ura1 = x; // int&auto&& ura2 = ca; // const int&auto&& ura3 = 10; // int&&const int b[10];auto arr1 = b; // const int*auto& arr2 = b; // const int(&)[10]
如果没有注意 auto
类型推导时忽略引用,可能引入难以发现的性能问题:
std::vector<std::string> v;auto s1 = v[0]; // auto 推导为 std::string,拷贝 v[0]
如果使用auto
定义接口,如头文件中的常量,可能因为开发人员修改了值,而导致类型发生变化。
override
或final
关键字理由 override
和final
关键字都能保证函数是虚函数,且重写了基类的虚函数。如果子类函数与基类函数原型不一致,则产生编译告警。final
还保证虚函数不会再被子类重写。
使用override
或final
关键字后,如果修改了基类虚函数原型,但忘记修改子类重写的虚函数,在编译期就可以发现。也可以避免有多个子类时,重写虚函数的修改遗漏。
示例
class Base {public: virtual void Foo(); virtual void Foo(int var); void Bar();};class Derived : public Base {public: void Foo() const override; // 编译失败: Derived::Foo 和 Base::Foo 原型不一致,不是重写 void Foo() override; // 正确: Derived::Foo 重写 Base::Foo void Foo(int var) final; // 正确: Derived::Foo(int) 重写 Base::Foo(int),且Derived的派生类不能再重写此函数 void Bar() override; // 编译失败: Base::Bar 不是虚函数};
总结
virtual
关键字override
或final
关键字(但不要两者一起使用),并且不使用virtual
关键字virtual
、override
和final
都不使用delete
关键字删除函数理由 相比于将类成员函数声明为private
但不实现,delete
关键字更明确,且适用范围更广。
示例
class Foo {private: // 只看头文件不知道拷贝构造是否被删除 Foo(const Foo&);};class Foo {public: // 明确删除拷贝赋值函数 Foo& operator=(const Foo&) = delete;};
delete
关键字还支持删除非成员函数
template<typename T>void Process(T value);template<>void Process<void>(void) = delete;
nullptr
,而不是NULL
或0
理由 长期以来,C++没有一个代表空指针的关键字,这是一件很尴尬的事:
#define NULL ((void *)0)char* str = NULL; // 错误: void* 不能自动转换为 char*void(C::*pmf)() = &C::Func;if (pmf == NULL) {} // 错误: void* 不能自动转换为指向成员函数的指针
如果把NULL
被定义为0
或0L
。可以解决上面的问题。
或者在需要空指针的地方直接使用0
。但这引入另一个问题,代码不清晰,特别是使用auto
自动推导:
auto result = Find(id);if (result == 0) { // Find() 返回的是 指针 还是 整数? // do something}
0
字面上是int
类型(0L
是long
),所以NULL
和0
都不是指针类型。 当重载指针和整数类型的函数时,传递NULL
或0
都调用到整数类型重载的函数:
void F(int);void F(int*);F(0); // 调用 F(int),而非 F(int*)F(NULL); // 调用 F(int),而非 F(int*)
另外,sizeof(NULL) == sizeof(void*)
并不一定总是成立的,这也是一个潜在的风险。
总结: 直接使用0
或0L
,代码不清晰,且无法做到类型安全;使用NULL
无法做到类型安全。这些都是潜在的风险。
nullptr
的优势不仅仅是在字面上代表了空指针,使代码清晰,而且它不再是一个整数类型。
nullptr
是std::nullptr_t
类型,而std::nullptr_t
可以隐式的转换为所有的原始指针类型,这使得nullptr
可以表现成指向任意类型的空指针。
void F(int);void F(int*);F(nullptr); // 调用 F(int*)auto result = Find(id);if (result == nullptr) { // Find() 返回的是 指针 // do something}
using
而非typedef
在C++11
之前,可以通过typedef
定义类型的别名。没人愿意多次重复std::map<uint32_t, std::vector<int>>
这样的代码。
typedef std::map<uint32_t, std::vector<int>> SomeType;
类型的别名实际是对类型的封装。而通过封装,可以让代码更清晰,同时在很大程度上避免类型变化带来的散弹式修改。 在C++11
之后,提供using
,实现声明别名(alias declarations)
:
using SomeType = std::map<uint32_t, std::vector<int>>;
对比两者的格式:
typedef Type Alias; // Type 在前,还是 Alias 在前using Alias = Type; // 符合'赋值'的用法,容易理解,不易出错
如果觉得这点还不足以切换到using
,我们接着看看模板别名(alias template)
:
// 定义模板的别名,一行代码template<class T>using MyAllocatorVector = std::vector<T, MyAllocator<T>>;MyAllocatorVector<int> data; // 使用 using 定义的别名template<class T>class MyClass {private: MyAllocatorVector<int> data_; // 模板类中使用 using 定义的别名};
而typedef
不支持带模板参数的别名,只能”曲线救国”:
// 通过模板包装 typedef,需要实现一个模板类template<class T>struct MyAllocatorVector { typedef std::vector<T, MyAllocator<T>> type;};MyAllocatorVector<int>::type data; // 使用 typedef 定义的别名,多写 ::typetemplate<class T>class MyClass {private: typename MyAllocatorVector<int>::type data_; // 模板类中使用,除了 ::type,还需要加上 typename};
从字面上看,std::move
的意思是要移动一个对象。而const对象是不允许修改的,自然也无法移动。因此用std::move
操作const对象会给代码阅读者带来困惑。 在实际功能上,std::move
会把对象转换成右值引用类型;对于const对象,会将其转换成const的右值引用。由于极少有类型会定义以const右值引用为参数的移动构造函数和移动赋值操作符,因此代码实际功能往往退化成了对象拷贝而不是对象移动,带来了性能上的损失。
错误示例:
std::string g_string;std::vector<std::string> g_stringList;void func(){ const std::string myString = "String content"; g_string = std::move(myString); // bad:并没有移动myString,而是进行了复制 const std::string anotherString = "Another string content"; g_stringList.push_back(std::move(anotherString)); // bad:并没有移动anotherString,而是进行了复制}
理由 避免资源泄露。
示例
void Use(int i){ auto p = new int {7}; // 不好: 通过 new 初始化局部指针 auto q = std::make_unique<int>(9); // 好: 保证释放内存 if (i > 0) { return; // 可能 return,导致内存泄露 } delete p; // 太晚了}
例外 在性能敏感、兼容性等场景可以使用原始指针。
unique_ptr
而不是shared_ptr
理由
shared_ptr
引用计数的原子操作存在可测量的开销,大量使用shared_ptr
影响性能。std::make_unique
而不是new
创建unique_ptr
理由
make_unique
提供了更简洁的创建方式示例
// 不好:两次出现 MyClass,重复导致不一致风险std::unique_ptr<MyClass> ptr(new MyClass(0, 1));// 好:只出现一次 MyClass,不存在不一致的可能auto ptr = std::make_unique<MyClass>(0, 1);
重复出现类型可能导致非常严重的问题,且很难发现:
// 编译正确,但new和delete不配套std::unique_ptr<uint8_t> ptr(new uint8_t[10]);std::unique_ptr<uint8_t[]> ptr(new uint8_t);// 非异常安全: 编译器可能按如下顺序计算参数:// 1. 分配 Foo 的内存,// 2. 构造 Foo,// 3. 调用 Bar,// 4. 构造 unique_ptr<Foo>.// 如果 Bar 抛出异常, Foo 不会被销毁,产生内存泄露。F(unique_ptr<Foo>(new Foo()), Bar());// 异常安全: 调用函数不会被打断.F(make_unique<Foo>(), Bar());
例外 std::make_unique
不支持自定义deleter
。 在需要自定义deleter
的场景,建议在自己的命名空间实现定制版本的make_unique
。 使用new
创建自定义deleter
的unique_ptr
是最后的选择。
std::make_shared
而不是new
创建shared_ptr
理由 使用std::make_shared
除了类似std::make_unique
一致性等原因外,还有性能的因素。 std::shared_ptr
管理两个实体:
deleter
等)std::make_shared
创建std::shared_ptr
,会一次性在堆上分配足够容纳控制块和管理对象的内存。而使用std::shared_ptr<MyClass>(new MyClass)
创建std::shared_ptr
,除了new MyClass
会触发一次堆分配外,std::shard_ptr
的构造函数还会触发第二次堆分配,产生额外的开销。
例外 类似std::make_unique
,std::make_shared
不支持定制deleter
lambda
(捕获局部变量,或编写局部函数)理由 函数无法捕获局部变量或在局部范围内声明;如果需要这些东西,尽可能选择lambda
,而不是手写的functor
。 另一方面,lambda
和functor
不会重载;如果需要重载,则使用函数。 如果lambda
和函数都可以的场景,则优先使用函数;尽可能使用最简单的工具。
示例
// 编写一个只接受 int 或 string 的函数// -- 重载是自然的选择void F(int);void F(const string&);// 需要捕获局部状态,或出现在语句或表达式范围// -- lambda 是自然的选择vector<Work> v = LotsOfWork();for (int taskNum = 0; taskNum < max; ++taskNum) { pool.Run([=, &v] {...});}pool.Join();
lambdas
,避免使用按引用捕获理由 非局部范围使用lambdas
包括返回值,存储在堆上,或者传递给其它线程。局部的指针和引用不应该在它们的范围外存在。lambdas
按引用捕获就是把局部对象的引用存储起来。如果这会导致超过局部变量生命周期的引用存在,则不应该按引用捕获。
示例
// 不好void Foo(){ int local = 42; // 按引用捕获 local. // 当函数返回后,local 不再存在, // 因此 Process() 的行为未定义! threadPool.QueueWork([&]{ Process(local); });}// 好void Foo(){ int local = 42; // 按值捕获 local。 // 因为拷贝,Process() 调用过程中,local 总是有效的 threadPool.QueueWork([=]{ Process(local); });}
this
,则显式捕获所有变量理由 在成员函数中的[=]
看起来是按值捕获。但因为是隐式的按值获取了this
指针,并能够操作所有成员变量,数据成员实际是按引用捕获的,一般情况下建议避免。如果的确需要这样做,明确写出对this
的捕获。
示例
class MyClass {public: void Foo() { int i = 0; auto Lambda = [=]() { Use(i, data_); }; // 不好: 看起来像是拷贝/按值捕获,成员变量实际上是按引用捕获 data_ = 42; Lambda(); // 调用 use(42); data_ = 43; Lambda(); // 调用 use(43); auto Lambda2 = [i, this]() { Use(i, data_); }; // 好,显式指定按值捕获,最明确,最少的混淆 }private: int data_ = 0;};
理由 lambda表达式提供了两种默认捕获模式:按引用(&)和按值(=)。 默认按引用捕获会隐式的捕获所有局部变量的引用,容易导致访问悬空引用。相比之下,显式的写出需要捕获的变量可以更容易的检查对象生命周期,减小犯错可能。 默认按值捕获会隐式的捕获this指针,且难以看出lambda函数所依赖的变量是哪些。如果存在静态变量,还会让阅读者误以为lambda拷贝了一份静态变量。 因此,通常应当明确写出lambda需要捕获的变量,而不是使用默认捕获模式。
错误示例
auto func(){ int addend = 5; static int baseValue = 3; return [=]() { // 实际上只复制了addend ++baseValue; // 修改会影响静态变量的值 return baseValue + addend; };}
正确示例
auto func(){ int addend = 5; static int baseValue = 3; return [addend, baseValue = baseValue]() mutable { // 使用C++14的捕获初始化拷贝一份变量 ++baseValue; // 修改自己的拷贝,不会影响静态变量的值 return baseValue + addend; };}
参考:《Effective Modern C++》:Item 31: Avoid default capture modes.
T*
或T&
作为参数,而不是智能指针理由
this
)。示例
// 接受任何 int*void F(int*);// 只能接受希望转移所有权的 intvoid G(unique_ptr<int>);// 只能接受希望共享所有权的 intvoid G(shared_ptr<int>);// 不改变所有权,但需要特定所有权的调用者void H(const unique_ptr<int>&);// 接受任何 intvoid H(int&);// 不好void F(shared_ptr<Widget>& w){ // ... Use(*w); // 只使用 w -- 完全不涉及生命周期管理 // ...};
Copyright © 2003-2013 www.wpsshop.cn 版权所有,并保留所有权利。