博客班级 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.

  • 1.林泽峰同学(可点此进同学博客)

    • GitHub地址
    • 问题:代码的注释太少,可标出一些函数的用途增加可读性。
    • 全篇没有注释,但仔细的一点点边看边思考反而学到了一个新的知识点:
      - 这里可以将html的本地读取变成线上读取,这是我与其他同学的代码可以优化的地方。
  • 2.余奇棚同学(可点此进同学博客)

    • GitHub地址
    • 问题:代码优化,可以将html的本地读取变成线上读取
    • 基本没有什么问题,可运行有注释代码之间没有太大的冗余,就提了优化的建议。因为其代码不优化其本身也不错,在线下去找他,学习改自己的代码了。
    • 学完之后反手一个从上面同学学来的优化建议扔过去
  • 3.陈冠华同学(可点此进同学博客)

    • GitHub地址
    • 问题:代码中使用太多if嵌套,很多可以简单化。
    • 看了看优秀同学们的代码简化的大致分为这几种(部分):


    • 然后照着改了改自己的

“容我再改一改!”

1.仓库未指定开源许可证,已开启。
lssue


2.注释问题,对每一步的步骤进行更详细的注释
第一个同学

第二个同学

解决:


3.对所有变量名或函数名使用了驼峰命名,增加了识别性和可读性


4.将包名改为英文且在上传仓库时删除,方便代码处理命令行传入的参数
按时间顺序,这是第二个提出的包名问题,明白上传到仓库里的应该无包名

这是第一个同学,从这里明白包名不建议使用中文,不然代码无法处理命令行传入的参数,改完之后通知了同学上传到仓库里的应该无包名


5.增加命令行代码
......这个暂时还不会,这还是上次老师讲课才意识到还有命令行代码,但是目前还不是很懂怎么做。

9/23更新:打算用vue试一下.


6.路径加上../更加规范


“让我再看一看!”

  1. 林泽峰同学(已采用)
  2. 林鑫宇同学(已采用)

这个部分,以后持续跟进,因为前端框架作业、人机交互作业、php撰写代码、数据库加上这个作业本身,20号晚上还有考试,感觉都很忙,等三天之后再去其仓库看看,并以后会为了这部分修改博客的

 posted on 2020-09-20 18:24  32泽  阅读(117)  评论(0编辑  收藏  举报