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

博客班级 2018软件工程2班 (福州大学至诚学院 - 计算机)
作业要求 第二次个人编程作业:代码互改
作业目标 学会使用Issue和pull request操作,对同学的代码进行评价, 对收到的Issue进行回复
作业源代码 https://gitee.com/chenbartender/personal
学号 211806306

“让我看看你的代码!”

1.崔亚明

问题 提取分数部分的代码基本相同,大部分代码重复,整体看起来繁杂
建议 建议将提取分数相关的代码封装成方法,在主方法中调用
码云地址 https://gitee.com/cui-yaming/personal

2.陈材

问题 没有考虑经验中有“互评”分数时的提取
建议 建议增加判断是否存在“互评分数”,若有则提取参与分和互评分,若无则提取参与分
码云地址 https://gitee.com/chen--cai/personal

3.林文基

问题 提取分数时,没有考虑到“互评”情况下的分数,导致最终结果产生偏差
建议 建议增加判断,是否有互评分数
码云地址 https://gitee.com/lin-wenji/personal

4.黄龙骏

问题 提取分数部分的代码没有考虑到“互评”情况下的分数提取
建议 建议增加判断,是否有互评分数
码云地址 https://gitee.com/hljhwc/personal

5.王浪浪

问题 提取分数部分的代码基本相同,大部分代码重复,整体看起来繁杂
建议 建议将提取分数相关的代码封装成方法,在主方法中调用,使代码更简洁
码云地址 https://gitee.com/wang-langlang/personal

“容我再改一改!”

commit记录

同学提的Issues

问题1

代码中很多变量采用拼音命名法,建议使用驼峰命名法。

我的修改

      Double ProgramMy,AddMy,BaseMy,TestMy,BeforeMy;
      ProgramMy=AddMy=BaseMy=TestMy=BeforeMy=0.0;

我的回复

问题2

可以稍微多一点备注,这样便于同学更好的理解你的代码内容,多出变量为拼音

我的修改

		pz.load(new FileInputStream("total.properties"));//导入分数配置文件
		double BeforeSure = Double.parseDouble(pz.getProperty("before"));//提取赢得课前自测分数
		double BaseSure = Double.parseDouble(pz.getProperty("base"));//提取课堂完成应得分数
		double TestSure = Double.parseDouble(pz.getProperty("test"));//提取课堂小测应得分数
		double ProgramSure = Double.parseDouble(pz.getProperty("program"));//提取编程题应得分数
		double AddSure = Double.parseDouble(pz.getProperty("add"));//提取附加题应得分数

我的回复

问题3

代码中课前自测部分是默认了已经参与,直接累加经验值,没有考虑到未参与的情况

我的修改

	public Double Allfenshu(Elements allelem,String leixing) {
		Scanner reader=null;
		Double FS=0.0;
		for (int i = 0; i < allelem.size(); i++) {
			Element allzhang = allelem.get(i);
			if(allzhang.toString().contains("课前自测")) {
				Elements span = allzhang.getElementsByTag("span");
				for(int j = 0; j < span.size(); j++) {
					if(span.get(j).text().contains("经验") && span.get(j).toString().contains("color:#8FC31F")) {
						reader = new Scanner(span.get(j).text());
						FS += reader.nextDouble();
						break;
					}
				}
			}
		}
		return FS;
	}

我的回复

注:其他ISSUES重复或无实质性建议,不采纳

让我再看一看!

1.黄龙骏同学进行了修改


黄龙骏同学在我提出ISSUES后,马上投入修改代码中,完善了算法。

注:其余同学正在修改中

我的Issue模板和pull request

pull request帮助谷同学修改代码

由于我与谷同学提取经验部分代码思路相同,用法也相同。但他的代码没有进行封装,代码看上去过于繁琐。故经邀请,我修改了谷同学的代码,封装方法简化代码

总结

通过观察同学的代码,以及同学对我提出的ISSUES,我总结出了以上常见的问题,在今后的编写代码过程中,我将会多注重以上问题

posted @ 2020-09-20 19:12  酒保陈某某  阅读(187)  评论(1编辑  收藏  举报