instanceof vs isAnX()

Question

在 java 中，我正在编写一些 DTO 对象，它们都继承自 AllocationDTO。
然后将这些对象的列表传递到 DAO 对象中以保存到数据库中。

根据要保存AllocationDTO 的哪个子类型，保存逻辑会发生变化（例如，要保存到数据库中的哪个表等）

我发现自己使用这样的代码：

for (AllocationDTO x : listOfAllocationDtos) {
    if (x instanceof ManagerAllocationDTO) {
         Manager m = (x(ManagerAllocationDTO)).getManager();
         // save manager etc to managerallocs
    } else if (x.getId() == AllocationDTO.TYPE_SPECIAL1) {
         // save to specialAlloc1 table
    } else if (x.getId() == AllocationDTO.TYPE_SPECIAL2) {
         // save to specialAlloc2 table
    }
}

ManagerAllocationDTO 有一个额外的字段将分配与经理相关联，但对于 specialalloc1/2 的情况，我没有创建子类型，因为数据的唯一区别是它保存到的表。

我的问题是一个软设计问题 - 这是最好的方法吗？

Answer 1

在没有instanceOf 和 if-else-cascade 的情况下分隔不同实例的一种方法是使用Visitor Design pattern。

• 新接口：AllocationVisitor 为 AllocationDTO 的每个具体子类提供一个方法：

  • 访问(TYPE_SPCIAL1 dto)
  • 访问(TYPE_SPCIAL2 dto)

• AllocationDTO: abstract void acceptVisitor(AllocationVisitor visitor)

• AllocationDTO 的每个 concreate 子类都以这种方式实现 acceptVisitor()： void acceptVisitor(AllocationVisitor visitor){visit(this);} //编译时类型选择正确的访问方式。
• 您的 DTO 实现了 AllocationVisitor 接口（带有一个内部类），创建它的一个实例，并将其传递给 listOfAllocationDtos 的元素。

道：

AllocationVisitor saveVisitor = new AllocationVisitor() {
  visit(TYPE_SPCIAL1 dto) {//what ever you need}
  visit(TYPE_SPCIAL2 dto) {//what ever TYPE_SPCIAL2 needs}
}
for (AllocationDTO x : listOfAllocationDtos) {
  x.visit(saveVisitor);
}