在java中避免许多if else语句的最好方法

Question

我有一个方法，实现如下

public void setAttributes(Data d) {
  Model m;
  String type = d.getType();
  if (type.equals("TYPE1")) {
    m.setType(type);
    m.setDuration(d.getDuration())
    m.setBenefit(d.getBenefit())
    m.setPermission(d.getPermission());
  } else if (type.equals("TYPE2")) { /* Here we wont be having duration, benefit */
    m.setType(type);
    m.setLab(true)
  }
}

这看起来不是可伸缩的，因为如果将来出现更多的TYPES，那么我们需要不断添加if else块。我能不能重构一下。注意:不同类型的属性会有所不同

Answer 1

您可以为每种类型创建Data的子类，并使用重写将行为单独附加到每个类。如果你计划有很多数据类型和很多相关的逻辑，这是有意义的。

public abstract class Data {
    ...
    public abstract void setAttributes(Model m);
}

Answer 2

考虑为可能的类型定义一个enum。

将getType()的类型修改为enum的类型。

将您的if块转换为switch块，以打开enum。

使用String是一种较弱的实现技术，因为它不是那么类型安全，如果你明白我的意思。

Answer 3

您可以使用switch

switch(type) {
    case "TYPE1":
        m.setType(type);
        m.setDuration(d.getDuration())
        m.setBenefit(d.getBenefit())
        m.setPermission(d.getPermission());
        break;
    case "TYPE2":
       m.setType(type);
       m.setLab(true)
       break;
}

Answer 4

也许switch-case语句适合您(参见:https://docs.oracle.com/javase/tutorial/java/nutsandbolts/switch.html)

这意味着对于您的代码示例:

public void setAttributes(Data d) {
   Model m;
   String type = d.getType();
   switch(type) {
       case "Type1":
           m.setType(type);
           m.setDuration(d.getDuration())
           m.setBenefit(d.getBenefit())
           m.setPermission(d.getPermission());
           break;
       case "TYPE2": 
          m.setType(type);
          m.setLab(true);
          break;
  }
}

希望能有所帮助