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

今天开始了结对编程项目，在开展结对编程项目之前我们需要对队友编写的个人项目进行点评。在仔细阅读队友的整个工程的代码后，我有了以下观点。

题目需求

1、命令行输入用户名和密码，两者之间用空格隔开（程序预设小学、初中和高中各三个账号，具体见附表），如果用户名和密码都正确，将根据账户类型显示“当前选择为XX出题”，XX为小学、初中和高中三个选项中的一个。否则提示“请输入正确的用户名、密码”，重新输入用户名、密码；
2、登录后，系统提示“准备生成XX数学题目，请输入生成题目数量（输入-1将退出当前用户，重新登录）：”，XX为小学、初中和高中三个选项中的一个，用户输入所需出的卷子的题目数量，系统默认将根据账号类型进行出题。每道题目的操作数在1-5个之间，操作数取值范围为1-100；
3、题目数量的有效输入范围是“10-30”（含10,30，或-1退出登录），程序根据输入的题目数量生成符合小学、初中和高中难度的题目的卷子（具体要求见附表）。同一个老师的卷子中的题目不能与以前的已生成的卷子中的题目重复（以指定文件夹下存在的文件为准，见5）；
4、在登录状态下，如果用户需要切换类型选项，命令行输入“切换为XX”，XX为小学、初中和高中三个选项中的一个，输入项不符合要求时，程序控制台提示“请输入小学、初中和高中三个选项中的一个”；输入正确后，显示“”系统提示“准备生成XX数学题目，请输入生成题目数量”，用户输入所需出的卷子的题目数量，系统新设置的类型进行出题；
5、生成的题目将以“年-月-日-时-分-秒.txt”的形式保存，每个账号一个文件夹。每道题目有题号，每题之间空一行；

结果测试

请输入用户名与密码（空格隔开）
张三1 123
当前选择为小学出题
准备生成小学数学题目，请输入题目数量（题数有效范围为10~30；输入-1将退出当前用户，重新登录）
20
小学试卷生成完毕
........等待切换中（输入-1退出登录，输入0退出系统）.......
-1请输入用户名与密码（空格隔开）
王五2 123
当前选择为高中出题
准备生成高中数学题目，请输入题目数量（题数有效范围为10~30；输入-1将退出当前用户，重新登录）
20
高中试卷生成完毕
........等待切换中（输入-1退出登录，输入0退出系统）.......
切换为小学
准备生成小学数学题目，请输入题目数量（题数有效范围为10~30；输入-1将退出当前用户，重新登录）
5
输入有误，请重新输入
25
小学试卷生成完毕
........等待切换中（输入-1退出登录，输入0退出系统）.......
0
成功退出系统文件部分内容：
//初中试卷
11、43*85*(49+53) = 12、29-62+29/9/76 =
//高中试卷
8、cos(49^2+cos(^0.5)89^0.5)*23/sin(^0.5)6^0.5 = 9、(17+tan(tan67^2*cos98^0.5)+95+tan66^0.5) =

代码分析

程序思路
阅读完队友的代码之后，我发现一个我们两个最大的不同点就在于数学题目生成的思路：
①笔者思路
先生成一定数量的操作数，然后循环判断每个操作数是否需要添加平方，根号以及三角函数符号，接着再判断是否需要生成括号，如果需要则要随机生成左右括号的位置，最后再利用循环将四则运算符号添加生成最终的题目。

②队友思路
每次循环随机生成一个操作数，然后对操作数现在的状态进行判断：操作数后面是否有括号，没有括号时操作数后面是否需要添加平方，根号；三角函数符号后是否有括号…当对操作数作出一系列的判断之后再添加四则运算符号，然后继续循环执行上述操作。

代码点评

①优点
1、注释简明易懂：队友的每个函数头都写了注释，且稍微难理解的题目生成函数的函数体中的注释也有相应的注释。因为阅读别人的代码其实是一个难以理解的过程，而在关键代码处注明了一些注释，这就可以帮助团队中的其他成员更好的理解，进而提高完成工程的效率。

2、用户体验好：可以看到在程序运行的过程中有很多提示性的话语，这些话语就能告诉用户在运行到不同阶段时应该输入的不同操作，也能知道程序此时正在执行的操作，以及相关操作是否成功。

3、异常输入处理到位：在生成高中题目的时候考虑了tan90无法求解的情况，而且当输入题目数量的时候考虑了用户输入不是数字的情况，并对每一种用户的异常输入都进行了异常处理，这就减少了用户使用过程中会出现的异常情况，也就相当于提高了用户体验感。

②不足
1、条件判断过多：在生成数学题目的时候，因为要对当前操作数进行多种情况的判断因此需要进行非常多的条件判断，这就让人有一定的困难去理解代码。

2、函数体太长，模块化的程度不高，建议将生成题目的代码与将题目写入文件的代码分成不同的函数，提高程序的模块化程度，进而帮助团队中的其他成员理解代码。

3、局部变量命名不是很规范，所以阅读代码的过程中要经常查看注释去理解变量的定义，因此建议使用驼峰命名法进行命名，且使用相对完整的英文去对变量进行命名，这就可以让团队中的成员一看到变量就知道变量的含义。