第二次个人编程作业:代码互改

博客班级 https://edu.cnblogs.com/campus/fzzcxy/2018SE2/
作业要求 https://edu.cnblogs.com/campus/fzzcxy/2018SE2/homework/11199
作业目标 <熟练使用 git,通过 Issue 向同学提出优化代码建议>
作业源代码 https://gitee.com/Chen-J-0123/personal
学号 <211806305>

我的Iusse模板

我的pull request模板

我的Commit提交信息

“让我看看你的代码!”

“容我再改一改!”

  • 问题1:使用绝对路径
    • Issue截图:
    • 代码修改如下:
  • 问题2:没有直接导包,使代码看起来过长
    • Issue截图:
    • 代码修改如下:

  • 问题3:代码不够简洁
    • Issue截图:
    • 代码修改如下:
  • 问题4:部分条件判断代码过于冗长
    • Issue截图:
    • 代码修改如下:
        if (smallSource != null && allSource != null) {
              Elements small = smallSource.getElementsByClass("interaction-row");
            small.addAll(allSource.getElementsByClass("interaction-row"));
      	for (int i = 0; i < small.size(); i++) {
      	      if (small.get(i).toString().contains("已参与&nbsp;")){
      		if(small.get(i).toString().contains("课前自测")) {
      		Elements span = small.get(i).getElementsByTag("span");
      		for (int j = 0; j < span.size(); j++) {
      		      if (span.get(j).text().contains("经验")) {
      		      Scanner sc = new Scanner(span.get(j).text());
      		      myBefore = myBefore + sc.nextDouble();
      		      break;
      		      }
      		}
      	     }else if(small.get(i).toString().contains("课堂完成")){
      	        Elements span = small.get(i).getElementsByTag("span");
      		for (int j = 0; j < span.size(); j++) {
      		      if (span.get(j).text().contains("经验")) {
      			Scanner sc = new Scanner(span.get(j).text());
      			myBase = myBase + sc.nextDouble();
      			break;
      		      }
      		}
      	     }else if (small.get(i).toString().contains("课堂小测")) {
      		Elements span = small.get(i).getElementsByTag("span");
      		for (int j = 0; j < span.size(); j++) {
      		      if (span.get(j).text().contains("经验")) {
      			Scanner sc = new Scanner(span.get(j).text());
      			myTest = myTest + sc.nextDouble();
      			break;
      		      }
      		}
      	     }else if (small.get(i).toString().contains("编程题")) {
      		Elements span = small.get(i).getElementsByTag("span");
      		for (int j = 0; j < span.size(); j++) {
      			if (span.get(j).text().contains("经验")) {
      				Scanner sc = new Scanner(span.get(j).text());
      				myProgram = myProgram + sc.nextDouble();
      				break;
      		          }
      		}
      	    }else if (small.get(i).toString().contains("附加题")) {
      		Elements span = small.get(i).getElementsByTag("span");
      		for (int j = 0; j < span.size(); j++) {
      		      if (span.get(j).text().contains("经验")) {
      			Scanner sc = new Scanner(span.get(j).text());
      			myAdd = myAdd + sc.nextDouble();
      			break;
      			}
      		 }
      	     }
      	}
      
  • 问题5:缺少命令行
    • Issue截图:
    • 修改如下:

“让我再看一看!”

个人总结

在仔细查看别人的代码的时候,不仅能够学习到别人的方法和思路,还可以发现自己代码的问题,从而再次完善自己的代码。

参考文献

https://gitee.com/help/articles/4145#article-header0

posted @ 2020-09-20 00:43  来日可期Cy  阅读(246)  评论(4编辑  收藏  举报