第二次我的编程做业：代码互改
| 这个做业属于哪一个课程 | https://edu.cnblogs.com/campus/fzzcxy/2018SE1 |
|---|---|
| 这个做业要求在哪里 | https://edu.cnblogs.com/campus/fzzcxy/2018SE1/homework/11195html |
| 这个做业的目标 | 对本身代码进行修改优化，对他人代码提出见解,issues模板的创建 java |
| 做业源代码 | https://gitee.com/RongC_Zhang/personal/tree/master/firstHomework |
| 学号 | 211806360 git |
issues模板
“让我看看你的代码！”
1.叶明浦的gitee：路径引用
2.胡玉彬的gitee：代码优化
3.刘慧的gitee：代码优化
4.王晓玲的gitee：变量名的命名
5.黄世辉的gitee：
6.陈颖鹏的gitee：代码优化
1.对于他人提交Pull Requests
胡玉斌
经过main主方法调用函数实现配置文件的读取
“容我再改一改！”
1.对于他人提交的issues
个人score.java源码
1.没有计算互评经验
经过判断最后1个span是否含有互评增长额外增长互评经验值
if(rows.indexOf("小测")!=-1) {
if(testEvaluate.indexOf("互评")!=-1 && joinSec.indexOf("已参与")!=-1) {
String testGet=activities.get(i).select("span").get(spanSize-3).text();
selfTest+=isNum(testGet);
String testGet2=activities.get(i).select("span").get(spanSize-1).text();
selfTest+=isNum(testGet2);
}
else if(join.indexOf("已参与")!=-1){
String testGet=activities.get(i).select("span").get(spanSize-1).text();
selfTest+=isNum(testGet);
}
}
2.变量名称须要按照驼峰命名法来命名
对定义的每个变量用驼峰命名法更改(例如：proGet、testEvaluate、joinSec等)函数
3.引用文件的路径最好用相对路径
2个html的地址引用"./src/small.html"、"./src/all.html"以及配置文件的地址引用"./src/total.properties"已更改学习
4.代码重复
2个代码皆是计算span的个数，已删除其中一个 已更改优化
5.用方法封装解析网页和提取经验值、配置文件
已经经过myScore()函数提取经验值并计算;经过ownScore()函数读取配置文件;经过countFinalScore()函数计算总分数解决(源代码可看，在标题下面已贴)lua
6.合并HTML以及多设置几个类
合并html：经过Elements activities=small.getElementsByClass("interaction-row");
activities.addAll(all.getElementsByClass("interaction-row"));实现合并(源代码可看，在标题下面已贴)
设置多个类：相似问题5，已解决
2.自提自改
问题1：没有对“已参与”进行判断
经过join=activities.get(i).select("span").get(spanSize-2).text();
join.indexOf("已参与")!=-1更改
问题2：没有实如今命令行运行(文件命名不规范，还存在中文，下次会注意)
“让我再看一看！”
除胡玉斌还未回复外，其他人都进行了修改；陈颖鹏对问题进行了有道理的拒绝。
“遇到的问题”
经过此次的互评发现本身的代码还有许多须要改进的地方
问题1：整理代码
将以前全部存在main主方法里的代码用多个函数进行封装修改
问题2：jar包的生成
经过查找资料，jar包如何生成(感谢王晓玲同窗的帮助)
生成jar包点击finish后出现了jar export finish with warning，最后发现是由于Score类的黄色错误没有修改。
问题3：命令行不能执行
经过屡次的修改实现，也遇到了比较麻烦的问题
以后发现，配置文件应该与jar包放在同一个文件夹内(应该是相对地址引用的问题)。spa