Java中我如何去除if...else...语句？

水深无声 2022-03-19 13:59 445阅读 0赞

###### 描述 ######

1.  读牛人技术博客 [A Java Geek][],最开始觉得这样的想法很有创意。提前使用静态代码块把对象存入map容器中，在需要的时候在取。他也有提到可以使用DI的方式把需要的对象提前注入好，但是这两种方式都会造成内存的浪费，因为有一些对象可能是频繁使用，而有些对象用的概率小甚至一次都没有用到，那么这样的方式是不好的。而且，我们是去除if…else…的语句，这样的方式虽然好像没有了if…else…语句，但是本质上并不是最好的方式，只是提供了一种思维方式。
2.  读《重构 改善既有代码的设计》有一条就是，**以多态取代条件表达式**。这是才是最本质的解决方式。
3.  这里的去除if…else…语句，不是遇见了if…else…语句就去除。这里是这样描述的：你手上有个条件表达式，**它根据对象类型的不同而选择不同的行为**。而不是平时编写代码遇见一些普通的条件表达式就去把它去除，不是这样的。
4.  而且，一般来说编程语言都有switch语句去替代if…esle…语句。从性能上看这两个语法也只有非常细微的差别，根本无需关心自己使用了哪个语法。

--------------------

###### 以多态取代条件表达式 ######

1.  代码的坏味道一：

double getSpeed() {
       switch_ (_type) {
       	case EUROPEAN:
       		return getBaseSpeed();
       	case AFRICAN:
       		return getBaseSpeed() - getLoadFactor() * _numberOfCoconuts;
       	case NORWEGIAN_BLUE:
       		return (is_Nailed) ? 0 : getBaseSpeed(_voltage);
       }
       throw new RuntimeException("Should be unreachable");
    }

1.  代码的坏味道二：

public Foo getFoo(Bar bar) {
    
       if (bar instanceof BarA) {
           return new FooA();
       } else if (bar instanceof BarB) {
           return new FooB();
       } else if (bar instanceof BarC) {
           return new FooC();
       } else if (bar instanceof BarD) {
           return new FooD();
       }
       throw new BarNotFoundException();
    }

1.  **思路**  
    将这个条件表达式的每个分支放进一个子类内的覆写函数中，然后将原始函数声明位抽象函数。
2.  **动机**（为什么我们要使用多态取代条件表达式？）  
    多态最根本的好处就是：如果你需要根据对象的不同类型而采取不同的行为，多态使你不必编写明显的条件表达式。
3.  范例来自《重构 改善既有代码设计》，因为去除if…else…语句是很重要的，所以手动记录下来。“员工与薪资”的例子。
4.  类图  
    ![在这里插入图片描述][watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L291dHNhbmRpbmc_size_16_color_FFFFFF_t_70]

--------------------

1.  **根据代码的坏味道一和二以及类图关系，我们可以得到一个初始版本的代码**

**Employee**

class Employee...
       int payAmount() {
       	switch (getType()) {
       		case EmployeeType.ENGINEER:
       			return _monthlySalary;
       		case EmployeeType.SALESMAN:
       			return _monthlySalary + _commission;
       		case EmployeeType.MANAGER:
       			return _monthlySalary + _bonus;
       		default:
       			throw new RuntimeException("Incorrect Employee");
       	}
       } 
       int getType() {
       	return _type.getTypeCode();
       }
       private EmployeeType _type;

EmployeeType

abstract class EmployeeType...
       abstract int getTypeCode();

Engineer

class Engineer extends EmployeeType...
    	int getTypeCode() {
    		return Employee.ENGINEER;
    	}

Manager

class Manager extends EmployeeType...
    	int getTypeCode() {
    		return Employee.MANAGER;
    	}

Salesman

class Salesman extends EmployeeType...
    	int getTypeCode() {
    		return Employee.SALESMAN;
    	}

--------------------

1.  **最终结果代码布局**  
    Employee

class Employee...
       int payAmount() {
       	return _type.payAmount(this);
       }
       private EmployeeType _type;

EmployeeType

class EmployeeType...
       abstract int payAmount(Employee emp);

Engineer

class Engineer extends EmployeeType...
    	int payAmount(Employee emp) {
    		return emp.getMonthlySalary();
    	}

Manager

calss Manager extends EmployeeType...
    	int payAmount(Employee emp) {
    		return emp.getMonthlySalary() + emp.getBouns();
    	}

Salseman

class Salseman extends EmployeeType...
       	int payAmount(Employee emp) {
       		return emp.getMonthlySalary() + emp.getCommission();
       	}

1.  理解  
    a, 把Employee中的原来的属性\_monthlySalary，\_commission，\_bonus变成最终的采用一个函数获取。  
    b, 注意的地方就是谁组合了谁。这里就是Employee组合了EmployeeType。或者可以理解为谁持有了谁的引用。  
    c, 其次就是一个继承关系。

##### 小结 #####

1.  由于偶然读到别人技术博客关于if…else…的取代文章，发现这也是一个思路，但是不够那么好，于是记录下《重构 改善既有代码设计》的一个**以多态取代条件表达式**的范例。
2.  注意什么时候可以用到这个范例。以及理解到它的动机。
3.  虽然这样确实处理了if…else…语句，但是一旦使用多态取代条件表达式的方式，必定会引入一个继承或者实现体系，其实，这是增加了理解的复杂度。
4.  面向对象的编程，如果增加了扩展性必定会引入设计模式，而一旦引入设计模式必定增加理解的复杂度，因为总有继承体系和实现体系结构，这种结构也是面向对象编程的特色。面向对象编程不一定比面向过程好。在实际的编码过程中，还是需要权衡这两种方式。

[A Java Geek]: https://blog.frankel.ch/avoid-sequences-if-else/
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L291dHNhbmRpbmc_size_16_color_FFFFFF_t_70]: /images/20220319/4ba50d507c6847678e6dc31144bfd988.png