首页 > 代码库 > Windows客户端C/C++编程规范“建议”——变量和常量

Windows客户端C/C++编程规范“建议”——变量和常量

8 变量和常量

8.1 尽量不要使用全局变量

等级:【要求】
说明:全局变量的滥用和goto的滥用一样,都是一种灾难。它将使得逻辑变得难以调试和控制。

8.2 不涉及外部使用的全局变量需要使用static关键字修饰

等级:【要求】
说明:这样可以避免冲突。

8.3 变量需初始化后才能使用

等级:【必须】

说明:变量最好在定义时初始化。如果定义时无法初始化,那就应该在定义后立即初始化。

        以下我们来分析下变量未初始化而被使用的场景:

        在debug环境下,我们声明一个变量,编译器会使用0xCC填充变量所在空间。而在Release环境下,变量所在空间是不会被初始化的。所以我们经常会发现一些奇怪的现象:在debug环境下逻辑是正确的,release环境下逻辑是错误的。
        变量没有初始化存在如下危害:

  1. 产生脏数据。由于变量空间没有初始化,会导致脏数据的产生。可以见下例。
  2. 影响正常逻辑。脏数据将会导致之后使用相关数据的地方出现逻辑错误。
  3. 间接导致崩溃。可以见下例:快哭了

void MyPrintf( char* p ) {
	strcat(p, "abcdefghijklmnopqrstuvwxyz"); 
	printf("%s", p);
}

int _tmain(int argc, _TCHAR* argv[])
{
	char szBuffer[32];  
	std::string str = "01234";
	memcpy_s( szBuffer, _countof(szBuffer), str.c_str(), str.length() );  
	MyPrintf(szBuffer);
	return 0;
}

        该例中,栈上分配了一段32字节的空间。由于该空间没有初始化,所以在一定概率下,这段内存中不会出现0x00。于是之后的strcat将会导致堆栈溢出。我这儿将strcat放到一个函数内部,是为了比较方便的触发崩溃。


        执行strcat后。



        基于以上危害,我们在申明变量时需要初始化。类的成员变量,需要在构造函数中初始化。

        这儿有个需要特别说明的,静态数组初始化请使用:微笑

char szPath[MAX_PATH] = {0};
        而以下这种写法就没有上面这种写法方便和效率高:
char szPath[MAX_PATH];  
memset( szPath, 0, sizeof(szPath) );

8.4 一行只定义一个变量

等级:【要求】
说明:一行只定义一个变量,将会使的变量的初始化、维护变得方便。

8.5 不要直接使用常量参与运算

等级:【必须】
说明:在代码逻辑中直接使用常量,将导致代码逻辑非常难读懂。因为常量不具有表意性。
        例子:快哭了
RECT mainrc;
mainrc.left += 124;
mainrc.top += 56;
        该例子,试图计算出一个窗口中某个控件的位置(无窗口控件)。但是对于第一次读这样的代码的同学,谁知道那两个常量是什么意思?可以这么修改:微笑
RECT mainrc;
const unsigned int unCloseBtnOffX = 124;
const unsigned int unCloseBtnOffY = 56;
mainrc.left += unCloseBtnOffX;
mainrc.top += unCloseBtnOffY;

8.6 变量定义在接近第一次使用处(C除外)

等级:【要求】

说明:这样设计将使得代码可读性加强。


(转载请指明出于breaksoftware的csdn博客)

Windows客户端C/C++编程规范“建议”——变量和常量