自学内容网 自学内容网

架构师应如何考虑重构

目录

重构的目的、时机、难点

 1.1重构的目的

1.2何时重构

    1.2.1 添加新功能的时候对周边历史进行小型重构

   1.2.2 cide review 时

  1.2.3 有计划有目的的重构

 1.2.5 何时不该重构

1.3、重构的难点

  1.3.1 保证重构前后行为一致。

1.3.2 减少出现问题带来的影响

1.4 常见的重构场景与方式

 1.4.1 策略模式取代条件表达式


重构的目的、时机、难点

 1.1重构的目的

   ① 优化代码结构、提高可读性

   ② 提高扩展效率

   ③降低修改代码的风险

1.2何时重构

       第一次做某件事的时候只管去做,第二次做类似的事情的时候会反感,但无论如何还是可以去做,第三次再做类似的事情,你应该去重构,正如老话说的:事不过三,三则重构

    1.2.1 添加新功能的时候对周边历史进行小型重构

     ① 当差不多代码赋值粘贴了3-5遍的时候

     ② 比如方法提炼、变量提炼、优化方法参数、消除重复逻辑

     ③ 当然也要取舍,对于简单影响小的可以立即重构,如果比较复杂的有风险的可以先做几路,完成当前任务后或者另找时间重构。

   1.2.2 cide review 时

    让有经验的同学把知识传递给编写代码的同学,从而给予改进的灵感(老同学对业务更加熟悉,也更了解业务的变化点有助于作出合理的设计)

  1.2.3 有计划有目的的重构

   对于中小型重构通常在需求中见缝插针进行重构就可以了,但对于大型重构难度和影响相对要大一些,所以就要做好设计,确定影响范围,这通常需要安排整块时间。

  1.2.4 出现线上问题

      发生线上问题可以暴露出一些问题,这也是改进的好时机,比如上下游系统出现故障影响到你的系统,就可以思考耦合性太强了能不能解耦。

 1.2.5 何时不该重构

     ① 重写比重构还容易(到这种程度重构的风险非常高)

     ② 隐藏在某个接口下运行稳定且极少修改的丑陋代码(难以看到收益)

1.3、重构的难点

  1.3.1 保证重构前后行为一致。

     ① 使用IDEA 重构功能进行安全重构

     ② 单元测试

     ③功能测试

     ④回归测试后

1.3.2 减少出现问题带来的影响

   ① 灰度并且 开关

   ② 监控报警快速发现问题

1.4 常见的重构场景与方式

 1.4.1 策略模式取代条件表达式

    增加策略类,用于一些更复杂的场景。

   

public class AccessChecker {

    public static String checkAccess(String role) {

        if("admin".equals(role)) {
            return "管理原可以访问所有功能";
        } else if("user".equals(role)) {

          return "普通用户只能访问部分功能";
       } else if("guest".equals(role)) {
         return "访客仅能浏览首页";

       } else {
         return "未知角色,请联系管理员";
       }

    }


}

使用策略模式 + 工厂模式优化

// 定义策略接口
public interface AccessStrategy {
  
     String checkAccess();

}

// 实现策略类
public class AdminAccessStrategy implements AccessStrategy  {

    @Override
    public String checkAccess() {
       return "管理员可以访问所有功能" 
    } 

}


public class UserAccessStrategy implements AccessStrategy {
   @Override 
   public String checkAccess() {
     return "普通用户只能访问部分功能";
   }

}

// 创建工厂类
public class AccessStrategyFactory {
    public static AccessStrategy getStrategy(String role) {
       switch(role) {
           case "admin":
              return new AdminAccessStrategy();
           case "user":
               return new UserAccessStrategy();
      
           default:
             throw new IllegalArgumentException("Unknown role:" + role);

       } 
    }
}


//应用策略
public class AccessManager {
   public String getAccessLevel(String role) {
       return AccessStrategyFactory.getStrategy(role).checkAccess();
   }
}

或者

public  enum UserRole {

   ADMIN("admin", new AdminAccessStrategy()),
   
   USER("user", new UserAccessStrategy()),
 
   GUEST("guest", new GuestAccessStrategy());

   
   private final String code;
 
   private final AccessStrategy strategy;

   
   UserRole(String code, AccessStrategy strategy) {
       this.code =code;
       this.strategy = strategy;
   }

   public String getCode() {
      return code;
   }
    
   public AccessStrategy getStrategy() {
      return strategy;
   }

   // 惊天方法,根据code 枚举对象
   public static UserRole getByCode(String code) {
      for(UserRole role : values()) {
          if(role.getCode().equalsIgnoreCase(code)) {
              return role;
           }
      }
      throw new IllegalArgumentException("Unknown role code:"  +code);
       
   }
    
}

或者Spring 的依赖注入

@Autowired
private Map<String, AccessStrategy> strategies;


public AccessStrategy getStrategy(String roleCode) {
     return strategies.get(roleCode.toLowerCase()+"strategy"));

}

 


原文地址:https://blog.csdn.net/qq_38345598/article/details/144647116

免责声明:本站文章内容转载自网络资源,如本站内容侵犯了原著者的合法权益,可联系本站删除。更多内容请关注自学内容网(zxcms.com)!