软件工程UML第二次作业-代码的优化

博客班级	<填写这份作业所在的博客班级的链接>
作业要求	https://edu.cnblogs.com/campus/fzzcxy/2018SE2/homework/11199
作业目标	<实现自身代码的优化和重构，学会Issue的提出>
作业源代码	https://gitee.com/huang-cy/software-engineering-uml
学号	<211814113>

1. 问题分析

1.1 对别人的第一次作业提出Issue

1.2 重构自己第一次作业

1.3 在Gitee上，对别人的代码提出Pull Requests（附加）

2. Issue的提出

2.1 First

• 码云地址：https://gitee.com/xu-qianlong/personal
• 改进建议：
  • 分数计算的占比有一些问题。编程题最高分题目调为95，在计算的时候也应该进行调整，将100 -> 95。同理附加题 100 -> 90。
  • 代码整体观看体感感不佳，将grap进一步模块化，可读性会强很多。
• Issue截图：

2.2 Second

• 码云地址：https://gitee.com/daydreamer1/personal
• 提出建议：将代码相同的一块，定义成变量，增加可读性和简洁性。 比如这样定义 eg Element QuestionType = es.get(i).child(1).child(0).toString()
• Issue截图：

2.3Third

• 码云地址：https://gitee.com/yu1392250489/personal
• 提出建议：将代码具体功能进行模块化。
• Issue截图：

2.4 Fourth

• 码云地址：https://gitee.com/gu-qingyao/gqy/tree/master/211806316_uml
• 提出建议：子元素查找，效率低下。可以换另一种查找方法，比如选择器查找。
• Issue截图：

2.5 Fifth

• 码云地址：https://gitee.com/l258/personal/tree/master/11
• 提出建议：对绝对路径进行优化，变成相对路径，增加代码可移植性。
• Issue截图：

3.重构代码

3.1别人提出的Issue

• 对代码进行注释
• 使用选择器代替子元素查找

3.2 自己的优化

• 对 properties 问价加载的代码进行优化

3.3 代码优化

• properties 文件的加载。

• Properties con = new Properties();
          con.load(new FileInputStream("UML/src/first/total.properties"));
          Enumeration fileName = con.propertyNames();
          //加载配置 properties 文件
  
          beforeTotal = Integer.parseInt(con.getProperty("before"));      //读取课前自测总分
          baseTotal = Integer.parseInt(con.getProperty("base"));          //读取课堂完成总分
          testTotal = Integer.parseInt(con.getProperty("test"));          //读取课堂小测总分
          programTotal = Integer.parseInt(con.getProperty("program"));    //读取编程题总分
          addTotal = Integer.parseInt(con.getProperty("add"));            //读取附加题总分
          System.out.println(beforeTotal);
  

• 子元素查找换成选择器查找 - > 优化例子。

• Elements allElement = allDocument.getElementsByClass("interaction-row");
          Elements smallElement = smallDocument.getElementsByClass("interaction-row");
          int sumScore = 0;
          for(int i = 0;i < allElement.size();i++){
              if(allElement.get(i).toString().contains("课前自测")){
                  System.out.println(allElement.get(i).toString());
                  System.out.println(allElement.get(i).toString());
                  Elements span = allElement.get(i).getElementsByTag("span");
                  for(int j = 0;j < span.size();j++){
                      if(span.get(j).toString().contains("经验")){
                          Scanner sc = new Scanner(span.get(j).text());
                          sumScore += sc.nextInt();
                          break;
                      }
                  }
              }
            }
  

4. 小结

• 对代码要实现的功能尽量模块化，减少 main() 函数的代码，方便维护。
• 使用相对路径代替绝对路径，增加代码可移植性
• 子元素查找效率太低，尽量不使用。