第二次个人编程作业:代码互改
博客班级 | 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的截图:
“容我再改一改!”
- 陈洁的意见以及我的修改
- 子怡的意见以及我的修改
- 雪凡给我的意见
-
修改后的代码
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; } } }
}
}
- 贝贝给我的建议
- 修改后:已经在源代码里面添加更多的注释
“让我再看一看!”
目前还没有看到有同学修改。
总结
在发现别人的问题时也能发现自己的问题,并且刚开始大家都会范很基础的问题,例如:没有采用驼峰命名法,使用绝对路径而不是相对路径等。