负笈前行

每个公司都会有自己代码风格或者编程规范，都旨在防范编程语言的一些陷阱或者提高代码效率，还有就是保持一致编码风格来提高代码可读性，方便code review；

或者说代码的一种美学，比如python也就有pep8约定，还有一些精致编码来表示你是否Pythonic。

DPDK也有自己的编码风格，看完颇有受益，这里提取一些个人比较喜欢的风格来参考，尤其是关于结构体部分。

1、注释风格，参考doxygen的使用

单行注释：

/*

* VERY important single-line comments look like this.

*/

或者

/* Most single-line comments look like this. */

多行注释：

/*

* Multi-line comments look like this. Make them real sentences. Fill

* them so they look like real paragraphs.

*/

2、头文件包含

标准库和dpdk提供的库头文件都使用<>包含，其他应用程序的头文件都使用 ''包含

1 #include 2 #include 3 #include 'application.h'

3、预编译头

1 #ifndef _FILE_H_2 #define _FILE_H_3 /* Code */4 #endif /* _FILE_H_ */

4、结构体定义

（1）声明结构体成员变量的时候，首先按方便使用排序，然后按字节大小排序（从大到小），然后再按字典表排序，并且考虑最小的字节填充来保证字节对齐。

按方便使用排序是意思是把常用的成员变量放在一起，突出整个结构体的逻辑性

（2）对于已经声明好的结构体，如果要添加新的成员，应该添加在结构体的尾部来保证向后兼容，但我个人觉得这个在遵守第(1)条的情况下，才考虑这条。

（3）结构体的每个元素都应该在单独的一行。

（4）成员变量的类型起始对齐，变量名使用空格对齐第一个字符。

（5）对于类型特别长的变量，不方便对齐，就用一个空格的间隔书写。

1 struct foo {2 struct foo *next; /* List of active foo. */3 struct mumble amumble; /* Comment for mumble. */4 int bar; /* Try to align the comments. */5 struct verylongtypename *baz; /* Won't fit with other members */6 };

（6）结构体声明放在所使用文件的代码起始部分，或者在一个单独的头文件里，方便其他文件包含。

（7）变量的声明应该与结构体的声明分开，如果在头文件声明变量，应该使用extern。

（8）结构体的定义要避免名字空间冲突，需要加上特有的前缀，比如dpdk是 rte_ 。

5、使用linux的queue macro会比自己实现的list rolling要好。

1 #include 2 struct foo {3 LIST_ENTRY(foo) link; /* Use queue macros for foo lists. */4 struct mumble amumble; /* Comment for mumble. */5 int bar; /* Try to align the comments. */6 struct verylongtypename *baz; /* Won't fit with other members */7 };8 LIST_HEAD(, foo) foohead; /* Head of global foo list. */

6、typedef的使用

避免使用typedef来声明结构体类型。

如下声明

1 struct my_struct_type {2 /* ... */3 };4 struct my_struct_type my_var;

比下面的好：

typedef struct my_struct_type {/* ... */} my_struct_type;my_struct_type my_var

（1）typedef的问题是它会不恰当的隐藏原有的结构体类型。比如，你不知道它define的是结构体自身还是结构体的指针（有人可能会说通过类型+前缀/后缀来区分，众口难调，我也不喜欢那种如 int_ptr 之类的写法）

（2）typedef只能声明一次，而结构体可以通过不完整的形式声明多次，比如 struct my_struct_type；

（3）要使用typedef的类型必须包含它所在的头文件，这个可能造成名词空间污染。

（4）不要使用define来替代typedef，会引起更多的问题，比如 #define pint int * 就会出现连续的变量声明问题。

（5）推荐使用typedef来声明函数指针，可以提高代码可读性

1 /**2 * Definition of a remote launch function.3 */4 typedef int (lcore_function_t)(void *);5 /* launch a function of lcore_function_t type */6 int rte_eal_remote_launch(lcore_function_t *f, void *arg, unsigned slave_id);