OOP大作业二轮总结

miaoxiadeblog / 2024-06-09 / 原文

第二轮下来最大的体会是自己的代码相较之前精炼了很多,从第四次作业开始注重了遵循单一职责原则。在做需求分析这方面也花了更多心思以更好地面对之后的迭代。
关于OOP中的单一职责:
前一轮大作业我把用到的方法全部堆在了主函数里,一个主函数能有四五百行。这次学会了用“代理者”Agent,主函数只负责输入和输出,代码可读性大大增加了。做题前要确认好类间关系以及各种属性应该安排在哪个类中。

三次作业的bug分析:

第四次大作业:
1.没有设置一个学生类,导致多个学生作答的情况的测试点只过一两个;(果然还是需求分析这方面没做好= =)
2.多人多试卷乱序输入时逻辑过于混乱导致输出也是完全错的
3.删除已经储存的信息后会出错(没有用List类而用的普通的数组)
第五次大作业:
1.最初没考虑档位上下限,做出了一个能无限升档和降档的调速器= =
第六次大作业:
1.输出的某个数据的格式不符合要求(但是结果提示的是答案错误不是格式错误害得我改了三个多小时没发现是格式错误555)
2.最初的需求分析没做好,没有做到普遍性
对bug的总结:
1.在做需求分析的时候一定要全面考虑,要追求普遍情况而非特殊情况,比如第四次作业潜意识认为只有一个学生,多个学生的情况完全没有办法。
2.不能疏忽题目要求,在代码没有逻辑错误和算术错误的情况下最应该先检查数据输出格式。
3.每一个类有什么限制都要了解清楚,不然真的贻笑大方了(我那档位爆炸的调速器)

下面是每次大作业的具体分析:

第四次大作业:

第四次大作业仍然是答题判题程序,通过增加题目种类(多选题,单选题,填空题)以考察继承与多态的学习成果。
SourceMonitor给到的代码分析:

从代码行数也能看出MatcherAndSeparation是比较关键的类,起到了一个代理的作用,最主要的逻辑也是写在其中。
类图:

踩坑心得

在写完多选题类后跑程序发现得到的答案都是正确的,但是一分没拿到,改了很久才发现是题目输入样例自带了个空格在答案后面,输出时我把那个空格一并输出了导致格式错误。很多时候在答案正确情况下最应该先检查格式问题。

改进建议

程序里同时存在一个抽象类Question和一个Question应该是当时我想把储存好的三种题型再总和到一起,但类图显示抽象类Question和Question两者之间没有联系,现在想想做成继承关系的话也能省不少代码了= =

第五、六次大作业:

第五次比较简单,就和迭代后的第六次放一起分析了
第六次大作业的代码分析:

主要方法放在了Agent函数里,写了四百多行,已经是优化了一遍后的行数了,问了下身边同学的总行数和我差不多,都是九百多行。
类图:

在第五次作业的串联电路基础上增加了并联电路的实现。

踩坑心得:

一开始思维没有打开,下意识认为题目就是局限在只有两条支路的并联电路,于是写的情况也是针对只有两条支路的情况,写的代码行数很多拿到的分却很少。后来全部重写了一遍,做成普遍情况后有n条电路都没问题。
总体还是偏简单的,但我卡壳了几个小时,最后是找了好几个大佬互测了一下才发现自己是数据格式输出错误,编译器写的是正确的格式,pta上的是错误的版本,导致我把其他人给的样例过了一遍发现计算结果都没问题还是没拿到分。
改正后36个测试点还有6个没过,但和其他人交流时发现即使是满分代码也有bug,比如断路情况下电路中的电器仍然能工作,所以这几个测试点一时半会真不知道到底是什么问题。

改进建议:

Agent函数里重复的代码太多了,几乎全是同样的for循环,如果把重复的地方单独做成一个方法类的话能让代码精简很多。

二轮下来的总结

