第二次个人编程作业:代码互改
博客班级 | 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提交信息
“让我看看你的代码!”
- 林雪凡
- 码云仓库地址:https://gitee.com/lin-xuefan/personal
- 问题所在:缺少命令行工具
- 提交Issue 的截图:
- 码云仓库地址:https://gitee.com/lin-xuefan/personal
- 石云凤
- 码云仓库地址:https://gitee.com/shi-yunfeng/personal
- 改进建议:可以具体列个表格
- 提交Issue 的截图:
- 码云仓库地址:https://gitee.com/shi-yunfeng/personal
- 谢菲菲
- 码云仓库地址:https://gitee.com/xie-feifei/personal
- 问题所在:代码命名不规范,建议使用驼峰命名法
- 提交Issue 的截图:
- 码云仓库地址:https://gitee.com/xie-feifei/personal
- 吴越
- 码云仓库地址:https://gitee.com/wu-yue/personal
- 问题所在:应将绝对路径改为相对路径,否则在其他环境中无法正常读取及运行
- 提交Issue 的截图:
- 码云仓库地址:https://gitee.com/wu-yue/personal
- 邱晴
- 码云仓库地址:https://gitee.com/daydreamer1/personal
- 问题所在:计算课堂小测部分没有考虑互评的经验
- 提交Issue 的截图:
- 码云仓库地址:https://gitee.com/daydreamer1/personal
“容我再改一改!”
- 问题1:使用绝对路径
- Issue截图:
- 代码修改如下:
- Issue截图:
- 问题2:没有直接导包,使代码看起来过长
- Issue截图:
- 代码修改如下:
- Issue截图:
- 问题3:代码不够简洁
- Issue截图:
- 代码修改如下:
- 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("已参与 ")){ 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; } } } }
- Issue截图:
- 问题5:缺少命令行
- Issue截图:
- 修改如下:
- Issue截图:
“让我再看一看!”
- 谢菲菲
- 码云仓库地址:https://gitee.com/xie-feifei/personal
- 改进前
- 改进后:
- 码云仓库地址:https://gitee.com/xie-feifei/personal
- 蒲子怡
- 码云仓库地址:https://gitee.com/pu-ziyi/personal
- 改进后:
- 码云仓库地址:https://gitee.com/pu-ziyi/personal
- 吴越
- 码云仓库地址:https://gitee.com/wu-yue/personal
- 改进后:
- 码云仓库地址:https://gitee.com/wu-yue/personal
个人总结
在仔细查看别人的代码的时候,不仅能够学习到别人的方法和思路,还可以发现自己代码的问题,从而再次完善自己的代码。