1# C语言编程规范 2 3## <a name="c0-1"></a>目的 4规则并不是完美的,通过禁止在特定情况下有用的特性,可能会对代码实现造成影响。但是我们制定规则的目的“为了大多数程序员可以得到更多的好处”, 如果在团队运作中认为某个规则无法遵循,希望可以共同改进该规则。 5参考该规范之前,希望您具有相应的C语言基础能力,而不是通过该文档来学习C语言。 61. 了解C语言的ISO标准; 72. 熟知C语言的基本语言特性; 83. 了解C语言的标准库; 9 10## <a name="c0-2"></a>总体原则 11代码需要在保证功能正确的前提下,满足**可读、可维护、安全、可靠、可测试、高效、可移植**的特征要求。 12 13## <a name="c0-3"></a>约定 14 15**规则**:编程时必须遵守的约定 16**建议**:编程时必须加以考虑的约定 17 18无论是“规则”还是“建议”,都必须理解该条目这么规定的原因,并努力遵守。 19 20## <a name="c0-4"></a>例外 21 22在不违背总体原则,经过充分考虑,有充足的理由的前提下,可以适当违背规范中约定。 23例外破坏了代码的一致性,请尽量避免。“规则”的例外应该是极少的。 24 25下列情况,应风格一致性原则优先: 26**修改外部开源代码、第三方代码时,应该遵守开源代码、第三方代码已有规范,保持风格统一。** 27 28 29# <a name="c1"></a>1 命名 30 31命名包括文件、函数、变量、类型、宏等命名。 32 33命名被认为是软件开发过程中最困难,也是最重要的事情。 34标识符的命名要清晰、明了,有明确含义,符合阅读习惯,容易理解。 35 36统一的命名风格是一致性原则最直接的体现。 37 38 39## <a name="c1-1"></a>总体风格 40 41**驼峰风格(CamelCase)** 42大小写字母混用,单词连在一起,不同单词间通过单词首字母大写来分开。 43按连接后的首字母是否大写,又分: **大驼峰(UpperCamelCase)**和**小驼峰(lowerCamelCase)** 44 45**内核风格(unix_like)** 46内核风格又称蛇形风格。单词小写,用下划线分割。 47如:'test_result' 48 49### <a name="r1-1"></a>规则1.1 标识符命名使用驼峰风格 50 51| 类型 | 命名风格 | 52|-|-| 53| 函数,结构体类型,枚举类型,联合体类型 | 大驼峰 | 54| 变量,函数参数,宏参数,结构体中字段,联合体中成员 | 小驼峰 | 55| 宏,常量,枚举值,goto 标签 | 全大写,下划线分割 | 56 57注意: 58上表中`常量`是指,全局作用域下,const 修饰的基本数据类型、枚举、字符串类型的变量,不包括数组、结构体和联合体。 59上表中`变量`是指除常量定义以外的其他变量,均使用小驼峰风格。 60对于更亲和Linux/Unix的代码,可以使用内核风格。 61已使用内核命名风格的代码,可以选择继续使用内核风格。 62不管什么样的命名风格,都应该保证同一函数或结构体、联合体内的命名风格是一致的。 63 64### <a name="a1-1"></a>建议1.1 作用域越大,命名应越精确 65 66C 与 C++ 不同,没有名字空间,没有类,所以全局作用域下的标识符命名要考虑不要冲突。 67对于全局函数、全局变量、宏、类型名、枚举名的命名,应当精确描述并全局唯一。 68 69例: 70```c 71int GetCount(void); // Bad: 描述不精确 72int GetActiveConnectCount(void); // Good 73``` 74 75为了命名更精确,必要时可以增加模块前缀。 76模块前缀与命名主体之间,按驼峰方式连接。 77示例: 78```c 79int PrefixFuncName(void); // OK: 驼峰方式,形式上无前缀,内容上有前缀 80 81enum XxxMyEnum { // OK. 82 ... 83}; 84``` 85 86## <a name="c1-2"></a>文件命名 87 88### <a name="a1-2"></a>建议1.2 文件命名统一采用小写字符 89 90文件名命名只允许使用小写字母、数字以及下划线(\_)。 91文件名应尽量简短、准确、无二义性。 92不大小写混用的原因是,不同系统对文件名大小写处理会不同(如 MicroSoft 的 DOS, Windows 系统不区分大小写,但是 Unix / Linux, Mac 系统则默认区分)。 93 94好的命名举例: 95`dhcp_user_log.c` 96 97坏的命名举例: 98`dhcp_user-log.c`: 不推荐用'\-'分隔 99`dhcpuserlog.c`: 未分割单词,可读性差 100 101## <a name="c1-3"></a>函数命名 102 103函数命名统一使用大驼峰风格。 104 105### <a name="a1-3"></a>建议1.3 函数的命名遵循阅读习惯 106 107动作类函数名,可以使用动宾结构。如: 108```c 109AddTableEntry() // OK 110DeleteUser() // OK 111GetUserInfo() // OK 112``` 113判断型函数,可以用形容词,或加 is: 114```c 115DataReady() // OK 116IsRunning() // OK 117JobDone() // OK 118``` 119数据型函数: 120```c 121TotalCount() // OK 122GetTotalCount() // OK 123``` 124 125## <a name="c1-4"></a>变量命名 126 127变量命名使用小驼峰风格,包括全局变量,局部变量,函数声明或定义中的参数,带括号宏中的参数。 128 129### <a name="r1-2"></a>规则1.2 全局变量应增加 'g_' 前缀,函数内静态变量命名不需要加特殊前缀 130 131全局变量应当尽量少使用,使用时应特别注意,所以加上前缀用于视觉上的突出,促使开发人员对这些变量的使用更加小心。 132全局静态变量命名与全局变量相同,函数内的静态变量命名与普通局部变量相同。 133 134```c 135int g_activeConnectCount; 136 137void Func(void) 138{ 139 static int pktCount = 0; 140 ... 141} 142``` 143 144注意: 145常量本质也是全局变量,但如果命名风格是全大写,下划线连接的格式,则不适用当前规则。 146 147### <a name="a1-4"></a>建议1.4 局部变量应该简短,且能够表达相关含义 148 149函数局部变量的命名,在能够表达相关含义的前提下,应该简短。 150 151如下: 152```c 153int Func(...) 154{ 155 enum PowerBoardStatus powerBoardStatusOfSlot; // Not good: 局部变量有点长 156 powerBoardStatusOfSlot = GetPowerBoardStatus(slot); 157 if (powerBoardStatusOfSlot == POWER_OFF) { 158 ... 159 } 160 ... 161} 162``` 163更好的写法: 164```c 165int Func(...) 166{ 167 enum PowerBoardStatus status; // Good: 结合上下文,status 已经能明确表达意思 168 status = GetPowerBoardStatus(slot); 169 if (status == POWER_OFF) { 170 ... 171 } 172 ... 173} 174``` 175类似的, tmp 可以用来称呼任意类型的临时变量。 176过短的变量命名应慎用,但有时候,单字符变量也是允许的,如用于循环语句中的计数器变量: 177```c 178int i; 179... 180for (i = 0; i < COUNTER_RANGE; i++) { 181 ... 182} 183``` 184或一些简单的数学计算函数中的变量: 185```c 186int Mul(int a, int b) 187{ 188 return a * b; 189} 190``` 191 192## <a name="c1-5"></a>类型命名 193 194类型命名采用大驼峰命名风格。 195类型包括结构体、联合体、枚举类型名。 196 197例: 198```c 199struct MsgHead { 200 enum MsgType type; 201 int msgLen; 202 char *msgBuf; 203}; 204 205union Packet { 206 struct SendPacket send; 207 struct RecvPacket recv; 208}; 209 210enum BaseColor { 211 RED, // 注意,枚举类型是大驼峰,枚举值应使用宏风格 212 GREEN, 213 BLUE 214}; 215 216typedef int (*NodeCmpFunc)(struct Node *a, struct Node *b); 217``` 218 219通过 typedef 对结构体、联合体、枚举起别名时,尽量使用匿名类型。 220若需要指针自嵌套,可以增加 'tag' 前缀或下划线后缀。 221```c 222typedef struct { // Good: 无须自嵌套,使用匿名结构体 223 int a; 224 int b; 225} MyType; // 结构体别名用大驼峰风格 226``` 227```c 228typedef struct tagNode { // Good: 使用 tag 前缀。这里也可以使用 'Node_'代替也可以。 229 struct tagNode *prev; 230 struct tagNode *next; 231} Node; // 类型主体用大驼峰风格 232``` 233 234## <a name="c1-6"></a>宏、常量、枚举命名 235 236宏、枚举值采用全大写,下划线连接的格式。 237常量推荐采用全大写,下划线连接风格。作为全局变量,也可以保持与普通全局变量命名风格相同。 238这里常量如前文定义,是指基本数据类型、枚举、字符串类型的全局 const 变量。 239 240函数式宏,命名风格,采用全大写,下划线连接风格。 241例外情况: 2421、用宏实现泛型功能的函数。如:实现list,map等功能的宏。可以与函数的命名方式相同,使用大驼峰命名风格。 2432、函数接口发生变更为兼容老版本时,使用函数同宏进行替代。可以与函数的命名方式相同,使用大驼峰命名风格。 2443、日志打印宏。可以与函数的命名方式相同,使用大驼峰命名风格。 245注:使用大驼峰命名的函数式宏,需要在接口说明中标注为宏。 246 247宏举例: 248```c 249#define PI 3.14 250#define MAX(a, b) (((a) < (b)) ? (b) : (a)) 251``` 252```c 253#ifdef SOME_DEFINE 254void Bar(int); 255#define Foo(a) Bar(a) // 特殊场景,用大驼峰风格命名函数式宏 256#else 257void Foo(int); 258#endif 259``` 260 261常量举例: 262```c 263const int VERSION = 200; // OK. 264 265const enum Color DEFAULT_COLOR = BLUE; // OK. 266 267const char PATH_SEP = '/'; // OK. 268 269const char * const GREETINGS = "Hello, World!"; // OK. 270``` 271 272非常量举例: 273```c 274// 结构体类型,不符合常量定义 275const struct MyType g_myData = { ... }; // OK: 用小驼峰 276 277// 数组类型,不符合常量定义 278const int g_xxxBaseValue[4] = { 1, 2, 4, 8 }; // OK: 用小驼峰 279 280int Foo(...) 281{ 282 // 局部作用域,不符合常量定义 283 const int bufSize = 100; // OK: 用小驼峰 284 ... 285} 286``` 287 288枚举举例: 289```c 290// 注意,枚举类型名用大驼峰,其下面的取值是全大写,下划线相连 291enum BaseColor { 292 RED, 293 GREEN, 294 BLUE 295}; 296``` 297 298### <a name="a1-6"></a>建议1.5 避免函数式宏中的临时变量命名污染外部作用域 299 300首先,**定义函数式宏前,应考虑能否定义为函数。如果可以则不要定义为函数式宏。** 301 302当函数式宏需要定义局部变量时,为了防止跟外部函数中的局部变量有命名冲突。 303 304后置下双划线,是一种解决方案。 例: 305```c 306#define SWAP_INT(a, b) do { \ 307 int tmp__ = a; \ 308 a = b; \ 309 b = tmp__; \ 310} while (0) 311``` 312 313# <a name="c2"></a>2 排版格式 314 315## <a name="c2-1"></a>行宽 316 317### <a name="r2-1"></a>规则2.1 行宽不超过 120 个字符 318 319代码行宽不宜过长,否则不利于阅读。 320控制行宽长度可以间接的引导开发去缩短函数、变量的命名,减少嵌套的层数,提升代码可读性。 321要求每行字符数不要超过 **120** 个;除非超过 **120** 能显著增加可读性,并且不会隐藏信息。 322虽然现代显示器分辨率已经很高,但是行宽过长,反而提高了阅读理解的难度;跟本规范提倡的“清晰”、“简洁”原则相背。 323 324如下场景不宜换行,可以例外: 325- 换行会导致内容截断,无法被方便查找(grep)的字符串,如命令行或 URL 等等。包含这些内容的代码或注释,可以适当例外。 326- \#include / \#error 语句可以超出行宽要求,但是也需要尽量避免。 327 328例: 329```c 330#ifndef XXX_YYY_ZZZ 331#error Header aaaa/bbbb/cccc/abc.h must only be included after xxxx/yyyy/zzzz/xyz.h 332#endif 333``` 334## <a name="c2-2"></a>缩进 335 336### <a name="r2-2"></a>规则2.2 使用空格进行缩进,每次缩进4个空格 337 338只允许使用空格(space)进行缩进,每次缩进为 **4** 个空格。不允许使用Tab键进行缩进。 339当前几乎所有的集成开发环境(IDE)和代码编辑器都支持配置将Tab键自动扩展为**4**空格输入,请配置你的代码编辑器支持使用空格进行缩进。 340 341## <a name="c2-3"></a>大括号 342 343### <a name="r2-3"></a>规则2.3 使用 K&R 缩进风格 344 345**K&R风格** 346换行时,函数左大括号另起一行放行首,并独占一行;其他左大括号跟随语句放行末。 347右大括号独占一行,除非后面跟着同一语句的剩余部分,如 do 语句中的 while,或者 if 语句的 else/else if,或者逗号、分号。 348 349如: 350```c 351struct MyType { // Good: 跟随语句放行末,前置1空格 352 ... 353}; // Good: 右大括号后面紧跟分号 354 355int Foo(int a) 356{ // Good: 函数左大括号独占一行,放行首 357 if (...) { 358 ... 359 } else { // Good: 右大括号与 else 语句在同一行 360 ... 361 } // Good: 右大括号独占一行 362} 363``` 364 365## <a name="c2-4"></a>函数声明和定义 366 367### <a name="r2-4"></a>规则2.4 函数声明、定义的返回类型和函数名在同一行;函数参数列表换行时应合理对齐 368 369在声明和定义函数的时候,函数的返回值类型应该和函数名在同一行。 370 371函数参数列表换行时,应合理对齐。 372参数列表的左圆括号总是和函数名在同一行,不要单独一行;右圆括号总是跟随最后一个参数。 373 374换行举例: 375```c 376ReturnType FunctionName(ArgType paramName1, ArgType paramName2) // Good:全在同一行 377{ 378 ... 379} 380 381ReturnType VeryVeryVeryLongFunctionName(ArgType paramName1, // 行宽不满足所有参数,进行换行 382 ArgType paramName2, // Good:和上一行参数对齐 383 ArgType paramName3) 384{ 385 ... 386} 387 388ReturnType LongFunctionName(ArgType paramName1, ArgType paramName2, // 行宽限制,进行换行 389 ArgType paramName3, ArgType paramName4, ArgType paramName5) // Good: 换行后 4 空格缩进 390{ 391 ... 392} 393 394ReturnType ReallyReallyReallyReallyLongFunctionName( // 行宽不满足第1个参数,直接换行 395 ArgType paramName1, ArgType paramName2, ArgType paramName3) // Good: 换行后 4 空格缩进 396{ 397 ... 398} 399``` 400 401## <a name="c2-5"></a>函数调用 402 403### <a name="r2-5"></a>规则2.5 函数调用参数列表换行时保持参数进行合理对齐 404 405函数调用时,函数参数列表如果换行,应该进行合理的参数对齐。 406左圆括号总是跟函数名,右圆括号总是跟最后一个参数。 407 408换行举例: 409```c 410ReturnType result = FunctionName(paramName1, paramName2); // Good:函数参数放在一行 411 412ReturnType result = FunctionName(paramName1, 413 paramName2, // Good:保持与上方参数对齐 414 paramName3); 415 416ReturnType result = FunctionName(paramName1, paramName2, 417 paramName3, paramName4, paramName5); // Good:参数换行,4 空格缩进 418 419ReturnType result = VeryVeryVeryLongFunctionName( // 行宽不满足第1个参数,直接换行 420 paramName1, paramName2, paramName3); // 换行后,4 空格缩进 421``` 422 423如果函数调用的参数存在内在关联性,按照可理解性优先于格式排版要求,对参数进行合理分组换行。 424```c 425// Good:每行的参数代表一组相关性较强的数据结构,放在一行便于理解 426int result = DealWithStructureLikeParams(left.x, left.y, // 表示一组相关参数 427 right.x, right.y); // 表示另外一组相关参数 428``` 429 430## <a name="c2-6"></a>条件语句 431 432### <a name="r2-6"></a>规则2.6 条件语句必须要使用大括号 433 434我们要求条件语句都需要使用大括号,即便只有一条语句。 435理由: 436- 代码逻辑直观,易读; 437- 在已有条件语句代码上增加新代码时不容易出错; 438- 对于在条件语句中使用函数式宏时,没有大括号保护容易出错(如果宏定义时遗漏了大括号)。 439 440```c 441if (objectIsNotExist) { // Good:单行条件语句也加大括号 442 return CreateNewObject(); 443} 444``` 445 446### <a name="r2-7"></a>规则2.7 禁止 if/else/else if 写在同一行 447 448条件语句中,若有多个分支,应该写在不同行。 449 450如下是正确的写法: 451```c 452if (someConditions) { 453 ... 454} else { // Good: else 与 if 在不同行 455 ... 456} 457``` 458 459下面是不符合规范的案例: 460```c 461if (someConditions) { ... } else { ... } // Bad: else 与 if 在同一行 462``` 463 464## <a name="c2-7"></a>循环 465 466### <a name="r2-8"></a>规则2.8 循环语句必须使用大括号 467 468和条件表达式类似,我们要求for/while循环语句必须加上大括号,即便循环体是空的,或循环语句只有一条。 469```c 470for (int i = 0; i < someRange; i++) { // Good: 使用了大括号 471 DoSomething(); 472} 473``` 474```c 475while (condition) { } // Good:循环体是空,使用大括号 476``` 477```c 478while (condition) { 479 continue; // Good:continue 表示空逻辑,使用大括号 480} 481``` 482 483坏的例子: 484```c 485for (int i = 0; i < someRange; i++) 486 DoSomething(); // Bad: 应该加上括号 487``` 488```c 489while (condition); // Bad:使用分号容易让人误解是while语句中的一部分 490``` 491 492## <a name="c2-8"></a>switch语句 493 494### <a name="r2-9"></a>规则2.9 switch 语句的 case/default 要缩进一层 495 496switch 语句的缩进风格如下: 497```c 498switch (var) { 499 case 0: // Good: 缩进 500 DoSomething1(); // Good: 缩进 501 break; 502 case 1: { // Good: 带大括号格式 503 DoSomething2(); 504 break; 505 } 506 default: 507 break; 508} 509``` 510 511```c 512switch (var) { 513case 0: // Bad: case 未缩进 514 DoSomething(); 515 break; 516default: // Bad: default 未缩进 517 break; 518} 519``` 520 521## <a name="c2-9"></a>表达式 522 523### <a name="a2-1"></a>建议2.1 表达式换行要保持换行的一致性,操作符放行末 524 525较长的表达式,不满足行宽要求的时候,需要在适当的地方换行。一般在较低优先级操作符或连接符后面截断,操作符或连接符放在行末。 526操作符、连接符放在行末,表示“未结束,后续还有”。 527 528例: 529```c 530// 假设下面第一行已经不满足行宽要求 531if ((currentValue > MIN) && // Good:换行后,布尔操作符放在行末 532 (currentValue < MAX)) { 533 DoSomething(); 534 ... 535} 536 537int result = reallyReallyLongVariableName1 + // Good: 加号留在行末 538 reallyReallyLongVariableName2; 539``` 540 541表达式换行后,注意保持合理对齐,或者4空格缩进。参考下面例子 542```c 543int sum = longVariableName1 + longVariableName2 + longVariableName3 + 544 longVariableName4 + longVariableName5 + longVariableName6; // OK: 4空格缩进 545 546int sum = longVariableName1 + longVariableName2 + longVariableName3 + 547 longVariableName4 + longVariableName5 + longVariableName6; // OK: 保持对齐 548``` 549 550## <a name="c2-10"></a>变量赋值 551 552### <a name="r2-10"></a>规则2.10 多个变量定义和赋值语句不允许写在一行 553 554每行最好只有一个变量初始化的语句,更容易阅读和理解。 555 556```c 557int maxCount = 10; 558bool isCompleted = false; 559``` 560 561下面是不符合规范的示例: 562```c 563int maxCount = 10; bool isCompleted = false; // Bad:多个初始化放在了同一行 564int x, y = 0; // Bad:多个变量定义需要分行,每行一个 565 566int pointX; 567int pointY; 568... 569pointX = 1; pointY = 2; // Bad:多个变量赋值语句放同一行 570``` 571 572例外情况: 573对于多个相关性强的变量定义,且无需初始化时,可以定义在一行,减少重复信息,以便代码更加紧凑。 574```c 575int i, j; // Good:多变量定义,未初始化,可以写在一行 576for (i = 0; i < row; i++) { 577 for (j = 0; j < col; j++) { 578 ... 579 } 580} 581``` 582 583## <a name="c2-11"></a>初始化 584 585初始化包括结构体、联合体及数组的初始化 586 587### <a name="r2-11"></a>规则2.11 初始化换行时要有缩进,或进行合理对齐 588 589结构体或数组初始化时,如果换行应保持4空格缩进。 590从可读性角度出发,选择换行点和对齐位置。 591```c 592// Good: 满足行宽要求时不换行 593int arr[4] = { 1, 2, 3, 4 }; 594 595// Good: 行宽较长时,换行让可读性更好 596const int rank[] = { 597 16, 16, 16, 16, 32, 32, 32, 32, 598 64, 64, 64, 64, 32, 32, 32, 32 599}; 600``` 601 602对于复杂结构数据的初始化,尽量清晰、紧凑。 603参考如下格式: 604```c 605int a[][4] = { 606 { 1, 2, 3, 4 }, { 2, 2, 3, 4 }, // OK. 607 { 3, 2, 3, 4 }, { 4, 2, 3, 4 } 608}; 609 610int b[][8] = { 611 { 1, 2, 3, 4, 5, 6, 7, 8 }, // OK. 612 { 2, 2, 3, 4, 5, 6, 7, 8 } 613}; 614``` 615```c 616int c[][8] = { 617 { 618 1, 2, 3, 4, 5, 6, 7, 8 // OK. 619 }, { 620 2, 2, 3, 4, 5, 6, 7, 8 621 } 622}; 623``` 624 625注意: 626- 左大括号放行末时,对应的右大括号需另起一行 627- 左大括号被内容跟随时,对应的右大括号也应跟随内容 628 629### <a name="r2-12"></a>规则2.12 结构体和联合体在按成员初始化时,每个成员初始化单独一行 630 631C99标准支持结构体和联合体按照成员进行初始化,标准中叫"指定初始化"(designated initializer)。 如果按照这种方式进行初始化,每个成员的初始化单独一行。 632```c 633struct Date { 634 int year; 635 int month; 636 int day; 637}; 638 639struct Date date = { // Good:使用指定初始化方式时,每行初始化一个 640 .year = 2000, 641 .month = 1, 642 .day = 1 643}; 644``` 645 646## <a name="c2-12"></a>指针 647 648### <a name="a2-2"></a>建议2.2 指针类型"\*"跟随变量名或者类型,不要两边都留有空格或都没有空格 649 650声明或定义指针变量或者返回指针类型函数时,"\*" 靠左靠右都可以,但是不要两边都有或者都没有空格。 651```c 652int *p1; // OK. 653int* p2; // OK. 654 655int*p3; // Bad: 两边都没空格 656int * p4; // Bad: 两边都有空格 657``` 658 659选择一种风格,并保持一致性。 660 661选择"\*"跟随类型风格时,避免一行同时声明带指针的多个变量。 662```c 663int* a, b; // Bad: 很容易将 b 误理解成指针 664``` 665 666选择"\*"跟随变量风格时,可能会存在无法紧跟的情况。 667无法跟随时就不跟随,不要破坏风格一致性。 668```c 669char * const VERSION = "V100"; // OK. 670int Foo(const char * restrict p); // OK. 671``` 672注意,任何时候 "\*" 不要紧跟 const 或 restrict 关键字。 673 674## <a name="c2-13"></a>编译预处理 675 676### <a name="r2-13"></a>规则2.13 编译预处理的"#"默认放在行首,嵌套编译预处理语句时,"#"可以进行缩进 677 678编译预处理的"#"统一放在行首;即便编译预处理的代码是嵌入在函数体中的,"#"也应该放在行首。 679注意,开发过程尽量不要使用编译预处理宏。如果需使用,则应由专人进行统一管理。 680 681## <a name="c2-14"></a>空格和空行 682 683### <a name="r2-14"></a>规则2.14 水平空格应该突出关键字和重要信息,避免不必要的留白 684 685水平空格应该突出关键字和重要信息,每行代码尾部不要加空格。 总体规则如下: 686- if, switch, case, do, while, for 等关键字之后加空格; 687- 小括号内部的两侧,不要加空格 688- 二元操作符(= + ‐ < > * / % | & ^ <= >= == !=)左右两侧加空格 689- 一元操作符(& * + ‐ ~ !)之后不要加空格 690- 三目操作符(? :)符号两侧均需要空格 691- 结构体中表示位域的冒号,两侧均需要空格 692- 前置和后置的自增、自减(++ --)和变量之间不加空格 693- 结构体成员操作符(. ->)前后不加空格 694- 大括号内部两侧有无空格,左右必须保持一致 695- 逗号、分号、冒号(不含三目操作符和表示位域的冒号)紧跟前面内容无空格,其后需要空格 696- 函数参数列表的小括号与函数名之间无空格 697- 类型强制转换的小括号与被转换对象之间无空格 698- 数组的中括号与数组名之间无空格 699- 涉及到换行时,行末的空格可以省去 700 701对于大括号内部两侧的空格,**建议**如下: 702- 一般的,大括号内部两侧建议加空格 703- 对于空的,或单个标识符,或单个字面常量,空格不是必须 704如:'{}', '{0}', '{NULL}', '{"hi"}' 等 705- 连续嵌套的多重括号之间,空格不是必须 706如:'{{0}}', '{{ 1, 2 }}' 等 707错误示例:'{ 0, {1}}',不属于连续嵌套场景,而且最外侧大括号左右不一致 708 709常规情况: 710```c 711int i = 0; // Good:变量初始化时,= 前后应该有空格,分号前面不要留空格 712int buf[BUF_SIZE] = {0}; // Good:数组初始化时,大括号内空格可选 713int arr[] = { 10, 20 }; // Good: 正常大括号内部两侧建议加空格 714``` 715 716函数定义和函数调用: 717```c 718int result = Foo(arg1,arg2); 719 ^ // Bad: 逗号后面应该有空格 720 721int result = Foo( arg1, arg2 ); 722 ^ ^ // Bad: 小括号内部两侧不应该有空格 723``` 724 725指针和取地址 726```c 727x = *p; // Good:*操作符和指针p之间不加空格 728p = &x; // Good:&操作符和变量x之间不加空格 729x = r.y; // Good:通过.访问成员变量时不加空格 730x = r->y; // Good:通过->访问成员变量时不加空格 731``` 732 733操作符: 734```c 735x = 0; // Good:赋值操作的=前后都要加空格 736x = -5; // Good:负数的符号和数值之前不要加空格 737++x; // Good:前置和后置的++/--和变量之间不要加空格 738x--; 739 740if (x && !y) // Good:布尔操作符前后要加上空格,!操作和变量之间不要空格 741v = w * x + y / z; // Good:二元操作符前后要加空格 742v = w * (x + z); // Good:括号内的表达式前后不需要加空格 743``` 744 745循环和条件语句: 746```c 747if (condition) { // Good:if关键字和括号之间加空格,括号内条件语句前后不加空格 748 ... 749} else { // Good:else关键字和大括号之间加空格 750 ... 751} 752 753while (condition) {} // Good:while关键字和括号之间加空格,括号内条件语句前后不加空格 754 755for (int i = 0; i < someRange; ++i) { // Good:for关键字和括号之间加空格,分号之后加空格 756 ... 757} 758 759switch (var) { // Good: switch 关键字后面有1空格 760 case 0: // Good:case语句条件和冒号之间不加空格 761 ... 762 break; 763 ... 764 default: 765 ... 766 break; 767} 768``` 769 770注意:当前的集成开发环境(IDE)和代码编辑器都可以设置删除行尾的空格,请正确配置你的编辑器。 771 772### <a name="a2-3"></a>建议2.3 合理安排空行,保持代码紧凑 773 774减少不必要的空行,可以显示更多的代码,方便代码阅读。下面有一些建议遵守的规则: 775- 根据上下内容的相关程度,合理安排空行; 776- 函数内部、类型定义内部、宏内部、初始化表达式内部,不使用连续空行 777- 不使用连续 **3** 个空行,或更多 778- 大括号内的代码块首行之前和末行之后不要加空行。 779 780```c 781ret = DoSomething(); 782 783if (ret != OK) { // Bad: 返回值判断应该紧跟函数调用 784 return -1; 785} 786``` 787```c 788int Foo(void) 789{ 790 ... 791} 792 793 794 795int Bar(void) // Bad:最多使用连续2个空行。 796{ 797 ... 798} 799``` 800```c 801int Foo(void) 802{ 803 804 DoSomething(); // Bad:大括号内部首尾,不需要空行 805 ... 806 807} 808``` 809 810# <a name="c3"></a>3 注释 811 812一般的,尽量通过清晰的架构逻辑,好的符号命名来提高代码可读性;需要的时候,才辅以注释说明。 813注释是为了帮助阅读者快速读懂代码,所以要从读者的角度出发,**按需注释**。 814 815注释内容要简洁、明了、无二义性,信息全面且不冗余。 816 817**注释跟代码一样重要。** 818写注释时要换位思考,用注释去表达此时读者真正需要的信息。在代码的功能、意图层次上进行注释,即注释解释代码难以表达的意图,不要重复代码信息。 819修改代码时,也要保证其相关注释的一致性。只改代码,不改注释是一种不文明行为,破坏了代码与注释的一致性,让阅读者迷惑、费解,甚至误解。 820 821使用**英文**进行注释。 822 823必须要加注释说明场合如下(包含但不限于列举的场合): 8241、模块对外提供的接口头文件必须对函数进行注释。 8252、定义全局变量必须加注释。 8263、核心算法必须加注释。 8274、超过50行的函数必须加注释。 828 829## <a name="c3-1"></a>注释风格 830 831在 C 代码中,使用 `/*` `*/`和 `//` 都是可以的。 832按注释的目的和位置,注释可分为不同的类型,如文件头注释、函数头注释、代码注释等等; 833同一类型的注释应该保持统一的风格。 834 835注意:本文示例代码中,大量使用 '//' 后置注释只是为了更精确的描述问题,并不代表这种注释风格更好。 836 837## <a name="c3-2"></a>文件头注释 838 839### <a name="r3-1"></a>规则3.1 文件头注释必须包含版权许可 840 841/* 842 * Copyright (c) 2020 XXX 843 * Licensed under the Apache License, Version 2.0 (the "License"); 844 * you may not use this file except in compliance with the License. 845 * You may obtain a copy of the License at 846 * 847 * http://www.apache.org/licenses/LICENSE-2.0 848 * 849 * Unless required by applicable law or agreed to in writing, software 850 * distributed under the License is distributed on an "AS IS" BASIS, 851 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. 852 * See the License for the specific language governing permissions and 853 * limitations under the License. 854 */ 855 856## <a name="c3-3"></a>函数头注释 857 858### <a name="r3-2"></a>规则3.2 禁止空有格式的函数头注释 859 860并不是所有的函数都需要函数头注释; 861函数原型无法表达的信息,加函数头注释辅助说明; 862 863函数头注释统一放在函数声明或定义上方。 864选择使用如下风格之一: 865**使用'//'写函数头** 866```c 867// 单行函数头 868int Func1(void); 869 870// 多行函数头 871// 第二行 872int Func2(void); 873``` 874 875**使用'/\*' '\*/' 写函数头** 876```c 877/* 单行函数头 */ 878int Func1(void); 879 880/* 881 * 单行或多行函数头 882 * 第二行 883 */ 884int Func2(void); 885``` 886 887函数尽量通过函数名自注释,**按需**写函数头注释。 888不要写无用、信息冗余的函数头;不要写空有格式的函数头。 889 890函数头注释内容**可选**,但不限于:功能说明、返回值,性能约束、用法、内存约定、算法实现、可重入的要求等等。 891模块对外头文件中的函数接口声明,其函数头注释,应当将重要、有用的信息表达清楚。 892 893例: 894```c 895/* 896 * 返回实际写入的字节数,-1表示写入失败 897 * 注意,内存 buf 由调用者负责释放 898 */ 899int WriteString(char *buf, int len); 900``` 901 902坏的例子: 903```c 904/* 905 * 函数名:WriteString 906 * 功能:写入字符串 907 * 参数: 908 * 返回值: 909 */ 910int WriteString(char *buf, int len); 911``` 912上面例子中的问题: 913- 参数、返回值,空有格式没内容 914- 函数名信息冗余 915- 关键的 buf 由谁释放没有说清楚 916 917## <a name="c3-4"></a>代码注释 918 919### <a name="r3-3"></a>规则3.3 代码注释放于对应代码的上方或右边 920### <a name="r3-4"></a>规则3.4 注释符与注释内容间要有1空格;右置注释与前面代码至少1空格 921 922代码上方的注释,应该保持对应代码一样的缩进。 923选择并统一使用如下风格之一: 924**使用'//'** 925```c 926// 这是单行注释 927DoSomething(); 928 929// 这是多行注释 930// 第二行 931DoSomething(); 932``` 933**使用'/\*' '\*/'** 934```c 935/* 这是单行注释 */ 936DoSomething(); 937 938/* 939 * 这是单/多行注释 940 * 第二行 941 */ 942DoSomething(); 943``` 944 945代码右边的注释,与代码之间,至少留1空格,建议不超过4空格。 946通常使用扩展后的 TAB 键即可实现 1-4 空格的缩进。 947 948选择并统一使用如下风格之一: 949```c 950int foo = 100; // 放右边的注释 951int bar = 200; /* 放右边的注释 */ 952``` 953 954右置格式在适当的时候,上下对齐会更美观。 955对齐后的注释,离左边代码最近的那一行,保证1-4空格的间隔。 956例: 957```c 958#define A_CONST 100 /* 相关的同类注释,可以考虑上下对齐 */ 959#define ANOTHER_CONST 200 /* 上下对齐时,与左侧代码保持间隔 */ 960``` 961 962当右置的注释超过行宽时,请考虑将注释置于代码上方。 963 964### <a name="r3-5"></a>规则3.5 不用的代码段直接删除,不要注释掉 965 966被注释掉的代码,无法被正常维护;当企图恢复使用这段代码时,极有可能引入易被忽略的缺陷。 967正确的做法是,不需要的代码直接删除掉。若再需要时,考虑移植或重写这段代码。 968 969这里说的注释掉代码,包括用 /\* \*/ 和 //,还包括 #if 0, #ifdef NEVER_DEFINED 等等。 970 971### <a name="a3-1"></a>建议3.1 case语句块结束时如果不加break/return,需要有注释说明(fall-through) 972 973有时候需要对多个case标签做相同的事情,case语句在结束不加break或return,直接执行下一个case标签中的语句,这在C语法中称之为"fall-through"。 974这种情况下,需要在"fall-through"的地方加上注释,清晰明确的表达出这样做的意图;或者至少显式指明是 "fall-through"。 975 976例,显式指明 fall-through: 977```c 978switch (var) { 979 case 0: 980 DoSomething(); 981 /* fall-through */ 982 case 1: 983 DoSomeOtherThing(); 984 ... 985 break; 986 default: 987 DoNothing(); 988 break; 989} 990``` 991 992如果 case 语句是空语句,则可以不用加注释特别说明: 993```c 994switch (var) { 995 case 0: 996 case 1: 997 DoSomething(); 998 break; 999 default: 1000 DoNothing(); 1001 break; 1002} 1003``` 1004 1005# <a name="c4"></a>4 头文件 1006 1007**对于C语言来说,头文件的设计体现了大部分的系统设计**。 1008正确使用头文件可使代码在可读性、文件大小和编译构建性能上大为改观。 1009 1010本章从编程规范的角度总结了一些方法,可用于帮助合理规划头文件。 1011 1012## <a name="c4-1"></a>头文件职责 1013 1014头文件是模块或文件的对外接口。 1015头文件中适合放置接口的声明,不允许放置实现(内联函数除外)。 1016头文件应当职责单一。头文件过于复杂,依赖过于复杂还是导致编译时间过长的主要原因。 1017 1018### <a name="a4-1"></a>建议4.1 每一个.c文件都应该有相应的.h文件,用于声明需要对外公开的接口 1019 1020通常情况下,每个.c文件都有一个相应的.h(并不一定同名),用于放置对外提供的函数声明、宏定义、类型定义等。 1021如果一个.c文件不需要对外公布任何接口,则其就不应当存在。 1022 1023例外:程序的入口(如main函数所在的文件),单元测试代码,动态库代码。 1024 1025示例: 1026foo.h 内容 1027```c 1028#ifndef FOO_H 1029#define FOO_H 1030 1031int Foo(void); // Good:头文件中声明对外接口 1032 1033#endif 1034``` 1035 1036foo.c 内容 1037```c 1038static void Bar(void); // Good: 对内函数的声明放在.c文件的头部,并声明为static限制其作用域 1039 1040void Foo(void) 1041{ 1042 Bar(); 1043} 1044 1045static void Bar(void) 1046{ 1047 // Do something; 1048} 1049``` 1050 1051内部使用的函数声明,宏、枚举、结构体等定义不应放在头文件中。 1052 1053有些产品中,习惯一个.c文件对应两个.h文件,一个用于存放对外公开的接口,一个用于存放内部需要用到的定义、声明等,以控制.c文件的代码行数。 1054不提倡这种风格,产生这种风格的根源在于.c过大,应当首先考虑拆分.c文件。 1055另外,一旦把私有定义、声明放到独立的头文件中,就无法从技术上避免别人包含。 1056 1057本规则反过来并不一定成立。比如: 1058有些特别简单的头文件,如命令 ID 定义头文件,不需要有对应的.c存在。 1059同一套接口协议下,有多个实例,由于接口相同且稳定,所以允许出现一个.h对应多个.c文件。 1060 1061### <a name="a4-2"></a>建议4.2 头文件的扩展名只使用.h,不使用非习惯用法的扩展名,如.inc 1062 1063有些产品中使用了 .inc 作为头文件扩展名,这不符合C语言的习惯用法。 1064在使用 .inc 作为头文件扩展名的产品,习惯上用于标识此头文件为私有头文件。 1065但是从产品的实际代码来看,这一条并没有被遵守,一个 .inc 文件被多个 .c 包含。 1066本规范不提倡将私有定义单独放在头文件中,具体见[建议4.1](#a4-1)。 1067 1068## <a name="c4-2"></a>头文件依赖 1069 1070头文件包含是一种依赖关系,头文件应向稳定的方向包含。 1071一般来说,应当让不稳定的模块依赖稳定的模块,从而当不稳定的模块发生变化时,不会影响(编译)稳定的模块。 1072 1073依赖的方向应该是:产品依赖于平台,平台依赖于标准库。 1074 1075除了不稳定的模块依赖于稳定的模块外,更好的方式是每个模块都依赖于接口,这样任何一个模块的内部实现更改都不需要重新编译另外一个模块。 1076在这里,假设接口本身是最稳定的。 1077 1078### <a name="r4-1"></a>规则4.1 禁止头文件循环依赖 1079 1080头文件循环依赖,指 a.h 包含 b.h,b.h 包含 c.h,c.h 包含 a.h, 导致任何一个头文件修改,都导致所有包含了a.h/b.h/c.h的代码全部重新编译一遍。 1081而如果是单向依赖,如a.h包含b.h,b.h包含c.h,而c.h不包含任何头文件,则修改a.h不会导致包含了b.h/c.h的源代码重新编译。 1082 1083头文件循环依赖直接体现了架构设计上的不合理,可通过架构优化来避免。 1084 1085 1086### <a name="r4-4"></a>规则4.2 头文件必须编写#define保护,防止重复包含 1087 1088为防止头文件被多重包含,所有头文件都应当使用 #define 作为包含保护;不要使用 #pragma once 1089 1090定义包含保护符时,应该遵守如下规则: 1091- 保护符使用唯一名称;统一使用子系统名_部件名_文件名的定义规则。 1092- 不要在受保护部分的前后放置代码或者注释,文件头注释除外。 1093 1094假定 util 子系统 timer 部件的 timer.h,其目录为 `timer/include/timer.h`。其保护符若使用 'TIME_H' 很容易不唯一,按规则定义如: 1095```c 1096#ifndef UTIL_TIMER_TIMER_H 1097#define UTIL_TIMER_TIMER_H 1098 1099... 1100 1101#endif // UTIL_TIMER_TIMER_H 1102``` 1103 1104### <a name="r4-5"></a>规则4.3 禁止通过声明的方式引用外部函数接口、变量 1105 1106只能通过包含头文件的方式使用其他模块或文件提供的接口。 1107通过 extern 声明的方式使用外部函数接口、变量,容易在外部接口改变时可能导致声明和定义不一致。 1108同时这种隐式依赖,容易导致架构腐化。 1109 1110不符合规范的案例: 1111a.c 内容 1112```c 1113extern int Foo(void); // Bad: 通过 extern 的方式引用外部函数 1114 1115void Bar(void) 1116{ 1117 int i = Foo(); // 这里使用了外部接口 Foo 1118 ... 1119} 1120``` 1121 1122应该改为: 1123a.c 内容 1124```c 1125#include "b.h" // Good: 通过包含头文件的方式使用其他.c提供的接口 1126 1127void Bar(void) 1128{ 1129 int i = Foo(); 1130 ... 1131} 1132``` 1133 1134b.h 内容 1135```c 1136int Foo(void); 1137``` 1138 1139b.c内容 1140```c 1141int Foo(void) 1142{ 1143 // Do something 1144} 1145``` 1146 1147例外,有些场景需要引用其内部函数,但并不想侵入代码时,可以 extern 声明方式引用。 1148如: 1149针对某一内部函数进行单元测试时,可以通过 extern 声明来引用被测函数; 1150当需要对某一函数进行打桩、打补丁处理时,允许 extern 声明该函数。 1151 1152### <a name="r4-6"></a>规则4.4 禁止在 extern "C" 中包含头文件 1153 1154在 extern "C" 中包含头文件,有可能会导致 extern "C" 嵌套,部分编译器对 extern "C" 嵌套层次有限制,嵌套层次太多会编译错误。 1155 1156extern "C" 通常出现在 C,C++ 混合编程的情况下,在 extern "C" 中包含头文件,可能会导致被包含头文件的原有意图遭到破坏,比如链接规范被不正确地更改。 1157 1158示例,存在a.h和b.h两个头文件: 1159a.h 内容 1160```c 1161... 1162#ifdef __cplusplus 1163void Foo(int); 1164#define A(value) Foo(value) 1165#else 1166void A(int) 1167#endif 1168``` 1169 1170b.h 内容 1171```c 1172... 1173#ifdef __cplusplus 1174extern "C" { 1175#endif 1176 1177#include "a.h" 1178void B(void); 1179 1180#ifdef __cplusplus 1181} 1182#endif 1183``` 1184 1185使用C++预处理器展开b.h,将会得到 1186```c 1187extern "C" { 1188 void Foo(int); 1189 void B(void); 1190} 1191``` 1192 1193按照 a.h 作者的本意,函数 Foo 是一个 C++ 自由函数,其链接规范为 "C++"。 1194但在 b.h 中,由于 `#include "a.h"` 被放到了 `extern "C"` 的内部,函数 Foo 的链接规范被不正确地更改了。 1195 1196例外: 1197如果在 C++ 编译环境中,想引用纯C的头文件,这些C头文件并没有 `extern "C"` 修饰。非侵入式的做法是,在 `extern "C"` 中去包含C头文件。 1198 1199# <a name="c5"></a>5 函数 1200 1201函数的作用:避免重复代码、增加可重用性;分层,降低复杂度、隐藏实现细节,使程序更加模块化,从而更有利于程序的阅读,维护。 1202 1203函数应该简洁、短小。 1204一个函数只完成一件事情。 1205 1206## <a name="c5-1"></a>函数设计 1207 1208函数设计的精髓:编写整洁函数,同时把代码有效组织起来。代码简单直接、不隐藏设计者的意图、用干净利落的抽象和直截了当的控制语句将函数有机组织起来。 1209 1210### <a name="r5-1"></a>规则5.1 避免函数过长,函数不超过50行(非空非注释) 1211 1212函数应该可以一屏显示完 (50行以内),只做一件事情,而且把它做好。 1213 1214过长的函数往往意味着函数功能不单一,过于复杂,或过分呈现细节,未进行进一步抽象。 1215 1216例外: 1217考虑代码的聚合性与功能的全面性,某些函数可能会超过50行,但前提是不影响代码的可读性与简洁。 1218这些例外的函数应该是极少的,例如特定算法处理。 1219 1220即使一个长函数现在工作的非常好, 一旦有人对其修改, 有可能出现新的问题, 甚至导致难以发现的bug。 1221建议将其拆分为更加简短并易于管理的若干函数,以便于他人阅读和修改代码。 1222 1223### <a name="r5-2"></a>规则5.2 避免函数的代码块嵌套过深,不要超过4层 1224 1225函数的代码块嵌套深度指的是函数中的代码控制块(例如:if、for、while、switch等)之间互相包含的深度。 1226每级嵌套都会增加阅读代码时的脑力消耗,因为需要在脑子里维护一个“栈”(比如,进入条件语句、进入循环等等)。 1227应该做进一步的功能分解,从而避免使代码的阅读者一次记住太多的上下文。 1228 1229使用`卫语句`可以有效的减少 if 相关的嵌套层次。例: 1230原代码嵌套层数是 3: 1231```c 1232int Foo(...) 1233{ 1234 if (received) { 1235 type = GetMsgType(msg); 1236 if (type != UNKNOWN) { 1237 return DealMsg(...); 1238 } 1239 } 1240 return -1; 1241} 1242``` 1243 1244使用`卫语句`重构,嵌套层数变成 2: 1245```c 1246int Foo(...) 1247{ 1248 if (!received) { // Good: 使用'卫语句' 1249 return -1; 1250 } 1251 1252 type = GetMsgType(msg); 1253 if (type == UNKNOWN) { 1254 return -1; 1255 } 1256 1257 return DealMsg(..); 1258} 1259``` 1260例外: 1261考虑代码的聚合性与功能的全面性,某些函数嵌套可能会超过4层,但前提是不影响代码的可读性与简洁。 1262这些例外的函数应该是极少的。 1263 1264### <a name="a5-1"></a>建议5.1 对函数的错误返回码要全面处理 1265 1266一个函数(标准库中的函数/第三方库函数/用户定义的函数)能够提供一些指示错误发生的方法。 1267这可以通过使用错误标记、特殊的返回数据或者其他手段,不管什么时候函数提供了这样的机制,调用程序应该在函数返回时立刻检查错误指示。 1268 1269示例: 1270```c 1271char fileHead[128]; 1272ReadFileHead(fileName, fileHead, sizeof(fileHead)); // Bad: 未检查返回值 1273 1274DealWithFileHead(fileHead, sizeof(fileHead)); // fileHead 可能无效 1275``` 1276 1277正确写法: 1278```c 1279char fileHead[128]; 1280ret = ReadFileHead(fileName, fileHead, sizeof(fileHead)); 1281if (ret != OK) { // Good: 确保 fileHead 被有效写入 1282 return ERROR; 1283} 1284 1285DealWithFileHead(fileHead, sizeof(fileHead)); // 处理文件头 1286``` 1287 1288 1289注意,当函数返回值被大量的显式(void)忽略掉时,应当考虑函数返回值的设计是否合理。 1290如果所有调用者都不关注函数返回值时,请将函数设计成`void`型。 1291 1292## <a name="c5-2"></a>函数参数 1293 1294### <a name="a5-2"></a>建议5.2 设计函数时,优先使用返回值而不是输出参数 1295 1296使用返回值而不是输出参数,可以提高可读性,并且通常提供相同或更好的性能。 1297 1298函数名为 GetXxx、FindXxx、IsXxx、OnXxx或直接名词作函数名的函数,直接返回对应对象,可读性更好。 1299例外: 13001、多值返回时,可以设计为输出参数返回。 13012、涉及内存分配时,可以设计为输出参数返回。调用者将申请的内存做为出参传入,而函数内由不再分配内存。 1302 1303### <a name="a5-3"></a>建议5.3 设计函数的参数时,统一按输入、输出、出入的顺序定义参数。 1304 1305函数参数的定义统一按输入参数、输出参数、出入参的顺序进行定义。 1306 1307### <a name="r5-3"></a>规则5.3 设计函数的资源时,涉及内存、锁、队列等资源分配的,需要同时提供释放函数。 1308 1309本着资源从那儿申请,向那儿释放的原则。如果函数申请了内存、锁、队列等资源,则模块需要同时提供资源的函数。 1310 1311### <a name="a5-4"></a>建议5.4 使用强类型参数,避免使用void\* 1312 1313尽管不同的语言对待强类型和弱类型有自己的观点,但是一般认为c/c++是强类型语言,既然我们使用的语言是强类型的,就应该保持这样的风格。 1314好处是尽量让编译器在编译阶段就检查出类型不匹配的问题。 1315 1316使用强类型便于编译器帮我们发现错误,如下代码中注意函数 `FooListAddNode` 的使用: 1317```c 1318struct FooNode { 1319 struct List link; 1320 int foo; 1321}; 1322 1323struct BarNode { 1324 struct List link; 1325 int bar; 1326} 1327 1328void FooListAddNode(void *node) // Bad: 这里用 void * 类型传递参数 1329{ 1330 FooNode *foo = (FooNode *)node; 1331 ListAppend(&g_fooList, &foo->link); 1332} 1333 1334void MakeTheList(...) 1335{ 1336 FooNode *foo; 1337 BarNode *bar; 1338 ... 1339 1340 FooListAddNode(bar); // Wrong: 这里本意是想传递参数 foo,但错传了 bar,却没有报错 1341} 1342 1343``` 1344 1345上述问题有可能很隐晦,不易轻易暴露,从而破坏性更大。 1346如果明确 `FooListAddNode` 的参数类型,而不是 `void *`,则在编译阶段就能发现上述问题。 1347```c 1348void FooListAddNode(FooNode *foo) 1349{ 1350 ListAppend(&g_fooList, &foo->link); 1351} 1352``` 1353 1354例外:某些通用泛型接口,需要传入不同类型指针的,可以用 `void *` 入参。 1355 1356### <a name="a5-5"></a>建议5.5 模块内部函数参数的合法性检查,由调用者负责 1357 1358对于模块外部传入的参数,必须进行合法性检查,保护程序免遭非法输入数据的破坏。 1359模块内部函数调用,缺省由调用者负责保证参数的合法性,如果都由被调用者来检查参数合法性,可能会出现同一个参数,被检查多次,产生冗余代码,很不简洁。 1360 1361由调用者保证入参的合法性,这种契约式编程能让代码逻辑更简洁,可读性更好。 1362示例: 1363```c 1364int SomeProc(...) 1365{ 1366 int data; 1367 1368 bool dataOK = GetData(&data); // 获取数据 1369 if (!dataOK) { // 检查上一步结果,其实也就保证了数据合法 1370 return -1; 1371 } 1372 1373 DealWithData(data); // 调用数据处理函数 1374 ... 1375} 1376 1377void DealWithData(int data) 1378{ 1379 if (data < MIN || data > MAX) { // Bad: 调用者已经保证了数据合法性 1380 return; 1381 } 1382 1383 ... 1384} 1385``` 1386 1387### <a name="a5-5"></a>建议5.5 函数的指针参数如果不是用于修改所指向的对象就应该声明为指向const的指针 1388 1389const 指针参数,将限制函数通过该指针修改所指向对象,使代码更牢固、安全。 1390 1391示例:C99标准 7.21.4.4 中strncmp 的例子,不变参数声明为const。 1392```c 1393int strncmp(const char *s1, const char *s2, size_t n); // Good:不变参数声明为const 1394``` 1395 1396注意: 1397指针参数要不要加 const 取决于函数设计,而不是看函数实体内有没有发生“修改对象”的动作。 1398 1399### <a name="a5-6"></a>建议5.6 函数的参数个数不超过5个 1400 1401函数的参数过多,会使得该函数易于受外部(其他部分的代码)变化的影响,从而影响维护工作。函数的参数过多同时也会增大测试的工作量。 1402 1403函数的参数个数不要超过5个,如果超过可以考虑: 1404- 看能否拆分函数 1405- 看能否将相关参数合在一起,定义结构体 1406 1407## <a name="c5-3"></a>内联函数 1408 1409内联函数是**C99**引入的一种函数优化手段。函数内联能消除函数调用的开销;并得益于内联实现跟调用点代码的合并,编译器有更大的视角,从而完成更多的代码优化。内联函数跟函数式宏比较类似,两者的分析详见[建议6.1](#a6-1)。 1410 1411### <a name="a5-7"></a>建议5.7 内联函数不超过10行(非空非注释) 1412 1413将函数定义成内联一般希望提升性能,但是实际并不一定能提升性能。 1414如果函数体短小,则函数内联可以有效的缩减目标代码的大小,并提升函数执行效率。 1415反之,函数体比较大,内联展开会导致目标代码的膨胀,特别是当调用点很多时,膨胀得更厉害,反而会降低执行效率。 1416内联函数规模建议控制在 **10** 行以内。 1417 1418不要为了提高性能而滥用内联函数。不要过早优化。一般情况,当有实际测试数据证明内联性能更高时,再将函数定义为内联。 1419对于类似 setter/getter 短小而且调用频繁的函数,可以定义为内联。 1420 1421### <a name="r5-3"></a>规则5.3 被多个源文件调用的内联函数要放在头文件中定义 1422 1423内联函数是在编译时内联展开,因此要求内联函数定义必须在调用此函数的每个源文件内可见。 1424如下所示代码,inline.h 只有`SomeInlineFunc`函数的声明而没有定义。other.c包含inline.h,调用`SomeInlineFunc`时无法内联。 1425 1426inline.h 1427```c 1428inline int SomeInlineFunc(void); 1429``` 1430 1431inline.c 1432```c 1433inline int SomeInlineFunc(void) 1434{ 1435 // 实现代码 1436} 1437``` 1438 1439other.c 1440```c 1441#include "inline.h" 1442int OtherFunc(void) 1443{ 1444 int ret = SomeInlineFunc(); 1445} 1446``` 1447 1448由于这个限制,多个源文件如果要调用同一个内联函数,需要将内联函数的定义放在头文件中。 1449**gnu89** 在内联函数实现上跟**C99**标准有差异,兼容做法是将函数声明为 **static inline**。 1450 1451# <a name="c6"></a>6 宏 1452 1453## <a name="c6-1"></a>函数式宏(function-like macro) 1454 1455函数式宏是指形如函数的宏(示例代码如下所示),其包含若干条语句来实现某一特定功能。 1456```c 1457#define ASSERT(x) do { \ 1458 if (!(x)) { \ 1459 printk(KERN_EMERG "assertion failed %s: %d: %s\n", \ 1460 __FILE__, __LINE__, #x); \ 1461 BUG(); \ 1462 } \ 1463} while (0) 1464``` 1465 1466### <a name="a6-1"></a>建议6.1 使用函数代替函数式宏 1467 1468定义函数式宏前,应考虑能否用函数替代。对于可替代场景,建议用函数替代宏。 1469函数式宏的缺点如下: 1470 1471- 函数式宏缺乏类型检查,不如函数调用检查严格。示例代码[见下](#macro_lack_of_type_check__example)。 1472- 宏展开时宏参数不求值,可能会产生非预期结果,详见[规则6.1](#r6-1)和[规则6.3](#r6-3)。 1473- 宏没有独立的作用域,跟控制流语句配合时,可能会产生如[规则6.2](#r6-2)描述的非预期结果。 1474- 宏的技巧性太强(参见下面的规则),例如`#`的用法和无处不在的括号,影响可读性。 1475- 在特定场景下必须用特定编译器对宏的扩展,如 `gcc` 的 `statement expression`,可移植性也不好。 1476- 宏在预编译阶段展开后,在其后编译、链接和调试时都不可见;而且包含多行的宏会展开为一行。函数式宏难以调试、难以打断点,不利于定位问题。 1477- 对于包含大量语句的宏,在每个调用点都要展开。如果调用点很多,会造成代码空间的膨胀。 1478 1479<a name="macro_lack_of_type_check__example"></a>函数式宏缺乏类型检查的示例代码: 1480 1481```c 1482#define MAX(a, b) (((a) < (b)) ? (b) : (a)) 1483 1484int Max(int a, int b) 1485{ 1486 return (a < b) ? b : a; 1487} 1488 1489int TestMacro(void) 1490{ 1491 unsigned int a = 1; 1492 int b = -1; 1493 1494 (void)printf("MACRO: max of a(%u) and b(%d) is %d\n", a, b, MAX(a, b)); 1495 (void)printf("FUNC : max of a(%u) and b(%d) is %d\n", a, b, Max(a, b)); 1496 return 0; 1497} 1498``` 1499 1500由于宏缺乏类型检查,`MAX`中的`a`和`b`的比较提升为无符号数的比较,结果是**a < b**。输出结果是: 1501 1502``` 1503MACRO: max of a(1) and b(-1) is -1 1504FUNC : max of a(1) and b(-1) is 1 1505``` 1506 1507函数没有宏的上述缺点。但是,函数相比宏,最大的劣势是执行效率不高(增加函数调用的开销和编译器优化的难度)。 1508为此,C99标准引入了内联函数(gcc在标准之前就引入了内联函数)。 1509 1510内联函数跟宏类似,也是在调用点展开。不同之处在于内联函数是在编译时展开。 1511内联函数兼具函数和宏的优点: 1512 1513- 内联函数/函数执行严格的类型检查 1514- 内联函数/函数的入参求值只会进行一次 1515- 内联函数就地展开,没有函数调用的开销 1516- 内联函数比函数优化得更好 1517 1518对于性能敏感的代码,可以考虑用内联函数代替函数式宏。 1519函数和内联函数不能完全替代函数式宏,函数式宏在某些场景更适合。 1520比如,在日志记录场景下,使用带可变参和默认参数的函数式宏更方便: 1521 1522```c 1523int ErrLog(const char *file, unsigned long line, const char *fmt, ...); 1524#define ERR_LOG(fmt, ...) ErrLog(__FILE__, __LINE__, fmt, ##__VA_ARGS__) 1525``` 1526 1527 1528### <a name="r6-1"></a>规则6.1 定义宏时,宏参数要使用完备的括号 1529 1530宏参数在宏展开时只是文本替换,在编译时再求值。文本替换后,宏包含的语句跟调用点代码合并。 1531合并后的表达式因为操作符的优先级和结合律,可能会导致计算结果跟期望的不同,尤其是当宏参数在一个表达式中时。 1532 1533如下所示,是一种错误的写法: 1534```c 1535#define SUM(a, b) a + b // Bad. 1536``` 1537 1538下面这样调用宏,执行结果跟预期不符: 1539`100 / SUM(2, 8)` 将扩展成 `(100 / 2) + 8`,预期结果则是`100 / (2 + 8)`。 1540这个问题可以通过将整个表示式加上括号来解决,如下所示: 1541```c 1542#define SUM(a, b) (a + b) // Bad. 1543``` 1544 1545但是这种改法在下面这种场景又有问题: 1546`SUM(1 << 2, 8)`扩展成`1 << (2 + 8)`(因为`<<`优先级低于`+`),跟预期结果`(1 << 2) + 8`不符。 1547 1548这个问题可以通过将每个宏参数都加上括号来解决,如下所示: 1549```c 1550#define SUM(a, b) (a) + (b) // Bad. 1551``` 1552 1553再看看第三种问题场景:`SUM(2, 8) * 10` 。扩展后的结果为 `(2) + ((8) * 10)`,跟预期结果`(2 + 8) * 10`不符。 1554 1555综上所述,正确的写法如下: 1556```c 1557#define SUM(a, b) ((a) + (b)) // Good. 1558``` 1559 1560但是要避免滥用括号。如下所示,单独的数字或标识符加括号毫无意义。 1561```c 1562#define SOME_CONST 100 // Good: 单独的数字无需括号 1563#define ANOTHER_CONST (-1) // Good: 负数需要使用括号 1564 1565#define THE_CONST SOME_CONST // Good: 单独的标识符无需括号 1566``` 1567 1568下列情况需要注意: 1569- 宏参数参与 '#', '##' 操作时,不要加括号 1570- 宏参数参与字符串拼接时,不要加括号 1571- 宏参数作为独立部分,在赋值(包括+=, -=等)操作的某一边时,无需括号 1572- 宏参数作为独立部分,在逗号表达式,函数或宏调用列表中,无需括号 1573 1574举例: 1575```c 1576#define MAKE_STR(x) #x // x 不要加括号 1577 1578#define HELLO_STR(obj) "Hello, " obj // obj 不要加括号 1579 1580#define ADD_3(sum, a, b, c) (sum = (a) + (b) + (c)) // a, b, c 需要括号;而 sum 无需括号 1581 1582#define FOO(a, b) Bar((a) + 1, b) // a 需要括号;而 b 无需括号 1583``` 1584 1585 1586### <a name="r6-2"></a>规则6.2 包含多条语句的函数式宏的实现语句必须放在 do-while(0) 中 1587 1588宏本身没有代码块的概念。当宏在调用点展开后,宏内定义的表达式和变量融合到调用代码中,可能会出现变量名冲突和宏内语句被分割等问题。 1589通过 do-while(0) 显式为宏加上边界,让宏有独立的作用域,并且跟分号能更好的结合而形成单条语句,从而规避此类问题。 1590 1591如下所示的宏是错误的用法(为了说明问题,下面示例代码稍不符规范): 1592 1593```c 1594// Not Good. 1595#define FOO(x) \ 1596 (void)printf("arg is %d\n", (x)); \ 1597 DoSomething((x)); 1598``` 1599 1600当像下面示例代码这样调用宏,for循环只执行了宏的第一条语句,宏的后一条语句只在循环结束后执行一次。 1601 1602```c 1603for (i = 1; i < 10; i++) 1604 FOO(i); 1605``` 1606 1607用大括号将`FOO`定义的语句括起来可以解决上面的问题: 1608 1609```c 1610#define FOO(x) { \ 1611 (void)printf("arg is %d\n", (x)); \ 1612 DoSomething((x)); \ 1613} 1614``` 1615 1616由于大括号跟分号没有关联。大括号后紧跟的分号,是另外一个语句。 1617如下示例代码,会出现'悬挂else' 编译报错: 1618 1619```c 1620if (condition) 1621 FOO(10); 1622else 1623 FOO(20); 1624``` 1625 1626正确的写法是用 do-while(0) 把执行体括起来,如下所示: 1627 1628```c 1629// Good. 1630#define FOO(x) do { \ 1631 (void)printf("arg is %d\n", (x)); \ 1632 DoSomething((x)); \ 1633} while (0) 1634``` 1635 1636例外: 1637- 包含 break, continue 语句的宏可以例外。使用此类宏务必特别小心。 1638- 宏中包含不完整语句时,可以例外。比如用宏封装 for 循环的条件部分。 1639- 非多条语句,或单个 if/for/while/switch 语句,可以例外。 1640 1641### <a name="r6-3"></a>规则6.3 不允许把带副作用的表达式作为参数传递给函数式宏 1642 1643由于宏只是文本替换,对于内部多次使用同一个宏参数的函数式宏,将带副作用的表达式作为宏参数传入会导致非预期的结果。 1644如下所示,宏`SQUARE`本身没有问题,但是使用时将带副作用的`a++`传入导致`a`的值在`SQUARE`执行后跟预期不符: 1645 1646```c 1647#define SQUARE(a) ((a) * (a)) 1648 1649int a = 5; 1650int b; 1651b = SQUARE(a++); // Bad: 实际 a 自增加了 2 次 1652``` 1653 1654`SQUARE(a++)`展开后为`((a++) * (a++))`,变量`a`自增了两次,其值为`7`,而不是预期的`6`。 1655 1656正确的写法如下所示: 1657 1658```c 1659b = SQUARE(a); 1660a++; // 结果:a = 6,只自增了一次。 1661``` 1662 1663此外,如果参数包含函数调用,宏展开后,函数可能会被重复调用。 1664如果函数执行结果相同,则存在浪费;如果函数多次调用结果不一样,执行结果可能不符合预期。 1665 1666 1667### <a name="a6-2"></a>建议6.2 函数式宏定义中慎用 return、goto、continue、break 等改变程序流程的语句 1668 1669宏中使用 return、goto、continue、break 等改变流程的语句,虽然能简化代码,但同时也隐藏了真实流程,不易于理解,容易导致资源泄漏等问题。 1670 1671首先,宏封装 return 容易导致过度封装和使用。 1672如下代码,`status`的判断是主干流程的一部分,用宏封装起来后,变得不直观了,阅读时习惯性把`RETURN_IF`宏忽略掉了,从而导致对主干流程的理解有偏差。 1673 1674```c 1675#define LOG_AND_RETURN_IF_FAIL(ret, fmt, ...) do { \ 1676 if ((ret) != OK) { \ 1677 (void)ErrLog(fmt, ##__VA_ARGS__); \ 1678 return (ret); \ 1679 } \ 1680} while (0) 1681 1682#define RETURN_IF(cond, ret) do { \ 1683 if (cond) { \ 1684 return (ret); \ 1685 } \ 1686} while (0) 1687 1688ret = InitModuleA(a, b, &status); 1689LOG_AND_RETURN_IF_FAIL(ret, "Init module A failed!"); // OK. 1690 1691RETURN_IF(status != READY, ERR_NOT_READY); // Bad: 重要逻辑不明显 1692 1693ret = InitModuleB(c); 1694LOG_AND_RETURN_IF_FAIL(ret, "Init module B failed!"); // OK. 1695``` 1696 1697其次,宏封装 return 也容易引发内存泄漏。再看一个例子: 1698```c 1699#define CHECK_PTR(ptr, ret) do { \ 1700 if ((ptr) == NULL) { \ 1701 return (ret); \ 1702 } \ 1703} while (0) 1704 1705... 1706 1707mem1 = MemAlloc(...); 1708CHECK_PTR(mem1, ERR_CODE_XXX); 1709 1710mem2 = MemAlloc(...); 1711CHECK_PTR(mem2, ERR_CODE_XXX); // Wrong: 内存泄漏 1712``` 1713 1714如果 `mem2` 申请内存失败了,`CHECK_PTR` 会直接返回,而没有释放 `mem1`。 1715除此之外,`CHECK_PTR` 宏命名也不好,宏名只反映了检查动作,没有指明结果。只有看了宏实现才知道指针为空时返回失败。 1716 1717综上所述: 1718不推荐宏定义中封装 return、goto、continue、break 等改变程序流程的语句; 1719对于返回值判断等异常处理场景可以例外。 1720 1721注意: 1722**包含 return、goto、continue、break 等改变流程语句的宏命名,务必要体现对应关键字。** 1723 1724 1725### <a name="a6-3"></a>建议6.3 函数式宏不超过10行(非空非注释) 1726 1727函数式宏本身的一大问题是比函数更难以调试和定位,特别是宏过长,调试和定位的难度更大。 1728而且宏扩展会导致目标代码的膨胀。建议函数式宏不要超过10行。 1729 1730# <a name="c7"></a>7 变量 1731 1732在C语言编码中,除了函数,最重要的就是变量。 1733变量在使用时,应始终遵循“职责单一”原则。 1734按作用域区分,变量可分为全局变量和局部变量。 1735 1736## <a name="c7-1"></a>全局变量 1737 1738尽量不用或少用全局变量。 1739在程序设计中,全局变量是在所有作用域都可访问的变量。通常,使用不必要的全局变量被认为是坏习惯。 1740 1741使用全局变量的缺点: 1742- 破坏函数的独立性和可移植性,使函数对全局变量产生依赖,存在耦合; 1743- 降低函数的代码可读性和可维护性。当多个函数读写全局变量时,某一时刻其取值可能不是确定的,对于代码的阅读和维护不利; 1744- 在并发编程环境中,使用全局变量会破坏函数的可重入性,需要增加额外的同步保护处理才能确保数据安全。 1745 1746如不可避免,对全局变量的读写应集中封装。 1747 1748### <a name="r7-1"></a>规则7.1 模块间,禁止使用全局变量作接口 1749 1750全局变量是模块内部的具体实现,不推荐但允许跨文件使用,但禁止作为模块接口暴露出去。 1751对全局变量的使用应该尽量集中,如果本模块的数据需要对外部模块开放,应提供对应函数接口。 1752 1753## <a name="c7-2"></a>局部变量 1754 1755### <a name="r7-2"></a>规则7.2 严禁使用未经初始化的变量 1756 1757这里的变量,指的是局部动态变量,并且还包括内存堆上申请的内存块。 1758因为他们的初始值都是不可预料的,所以禁止未经有效初始化就直接读取其值。 1759 1760```c 1761void Foo(...) 1762{ 1763 int data; 1764 Bar(data); // Bad: 未初始化就使用 1765 ... 1766} 1767``` 1768 1769如果有不同分支,要确保所有分支都得到初始化后才能使用: 1770```c 1771void Foo(...) 1772{ 1773 int data; 1774 if (...) { 1775 data = 100; 1776 } 1777 Bar(data); // Bad: 部分分支该值未初始化 1778 ... 1779} 1780``` 1781 1782未经初始化就使用,一般静态检查工具是可以检查出来的。 1783如 PCLint 工具,针对上述两个例子分别会报错: 1784>Warning 530: Symbol 'data' (line ...) not initialized 1785>Warning 644: Variable 'data' (line ...) may not have been initialized 1786 1787### <a name="r7-3"></a>规则7.3 禁止无效、冗余的变量初始化 1788 1789如果没有确定的初始值,而仍然进行初始化,不仅不简洁,反而不安全,可能会引入更难发现的问题。 1790 1791常见的冗余初始化: 1792```c 1793int cnt = 0; // Bad: 冗余初始化,将会被后面直接覆盖 1794... 1795cnt = GetXxxCnt(); 1796... 1797``` 1798 1799对于后续有条件赋值的变量,可以在定义时初始化成默认值 1800```c 1801char *buf = NULL; // Good: 这里用 NULL 代表默认值 1802if (condition) { 1803 buf = malloc(MEM_SIZE); 1804} 1805... 1806if (buf != NULL) { // 判断是否申请过内存 1807 free(buf); 1808} 1809``` 1810 1811针对大数组的冗余清零,更是会影响到性能。 1812```c 1813char buf[VERY_BIG_SIZE] = {0}; 1814memset(buf, 0, sizeof(buf)); // Bad: 冗余清零 1815``` 1816 1817无效初始化,隐藏更大问题的反例: 1818```c 1819void Foo(...) 1820{ 1821 int data = 0; // Bad: 习惯性的进行初始化 1822 1823 UseData(data); // 使用数据,本应该写在获取数据后面 1824 data = GetData(...); // 获取数据 1825 ... 1826} 1827``` 1828上例代码,如果没有赋 0 初始化,静态检查工具可以帮助发现“未经初始化就直接使用”的问题。 1829但因为无效初始化,“使用数据”与“获取数据”写颠倒的缺陷,不能被轻易发现。 1830 1831因此,应该写简洁的代码,对变量或内存块进行正确、必要的初始化。 1832 1833C99不再限制局部变量定义必须在语句之前,可以按需定义,即在靠近变量使用的地方定义变量。 1834这种简洁的做法,不仅将变量作用域限制更小,而且更方便阅读和维护,还能解决定义变量时不知该怎么初始化的问题。 1835如果编译环境支持,建议按需定义。 1836 1837**例外:** 1838**遵从“安全规范”要求,指针变量、表示资源描述符的变量、BOOL变量不作要求。** 1839 1840### <a name="r7-4"></a>规则7.4 不允许使用魔鬼数字 1841 1842所谓魔鬼数字即看不懂、难以理解的数字。 1843魔鬼数字并非一个非黑即白的概念,看不懂也有程度,需要结合代码上下文和业务相关知识来判断 1844 1845例如数字 12,在不同的上下文中情况是不一样的: 1846`type = 12;` 就看不懂,但 `month = year * 12;` 就能看懂。 1847数字 0 有时候也是魔鬼数字,比如 `status = 0;` 并不能表达是什么状态。 1848 1849解决途径: 1850对于单点使用的数字,可以增加注释说明 1851对于多处使用的数字,必须定义宏或const 变量,并通过符号命名自注释。 1852 1853禁止出现下列情况: 1854没有通过符号来解释数字含义,如 `#define ZERO 0` 1855符号命名限制了其取值,如 `#define XX_TIMER_INTERVAL_300MS 300` 1856 1857# <a name="c8"></a>8 编程实践 1858 1859## <a name="c8-1"></a>表达式 1860 1861### <a name="a8-1"></a>建议8.1 表达式的比较,应当遵循左侧倾向于变化、右侧倾向于不变的原则 1862 1863当变量与常量比较时,如果常量放左边,如 `if (MAX == v)` 不符合阅读习惯,而 `if (MAX > v)` 更是难于理解。 1864应当按人的正常阅读、表达习惯,将常量放右边。写成如下方式: 1865```c 1866if (v == MAX) ... 1867if (v < MAX) ... 1868``` 1869 1870也有特殊情况,如:`if (MIN < v && v < MAX)` 用来描述区间时,前半段是常量在左的。 1871 1872不用担心将 '==' 误写成 '=',因为 `if (v = MAX)` 会有编译告警,其他静态检查工具也会报错。让工具去解决笔误问题,代码要符合可读性第一。 1873 1874### <a name="r8-1"></a>规则8.1 含有变量自增或自减运算的表达式中禁止再次引用该变量 1875 1876含有变量自增或自减运算的表达式中,如果再引用该变量,其结果在C标准中未明确定义。各个编译器或者同一个编译器不同版本实现可能会不一致。 1877为了更好的可移植性,不应该对标准未定义的运算次序做任何假设。 1878 1879注意,运算次序的问题不能使用括号来解决,因为这不是优先级的问题。 1880 1881示例: 1882```c 1883x = b[i] + i++; // Bad: b[i]运算跟 i++,先后顺序并不明确。 1884``` 1885正确的写法是将自增或自减运算单独放一行: 1886```c 1887x = b[i] + i; 1888i++; // Good: 单独一行 1889``` 1890 1891函数参数: 1892```c 1893Func(i++, i); // Bad: 传递第2个参数时,不确定自增运算有没有发生 1894``` 1895正确的写法: 1896```c 1897i++; // Good: 单独一行 1898x = Func(i, i); 1899``` 1900 1901 1902### <a name="a8-2"></a>建议8.2 用括号明确表达式的操作顺序,避免过分依赖默认优先级 1903 1904可以使用括号强调表达式操作顺序,防止因默认的优先级与设计思想不符而导致程序出错。 1905然而过多的括号会分散代码使其降低了可读性,应适度使用。 1906 1907当表达式包含不常用,优先级易混淆的操作符时,推荐使用括号,比如位操作符: 1908```c 1909c = (a & 0xFF) + b; /* 涉及位操作符,需要括号 */ 1910``` 1911 1912## <a name="c8-2"></a>语句 1913 1914### <a name="r8-2"></a>规则8.2 switch语句要有default分支 1915 1916大部分情况下,switch语句中要有default分支,保证在遗漏case标签处理时能够有一个缺省的处理行为。 1917 1918特例: 1919如果switch条件变量是枚举类型,并且 case 分支覆盖了所有取值,则加上default分支处理有些多余。 1920现代编译器都具备检查是否在switch语句中遗漏了某些枚举值的case分支的能力,会有相应的warning提示。 1921```c 1922enum Color { 1923 RED, 1924 BLUE 1925}; 1926 1927// 因为switch条件变量是枚举值,这里可以不用加default处理分支 1928switch (color) { 1929 case RED: 1930 DoRedThing(); 1931 break; 1932 case BLUE: 1933 DoBlueThing(); 1934 ... 1935 break; 1936} 1937``` 1938 1939### <a name="a8-3"></a>建议8.3 慎用 goto 语句 1940 1941goto语句会破坏程序的结构性,所以除非确实需要,最好不使用goto语句。使用时,也只允许跳转到本函数goto语句之后的语句。 1942 1943goto语句通常用来实现函数单点返回。 1944同一个函数体内部存在大量相同的逻辑但又不方便封装成函数的情况下,譬如反复执行文件操作, 1945对文件操作失败以后的处理部分代码(譬如关闭文件句柄,释放动态申请的内存等等), 1946一般会放在该函数体的最后部分,在需要的地方就goto到那里,这样代码反而变得清晰简洁。 1947实际也可以封装成函数或者封装成宏,但是这么做会让代码变得没那么直接明了。 1948 1949示例: 1950```c 1951// Good: 使用 goto 实现单点返回 1952int SomeInitFunc(void) 1953{ 1954 void *p1; 1955 void *p2 = NULL; 1956 void *p3 = NULL; 1957 1958 p1 = malloc(MEM_LEN); 1959 if (p1 == NULL) { 1960 goto EXIT; 1961 } 1962 1963 p2 = malloc(MEM_LEN); 1964 if (p2 == NULL) { 1965 goto EXIT; 1966 } 1967 1968 p3 = malloc(MEM_LEN); 1969 if (p3 == NULL) { 1970 goto EXIT; 1971 } 1972 1973 DoSomething(p1, p2, p3); 1974 return 0; // OK. 1975 1976EXIT: 1977 if (p3 != NULL) { 1978 free(p3); 1979 } 1980 if (p2 != NULL) { 1981 free(p2); 1982 } 1983 if (p1 != NULL) { 1984 free(p1); 1985 } 1986 return -1; // Failed! 1987} 1988``` 1989 1990## <a name="c8-3"></a>类型转换 1991 1992### <a name="a8-4"></a>建议8.4 尽量减少没有必要的数据类型默认转换与强制转换 1993 1994当进行数据类型强制转换时,其数据的意义、转换后的取值等都有可能发生变化,而这些细节若考虑不周,就很有可能留下隐患。 1995 1996如下赋值,多数编译器不产生告警,但值的含义还是稍有变化。 1997```c 1998char ch; 1999unsigned short int exam; 2000 2001ch = -1; 2002exam = ch; // Bad: 编译器不产生告警,此时exam为0xFFFF。 2003``` 2004