第二次个人编程作业:代码互改
| 博客班级 | https://edu.cnblogs.com/campus/fzzcxy/2018SE1 |
|---|---|
| 作业要求 | https://edu.cnblogs.com/campus/fzzcxy/2018SE1/homework/11195 |
| 作业目标 | 按要求在码云仓库上对其他同学的代码进行审查,并通过issues提出问题,修改自己的代码 |
| 作业源代码 | https://gitee.com/L1n-9s5/personal |
| 学号 | 211806392 |
Issues 模板 和 Commit 信息
最开始创建仓库的时候并没有初始化issus模板,后期添加上了,采用了比较通用的模板

修改代码之后添加了commit信息

让我康康!(对其他同学的代码仓库进行代码复审)
查看了其他五位同学的代码,并发现了一些问题





代码无法运行出正确的结果

发现了与之前一样的问题
改一改
目前位置在仓库中发现了5个issues,采纳了一些意见并且修改
- 陈甜同学的小提议

根据提议尝试修改自己的代码,发现switch并没有想象中的那么好用,它只接受int,stringm,enum类型的值,但是在查找titles时候用的是container函数,返回的是布尔值类型,又或者用正则表达式,但因为水平问题,无疑是加大了难度,所以并没有尝试。
- 蔡华翔同学给的建议

指出了互评经验没有提取并且给出了修改的建议,但是无法理解图中所给的代码的内容,所以没有采用,而是使用了select("span[style=color:#8FC31F;]")来选择互评的经验来解决问题
- 赵泽阳同学给的意见

不是很能理解没有用中文命名的意思,检查过了所有的文件都是以英语命名的
- 自我检查
在文件中发现了很多无用的代码,做了删除处理,并且发现了循环中冗余的代码部分,对此做了部分修改。
修改前

修改后

后续得到的回应
总共得到了两个回应,并且都采纳了
分别是许培腾和蔡华翔的代码
-
许培腾

-
蔡华翔



浙公网安备 33010602011771号