空间管理您的位置: 51Testing软件测试网 » pioneer of living space.......... » 日志

我们拒绝平庸，拒绝随波逐流，拒绝墨守成规，让梦想不再流浪。

C&C++编程军规

上一篇 / 下一篇 2014-06-23 15:53:41 / 个人分类：编程

查看( 1919 ) / 评论( 0 ) / 评分( 0 / 0 )

军规条例

oO&[^&?a-oPcmd051Testing软件测试网f`9sQ&a

2eJ$G m0D%I4z7ky9L6S0军规一：【禁止使用未经初始化的变量作为右值】

zwwGhmz1hqd0军规二：【禁止使用memset，memcpy直接对类实例进行初始化和拷贝】

军规三：【防止出现数组、指针内存访问越界，确保字符串以 ’\0’结束】

$ro(CpvpO h0军规四：【防止使用空指针和野指针】

军规五：【及时释放内存，避免重复释放内存】

军规六：【禁止返回函数中的局部变量的指针】

军规七：【防止类型转换溢出和精度丢失】

军规八：【多线程多进程访问共享资源时要防止重入和死锁】

6TI3X\JC`0军规九：【头文件包含防止循环依赖，防止包含不同版本的头文件】

.e0X"A R4Kn9h~,}-Y0军规十：【使用清晰简洁的表达式，加括号防止优先级混淆】

51Testing软件测试网8r"IiWl2ag

4GGU5aY"w/b051Testing软件测试网,JYS:j,I9q

*uR Jd3tLG@L0

军规说明

51Testing软件测试网!q-KN"_6pE

!|$xZ ?vz t e0军规一：【禁止使用未经初始化的变量作为右值】

+BM4b$Pr(Uc9b E0包括全局变量，局部变量和类成员变量等，变量必须赋有意义的初值。

对于类成员变量，必须在构造函数或专门的初始化函数中进行初始化。

[F d.~)L0错误示例：

VOS_UINT8 udwVADOption; >>>修正方案：此处进行初始化赋初值 51Testing软件测试网/G*W:N6^u5D7A

switch (ucRscType) 51Testing软件测试网 yWx+A8sL\B

nno4f)H2B*[[2Y0{

%e RN0nuzD0case MRP_VPU_SUB_RSC_TC: 51Testing软件测试网DIa:vW+S

udwVADOption = g_stMrpAtcParaTblRec.udwVADOption; 51Testing软件测试网w6y6HtE'C

1M*M7D,U$CQKC,P0break; 51Testing软件测试网/C5i'b2v/V-r!x

GQ.K#z;vT_X k-?0case MRP_VPU_SUB_RSC_CC: 51Testing软件测试网1h5PJk1N0`

{3J_ |rO @0udwVADOption = g_stMrpCcParaTblRec.udwVADSwitch;

break;

default: --> 进default分支将导致相应的局部变量没有赋值 51Testing软件测试网Rvki]`n

/*目前没有其他资源支持*/

break;

.n+VGv ]3Q0} 51Testing软件测试网w5JE6@ZO(?7g

5pCv+NZ$n0if (VOS_NULL_BYTE == pstIeInfo->ucVadPt)

9D}9R+l8{{/^5@4g0{

;vxX}+W]9w.Ft0pstIeInfo->ucVadPt = udwVADOption; --> 这里使用就有问题了 51Testing软件测试网/{&j5`,Sc!a6jw

|!eXO Mr-BS&ru0}

详见：

《C规范》“规则4.3 严禁使用未经初始化的变量作为右值”，51Testing软件测试网yPg'v)yLA%Q

《C++规范》“2.1 声明、定义与初始化”51Testing软件测试网g+x5J/Zi0A

军规二：【禁止使用memset，memcpy直接对类实例进行初始化和拷贝】51Testing软件测试网W5D1feQ\

C++的class必须实现对成员变量的具体操作。

~/kD/DQv#T!H0POD（Plain Old Data）是普通旧式数据，如结构/枚举/成员指针等，内存字节是连续的。非POD对象的内存字节不连续，中间可能包含vptr(虚函数指针表)等隐藏的数据。

错误的做法：使用memset(this, 0, sizeof(*this)) 对初始化对象，vptr被破坏，赋为0。删除对象实例时，通过vptr指针访问析构函数，导致coredump

正确做法：取消memset操作，按照变量声明的顺序一个个初始化对象的成员变量。

C++编译器经常会在多态对象中嵌入一些隐藏数据（如vptr），多重继承/虚拟继承会添加更多的内部指针。不要直接对C++的对象进行memset/memcpy等操作，应尽量使用类自身提供的赋值/复制功能。

!@'r cu9|7w0详见：51Testing软件测试网:@3C!vw$r?1^'}

H"b7K9A:@3fHM'J0《C++规范》“规则2.1 禁止用memcpy、memset初始化非POD对象”51Testing软件测试网M7l#\:Y'uH x

军规三：【防止数组、指针内存访问越界，确保字符串以 ’\0’结束】

数组、指针分配空间足够大小，操作时防止越界。

字符串要使用安全的函数来操作。

OJ7\#y!@.y`^};P0错误示例：

X1PVh[0说明：opt->paramlen是信元长度，是可变长的。

{%KUT{/~g0VOS_MemCpy(stCallRefNum.number, opt->param, opt->paramlen);

修正方案：

先判断opt->paramlen长度，避免超过stCallRefNum.number长度。

8hSK8D4d"P0在转换时，代码会直接根据对信元长度进行内存拷贝操作；

正常情况下，对局的信元长度是正常的，不会有问题。 51Testing软件测试网:jU7nu.S$?0W

异常情况：如果长度超长，则会发生内存越界，导致故障。51Testing软件测试网K/_.^6h#?@8}

错误示例：

\C;NSt cxV3R0pErrMsgBuffer = VOS_Malloc(MODULE_ID_MAPPERDRV, dwBufSize + 1);

|;O.?t-l6Q#ca0strncpy (pErrMsgBuffer, strErrMsg, dwBufSize); 51Testing软件测试网.TU)S]s6V

当源字符串长度超出dwBufSize时，strncpy拷贝dwBufSize长度的字符，但目标字符串的结尾没有被赋值为‟\0‟，以后引用字符串pErrMsgBuffer时可能出错。

避免使用危险函数sprintf /vsprintf/strcpy/strcat/gets操作字符串，使用相对安全的函数snprintf/strncpy/strncat/fgets代替。

详见：

jI9}AN:@;?pj;@w.G[0《C规范》“规则6.1 禁止内存操作越界”，“规则13.1 确保所有字符串是以NULL结束”51Testing软件测试网[(lu1q&f&~

军规四：【防止使用空指针和野指针】

指针变量在赋值后、使用前要进行判空。

s8h3I HEPs"Z0释放内存后，要立即将指针设置为空，防止产生野指针和访问已释放内存。

;Eh"B|{ r(C4c0错误示例：

void CSCP::AnalysisConnectRequest(CTI_RESOURCE_CONNECT_Msg* pMsg, CConnectNode* pNode,CConference* pOcuppyConf) 51Testing软件测试网$|h)K+KV~%c

mK6`.QG%q5R1Ko0{

CConnectObject* pObject = NULL;

... ...

5K+B&_s.Y J0if(pSourceObject != NULL && pDestObject != NULL) {

*KH`,Nw5iz2m0if(!pSourceObject->IsConference() && !pDestObject->IsConference()) {

CConference* pDevice = pOcuppyConf;

if(CONN_CONF_REQUEST==pObject->m_ucCurConnectContext) //使用指针时没有检查，导致使用空指针

修正方案：先给pObject做有效性判定再使用，如：赋初始值。

错误示例：指针托管后被误delete

#x m,ye ]Mx1m0NEW(pSession, CSubscribeSessionInfo);51Testing软件测试网Q^C*L|9j^Q

?)ki1][Y9t$@s0CSessionProcessMgr::instance()->event(pSession->getSessionType(), pSession, code);

m_pSessionControl->addSession(pSession)

//上面event处理完了有可能会话已结束

4OG+EUbSjg A/a;O0if (pSession->isSessionOver())

V^LOJd]Z8y0{

DELETE(pSession);//此时delete，那么托管的指针指向内存就被误释放了51Testing软件测试网~o{%Cd L

,E1bF%A-[C(o0return;51Testing软件测试网J)iPc%z+ro

}

修正方案：

PKw~z,oG0if (pSession->isSessionOver())

{

m_pSessionControl->removesession(pSession);51Testing软件测试网*Bj%i~;i

#Q)|a4T@(K~WO,n$G0return;

KF8NI^8C!o%i0}

mdqe^ [B0详见：

《C规范》“规则6.3 禁止引用已经释放的内存空间”，《C++规范》“规则6.2 释放内存后，要立即将指针设置为NULL，防止产生野指针”

军规五：【及时释放内存，避免重复释放内存】

O%V Cb [ `&F&U0尽量避免跨函数分配和释放，尽量避免跨类分配和释放。若因设计的需要跨作用域进行分配和释放，必须重点评审，禁止出现忘记释放导致内存泄漏，禁止重复释放。51Testing软件测试网QF"\%o1z~

yGbQT;X0错误示例：指针托管后重复delete51Testing软件测试网w*`0z)p!r-A

NEW(pLogMsg)51Testing软件测试网'U7qGl_2m

"o%u5P*M9[T6q7k0//…

if (false == app.postMsg(pLogMsg))

{

e8bZ5[ c"N!AXA7O0//…51Testing软件测试网Aiv/GI Jv

%I'`m7Sl0UO0DELETE(pLogMsg);

\;iFv_3ht4V]-z0}

7Je7umkY0TBool Application::postMsg(InnerMsg* pMsg)51Testing软件测试网6[j"V"x8Zk

{

2n jYhYJv |"D6E0if (false == dispatch(pMsg))

{51Testing软件测试网ug*jl6K%E@

+g2ki n%]A0//…51Testing软件测试网%Zda-N}2D9l

DELETE(pMsg);

NO u-L(G,HV0return False;51Testing软件测试网 m|7R\n1G

&_h&y*{ b0}

//...

pdVz&\ l0}51Testing软件测试网2|d(p~}yg

mCo&w&dC mzg0修正方案：统一释放原则。51Testing软件测试网/F4zS&}r;z

TBool Application::postMsg(InnerMsg* pMsg)

9Y\ E(x[n0{

IdI0fw{"^0if (false == dispatch(pMsg))51Testing软件测试网7[\P F8E

{

'C,[YS6ov;i0//…

(toc;~[8D)r0DELETE(pMsg);

return False;//托管失败时由上面的申请内存的函数负责释放

v'h ^GrU1I7D*{F4[ N0}51Testing软件测试网7Hd,xc%Xz2oe

a I"Q:c9}(X0//...

}

^P8p'JC.]0详见：

"h/m^!\2}2fg0《C规范》“规则6.2 禁止内存泄漏”,

《C++规范》“规则6.3 单个对象释放使用delete，数组对象释放使用delete []”“规则6.4 释放结构(类)指针时，首先释放其成员指针的内存空间”“规则6.5 释放指针数组时，首先释放数组每个元素指针的内存”51Testing软件测试网e1rs2]N4Fg

L#p#Y)?7z0军规六：【禁止返回函数中的局部变量的指针】

不得将函数中局部变量的地址或指针作为返回值。

错误示例：

char* GetParameter ()51Testing软件测试网:l KdN,?

(YWp!Nx:A0{51Testing软件测试网 w^$_V_

CDBConnect DBConnect;51Testing软件测试网!uh3p)xa4Y.v

//………………..

return DBConnect.GetString("ParamValue");

}

修正方案：返回一个副本

9t:zq;L8S H4Ni4I7D0String GetParameter ()

{51Testing软件测试网J2v%UT@P:@v

d)mv@v,l]*p4x0CDBConnect DBConnect;

//………………..51Testing软件测试网 Y;|EJv`

A|X1b!UKDb qc7B0return String( DBConnect.GetString("ParamValue") );

}

9v lhNF z0详见：

!h'[@\Y VL2n/d_SfO0《C++规范》“规则6.6 不要返回局部对象指针”51Testing软件测试网'nCk X2m3a+~

军规七：【防止类型转换溢出和精度丢失】

防止无符号数运算溢出，数据类型转换时精度丢失。

尽量避免不必要的类型转换。

q`4U Q%lm"`u0C++新增代码使用static_cast等C++风格的类型转换，以便于排错。51Testing软件测试网p NS%G1Z

[R(Rzvv/N0错误示例：51Testing软件测试网J]_P l

uiLargeNumber = GetLargeNumber();

G4e]u8Xaql z0uiSmallNumber = GetSmallNumber();51Testing软件测试网MAtwm \

unUkJ8K:c0unsigned int uiLow = uiLargeNumber - uiSmallNumber; //错误，无法保证uiLargeNumber一定比uiSmallNumber大。

修正方案：

uiLargeNumber = GetLargeNumber();

H"WlI J$nk0uiSmallNumber = GetSmallNumber();51Testing软件测试网1H&r ~ l/egj

[.W7N'F aG-r$o0if (uiLargeNumber > uiSmallNumber) //正确，先判断好范围。

{

uiLow = uiLargeNumber - uiSmallNumber;51Testing软件测试网[9sWD]G2}

?X!r2`&w5Lx"`0}

B*|8],J#{_5c{0else

|~ s3^2\0k0{

uiLow = 0;

zwW$qJ*['`HL)Y0}

U?z"J-}qo9[0详见：

《C规范》“规则13.3 避免整数溢出”，“规则13.4 避免符号错误”，“规则13.5：避免截断错误”, 51Testing软件测试网 lq ]G;{

R1B E&v$J![ w$C0《C++规范》“规则2.2 使用C++风格的类型转换，不要使用C风格的类型转换” 51Testing软件测试网h"g9m(J/Qy

$}^m/m|lS0军规八：【多线程多进程访问共享资源时要防止重入和死锁】

加锁注意不要遗漏程序分支，C++代码可以采用guard方法。

5f1r(~?7I*v0注意使用线程安全函数。

错误示例：

5}7g5w$Ghn D3|+X)P0//g_Exam是int型全局变量，函数Squre_Exam返回Exam平方值。那么如下函数不具有可重入性。 51Testing软件测试网z%zMK!N

...... 51Testing软件测试网([aE-[(upsf

int g_Exam;

unsigned int example( int para )

I&p!Gg'r6{6m-^B0{

Q$E7~3C%_0unsigned int temp;

N7u"k k5x0g_Exam = para; // （**）

temp = Square_Exam(g_Exam);

return temp;

Yw'qUC0}

u3~ yYH\l0......

正确示例：

...... 51Testing软件测试网W fY_K

int g_Exam;

^Qa+tM6o9?!D0semaphore smMutex = 1; /* 控制互斥访问的信号量 */

......

9rP[N8e/Y_`}0unsigned int example( int para )

/D7Uv$Z)}y\*x2HV?0{

unsigned int temp;

*xOp\W o@2o2`0P(smMutex); // 若申请不到“信号量”，说明另外的任务正处于给Exam赋值并计算其平

2Kf;qgA`9SKrky0g_Exam = para; // 方过程中（即正在使用此信号），本任务必须等待其释放信号后，才必须等待本

V(smMutex); // 任务释放号量后，才能再使用本信号。

wV1r H;dL+W+Zj0return temp;

\&q\{]4WM r0}51Testing软件测试网h$Ob%J qr

-@%Idt-ul|lNS0详见：51Testing软件测试网tS8[R d.u3u

6cLB i_rDV0《C++规范》“规则10.1 多线程、进程并行访问共享资源时，一定要加锁保护”,“规则10.3 锁范围尽量小，只锁对应资源操作代码”,“规则10.4 避免嵌套加锁” 51Testing软件测试网d fW9j]2\

军规九：【头文件包含防止循环依赖，防止包含不同版本的头文件】

at?2{|];n6_2W0头文件包含层次清晰，去掉不必要的包含与重复包含，需要提供外部使用的接口才放到头文件中。51Testing软件测试网U&{Ki7h#AZ3j

防止包含不同版本的API头文件，以免导致不必要的运行错误。

%B L9qIc5B)u&q%h0详见：

《C规范》“规则1.1 每一个.c文件应有一个同名.h文件，用于声明需要对外公开的接口”，“规则1.2 禁止头文件循环依赖”，“规则1.3 .c/.h文件禁止包含用不到的头文件”

6Il2T.x U8t2e*l.K0军规十：【使用清晰简洁的表达式，加括号防止优先级混淆】51Testing软件测试网o3MX(RBN

Z`T@d8e2her0注意运算符的优先级，并用括号明确表达式的操作顺序。51Testing软件测试网;JR c%HR

9C+zWH|s0错误示例：

……

if ((sqn.seq2 + 1) & 0x01000000 > 0) {

……

}

Y0Yb^:b(SI0q0pn:U0……

由于>运算符优先级大于&, 上述if语句恒成立。51Testing软件测试网+Wh8|su&k

正确示例：

……

vm w+p|oD0if (((sqn.seq2 + 1) & 0x01000000 ) > 0) { // 加括号

[ M\u"{u0……

}

5F;?e/?R6T0……51Testing软件测试网3KY F+kf!Yw

#u}{`jR0尽量避免使用复杂的表达式，除非性能要求，最好拆成几个语句。

收藏举报

TAG:

查看全部评论

丰卉者

用户菜单

我的栏目

标题搜索

日历

数据统计

访问量: 109268
日志数: 89
图片数: 1
文件数: 15
建立时间: 2013-03-01
更新时间: 2018-08-23

C&C++编程军规

用户菜单

我的栏目

标题搜索

日历

我的存档

数据统计

RSS订阅