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

博客班级 https://edu.cnblogs.com/campus/fzzcxy/2018SE2/
作业要求 https://edu.cnblogs.com/campus/fzzcxy/2018SE2/homework/11199
作业目标 <通过Issue向其他同学的代码提出修改建议,并对其他人对自己的代码的建议进行修改>
作业源代码 https://gitee.com/xie-feifei/personal
学号 <211808332>

我的Issue模板

我的Commit信息

“让我看看你的代码!”

1. 何喜

2.石云凤

3.毛菁菁

  • 码云仓库地址:https://gitee.com/mjjyjq/personal
  • 问题所在:代码里面没看到注释,建议加点注释方便阅读和修改。
  • 提交的Issue的截图:

4.林宇豪

5.张何毓

“容我再改一改!”

  1. 陈洁的意见以及的修改
  2. 子怡的意见以及的修改
  3. 雪凡给我的意见
  • 修改后的代码

    for (int i = 0; i < es.size(); i++) {

      Element child = smallSource.select("div[class=interaction-row]").get(i);
    
      if (child.select("span").get(1).toString().contains("课堂完成")) {
      	if (child.toString().contains("已参与")) {
      		Scanner sc = new Scanner(child.select("span").get(9).text());
      		temp = sc.nextInt();
      		myBase = myBase + temp;
      	}
      } else if (child.select("span").get(1).toString().contains("课堂小测")) {
      	if (child.toString().contains("已参与")) {
      		Scanner sc = new Scanner(child.select("span").get(9).text());
      		temp = sc.nextInt();
      		myTest = myTest + temp;
      		// 筛选互评分部分
      		if (child.toString().contains("+")) {
      			Scanner sc2 = new Scanner(child.select("span").get(11).text());
      			temp = sc2.nextLine().toString().charAt(3) - 48;
      			myTest = myTest + temp;
      		}
      	}
      } else if (child.select("span").get(1).toString().contains("编程题")) {
      	if (child.select("span").get(8).text().contains("已参与")) {
      		Scanner sc = new Scanner(child.select("span").get(9).text());
      		temp = sc.nextInt();
      		myProgram = myProgram + temp;
      	}
      } else if (child.select("span").get(1).toString().contains("附加题")) {
      	if (child.select("span").get(8).text().contains("已参与")) {
      		Scanner sc = new Scanner(child.select("span").get(9).text());
      		temp = sc.nextInt();
      		myAdd = myAdd + temp;
      	}
      } else {
      	if (child.select("span").get(1).toString().contains("课前自测")) {
      		if (child.toString().contains("color:#8FC31F;")) {
      			Scanner sc = new Scanner(child.select("span").get(12).text());
      			temp = sc.nextInt();
      			myBefore = myBefore + temp;
      		}
      	}
      }
    

    }
    }

  1. 贝贝给我的建议
  • 修改后:已经在源代码里面添加更多的注释

“让我再看一看!”

目前还没有看到有同学修改。

总结

在发现别人的问题时也能发现自己的问题,并且刚开始大家都会范很基础的问题,例如:没有采用驼峰命名法,使用绝对路径而不是相对路径等。

参考文献

https://github.com/lufg/KPM/issues/1

posted @ 2020-09-21 15:18  白梦川  阅读(167)  评论(0编辑  收藏  举报