第二次个人编程作业:代码互改
第二次个人编程作业:代码互改
博客班级 | https://edu.cnblogs.com/campus/fzzcxy/2018SE1 |
---|---|
作业要求 | https://edu.cnblogs.com/campus/fzzcxy/2018SE1/homework/11195 |
作业目标 | 学会给代码找bug“找茬”,并通过提交issue告知主人,必要时pull request |
作业源代码 | https://gitee.com/wang-bingwei/personal |
学号 | 211806342 |
目录
- 仓库 Issues 模板
- 码提交附记录和Commit 信息
- 对其他同学的代码进行审查
- 对收到的issues进行回复
- 重构与优化
- 跟进对其他同学提出的issues
- 心得
一.为自己的代码仓库添加 Issues 模板
--- >- bug报告模板
--- >- 功能改进模板
二.每次提交代码附上有意义的 Commit 信息
三.对其他同学的代码进行审查,并通过issues提出问题
--- - #### 赵泽阳#1
对读取两个html文件方式的建议
--- - #### 王晓玲#2
锦上添花的小建议
--- - #### 徐笑龙#3
对于读取文件路径的建议
--- - #### 邱建杰#4
对于代码大量重复的建议
private static int Interception(Elements es, String s, int score){
int temp;
if (es.child(1).child(2).toString().contains(s)) {
Scanner sc = new Scanner(es.get(i).child(1).child(2).children().get(0).children().get(7).text());
emp = sc.nextInt();
score += temp;
}
return score;
}
-
薛文化#5
小优化
while(!checkInput(firstFileName) || !checkInput(secondFileName)){
if(!checkInput(firstFileName)){
System.out.println("第一个文件名有错误,请重新输入");
}else{
System.out.println("第二个文件名有错误,请重新输入");
}
}
四.对自己的代码仓库中收到的issues进行回复
--- 1. ###### 给自己提交issue
--- 2. ###### 问题回复:码云用户名为中文导致gitpush失败如何解决
--- 3. ###### 问题回复:有关于驼峰命名法的变量名使用
--- 4. ###### 小问题:jar包问题和配置文件路径问题
--- 5. ###### 大问题:
1.没有计算互评经验
2.可读性不强
五.重构与优化
--- - 优化一
修复了之前没有计算互评经验的问题
--- - 优化二
重构了筛选方式,由原先多个.child(0)的筛选过程全部更改为为select("span").get(i)的查找方式,避免读起来会“绕蒙了”的不良现象(选取部分截图)
六.跟进你对其他同学提出的issues是否被采纳
--- - 采纳一
--- - 采纳二
--- - 采纳三
七.心得体会
本次作业令我更加领会到团队协作的重要性,同时也锻炼了阅读代码的能力,此外更是看到了不少同学极其优秀的代码,收获颇丰!但对Pull Request还不够熟悉,之后还要多加练习。同时希望更多同学来阅读自己的代码,不论是找毛病还是提建议,统统欢迎!