《编程规范与编写优质代码培训（海汇）》.pptVIP

编码规范培训 设计功能单一的函数 如果pv为NULL，那么realloc的作用相当于调用malloc(size)，并返回指向新分配内存块的指针，或者在该请求无法满足时返回NULL。 如果pv不是NULL，但新的块长为零，那么realloc的作用相当于调用free(pv)并且总是返回NULL。 如果pv为NULL且当前的内存块长为零，结果无定义 函数设计 设计功能单一的函数 这种设计的问题： 包含细节太多，不利于程序员的安全使用； 对入参没有限制，很难使函数防范无效的参数； 解决方法： 将多功能的函数分解为多个单一功能的函数 对于以上的realloc()函数可以分解为扩大内存块、缩小内存块、分配内存块和释放内存块四个函数；（对于每个函数，我们可以对入参进行严格的检查，而对于调用者也不会再发生使用的错误） 函数设计 设计功能单一的函数 函数接口设计要清晰易懂，没有异议 例子 下面的调用完成什么工作？ UnsignedToStr(u, str, TRUE); UnsignedToStr(u, str, FALSE); 改写调用语句 #define BASE10 1 #define BASE16 0 ……… UnsignedToStr(u, str, BASE10); UnsignedToStr(u, str, BASE16); 修改后程序清晰明了 函数设计 设计功能单一的函数 根据前面编写单一功能函数处理，可以将此函数写为两个函数，如下 void UnsignedToDecStr(unsigned u, char* str); void UnsignedToHexStr(unsigned u, char* str); 为了增加函数的灵活性，可改写为 void UnsignedToStr(unsigned u, char* str, unsigned base); 两个问题： 增加函数灵活性和函数功能单一相矛盾 函数参数中最好不要使用布尔值，它不利于程序的扩展 编写注释突出可能的异常情况 函数设计 数据上溢 上溢 bool func(size_t cbSize) { if (cbSize 1024) { // we never deal with a string trailing null char *buf = new char[cbSize-1]; memset(buf,0,cbSize-1); // do stuff delete [] buf; return true; } else { return false; } } 代码是验证 cbSize 不大于 1 KB，并且 new 或 malloc 应该始终正确地分配 1 Kb内存。 new 或 malloc 的返回值应该在此时进行检查。 cbSize 不能为负数，因为它是 size_t。但是，如果 cbSize 是零，又会如何呢？查看一下分配缓冲区的代码，它从缓冲区大小请求中减去一。从零减去一会产生 size_t 变量，这是一个无符号的整数，其限制为 0xFFFFFFFF（假设为 32 位的值）或者 4 GB。 数据下溢 下溢 例子： void * memchr( void *pv, unsigned char ch, size_t size ) { unsigned char *pch = (unsigned char *) pv; while( -- size =0 ) （修改为while( size-- 0 ) ） { if( *pch == ch ) return (pch ); pch++; } return( NULL ); } 经常反问：“这个变量表达式会上溢或下溢吗？” 避免使用嵌套的“?:“运算符 unsigned uCycleCheckBox(unsigned uCur) { return( (uCur=1)?(1-uCur):(uCur==4)?2:(uCur+1) ); } 改写为： usigned uCycleCheckBox(unsigned uCur) { unsigned uRet; if(uCur = 1) { if(uCur != 0) /* 处理0，1，0 …… 循环 */ uRet = 0; else uRet = 1; } else { if(uCur == 4) /* 处