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

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



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




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

posted @ 2020-09-17 04:01  栩岸  阅读(243)  评论(0编辑  收藏  举报