避免使用编译宏
宏在编译之前被预处理器所替换,从而使得调试非常困难,因为调试器无法知道源代码来自哪里。
// Bad Idea
#define PI 3.14159;
// Good Idea
namespace my_project {
class Constants {
public:
// if the above macro would be expanded, then the following line would be:
// static const double 3.14159 = 3.14159;
// which leads to a compile-time error. Sometimes such errors are hard to understand.
static constexpr double PI = 3.14159;
};
}
避免使用布尔值作为函数参数
在阅读代码时,布尔值无法提供任何额外含义。可以创建一个名称更有意义的独立函数,或者传递含义更明确的枚举值。
避免使用裸循环
了解和理解现有C++标准算法,并付诸实践。
-
参考cppreference[2]
-
观看C++ Seasoning[3]
将对[]
的调用看作是一种潜在的代码坏味道,表明没有在需要的地方使用合适的算法。
永远不要使用有副作用的assert
// Bad Idea
assert(set_value(something));
// Better Idea
[[maybe_unused]] const auto success = set_value(something);
assert(success);
在release版本中assert()
将会被删除,从而造成set_value
无法被调用。
虽然第二个版本更丑,但总比第一个错误版本好一点。
正确使用“override”和“final”
这些关键字使其他开发人员可以清楚知道虚函数可以被如何使用,如果虚函数的签名发生了变化,就可以捕获潜在错误,并有可能向编译器提示可以执行哪些优化