java - 通过创建依赖于特定条件的对象来避免 If-else 代码异味

Question

有没有比使用 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 code smell。

我也想避免反射，因为它感觉像是作弊，我确实认为它不是一个优雅、干净和可读的解决方案。

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

Answer 1

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

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

用 Condition,Product 对在静态初始化中填充它:

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

在运行时只查询 map :

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