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