From ff3a9591c02200894c1cd13eb575b57db420d6b8 Mon Sep 17 00:00:00 2001 From: Bernard Xiong Date: Fri, 9 Feb 2018 16:33:34 +0800 Subject: Update and rename coding_style_cn.txt to coding_style_cn.md --- documentation/coding_style_cn.md | 246 ++++++++++++++++++++++++++++++++++++++ documentation/coding_style_cn.txt | 222 ---------------------------------- 2 files changed, 246 insertions(+), 222 deletions(-) create mode 100644 documentation/coding_style_cn.md delete mode 100644 documentation/coding_style_cn.txt (limited to 'documentation') diff --git a/documentation/coding_style_cn.md b/documentation/coding_style_cn.md new file mode 100644 index 0000000000..0907723e73 --- /dev/null +++ b/documentation/coding_style_cn.md @@ -0,0 +1,246 @@ +# RT-Thread 编程风格 + +这是一份 RT-Thread 开发人员的开发指引。RT-Thread 做为一份开源软件，它需要由不同 +的人采用合作的方式完成，这份文档是开发人员的一个指引。RT-Thread 的开发人员请遵 +守这样的编程风格。同时对于使用 RT-Thread 的用户，也可通过这份文档了解 RT-Thread +代码内部一些约定从而比较容易的把握到 RT-Thread 的实现方式。 + +## 1.目录名称 + +目录名称如果无特殊的需求，请使用全小写的形式；目录名称应能够反应部分的意思，例 +如各芯片移植由其芯片名称构成或芯片类别构成；components 目录下能够反映组件的意义。 + +## 2.文件名称 + +文件名称如果无特殊的需求(如果是引用其他地方，可以保留相应的名称)，请使用全小写 +的形式。另外为了避免文件名重名的问题，一些地方请尽量不要使用通用化、使用频率高 +的名称。 + +## 3.头文件定义 + +C语言头文件为了避免多次重复包含，需要定义一个符号。这个符号的定义形式请采用如下 +的风格： + +```c + #ifndef __FILE_H__ + #define __FILE_H__ + /* header file content */ + #endif +``` + +即定义的符号两侧采用 "__" 以避免重名，另外也可以根据文件名中是否包含多个词语而 +采用 "_" 连接起来。 + +## 4.文件头注释 + +在每个源文件文件头上，应该包括相应的版权信息，Change Log 记录： + +```c + /* + * File : rtthread.h + * This file is part of RT-Thread RTOS + * COPYRIGHT (C) 2006 - 2012, RT-Thread Development Team + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + * + * Change Logs: + * Date Author Notes + * 2006-03-18 Bernard the first version + * 2006-04-26 Bernard add semaphore APIs + * … + */ +``` + +例如采用如上的形式。 + +## 5.结构体定义 + +结构体名称请使用小写英文名的形式，单词与单词之间采用 "_" 连接，例如： + +```c + struct rt_list_node + { + struct rt_list_node *next; + struct rt_list_node *prev; + }; +``` + +其中，"{"，"}" 独立占用一行，后面的成员定义使用缩进的方式定义。 + +结构体等的类型定义请以结构体名称加上 "_t" 的形式作为名称，例如： + +```c + typedef struct rt_list_node rt_list_t; +``` + +因为内核中对象引用方便的缘故，采用了对象内核指针作为类型定义的形式，例如： + +```c + typedef struct rt_timer* rt_timer_t; +``` + +## 6.宏定义 + +在RT-Thread中，请使用大写英文名称作为宏定义，单词之间使用 "_" 连接，例如： + +```c + #define RT_TRUE 1 +``` + +## 7.函数名称、声明 + +函数名称请使用小写英文的形式，单词之间使用 "_" 连接。提供给上层应用使用的 API +接口，必须在相应的头文件中声明；如果函数入口参数是空，必须使用 void 作为入口参 +数，例如： + +```c + rt_thread_t rt_thread_self(void); +``` + +## 8.注释编写 + +请使用英文做为注释，使用中文注释将意味着在编写代码时需要来回不停的切换中英文输 +入法从而打断编写代码的思路。并且使用英文注释也能够比较好的与中国以外的技术者进 +行交流。 + +源代码的注释不应该过多，更多的说明应该是代码做了什么，仅当个别关键点才需要一些 +相应提示性的注释以解释一段复杂的算法它是如何工作的。对语句的注释只能写在它的上 +方或右方，其他位置都是非法的。 + +## 9.缩进及分行 + +缩进请采用 4 个空格的方式。如果没有什么特殊意义，请在 "{" 后进行分行，并在下一 +行都采用缩进的方式，例如： + +```c + if (condition) + { + /* others */ + } +``` + +唯一的例外是 swtich 语句，switch-case 语句采用 case 语句与 swtich 对齐的方式， +例如： + +```c + switch (value) + { + case value1: + break; + } +``` + +case 语句与前面的 switch 语句对齐，后续的语句则采用缩进的方式。 + +分行上，如果没有什么特殊考虑，请**不要在代码中连续使用两个以上的空行**。 + +## 10.大括号与空格 + +从代码阅读角度，建议每个大括号单独占用一行，而不是跟在语句的后面，例如： + +```c + if (condition) + { + /* others */ + } +``` + +匹配的大括号单独占用一行，代码阅读起来就会有相应的层次而不会容易出现混淆的情况。 + +空格建议在非函数方式的括号调用前留一个空格以和前面的进行区分，例如： + +```c + if (x <= y) + { + /* others */ + } + + for (index = 0; index < MAX_NUMBER; index ++) + { + /* others */ + } +``` + +建议在括号前留出一个空格(涉及的包括 if、for、while、swtich 语句)，而运算表达式 +中，运算符与字符串间留一个空格。另外，不要在括号的表达式两侧留空格，例如： + +```c + if ( x <= y ) + { + /* other */ + } +``` + +这样括号内两侧的空格是不允许的。 + +## 11.trace、log信息 + +在 RT-Thread 中，普遍使用的 log 方式是 rt_kprintf。rt_kprintf 在 RT-Thread 被实 +现成一个采用轮询、非中断方式的字串输出，能够适合于在中断这类"即时"显示日志的场 +合。因为这种轮询方式的存在，也必然会影响到日志输出的时序关系。 + +建议在代码中不要频繁的使用 rt_kprintf 作为日志输出，除非你真正的明白，你的代码 +运行占用的时间多一些也没什么关系。 + +日志输出应该被设计成正常情况下是关闭状态(例如通过一个变量或宏就能够开启)，并且 +当真正输出日志时，日志是易懂易定位问题的方式。"天书式"的日志系统是糟糕的，不合 +理的。 + +## 12.函数 + +在内核编程中，函数应该尽量精简，仅完成相对独立的简单功能。函数的实现不应该太长 +，函数实现太长，应该反思能够如何修改(或拆分)使得函数更为精简、易懂。 + +## 13.对象 + +RT-Thread 内核采用了 C 语言对象化技术，命名表现形式是：对象名结构体表示类定义、 +对象名 + 动词短语形式表示类方法，例如： + +```c + struct rt_timer + { + struct rt_object parent; + /* other fields */ + }; + typedef struct rt_timer* rt_timer_t; +``` + +结构体定义 rt_timer 代表了 timer 对象的类定义； + +```c + rt_timer_t rt_timer_create(const char* name, + void (*timeout)(void* parameter), void* parameter, + rt_tick_t time, rt_uint8_t flag); + rt_err_t rt_timer_delete(rt_timer_t timer); + rt_err_t rt_timer_start(rt_timer_t timer); + rt_err_t rt_timer_stop(rt_timer_t timer); +``` + +rt_timer + 动词短语的形式表示能够应用于 timer 对象的方法。 + +在创建一个新的对象时，应该思考好，对象的内存操作处理：是否允许一个静态对象存在 +，或仅仅支持从堆中动态分配的对象。 + +## 14. 用 astyle 自动格式化代码 +参数：--style=allman + --indent=spaces=4 + --pad-oper + --pad-header + --unpad-paren + --suffix=none + --align-pointer=name + --lineend=linux + --convert-tabs + --verbose diff --git a/documentation/coding_style_cn.txt b/documentation/coding_style_cn.txt deleted file mode 100644 index 7119068f63..0000000000 --- a/documentation/coding_style_cn.txt +++ /dev/null @@ -1,222 +0,0 @@ - RT-Thread 编程风格 - -这是一份 RT-Thread 开发人员的开发指引。RT-Thread 做为一份开源软件，它需要由不同 -的人采用合作的方式完成，这份文档是开发人员的一个指引。RT-Thread 的开发人员请遵 -守这样的编程风格。同时对于使用 RT-Thread 的用户，也可通过这份文档了解 RT-Thread -代码内部一些约定从而比较容易的把握到 RT-Thread 的实现方式。 - - -1.目录名称 - -目录名称如果无特殊的需求，请使用全小写的形式；目录名称应能够反应部分的意思，例 -如各芯片移植由其芯片名称构成或芯片类别构成；components 目录下能够反映组件的意义 -。 - - -2.文件名称 - -文件名称如果无特殊的需求(如果是引用其他地方，可以保留相应的名称)，请使用全小写 -的形式。另外为了避免文件名重名的问题，一些地方请尽量不要使用通用化、使用频率高 -的名称。 - - -3.头文件定义 - -C语言头文件为了避免多次重复包含，需要定义一个符号。这个符号的定义形式请采用如下 -的风格： - - #ifndef __FILE_H__ - #define __FILE_H__ - /* header file content */ - #endif - -即定义的符号两侧采用 "__" 以避免重名，另外也可以根据文件名中是否包含多个词语而 -采用 "_" 连接起来。 - - -4.文件头注释 - -在每个源文件文件头上，应该包括相应的版权信息，Change Log 记录： - - /* - * File : rtthread.h - * This file is part of RT-Thread RTOS - * COPYRIGHT (C) 2006, RT-Thread Development Team - * - * The license and distribution terms for this file may be - * found in the file LICENSE in this distribution or at - * http://www.rt-thread.org/license/LICENSE. - * - * Change Logs: - * Date Author Notes - * 2006-03-18 Bernard the first version - * 2006-04-26 Bernard add semaphore APIs - * … - */ - -例如采用如上的形式。 - - -5.结构体定义 - -结构体名称请使用小写英文名的形式，单词与单词之间采用 "_" 连接，例如： - - struct rt_list_node - { - struct rt_list_node *next; - struct rt_list_node *prev; - }; - -其中，"{"，"}" 独立占用一行，后面的成员定义使用缩进的方式定义。 - -结构体等的类型定义请以结构体名称加上 "_t" 的形式作为名称，例如： - - typedef struct rt_list_node rt_list_t; - -因为内核中对象引用方便的缘故，采用了对象内核指针作为类型定义的形式，例如： - - typedef struct rt_timer* rt_timer_t; - - -6.宏定义 - -在RT-Thread中，请使用大写英文名称作为宏定义，单词之间使用 "_" 连接，例如： - - #define RT_TRUE 1 - - -7.函数名称、声明 - -函数名称请使用小写英文的形式，单词之间使用 "_" 连接。提供给上层应用使用的 API -接口，必须在相应的头文件中声明；如果函数入口参数是空，必须使用 void 作为入口参 -数，例如： - - rt_thread_t rt_thread_self(void); - - -8.注释编写 - -请使用英文做为注释，使用中文注释将意味着在编写代码时需要来回不停的切换中英文输 -入法从而打断编写代码的思路。并且使用英文注释也能够比较好的与中国以外的技术者进 -行交流。 - -源代码的注释不应该过多，更多的说明应该是代码做了什么，仅当个别关键点才需要一些 -相应提示性的注释以解释一段复杂的算法它是如何工作的。对语句的注释只能写在它的上 -方或右方，其他位置都是非法的。 - - -9.缩进 - -缩进请采用 TAB 字符或 4 个空格的方式进行，推荐使用4个空格的方式进行缩进。如果没 -有什么特殊意义，请在 "{" 的下一行都采用缩进的方式，例如： - - if (condition) - { - /* others */ - } - -唯一的例外是 swtich 语句，switch-case 语句采用 case 语句与 swtich 对齐的方式， -例如： - - switch (value) - { - case value1: - break; - } - -case 语句与前面的 switch 语句对齐，后续的语句则采用缩进的方式。 - - -10.大括号与空格 - -从代码阅读角度，建议每个大括号单独占用一行，而不是跟在语句的后面，例如： - - if (condition) - { - /* others */ - } - -匹配的大括号单独占用一行，代码阅读起来就会有相应的层次而不会容易出现混淆的情况 -。 - -空格建议在非函数方式的括号调用前留一个空格以和前面的进行区分，例如： - - if (x <= y) - { - /* others */ - } - - for (index = 0; index < MAX_NUMBER; index ++) - { - /* others */ - } - -建议在括号前留出一个空格(涉及的包括 if、for、while、swtich 语句)，而运算表达式 -中，运算符与字符串间留一个空格。另外，不要在括号的表达式两侧留空格，例如： - - if ( x <= y ) - { - /* other */ - } - -这样括号内两侧的空格是不允许的。 - - -11.trace、log信息 - -在 RT-Thread 中，普遍使用的 log 方式是 rt_kprintf。rt_kprintf 在 RT-Thread 被实 -现成一个采用轮询、非中断方式的字串输出，能够适合于在中断这类"即时"显示日志的场 -合。因为这种轮询方式的存在，也必然会影响到日志输出的时序关系。 - -建议在代码中不要频繁的使用 rt_kprintf 作为日志输出，除非你真正的明白，你的代码 -运行占用的时间多一些也没什么关系。 - -日志输出应该被设计成正常情况下是关闭状态(例如通过一个变量或宏就能够开启)，并且 -当真正输出日志时，日志是易懂易定位问题的方式。"天书式"的日志系统是糟糕的，不合 -理的。 - - -12.函数 - -在内核编程中，函数应该尽量精简，仅完成相对独立的简单功能。函数的实现不应该太长 -，函数实现太长，应该反思能够如何修改(或拆分)使得函数更为精简、易懂。 - - -13.对象 - -RT-Thread 内核采用了 C 语言对象化技术，命名表现形式是：对象名结构体表示类定义、 -对象名 + 动词短语形式表示类方法，例如： - - struct rt_timer - { - struct rt_object parent; - /* other fields */ - }; - typedef struct rt_timer* rt_timer_t; - -结构体定义 rt_timer 代表了 timer 对象的类定义； - - rt_timer_t rt_timer_create(const char* name, - void (*timeout)(void* parameter), void* parameter, - rt_tick_t time, rt_uint8_t flag); - rt_err_t rt_timer_delete(rt_timer_t timer); - rt_err_t rt_timer_start(rt_timer_t timer); - rt_err_t rt_timer_stop(rt_timer_t timer); - -rt_timer + 动词短语的形式表示能够应用于 timer 对象的方法。 - -在创建一个新的对象时，应该思考好，对象的内存操作处理：是否允许一个静态对象存在 -，或仅仅支持从堆中动态分配的对象。 - -14. 用 astyle 自动格式化代码 -参数：--style=allman - --indent=spaces=4 - --pad-oper - --pad-header - --unpad-paren - --suffix=none - --align-pointer=name - --lineend=linux - --convert-tabs - --verbose - -- cgit v1.3.1