个人项目：中小学数学卷子自动生成程序---队友代码点评

对队友的程序进行优缺点分析:　　

优点：

1、程序结构清晰

程序分为五个类，DataBase类，数据库类存放用户信息，paperSystem类生成试卷类，TeacherUesr类 用户类封装用户信息，Main主函数类 ，程序结构明确，逻辑清晰，便于设计和修改，非常吻合“高内聚，低耦合”。

2、测试完善

用户体验很好，考虑很多边界问题，例如输入的时候，考虑输入用户名和密码换行也可以读取的问题

 

 

3.代码性能好，效率比较高

队友的代码对性能考虑极为细致，例如在使用scanner读取输入是，我重复的实例化，但是他只用了一次，构思巧妙，让人眼前一亮

4.增加了附加任务

队友在原需求中，增加了题目的难度，附加了表达式中的括号，并且括号插入也做到了随机，并不局限于只包含两个操作

5、程序检测自动生成文件夹，方便统一管理。

6、代码规范做的很好，代码中个函数的命名和参数的命名都严格根据文档编写

 

缺点：

1.每个方法体没有加注释说明是说明功能，入口参数是什么，阅读起来较为麻烦

 

2.试题生成算法效率低

　　　　队友的算法是每次插入一个符号都需要遍历一遍现在的表达式，然后判断哪些位置可以插入，再取随机数去插入，这个算法的效率很低，更应该通过分层思想，先随机插入中级高级符号，然后使用低级符号链接上述的表达式雏形，使用字符串的拼接即可完成

 

3.试卷在eclipse里面查看为乱码，但是文件夹下是可以正常显示，和伙伴沟通之后，明白他是采用了两种编码，才出现这个问题，但不影响用户使用

4.查重时使用indexOf()，没有查询到返回-1，还得做if判断返回false，我使用的是contains()效率高，而且直接返回布尔变量

 

总结：

通过本次研读队友的代码，让我对代码风格和代码规范有了很深刻的印象，在之前没有理解老师严格要求的代码变量名命名规范，通过在阅读代码时，发现除了注释，如果变量命名混乱，给阅读也带来了极大的不便，另一方面深刻感受到，软件实现不能仅仅考虑需求的实现，更要考虑用户 使用体验，程序性能优化，程序是否便于扩展等深层次的问题