最大的缺点是代码中仍旧出现了很多的if语句和for循环(说人话其实就是垃圾代码),诸如此类:

if (arr.startsWith("#N:")) {
            if(arr.matches("#N:[0-9]+[ ]#Q:.+[ ]#A:.*")){
            	 String[] a = arr.split(" * #A: *| * #Q:*|#N:");
                 if(a.length == 3){
                	 single[Integer.parseInt(a[1])] = new SingleKind(Integer.parseInt(a[1]), a[2].trim(), "","s");
                 }
                 else {
                     String answer = a[3].replaceAll("\\s+$","");
                     single[Integer.parseInt(a[1])] = new SingleKind(Integer.parseInt(a[1]), a[2].trim(), answer.trim(),"s");
                 }
            }else {
            	System.out.println("wrong format:" + arr);
            }
    	}
    	else if (arr.startsWith("#K:")) {
                if(arr.matches("#K:[0-9]+[ ]#Q:.+[ ]#A:.*")){
                	String[] a = arr.split(" * #A: *| * #Q:*|#K:");
                    if(a.length == 3){
                   	 fillblank[Integer.parseInt(a[1])] = new FillblankKind(Integer.parseInt(a[1]), a[2].trim(), "","f");
                    }
                    else {
                        String answer = a[3].replaceAll("\\s+$","");
                        fillblank[Integer.parseInt(a[1])] = new FillblankKind(Integer.parseInt(a[1]), a[2].trim(), answer.trim(),"f");
                    }
                }else {
                	System.out.println("wrong format:" + arr);
                }
            }
            else if (arr.startsWith("#Z:")) {
                    if(arr.matches("#Z:[0-9]+[ ]#Q:.+[ ]#A:.*")){
                    	String[] a = arr.split(" * #A: *| * #Q:*|#Z:");
                        if(a.length == 3){
                       	 multiple[Integer.parseInt(a[1])] = new MultipleKind(Integer.parseInt(a[1]), a[2].trim(), "","m");
                        }
                        else {
                            String answer = a[3].replaceAll("\\s+$","");
                            multiple[Integer.parseInt(a[1])] = new MultipleKind(Integer.parseInt(a[1]), a[2].trim(), answer,"m");
                        }
                    }else {
                    	System.out.println("wrong format:" + arr);
                    }
        } 
                else if (arr.startsWith("#T:")) {
                	if(arr.matches("#T:[0-9]+([ ][0-9]+[-][0-9]+)*")){
                		 t++;
                		 int g = 1;
                         String[] a = arr.split("\\s+|-+|#T:");
                         paper[Integer.parseInt(a[1])] = new TestPaper(Integer.parseInt(a[1]), (a.length - 2) / 2);
                         for (int j = 0; j < a.length - 2; j += 2) {
                        	 paper[(Integer.parseInt(a[1]))].setStandardGrade(g, Integer.parseInt(a[3 + j]));
                             g++;
                         }
                         for (int j = 1, m = 2; j <= (a.length - 2) / 2; j += 1, m += 2) {
                        	 paper[(Integer.parseInt(a[1]))].saveQuestionNum( j, Integer.parseInt(a[m]));
                         }
                 }
               else {
            	   System.out.println("wrong format:" + arr);
            }
          }
                else if (arr.startsWith("#S:")) {
            	   if(arr.matches("#S:\\d+ \\d+( #A:\\d+-.*)*")){
            		   ans++;
                       int num = 0;
                       String[] a = arr.split(" *#S:*| *#A:*|[-]+");
                       if(a.length % 2 == 0) {
                           String[] a2 = a[1].split(" ");
                           this.answer[ans] = new AnswerPaper(Integer.parseInt(a2[0]), a2[1], (a.length - 2) / 2);
                           for (int i = 1, j = 3; j <= a.length; i++, j += 2) {
                        	   if(Character.isWhitespace(a[j].charAt(a[j].length()-1))) {
                            	   answer[ans].setStudentAnswer(i, a[j].substring(0, a[j].length() - 1));
                            	   }
                            	   else {
                            		   answer[ans].setStudentAnswer(i, a[j]);
                            	   }
                           }
                       }else{
                           String[] a2 = a[1].split(" ");
                           this.answer[ans]=new AnswerPaper(Integer.parseInt(a2[0]), a2[1], (a.length - 1) / 2);
                           for (int i = 1, j = 3; j <= a.length - 2; i++, j += 2) {
                        	   if(Character.isWhitespace(a[j].charAt(a[j].length()-1))) {
                        	   answer[ans].setStudentAnswer(i, a[j].substring(0, a[j].length() - 1));
                        	   }
                        	   else {
                        		   answer[ans].setStudentAnswer(i, a[j]);
                        	   }
                               num = i;
                           }
                           num += 1;
                           answer[ans].setStudentAnswer(num, "");
                       }
                       for (int i = 1, j = 2; j < a.length; i++, j+=2) {
                    	   answer[ans].setStudentQuestion(i, Integer.parseInt(a[j]));
                       }
            }else {
            	System.out.println("wrong format:" + arr);
            }
        }  
               else if (arr.startsWith("#X:")) {
            if(arr.matches("#X:[0-9]+([ ]\\w+[-][0-9]+)*[ ]\\w+")){
            	String[] a = arr.split("\\s+|-+|#X:");
                for (int j = 1, m = 2; j <= (a.length - 1) / 2+1; j += 2, m += 2) {
                	this.students.put(a[j], a[m]);
                	stu++;
                }
            }else {
            	System.out.println("wrong format:" + arr);
            }
        } 
          else if (arr.startsWith("#D:N-")) {
            if(arr.matches("#D:N-[0-9]+")){
            	String[] a = arr.split("#D:N-");
                int questionNum = Integer.parseInt(a[1]);
                this.deletedQuestions.add(questionNum);
                single[questionNum] = new SingleKind(questionNum, null, null,"s");
                multiple[questionNum] = new MultipleKind(questionNum, null, null,"m");
                fillblank[questionNum] = new FillblankKind(questionNum, null, null,"f");
            }else {
            	System.out.println("wrong format:" + arr);
            }
        } 
          else {
        	  System.out.println("wrong format:" + arr);
        			}
    	int k=1;
    	for(k=1;k<100;k++) {
    		if(single[k]!=null) {
    			question[k]=new Question(single[k].getQuestionNum(),single[k].getQuestionContent(),single[k].getStandAnswer(),"s");
    		}
    		if(multiple[k]!=null) {
    			question[k]=new Question(multiple[k].getQuestionNum(),multiple[k].getQuestionContent(),multiple[k].getStandAnswer(),"m");
    		}
    		if(fillblank[k]!=null) {
    			question[k]=new Question(fillblank[k].getQuestionNum(),fillblank[k].getQuestionContent(),fillblank[k].getStandAnswer(),"f");
    		}
    	}
}
缺点

学到的各种模式没使用过,为了在一周时间内边上课边写大作业于是只敢在“舒适圈”中用些自己比较习惯的写法,希望自己在之后空闲下来的时间去钻研和掌握课上学到的模式。

进步

在分析类间关系上更熟练了,继承和多态的使用和遵循了单一职责也使得未来迭代后不需要花太多时间去修改。在这方面也有实验的功劳,我在实验上花了不少心思后对做需求分析和设计类间关系上更加有方向了。还有就是对List类有了更多的了解,用起来很方便,之前一直用普通数组解决问题导致几乎无法增删情况。

未来展望:

多啃啃Java课本,学会更多方法有助于优化代码,和身边同学交流一下看看他们用的哪些好用迅速的方法还是很有必要的。要敢于去沟通交流互测代码才能及时发现自己的错误。要多多享受这种查缺补漏的过程。在大作业适当降低难度后我也能更快完成作业,空闲的时间也多了出来,对OOP的热情也是保持了。