第二次个人编程作业——代码互改
| 博客班级 | https://edu.cnblogs.com/campus/fzzcxy/2018SE2 |
|---|---|
| 作业要求 | https://edu.cnblogs.com/campus/fzzcxy/2018SE2/homework/11199 |
| 作业目标 | <对他人的代码提出issue并对别人提出的issue进行回复并修改代码> |
| 作业源代码 | https://gitee.com/hljhwc/personal |
| 学号 | <211806386> |
“让我看看你的代码！”
1.陈少龙同学：https://gitee.com/chenbartender/personal
*对于课前自测是否参与的判断问题，可以通过html中的字体颜色来判断，参与过字体为绿色，没有参与字体为红色
2.谷清尧同学：https://gitee.com/gu-qingyao/gqy
*代码文件中有一大部分被注释的无用代码，也许是在编程过程中用来测试或者被舍弃的部分，建议删除
3.何一山同学：https://gitee.com/he-yishan3/personal
*是编程习惯的问题，我本人也有这样的问题，虽然直接命名或者其他的命名法在自己编写代码时比较方便，但对别人
阅读代码会有一定的影响，所以建议使用驼峰命名法
4.林文基同学：https://gitee.com/lin-wenji/personal
*码云仓库中没有上次作业的独立文件夹，所有的文件都在同一个目录下，这样在以后作业文件更多时很容易弄混，同
时也不方便其他同学查看你的代码
5.陈材同学：https://gitee.com/chen--cai/personal
*其实和我是一样的问题，就是写代码时图方便，同样的判断语句直接在不同方法下复制多次，而不是用封装为函数的
方法，虽然方便了自己但让代码显得冗长，复杂。我自己也要在接下来的学习中养成这个习惯。
“容我再改一改！”
1.对于陈少龙同学提出的issue：
我检查了自己的代码，发现自己由于上次的疏忽，忘记考虑了在云班课中还有互评分数这一块的内容，难怪我计算出来
的分数那么低。特别感谢陈少龙同学提出的问题。
我重新打开云班课源代码，找到互评经验值的部分后，给累加分数的部分增加了以下判断条件。
boolean hp = smallel.get(i).text().contains("互评");
if(hp){
String s = (smallz.child(9).text().replaceAll("互评","").replaceAll("经验","").replaceAll(" ",""));
int ss =Integer.parseInt(s);
keqian+=ss;
}
这样就可以获取互评部分的经验值，并加到对应部分的总经验值中，从而计算出更准确的分数。
2.对于谷清尧同学提出的issue：
这也是我一直以来的一个坏习惯，对于变量想命名什么就取什么，但这样让我的代码可读性大大降低，以后我也会慢慢养成更好的编码习惯。
以下为已修改的截图
“让我再看一看！”
1.对陈少龙同学提出的issue，他已经进行了修改，增加了以下对于是否参与课前自测的判断语句
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;
}
陈少龙同学学习态度非常认真，几乎在我刚提出issue的时候就已经着手修改。
2.对何一山同学提出的issue，他已经修改变量名使用了驼峰命名法。
我的commit记录
issue模板截图
