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

博客班级	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. 何喜

• 码云仓库地址：https://gitee.com/he-xi/personal
• 问题所在：建议使用驼峰命名法，会方便阅读。
• 提交的Issue的截图：
  

2.石云凤

• 码云仓库地址：https://gitee.com/shi-yunfeng/personal
• 问题所在：if里面的代码过于冗长，可以适当优化。
• 提交的Issue的截图：
  

3.毛菁菁

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

4.林宇豪

• 码云仓库地址：https://gitee.com/l258/personal
• 问题所在：缺少命令行代码。
• 提交的Issue的截图：
  

5.张何毓

• 码云仓库地址：https://gitee.com/zhang-heyu/personal
• 问题所在：可以直接导入包，使代码更加简洁。
• 提交的Issue的截图：
  

“容我再改一改！”

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;
  }
  }
  }
  }
  }

4. 贝贝给我的建议
   

• 修改后：已经在源代码里面添加更多的注释
  

“让我再看一看！”

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

总结

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

参考文献

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