第二次编程作业（代码互改）

博客班级	https://edu.cnblogs.com/campus/fzzcxy/2018SE2
作业要求	https://edu.cnblogs.com/campus/fzzcxy/2018SE2/homework/11199
作业目标	<填写这份作业要达到哪些目标>
作业源代码	https://gitee.com/wang-langlang/personal
学号	<211806343>

@

目录

• “让我看看你的代码！”

  • 1、刘鸿宇

  • 2、柯致腾

  • 3、陈少龙

  • 4、廖鸿志

  • 5、陈才

• “容我再改一改！”

  • *

    来自陈少龙的issue

  • *

    来自 HoUdInI的issue

  • *

    来自谷清尧的issue

  • *

    来自Lhy的issue

• “让我再看一看！”

• 我的commit记录

• 我的Issues模板

• 总结

“让我看看你的代码！”

1、刘鸿宇

• 码云仓库地址：https://gitee.com/liuihongyu/personal/issues/I1VY76

• 建议：你的代码可以按功能分解，模块化。经验获取的那部分代码可以进行封装，减轻代码的冗余。
  对经验值的爬取，除了正则以外，是否还有更优得爬取方法。
  而且，我作为一个想要提高的学生，我光知道自己的分数，却不知道我需要提高的地方在哪，你可以试着在输出最后总分时，将个个部分的得分百分比显示，并给予这位同学些许建议。

• 截图：

2、柯致腾

• 码云仓库地址：https://gitee.com/ke-zhiteng/personal/issues/I1VY91

• 建议：没有任何的注释，这是一个非常不好得习惯，在开发环境中，你如果写代码没有注释，可是会被一堆人暴揍得。
  只有规范得编码，才能有高质量，高效率。

• 截图：
  

3、陈少龙

• 码云仓库地址：https://gitee.com/chenbartender/personal

• 建议：规范地编码，很明显得，出现了中文拼音式得变量命名。 当代码量很多，并且由他人进行后期维护得时候，这是非常不方便的。

• 截图：
  

4、廖鸿志

• 码云仓库地址：https://gitee.com/liao-hongzhi/Personal/issues/I1VY8Q

• 建议：对仓库文件存放的优化，建议以后将同一个项目的文件放在一个文件夹中commit，否者很容易出现仓库的文件混乱。

• 截图：
  

5、陈才

• 码云仓库地址：https://gitee.com/chen--cai/personal

• 建议：多次出现了重复的代码,在遇见这种情况时，可以使用封装方法，减轻代码得冗余。

• 截图：
  

“容我再改一改！”

自己issue记录

*

来自陈少龙的issue

• 问题：提取分数部分的代码基本相同，大部分代码重复，整体看起来繁杂
  建议：建议将提取分数相关的代码封装成方法，在主方法中调用，使代码更简洁

• 我的回复：
  

*

来自 HoUdInI的issue

• 问题：你所使用的正则表达式"\w* 经验"，可以更改为“\d* 经验”，然后在find后用Double.parseDouble(matcher.group(1))的方法直接转换把数字部分直接转换成double类型，不用将“经验”这两个字替换成null，再转换成double类型

• 我的回复：
  

*

来自谷清尧的issue

• 问题：正则的方法过于麻烦,可以考虑通过获取节点的方法来获取(https://images.gitee.com/uploads/images/2020/0920/003037_556131d8_7999398.jpeg "183410_5ed242bc_7993013.jpg")

• 我的回复：
  

*

来自Lhy的issue

• 问题：计算经验值的方法几乎一致，导致代码变得很多，可以将for代码可以整合在一起，if后面可以写成if else。

• 我的回复：
  

“让我再看一看！”

• 对陈少龙同学的建议，已经被采纳

• 复查其代码，发现修改得非常好。

我的commit记录

我的Issues模板

总结

• 

• 这些是我在观察同学代码时发现的问题，也是大多数同学都通有的问题。