我怎样才能避免这个 if 语句

Question

我有一个枚举

public enum Vehicle {
    CAR("CAR", "Car"), PUSHBIKE("PUSHBIKE", "PuschBike");

    public  boolean isCar()
{
    ...
}

    public  boolean isPushBike()
{
    ....
}
}

我有一个 2 DAO CarDAO 和 PushBikeDAO 正在实现一个 BaseDao

我有一个 JSF 托管 bean 这样的东西

public class JsfManagedBean {

    private Vehicle vehicle;

    private BaseDAO baseDao;

    public void Search()
    {
           //I need to get rid of this if statement
        if (vehicle.isCar())
        {
            baseDao = new CarDao;
            baseDao.search();
        }
        else if(vehicle.isPushBike())
        {
            baseDao = new PushBike;
            baseDao.search();
        }
      //Please Note each type of search is very different call to an other party's Jar
    }
}

我试图摆脱这个如果可能通过使用泛型或任何适当的面向对象技术来声明 可能类似于

baseDao = new baseDaoImpl<getClass(vehicle.getcode())> 

如果 vehicle.getcode() 返回字符串值 Car 我确实有一个模型类 Car。

只是大声思考（真的抓住了吸管:)）。

这是 我的

Answer 1

向枚举添加一个方法，调用 new 并返回正确的 dao。

Answer 2

让每个枚举常量定义它们各自的 DAO 类：

public enum Vehicle {
  CAR("CAR", "Car"){
    public BaseDAO getNewDAO(){
      return new CarDAO();
    }
  },
  PUSHBIKE("PUSHBIKE", "PuschBike"){
    public BaseDAO getNewDAO() {
      return new PushBikeDAO();
    }
  };

  Vehicle(String a, String b){/* ... */}
  //this forces every enum constant to implement a getNewDAO() method:
  abstract BaseDAO getNewDAO();
}

这样，您可以使用：

public void Search() {
  baseDao = vehicle.getNewDAO();
  baseDao.search();
}

看一下 Factory方法模式和策略模式（如果您想知道） 更多的。枚举是我使用后者的首选方式。

Answer 3

我会使用工厂方法，如下所示：

public class JsfManagedBean {

    private static final Map<Vehicle,BaseDAO> daos;
    static {
        Map<Vehicle,BaseDAO> tmp = new HashMap<Vehicle,BaseDAO>();
        tmp.put(Vehicle.CAR,new CarDAO());
        tmp.put(Vehicle.BIKE,new BikeDAO());
        daos = Collections.unmodifiableMap(tmp);
    }
    public static getDao(Vehicle v) {
        return daos.get(v);
    }

    private Vehicle vehicle;

    private BaseDAO baseDao;

    public void Search()
    {
            baseDao = getDao(vehicle);
            baseDao.search();
    }
}

Answer 4

除非您对 DAO 对象有更多用途，否则可以缩短此代码：

if (vehicle.isCar()) new CarDao().search();
else if(vehicle.isPushBike()) new PushbikeDao().search();

有两种选择，我将继续使用 if 语句。如果您确实有很多车辆变体，则可以使用由枚举值作为键的哈希表并存储 DAO 类：

Map<Vehicle, Class> DAOClass = new HashMap<...>();
...
DAOClass.get(vehicle).getConstructor().newInstance().search();

不在这里使用反射并不那么慢。