码迷,mamicode.com
首页 > 其他好文 > 详细

一位初入门的新员工的代码分析(1)

时间:2015-08-18 09:09:11      阅读:97      评论:0      收藏:0      [点我收藏+]

标签:

    公司刚来的新员工,刚毕业,经过几个月的实践,完成了一个实训编码练习。通过对其代码的阅读,发现了一些问题。这些问题对于初学者来说,是比较容易犯的错误,而且以他们目前的经验来说,不容易发现或理解。对此进行专门的解读和分享,希望各位初学者能够有所启发,早日提高编码水平。

    任务比较简单,是完成一个学生管理系统,现在很多大学生都自己玩这样的系统。要求使用链表、XML等考核点。

以下是其写的学生链表类头文件:

#include "Student.h"
#include "XMLWrapper.h"
struct StudentNode
{
 CStudent m_Student;
 StudentNode *next;
};
class CStudentList
{
public:
CStudentList();
~CStudentList();
 //初始化学生链表
void InitializeStudentList(int nSize,CStudent *Student);
    void InitializeStudentList();
 //添加学生
 void AddStudent(CStudent Student);
//删除学生
void DeleteStudent(CString StudentID);
//清空学生链表
void ClearStudentList();
//通过学号查找学生
CStudent* SelectStudent(CString strStudentID);
//查找班级内的学生
BOOL  SelectCalssStudent(int *Count,CStudent **Student,CString strMajor,CString strGrade,CString strClass);
//保存学生信息
BOOL SaveStudentInfomation(TiXmlElement *RootElement,CString strXML,CXMLWrapper *pTinyXML);
//检查学生链表是否相等
BOOL CheckStudentList(CStudentList *pStudentList);
private:
StudentNode *m_head;
};
对于这个头文件定义,初学者可以从自己目前的经验来看看有什么优缺点。然后再看下面的分析,可能会更有价值一些。









好吧,下面分析一下从这个头文件中看到的不足。
1。变量命名风格不统一
CString StudentID    VS  CString strStudentID
CStudent *Student   VS  CStudent **Student
可读性差,读者难以通过变量名来容易辨识变量类型,很容易混淆
建议:实在不知道该怎么办的话,一是找个编码规范学一学,二是看看MFC吧。

2。变量名称无意义
StudentNode *m_head
m_head能代表什么?不能达到顾名思义的效果。
建议:最优秀的注释就是不用写注释。不用写是指代码本身就可以自解释。

3。函数名有单词错误
SelectCalssStudent
班级--Class
建议:实在不清楚的话,可以查查字典啊。

4。返回值类型不合理
void InitializeStudentList(int nSize,CStudent *Student);
void AddStudent(CStudent Student);
操作是否成功,如何让用户了解?
建议:业务功能性操作,尽量考虑保留返回值,是否成功对外部的处理有较大的影响。

5。参数个数太多
BOOL  SelectCalssStudent(int *Count,CStudent **Student,CString strMajor,CString strGrade,CString strClass);
未来班级属性增加怎么办?
建议:关系比较紧密地参数,且数量较多,有扩展可能性的,建议使用结构,可保持接口不变
           另外,常见的OUT型参数,一般放在函数参数的末尾

6。类的职责不明确
BOOL SaveStudentInfomation(TiXmlElement *RootElement,CString strXML,CXMLWrapper *pTinyXML);
学生列表类是否负责学生信息的读写,还只是作为一个学生对象的容器?两种方案似乎都只完成了部分功能,封装还不够。

如果仅作为容器,是抽象的,与业务无关;而读写,属业务范畴
如果负责读写,那么应该有信息的读取处理部分,而不是只有保存部分
扩展考虑,存储未来可能存在不同的存储方式,这里直接固化为XML文件格式存储,未来无法扩展。
如果只作为容器,那么文件存储类应该移除。为了支持外部遍历容器进行相关处理,类应该增加遍历机制,如Prev(),Next(),Head(),Tail()等
另外,作为容器类,似乎还应该有修改学生信息的函数
相关性:设计模式中,有单一职责原则;有迭代器模式
其它建议:可将节点中的学生对象定义为指针类型



版权声明:本文为博主原创文章,未经博主允许不得转载。

一位初入门的新员工的代码分析(1)

标签:

原文地址:http://blog.csdn.net/happyparrot/article/details/47746093

(0)
(0)
   
举报
评论 一句话评论(0
登录后才能评论!
© 2014 mamicode.com 版权所有  联系我们:gaon5@hotmail.com
迷上了代码!