TL;DR：移除所有意外的 IF 条件

• 代码重复问题
• 可能的拼写错误及其他缺陷

相关代码异味：

编程初学者课程：控制结构。资深开发者建议：避免使用这些结构。

接下来的步骤

1. 寻找或创建多态层次
2. 将每个if语句的主体移动到相应的抽象中
3. 为抽象命名
4. 为方法命名
5. 用多态消息发送替换if语句

    public String handleMicrophoneState(String state) {  
        if (state.equals("off")) {  
            return "麦克风已关闭";  
        } else {  
            return "麦克风已开启";  
        }  
    }  

    /* 表示 'off' 状态的常量在整个代码中重复出现，增加了拼写错误的可能性。
    'else' 条件并没有显式检查 'on' 状态，而是隐式地处理任何其他状态；
    这种处理方式会导致在任何需要处理状态的地方重复使用 IF 条件，
    暴露了内部表示并违反了封装原则。
    该算法不便于扩展且封闭修改，这意味着添加新的状态将需要在代码的多个地方进行修改。 */

    // 步骤 1：找到或创建多态层次结构  

    abstract class MicrophoneState { }  
    final class On extends MicrophoneState { }  
    final class Off extends MicrophoneState { }  

    // 步骤 2：将每个 if 语句的主体移到相应的抽象类中  

    abstract class MicrophoneState {  
        public abstract String 来自if的多态方法();  
    }  

    final class On extends MicrophoneState {  
        @Override  
        public String 来自if的多态方法() {  
            return "麦克风开启";  
        }  
    }  

    final class Off extends MicrophoneState {  
        @Override  
        public String 来自if的多态方法() {  
            return "麦克风关闭";  
        }  
    }  

    // 步骤 3：命名抽象类  

    abstract class MicrophoneState {}  
    final class 开启的麦克风状态 extends MicrophoneState {}  
    final class 关闭的麦克风状态 extends MicrophoneState {}  

    // 步骤 4：命名方法  

    abstract class MicrophoneState {  
       public abstract String handle();  
    }  

    final class 开启的麦克风状态 extends MicrophoneState {  
        @Override  
        public String handle() {  
            return "麦克风开启";  
        }  
    }  

    final class 关闭的麦克风状态 extends MicrophoneState {  
        @Override  
        public String handle() {  
            return "麦克风关闭";  
        }  
    }  

    // 步骤 5：用多态消息替换 if 语句  

     public String handleMicrophoneState(String state) {  
            Map<String, MicrophoneState> states = new HashMap<>();  
            states.put("静音", new Muted());  
            states.put("录音", new Recording());  
            states.put("空闲", new Idle());  

            MicrophoneState microphoneState =   
                states.getOrDefault(state, new 空的麦克风状态());  
            return microphoneState.handle();  
        }

类型：

[X] 半自动

大多数步骤都是机械化的，这是一个相当安全的代码重构。

重构后的代码遵循开-闭原则，更倾向于利用多态，而不是通过IF语句进行条件判断。

你只需把它用在意外的情况中的IFs上。

将业务规则保留为 “领域 if”，不要进行这种重构。

一些情况 IFs

这张图片由 Renuagra 在 Pixabay 上分享。

这篇文章是重构系列文章的一部分。