软件工程（2018）结对编程第一次作业

代码链接

评论

1. 同伴的代码比较整洁，但有些地方代码缩进做的不是很好，如：
   

   　　while(newptr->sex!=’m’ && newptr->sex!=’f’)
   {
   cin.clear();
   cout<<”请输入正确的性别(m或f):”;//性别控制
   cin.ignore(10000,’\n’);
   cin>>newptr->sex;
   }

   且代码缩进方式不一。
2. 注释详细，但部分注释不符代码注释规范，如：
   

   case 1://///////////////////////////////////////////////////////////普通病人登记

   这样的写法可能是为了自己能看的清楚一些，可始终不好，因为项目是团队来完成，后期维护是由团队其他伙伴来负责，如果不遵循代码的规范，会让伙伴很为难。
3. 除此之外，同伴代码中存在命名不够规范的问题，还有一处实验代码忘记删除，如下：
   

   //const int max=5;//////上限
   //int In_num=0;
   //int leave_num=0;///全局变量，计算离队（就诊完毕）人数

   总的来说，同伴的代码还是很不错的，注释全面，可读性很好。

总结

再一次深刻地意识到代码审查的重要性，最容易蒙骗自己的就是我们本身，很多时候我们无法以一个客观的角度去审视代码，所以需要同伴来帮助完善代码。