中小学生试卷自动生成程序--jialin大佬代码分析

Posted 2020-11-13 hnufsh

结对编程代码评价

  有幸和小jialin结对编程。拿到jialin的代码后。

我先是尝试用idea运行。结果报了如下错误。

无法加载主类，再尝试用eclipse运行。

好的，可以运行，那为什么用idea跑不了呢？

度娘了一下，发现是环境变量设置的问题。

接下来步入正题，开始看代码。

打开工程，发现在只有一个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()，当输入非整数时，会抛出类型不匹配的异常，可以在这里进行一下异常处理，当输入不为整数的时候要求重新输入。

总结

总体来看，jialin大佬代码很好的完成了题目的各项需求，只是在代码风格上还有一些地方可以提高，整个工程的架构也可以做的更好，一个class写到底的方式调试的时候实在是太不友善了，jialin大佬凭着清晰的逻辑能力完成了工程，但是我看起来的时候就有点头疼了。看了jialin的代码之后，再反思一下我自己写的，发现自己写的一些东西其实是有些多余的，命令提示也可以做的更友好一些，整个工程的耦合程度也可以进一步降低。