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

博客班级	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，我总结出了以上常见的问题，在今后的编写代码过程中，我将会多注重以上问题