有幸和小jialin结对编程。拿到jialin的代码后。
我先是尝试用idea运行。结果报了如下错误。
无法加载主类，再尝试用eclipse运行。
度娘了一下，发现是环境变量设置的问题。
接下来步入正题，开始看代码。
打开工程，发现在只有一个class文件，头大，这样的架构肯定不够好。一个一个函数看下来吧。
第一个函数是 CheckNameAndPassword()，用于检测账号密码是否正确
boolean CheckNameAndPassword(User []user,User indata) {
for(int i=0;i<9;i++) {
if(indata.name.equals(user[i].name) && indata.password.equals(user[i].password)) {
indata.grade=user[i].grade;
return true;
}
}
return false;
}
可以看出来jialin大佬这里是把用户写死了的，先产生了满足需求的9个User，这样写的好处是执行的效率比较高，缺点是添加用户比较麻烦，要对代码进行改动。
接下来是TestDemo，小jialin把所有的逻辑都写在了这里面。看第一段我发现了一个代码风格上的问题。代码如下：
if(xx!=0) indata=Login();
grade=ChooseGrade(indata);
xx=PaperProduce(indata.name,grade);
在符号如“=”两边最好空格一下，这样代码看起来会更清晰。改后的如下：
if(xx != 0) indata = Login();
grade = ChooseGrade(indata);
xx = PaperProduce(indata.name, grade);
改了之后是不是好看了一点点，继续往下看。
接下来分别是选择出题难度，保存试卷，产生试卷，产生题目，判断是否重复的函数。
重点看一下查重函数itemIsExist()
static boolean itemIsExist(String problem, String username,String grade) {
String file_path= "papers\\"+username+"\\"+grade;
File folder = new File(file_path);
String all = "";
File files[] = folder.listFiles();
for(File f : files) {
InputStream is = null;
try {
is = new FileInputStream(f);
} catch (FileNotFoundException e) {
// TODO 自动生成的 catch 块
e.printStackTrace();
}
byte content[] = new byte[1024];
try {
is.read(content);
} catch (IOException e) {
// TODO 自动生成的 catch 块
e.printStackTrace();
}
try {
is.close();
} catch (IOException e) {
// TODO 自动生成的 catch 块
e.printStackTrace();
}
try {
all += new String((content),"utf-8");
} catch (UnsupportedEncodingException e) {
// TODO 自动生成的 catch 块
e.printStackTrace();
}
}
if(all.indexOf(problem) == -1) {
return false;
}else {
return true;
}
}
可以看到每次查重都要去读文件。将已经产生的题目全部读出，再一一对比。每次读文件都会使用比较多的资源，我觉得可以把产生的题目先存入字符串组，新产生题目与字符串组内题目对比，若重复则重新产生一道，最后将整字符串组写入，这样可以减少读写文件带来的消耗。
jialin大佬的产生题目的逻辑写得很好，可以随机的产生符合需求的各种形式的题目，还可以给题目加上括号。同时防止了不合理括号产生。
接下来实际运行体验一下。
基本满足需求，试卷也成功产生了。接下来输入一些奇怪的东西测试一下程序的稳定性。
发现了一点不稳定的地方，当要求输入试卷数量时如果没有输入数字，随意输入一个字符串，程序就会终止。这是因为在该逻辑中使用了Scanne输入的nextInt()，当输入非整数时，会抛出类型不匹配的异常，可以在这里进行一下异常处理，当输入不为整数的时候要求重新输入。
总结