一位初入门的新员工的代码分析(2)
来源:互联网 发布:腾龙网络秋个人珍藏 编辑:程序博客网 时间:2024/06/05 17:24
前文分析了一个类的头文件定义。从头文件定义中可以看出类的一个大致功能,可以分析该类的设计是否合理。其它还可以观察到编码规范、函数参数、返回值等类型的合理性。在类的设计中,一定要充分明确类的职责,不是此类职责范围内的事,不要随意让这个类来处理,保持类的独立性,降低耦合度,这点非常的重要。很多新人对如何设计类很模糊,很多情况下,类设计不合理,这需要逐步的积累经验,提高自己。
今天对函数体进行分析,通过函数体的分析,找出问题,完善自己的代码。
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新人能够有所收获,编写自己的强壮代码。
0 0
- 一位初入门的新员工的代码分析(2)
- 一位初入门的新员工的代码分析(1)
- 新员工的试题
- 新员工学习的技术
- 对新员工的谈话
- 新员工的晋升故事
- Android 6.0 分析 (一位网友的分析,不错)
- 任正非给新员工的信
- 新员工的培训与成本
- VSS上新员工的档案
- 华为致新员工的一封信
- 某公司对新员工的公开信
- Android代码规范----一位2年外包开发者的心声
- 一位VC爱好者的入门之路
- 一位码农写给老婆的代码
- 一位码农写给老婆的代码
- 一位码农写给老婆的代码
- 一位前辈的iOS 代码实践总结
- Linux查看物理CPU个数、核数、逻辑CPU个数
- 枚举思想
- 关于信号中断与慢系统调用的深度发现
- 一款功能强大的iphone购物应用源码
- unity和委托
- 一位初入门的新员工的代码分析(2)
- 分治思想
- 最短路
- 回溯思想
- CentOS 安装FTP服务
- 借助江苏电信欢go进行流量推广活动合作的工作规范(二)
- 线程同步与互斥:POSIX无名信号量
- 动态规划
- 从第一眼你就会爱上的bootstrap UI资源(下载)。