Question

我有一个java控制器类，当在任何项目中调用保存操作时，例如Product或coverage或Limit，它调用控制器保存方法并传递参数。控制器中的if逻辑检查参数并调用相应对象的save方法。 if逻辑日益增长。任何人都可以建议更好的设计模式吗？

代码：

public class Product {
public void save(PolicyData p){
    //logic here
}
}

public class Coverage {
    public void save(PolicyData p){
        //logic here
    }
}

public class Limit {
    public void save(PolicyData p){
        //logic here
    }
}

public class Controller {

    private Product pr=new Product();
    private Limit lim=new Limit();
    private Coverage cov=new Coverage();


    public void save(PolicyData p,String item){
    if(item.equals("Product")){
        pr.save(p);
    }if(item.equals("Coverage")){
        cov.save(p);
    }if(item.equals("Limit")){
        lim.save(p);
    }
    }
}

Answer 1

创建可保存的界面：

public interface Saveable {
    public void save(String p);
}

您的类实现了接口，然后创建了Map：

private Map<String, Saveable> saveMap = new HashMap<>();

用Savable对象填充它，然后根据String调用save方法。

public enum SaveableType {
    PRODUCT, COVERAGE, LIMIT
}

public class Controller {
    private Product pr = new Product();
    private Limit lim = new Limit();
    private Coverage cov = new Coverage();
    private Map<SaveableType, Saveable> saveableMap = new HashMap<>();

    public Controller() {
        saveableMap.put(SaveableType.PRODUCT, pr);
        saveableMap.put(SaveableType.LIMIT, lim);
        saveableMap.put(SaveableType.COVERAGE, cov);

    }

    // better to use enum for the 2nd parameter not a String
    public void save(PolicyData p, String item) {
        SaveableType saveables = SaveableType.valueOf(item.toUpperCase());
        saveableMap.get(saveables).save(p);
    }
}

Answer 2

不是真正的减少if-else丛林的解决方案。但这可能看起来更“漂亮”。

http://www.w3schools.com/js/js_switch.asp

所以在你的情况下：

switch(item) {
    case("Product"): pr.save(item); break;
    // and so on... 
}

Answer 3

创建具有保存方法的界面
在您的对象中实现该方法
在您的控制器中创建一个地图，以便根据您现在在if

代码：

gettext

如何从我的控制器减少if else jungle？

3 个答案: