模板方法模式

Question

如果我的 childClass 方法 getInfoFromDB() 和 saveToDB() 需要执行不同的逻辑，我可以知道如何创建 childClass 吗？

public abstract class BaseClass {
    public abstract Object doTransaction();
    public Object executeTrans() {
          //do something
          tx.begin();            
          this.doTransaction();
          tx.commit();

    }
}
public childClass extends BaseClass{
    @Override
    public Object doTransaction(){
        //overide to get something from database so can only be used for getInfoFromDB() and not for saveToDB()
        return something;
    }
    public List<String> getInfoFromDB(){
        super.executeTrans();
    }
    public void saveToDB(){
        super.executeTrans() ;
    }
}

Answer 1

在这种情况下，您应该使用模板模式，如下

public abstract class BaseClass 
{      
  public Object executeTrans(Template template) 
  {
    tx.begin();            
    template.doTransaction();
    tx.commit();    
  }
}

public interface Template
{
  public void doTransaction();
}

public childClass extends BaseClass
{
  public List<String> getInfoFromDB()
  {
    executeTrans(
      new Template()
      {
        public void doTransaction() 
        {
          ...do get info from DB here.
        }
      }
    );
  }

  public void saveToDB()
  {
    executeTrans(
      new Template()
      {
        public void doTransaction() 
        {
          ...do save to DB here.
        }
      }
    );
  }
}

所示： 话虽如此，我建议使用 Spring JDBC 模板类，而不是滚动您自己的 - 它们已经过尝试和测试，并且已经解决了您将遇到的问题嵌套事务等

Answer 2

将包含不同逻辑的 Runnable 传递给executeTrans() 方法。

但是，我不确定模板方法模式是否真的是您所需要的（以及它如何处理回滚？）。您可能想研究一个框架，例如 Spring，它允许 声明式交易。

Answer 3

缺口，
我要使用的“tx”如下所示。从代码来看，最佳实践是生命周期正常，因为它是由 savetodb() 和 getinfofromdb() 调用的

public abstract class BaseClass 
{      
  public Object executeTrans(Template template) 
  {
        // PersistenceManager pm = ...;
        Transaction tx = pm.currentTransaction();
        try {
            tx.begin();
             template.doTransaction();
            tx.commit();
        } finally {
            if (tx.isActive()) {
                tx.rollback();
            }
        }

  }
}