一位初入门的新员工的代码分析（2）

前文分析了一个类的头文件定义。从头文件定义中可以看出类的一个大致功能，可以分析该类的设计是否合理。其它还可以观察到编码规范、函数参数、返回值等类型的合理性。在类的设计中，一定要充分明确类的职责，不是此类职责范围内的事，不要随意让这个类来处理，保持类的独立性，降低耦合度，这点非常的重要。很多新人对如何设计类很模糊，很多情况下，类设计不合理，这需要逐步的积累经验，提高自己。 今天对函数体进行分析，通过函数体的分析，找出问题，完善自己的代码。

void CStudentList::InitializeStudentList(int nSize,CStudent *Student)
{
	m_head=new StudentNode;
	if(m_head!=NULL)
	{	
		m_head->m_Student=Student[0];
		m_head->next=NULL;
	}
	StudentNode *PreNode=NULL;
	PreNode=m_head;
	for(int i=1;i<nSize;i++)
	{
		StudentNode *NewStudentNode=NULL;
		NewStudentNode=new StudentNode;
		if(NewStudentNode!=NULL)
		{
			NewStudentNode->m_Student=Student[i];
			NewStudentNode->next=PreNode->next;
			PreNode->next=NewStudentNode;

		}
		PreNode=PreNode->next;
	}
}

请先思考不许偷看

好吧，你已经有自己的判断了吗？那么我们来分析一下吧。首先，m_head=new StudentNode 就有问题，如果m_head当前不为空，那么不就产生内存泄漏了吗？所以，在这句之前，应该先清空列表。其次，m_head->m_Student=Student[0]; 也有问题，确定Student这个参数不为空？应该先检查一下，代码要有自我保护意识，提高程序的健壮性。另外，既然if(m_head!=NULL)进行了判断，那么为什么不考虑条件不成立的时候呢？一旦不成立，因为m_head为空，后续的代码中，所有相应的指针处理都会导致崩溃。同样，后续代码中的if(NewStudentNode!=NULL)也存在同样的问题。通过对这个函数的分析，让我们认识到写代码不能只看到理想的一面，更多的要考虑各种例外，才能确保你的程序经得起考验。希望IT新人能够有所收获，编写自己的强壮代码。