博客班级 | https://edu.cnblogs.com/campus/fzzcxy/2018SE2 |
---|---|
作业要求 | https://edu.cnblogs.com/campus/fzzcxy/2018SE2/homework/11199 |
作业目标 | 为自己的代码仓库添加 Issues 模板,对其他同学代码进行审核提交 Issue,同时对于他人提交的 Issue 修改代码并提交到自己的仓库中,并使用有意义的 Commit 信息 |
作业源代码 | https://gitee.com/W32ZE/personal |
学号 | 211806411 |
我的commit信息 | |
========== | |
“让我看看你的代码!”
issues.
-
- GitHub地址
- 问题:代码的注释太少,可标出一些函数的用途增加可读性。
- 全篇没有注释,但仔细的一点点边看边思考反而学到了一个新的知识点:
- 这里可以将html的本地读取变成线上读取,这是我与其他同学的代码可以优化的地方。
-
- GitHub地址
- 问题:代码优化,可以将html的本地读取变成线上读取
- 基本没有什么问题,可运行有注释代码之间没有太大的冗余,就提了优化的建议。因为其代码不优化其本身也不错,在线下去找他,学习改自己的代码了。
学完之后反手一个从上面同学学来的优化建议扔过去
-
- GitHub地址
- 问题:代码中使用太多if嵌套,很多可以简单化。
看了看优秀同学们的代码简化的大致分为这几种(部分):
-
然后照着改了改自己的
-
- GitHub地址
- 问题:遗漏了互评成绩的计算
看代码和其他同学给其他同学的意见的时候,才发现还有这个???自己要改的东西真是越来越多了~
-
- GitHub地址
- 问题:代码优化,可以将html的本地读取变成线上读取
- 代码只有这一句注释,不容易阅读。
“容我再改一改!”
1.仓库未指定开源许可证,已开启。
lssue
2.注释问题,对每一步的步骤进行更详细的注释
第一个同学
第二个同学
解决:
3.对所有变量名或函数名使用了驼峰命名,增加了识别性和可读性
4.将包名改为英文且在上传仓库时删除,方便代码处理命令行传入的参数
按时间顺序,这是第二个提出的包名问题,明白上传到仓库里的应该无包名
这是第一个同学,从这里明白包名不建议使用中文,不然代码无法处理命令行传入的参数,改完之后通知了同学上传到仓库里的应该无包名
5.增加命令行代码
......这个暂时还不会,这还是上次老师讲课才意识到还有命令行代码,但是目前还不是很懂怎么做。
9/23更新:打算用vue试一下.
6.路径加上../更加规范
“让我再看一看!”
这个部分,以后持续跟进,因为前端框架作业、人机交互作业、php撰写代码、数据库加上这个作业本身,20号晚上还有考试,感觉都很忙,等三天之后再去其仓库看看,并以后会为了这部分修改博客的