最重要的一条规则
编写代码时最重要的一条规则是:检查周围的代码并尝试模仿它。
作为维护人员,如果收到的补丁明显与周围代码的编码风格不同,这是令人沮丧的。这是不尊重人的,就像某人穿着泥泞的鞋子走进一间一尘不染的房子。
因此,无论本文推荐的是什么,如果已经编写了代码并且您正在对其进行修补,请保持其当前的样式一致,即使它不是您最喜欢的样式。
一般性的规则
这里列出了最明显和最重要的一般规则。在你继续阅读其他章节之前,请仔细检查它们:
- 使用C99标准
- 不使用制表符,而是使用空格
- 每个缩进级别使用4个空格
- 在关键字和左括号之间使用一个空格
- 在函数名和左括号之间不要使用空格
int32_t a = sum(4, 3);
int32_t a = sum (4, 3);
- 不要在变量/函数/宏/类型中使用_或前缀。这是为C语言本身保留的。
- 对于严格的模块私有函数,使用prv_ name前缀。
- 对于包含下划线_ char的变量/函数/宏/类型,只能使用小写字母。
- 左花括号总是与关键字(for, while, do, switch, if,…)在同一行。
size_t i;
for (i = 0; i < 5; ++i) {
}
for (i = 0; i < 5; ++i){
}
for (i = 0; i < 5; ++i)
{
}
- 在比较操作符和赋值操作符之前和之后使用单个空格。
int32_t a;
a = 3 + 4;
for (a = 0; a < 5; ++a)
a=3+4;
a = 3+4;
for (a=0;a<5;++a)
- 每个逗号后用单空格。
func_name(5, 4);
func_name(4,3);
- 不要初始化静态和全局变量为0(或NULL),让编译器为您做。
static int32_t a;
static int32_t b = 4;
static int32_t a = 0;
void my_func(void) {
static int32_t* ptr;
static char abc = 0;
}
- 在同一行中声明所有相同类型的局部变量。
void my_func(void) {
char a;
char b;
char a, b;
}
- 按顺序声明局部变量
i. 自定义结构和枚举
ii. 整数类型,更宽的无符号类型优先
iii. 单/双浮点
int my_func(void) {
my_struct_t my;
my_struct_ptr_t* p;
uint32_t a;
int32_t b;
uint16_t c;
int16_t g;
char h;
double d;
float f;
}
- 总是在块的开头声明局部变量,在第一个可执行语句之前。
- 在for循环中声明计数器变量。
for (size_t i = 0; i < 10; ++i)
size_t i;
for (i = 0; i < 10; ++i) {
if (...) {
break;
}
}
if (i * 10) {
}
size_t i;
for (i = 0; i < 10; ++i) ...
- 避免在声明中使用函数调用来赋值变量,除了单个变量。
void a(void) {
int32_t a, b = sum(1, 2);
int32_t a, b;
b = sum(1, 2);
uint8_t a = 3, b = 4;
}
- 除了char、float或double之外,始终使用stdint.h标准库中声明的类型。例如,8位的uint8_t等
- 不要使用stdbool.h库。分别使用1或0表示真或假。
uint8_t status;
status = 0;
bool status = true;
- 永远不要与真实相比较。例如,使用if(check_func()){…}替换if (check_func() * 1)
- 总是将指针与空值进行比较。
void* ptr;
if (ptr * NULL || ptr != NULL) {
}
if (ptr || !ptr) {
}
- 总是使用前增量(和递减),而不是后增量(和递减)
int32_t a = 0;
...
a++;
++a;
for (size_t j = 0; j < 10; ++j) {}
- 总是使用size_t作为长度或大小变量
- 如果函数不应该修改指针所指向的内存,则总是使用const作为指针
- 如果不应该修改函数的形参或变量,则总是使用const
void
my_func(const void* d) {
}
void
my_func(const void* const d) {
}
void
my_func(const size_t len) {
}
void
my_func(void* const d) {
}
- 当函数可以接受任何类型的指针时,总是使用void *,不要使用uint8_t *。函数在实现时必须注意正确的类型转换。
void send_data(const void* data, size_t len) {
const uint8_t* d = data;
}
void send_data(const void* data, int len) {
}
- 总是使用括号和sizeof操作符
- 不要使用变长数组。使用动态内存分配代替标准C malloc和自由函数,或者如果库/项目提供了自定义内存分配,使用它的实现看看LwMEM,一个自定义内存管理库。
void
my_func(size_t size) {
int32_t* arr;
arr = malloc(sizeof(*arr) * n);
arr = malloc(sizeof *arr * n);
if (arr * NULL) {
}
free(arr);
}
void
my_func(size_t size) {
int32_t arr[size];
}
- 总是将variable与0进行比较,除非它被视为布尔类型
- 永远不要将布尔处理的变量与0或1进行比较。用NOT(!)代替
size_t length = 5;
uint8_t is_ok = 0;
if (length)
if (length > 0)
if (length * 0)
if (is_ok)
if (!is_ok)
if (is_ok * 1)
if (is_ok * 0)
- 对于注释,总是使用,即使是单行注释
- 在头文件中总是包含带有extern关键字的c++检查
- 每个函数都必须包含doxygen-enabled注释,即使函数是静态的
- 使用英文名称/文本的函数,变量,注释
- 变量使用小写字母
- 如果变量包含多个名称,请使用下划线。force_redraw。不要使用forceRedraw
- 对于C标准库的包含文件,请始终使用<和>。例如,# include < stdlib.h >
- 对于自定义库,请始终使用""。例如,# include“my_library.h”
- 当转换为指针类型时,总是将星号与类型对齐,例如。uint8_t* t = (uint8_t*)var_width_diff_type
- 始终尊重项目或库中已经使用的代码风格
注释相关的规则
- 不允许以//开头的注释。总是使用② comment */,即使是单行注释
- 对于多行注释,每行使用空格+星号
- 注释时使用12个缩进(12 * 4个空格)偏移量。如果语句大于12个缩进,将注释4-空格对齐(下面的例子)到下一个可用缩进
void my_func(void) {
char a, b;
a = call_func_returning_char_a(a);
b = call_func_returning_char_a_but_func_name_is_very_long(a);
}
函数定义的规则
- 每个可以从模块外部访问的函数都必须包含函数原型(或声明)
- 函数名必须小写,可以用下划线_分隔。(这个原则好像因人而异)
void my_func(void);
void myfunc(void);
void MYFunc(void);
void myFunc();
- 当函数返回指针时,将星号对齐到返回类型
const char* my_func(void);
my_struct_t* my_func(int32_t a, int32_t b);
const char *my_func(void);
my_struct_t * my_func(void);
- 对齐所有的功能原型(使用相同/相似的功能)以提高可读性
void set(int32_t a);
my_type_t get(void);
my_ptr_t* get_ptr(void);
void set(int32_t a);
const char * get(void);
- 函数实现必须在单独的行中包含返回类型和可选的其他关键字
int32_t
foo(void) {
return 0;
}
static const char*
get_string(void) {
return "Hello world!\r\n";
}
int32_t foo(void) {
return 0;
}
变量相关的规则
- 使变量名全部小写,下划线_字符可选
int32_t a;
int32_t my_var;
int32_t myvar;
int32_t A;
int32_t myVar;
int32_t MYVar;
- 按类型将局部变量分组在一起
void foo(void) {
int32_t a, b;
char a;
char b;
}
- 不要在第一个可执行语句之后声明变量
void foo(void) {
int32_t a;
a = bar();
int32_t b;
}
- 你可以在下一个缩进级别中声明新的变量
int32_t a, b;
a = foo();
if (a) {
int32_t c, d;
c = foo();
int32_t e;
}
- 用星号声明指针变量与类型对齐
char* a;
char *a;
char * a;
- 当声明多个指针变量时,可以使用星号对变量名进行声明
char *p, *n;
结构、枚举类型定义
- 结构名或枚举名必须小写,单词之间有下划线_字符
- 结构或枚举可以包含typedef关键字
- 所有结构成员都必须小写
- 所有枚举成员必须是大写的
- 结构/枚举必须遵循doxygen文档语法
- 在声明结构体时,它可以使用以下三种不同的选项之一:
1. 当结构体仅用名称声明时,它的名称后不能包含_t后缀。
struct struct_name {
char* a;
char b;
};
2. 当只使用typedef声明结构时,它的名称后面必须包含_t后缀。
typedef struct {
char* a;
char b;
} struct_name_t;
3. 当结构用name和typedef声明时,它不能包含t作为基本名称,它必须在它的名称后面包含t后缀作为typedef部分。
typedef struct struct_name {
char* a;
char b;
char c;
} struct_name_t;
- 错误声明的例子及其建议的纠正:
typedef struct {
int32_t a, b;
} a;
typedef struct {
int32_t a;
int32_t b;
} a_t;
struct name_t {
int32_t a;
int32_t b;
};
typedef enum {
MY_ENUM_TESTA,
my_enum_testb,
} my_enum_t;
- 在声明时初始化结构时,使用C99初始化风格
a_t a = {
.a = 4,
.b = 5,
};
a_t = {1, 2};
- 当为函数句柄引入new typedef时,使用_fn后缀
typedef uint8_t (*my_func_typedef_fn)(uint8_t p1, const char* p2);
复合语句规则
- 每个复合语句必须包括左花括号和右花括号,即使它只包含1个嵌套语句
- 每个复合语句必须包含单个缩进;嵌套语句时,每个嵌套包含1个缩进大小
if (c) {
do_a();
} else {
do_b();
}
if (c)
do_a();
else
do_b();
if (c) do_a();
else do_b();
- 在if或if-else-if语句的情况下,else必须与第一条语句的右括号在同一行
if (a) {
} else if (b) {
} else {
}
if (a) {
}
else {
}
if (a) {
}
else
{
}
} else {
}
if (a) {
}
else {
}
if (a) {
}
else
{
}
- 在do-while语句的情况下,while部分必须与do部分的右括号在同一行
do {
int32_t a;
a = do_a();
do_b(a);
} while (check());
do
{
} while (check());
do {
}
while (check());
- 每一个开括号都需要缩进
if (a) {
do_a();
} else {
do_b();
if (c) {
do_c();
}
}
- 不要做没有花括号的复合语句,即使是单个语句。下面的例子展示了一些不好的做法
if (a) do_b();
else do_c();
if (a) do_a(); else do_b();
- 空while循环、do-while循环或for循环必须包含花括号
while (is_register_bit_set()) {}
while (is_register_bit_set());
while (is_register_bit_set()) { }
while (is_register_bit_set()) {
}
- 如果while(或for、do-while等)为空(嵌入式编程中也可能是这种情况),请使用空的单行括号
while (*addr & (1 << 13)) {}
while (*addr & (1 << 13)) { }
while (*addr & (1 << 13)) {
}
while (*addr & (1 << 13));
- 尽量避免在循环块内递增变量,参见示例
int32_t a = 0;
while (a < 10) {
.
..
...
++a;
}
for (size_t a = 0; a < 10; ++a) {
}
for (size_t a = 0; a < 10; ) {
if (...) {
++a;
}
}
分支语句规则
- 为每个case语句添加单个缩进
- 使用额外的单缩进break语句在每个case或default
switch (check()) {
case 0:
do_a();
break;
case 1:
do_b();
break;
default:
break;
}
switch (check()) {
case 0:
do_a();
break;
case 1:
do_b();
break;
default:
break;
}
switch (check()) {
case 0:
do_a();
break;
case 1:
do_b();
break;
default:
break;
}
- 总是包含default语句
switch (var) {
case 0:
do_job();
break;
default: break;
}
switch (var) {
case 0:
do_job();
break;
}
- 如果需要局部变量,则使用花括号并在里面放入break语句。将左花括号放在case语句的同一行
switch (a) {
case 0: {
int32_t a, b;
char c;
a = 5;
break;
}
case 1:
{
int32_t a;
break;
}
case 2: {
int32_t a;
}
break;
}
宏和预处理指令
- 总是使用宏而不是文字常量,特别是对于数字
- 所有的宏必须是全大写的,并带有下划线_字符(可选),除非它们被明确标记为function,将来可能会被常规函数语法替换
- 总是用圆括号保护输入参数
- 总是用括号保护最终的宏计算
int32_t x = 5 * SUM(3, 4);
int32_t x = 5 * (3) + (4);
- 当宏使用多个语句时,使用do-while(0)语句保护它
typedef struct {
int32_t px, py;
} point_t;
point_t p;
SET_POINT(&p, 3, 4);
(&p)->px = (3); (&p)->py = (4);
if (a)
if (b)
SET_POINT(&p, 3, 4);
else
SET_POINT(&p, 5, 6);
if (a)
if (b)
(&p)->px = (3); (&p)->py = (4);
else
(&p)->px = (5); (&p)->py = (6);
if (a)
if (b)
(&p)->px = (3);
(&p)->py = (4);
else
(&p)->px = (5);
(&p)->py = (6);
(p)->px = (x); \
(p)->py = (y); \
} while (0)
if (a)
if (b)
do { (&p)->px = (3); (&p)->py = (4); } while (0);
else
do { (&p)->px = (5); (&p)->py = (6); } while (0);
if (a) {
if (b) {
SET_POINT(&p, 3, 4);
} else {
SET_POINT(&p, 5, 6);
}
}
- 不缩进子语句内#if语句
文档
- 文档化的代码允许doxygen解析和通用的html/pdf/latex输出,因此正确地执行是非常重要的。
- 对变量、函数和结构/枚举使用doxygen支持的文档样式
- 经常使用\作为doxygen,不要使用@
- 始终使用5x4空格(5个制表符)作为文本行开始的偏移量
static
type_t* list;
- 每个结构/枚举成员都必须包含文档
- 注释的开头使用12x4空格偏移量
typedef struct {
int32_t x;
int32_t y;
int32_t size;
} point_t;
typedef enum {
COLOR_RED,
COLOR_GREEN,
COLOR_BLUE,
} point_color_t;
- 函数的文档必须在函数实现中编写(通常是源文件)
- 函数必须包括简要和所有参数文档
- 如果每个参数分别为in或out输入和输出,则必须注意
- 如果函数返回某个值,则必须包含返回形参。这不适用于void函数
- 函数可以包含其他doxygen关键字,如note或warning
- 在参数名和描述之间使用冒号:
int32_t
sum(int32_t a, int32_t b) {
return a + b;
}
void
void_sum(int32_t a, int32_t b, int32_t* result) {
*result = a + b;
}
- 如果函数返回枚举的成员,则使用ref关键字指定哪个成员
typedef enum {
MY_ERR,
MY_OK
} my_enum_t;
my_enum_t
chec_value(void) {
return MY_OK;
}
- 对常量或数字使用符号(' NULL ' => NULL)
const void *
get_data(const void* in) {
return in;
}
- 宏的文档必须包括hideinitializer doxygen命令
头/源文件
- 在文件末尾留下一个空行
- 每个文件都必须包括文件的doxygen注释和后跟空行的简要描述(使用doxygen时)
- 每个文件(头文件或源文件)必须包含许可证(开始注释包括单个星号,因为doxygen必须忽略这个)
- 使用与项目/库已经使用的相同的许可证
- 头文件必须包含保护符#ifndef
- 头文件必须包含c++检查
- 在c++检查之外包含外部头文件
- 首先用STL C文件包含外部头文件,然后是应用程序自定义文件
- 头文件必须包含其他所有头文件,以便正确编译,但不能包含更多头文件(如果需要,.c应该包含其余的头文件)
- 头文件必须只公开模块公共变量/类型/函数
- 在头文件中使用extern作为全局模块变量,稍后在源文件中定义它们
extern int32_t my_variable;
int32_t my_variable;
- 不要把.c文件包含在另一个.c文件中
- .c文件应该首先包含相应的.h文件,然后是其他文件,除非另有明确的必要
- 在头文件中不包含模块私有声明
- 头文件示例(示例中没有license)
extern "C" {
}