结对项目之互评

前述

最近做了一个给中小学生出数学题的项目，学习了小伙伴的代码之后，总结一下对方的优点和缺点，取长补短，共同进步

项目介绍

用户：
小学、初中和高中数学老师。
功能：
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.DataRecord类
包含3个私有变量type、name和password，1个set（）方法以及3个get()方法。此类保存了用户的信息
2.SDataRecord类
保存了9个账号的信息，以及验证输入是否合法的函数
3. txtwriter类
创建txt文件保存题目
4.rubricnum 类
根据类型和题目数量生成相应的题目
5. Giword类
根据用户的输入调用相应的函数，与用户交互相关的类

优点分析

1.关于用户输入是否合法的判断：首先根据把用户的输入用空格拆分后得到的段数是否等于三来判断输入合法与否，然后再进一步检验库中有无相应账户。逻辑十分的清晰

String getju = scan.nextLine();
		String inline[]=getju.split(" ");
		int keylength=inline.length;
		if(keylength!=3)
		{
			System.out.println("输入不规范，请重新输入");
			gw();
		}

if (SDataRecord.inputcount(inline[0], inline[1], inline[2])==1)
		{
			System.out.println("当前选择为"+inline[0]+"出题");
		}
		else
		{
			System.out.println("请输入正确的用户名、密码");
			gw();
		}

public static int inputcount(String type,String name,String password){
		initalize();
		int flag=0;
		for(int i=0;i<9;i++)
		{
			if(type.equals(datarecord[i].gettype()))
			{
				if(name.equals(datarecord[i].getname()))
				{
					if(password.equals(datarecord[i].getpassword()))
					{
						flag=1;
					}
				}
			}
		}
		return flag;
	}

2.把所有生成过的题目保存在一个txt文件中，方便查重，因此在写文件的时候采用不覆盖原有文件内容的方式
3.通过set的contains（）方法查重，十分方便

不足之处

1.成功登录后，当提示用户输入题目的数量时，应该分别判断和-1，10，以及100的大小关系。举个例子：如果用户的输入大于100，那个系统应该提示重新输入题目数量，而不是退出登录状态
2.生成题目时，括号是根据随机数当作位置来插入的，是伪随机
3.初中和高中和小学部分的代码有很高的相似程度，但是小伙伴初中和高中部分代码没有复用其它部分的代码。
4.无用的代码，如：空白的构造函数

public DataRecord() {
	} 

5.txtwritter类的writeTxtFile（）方法，先读取原有文件的内容，然后再进行写入操作，其实这些可以通过FileOutputStream方法的第二个参数来实现

FileOutputStream(File file, boolean append)
创建文件输出流以写入由指定的File对象表示的文件。
第二个,boolean append,这个参数代表是否覆盖原有内容,默认为false,(意思为覆盖原有内容),我在我的构造函数中,传入参数为true,意思为不覆盖原有内容

总结

相比小伙伴的代码，我的代码在结构上还需要进一步的优化，可以使人很快的看懂，同时也应该多熟悉一点已有类型的方法，使自己能够更加有效率的解决问题。