编程语言
首页 > 编程语言> > java – 避免使用if-else代码来创建依赖于特定条件的对象

java – 避免使用if-else代码来创建依赖于特定条件的对象

作者:互联网

是否有更好的方法来处理依赖于另一个对象类型(条件)的对象(产品)的实例化,而不是使用if-else与instanceof配对,如下面的代码所示?

import java.util.ArrayList;
import java.util.List;

abstract class AbstractProduct {
    private AbstractCondition condition;
    public AbstractProduct(AbstractCondition condition) {
        this.condition = condition;
    }
    public abstract void doSomething();
}

class ProductA extends AbstractProduct {
    AbstractCondition condition;
    public ProductA(AbstractCondition condition) {
        super(condition);
    }

    @Override
    public void doSomething() {
        System.out.println("I'm Product A");
    }
}

class ProductB extends AbstractProduct {    
    public ProductB(AbstractCondition condition) {
        super(condition);
    }   

    @Override
    public void doSomething() {
        System.out.println("I'm Product B");
    }
}

class AbstractCondition { }

class ConditionA extends AbstractCondition { }

class ConditionB extends AbstractCondition { }

public class Try {
    public static void main(String[] args) {
        List<AbstractCondition> conditions = new ArrayList<AbstractCondition>();
        List<AbstractProduct> products = new ArrayList<AbstractProduct>();

        conditions.add(new ConditionA());               
        conditions.add(new ConditionB());               
        conditions.add(new ConditionB());               
        conditions.add(new ConditionA());

        for (AbstractCondition c : conditions) {
            tryDoSomething(c);
        }
    }

    public static void tryDoSomething(AbstractCondition condition) {
        AbstractProduct product = null;
        if (condition instanceof ConditionA) {
            product = new ProductA(condition);
        } else if (condition instanceof ConditionB) {
            product = new ProductB(condition);
        }
        product.doSomething();
    }
}

与我的实际代码的上述代码的不同之处在于:我没有直接控制AbstractCondition及其子类型(因为它们在库中),但AbstractProduct的具体子类型的创建取决于具体条件.

我的目标是:尽量避免tryDoSomething()中的if-else代码气味.

我也想避免反思,因为它感觉像作弊,我认为它不是一个优雅,清洁和可读的解决方案.

换句话说,我想用良好的OOP原则(例如利用多态性)解决这个问题,并解决一些设计模式(在这个特定情况下我显然不知道).

解决方法:

由于无法编辑原始对象,因此需要创建从条件类型到产品类型的静态映射:

private static HashMap< Class<? extends AbstractCondition>, 
                        Class<? extends AbstractProduct>
                      > conditionToProduct;`

使用条件,产品对进行静态初始化:

static { 
  conditionToProduct.put(ConditionA.class, ProductA.class); 
  ... 
} 

并在运行时只查询地图:

Class<? extends AbstractProduct> productClass = conditionToProduct.get(condition.getClass());
productClass.newInstance();

标签:java,oop,design-patterns,instanceof,java-6
来源: https://codeday.me/bug/20190519/1134280.html