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

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

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

目录

1. 仓库 Issues 模板
2. 码提交附记录和Commit 信息
3. 对其他同学的代码进行审查
4. 对收到的issues进行回复
5. 重构与优化
6. 跟进对其他同学提出的issues
7. 心得

一.为自己的代码仓库添加 Issues 模板

• bug报告模板

• 功能改进模板

二.每次提交代码附上有意义的 Commit 信息

三.对其他同学的代码进行审查，并通过issues提出问题

• 赵泽阳#1

  对读取两个html文件方式的建议

  仓库：https://gitee.com/xjjzzy/personal

• 王晓玲#2

  锦上添花的小建议

  仓库：https://gitee.com/keep-looking/personal

• 徐笑龙#3

  对于读取文件路径的建议

  仓库：https://gitee.com/xu-xiaolong/personal

• 邱建杰#4

  对于代码大量重复的建议

  仓库：https://gitee.com/qiu-jianjie/personal

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

  小优化

  https://gitee.com/xue-culture/personal

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还不够熟悉，之后还要多加练习。同时希望更多同学来阅读自己的代码，不论是找毛病还是提建议，统统欢迎！