0

我的应用程序使用的是 DAO 模式。我是否应该将实现方法设为“静态”?

我说的是这个具体的实现:

  • findByIdentifier(字符串标识符)

但也关于通用实现:

  • E getByID(K ID);
  • 列出 findAll();
  • 保存(E实体);
  • 更新(E实体);
  • 删除(E实体);

感谢您的建议!

public interface DaoBase<K, E> {
    E getByID(K ID);
    List<E> findAll();
    void save(E entity);
    void update(E entity);
    void delete(E entity);
}



public interface DaoLanguage extends DaoBase<Long, LanguageEntity> {    
    LanguageEntity findByIdentifier(String identifier);
}

我正在使用 JPA 来持久化所有实体类。

public class JpaDaoLanguage extends JpaDaoBase<Long, LanguageEntity> implements DaoLanguage {
    public LanguageEntity findByIdentifier(String identifier) {
        LanguageEntity language = (LanguageEntity) entityManager.createQuery(
                "select lan from LanguageEntity as lan where lan.identifier = ?1")
                .setParameter(1, identifier)
                .getSingleResult();
        return language;
    }
}

通用基础实现

public abstract class JpaDaoBase<K, E> implements DaoBase<K, E> {
    protected EntityManager entityManager;
    protected Class<E> entityClass;

    @SuppressWarnings("unchecked")
    public JpaDaoBase() {
        ParameterizedType genericSuperclass = (ParameterizedType) getClass()
                .getGenericSuperclass();
        this.entityClass = (Class<E>) genericSuperclass
                .getActualTypeArguments()[1];

        EntityManagerFactory factory = Persistence
                .createEntityManagerFactory("CyberLabPersistenceUnit");
        entityManager = factory.createEntityManager();
    }

    @Override
    public E getByID(K ID) {
        return entityManager.find(entityClass, ID);
    }

    @SuppressWarnings("unchecked")
    @Override
    public List<E> findAll() {
        Query q = entityManager.createQuery("SELECT e FROM "
                + entityClass.getName() + " e ");
        return (List<E>) q.getResultList();
    }

    @Override
    public void save(E entity) {
        entityManager.persist(entity);
    }

    @Override
    public void update(E entity) {
        entityManager.merge(entity);
    }

    @Override
    public void delete(E entity) {
        entityManager.remove(entity);
    }
}
4

1 回答 1

1

试试这个:为你拥有的使用这个 DAO 的代码编写一个单元测试。

您的目标之一应该是快速通过的测试,并且不依赖外部资源(一个好的单元测试不应该要求存在实际的数据库。)因此,不要让您的测试调用真正的 DAO,而是创建一个模拟 DAO,它返回一个固定的语言实体,并让您的测试代码使用模拟而不是真正的 DAO。这仍然有效吗?如果真实的 DAO 具有这些方法的静态实现,您能否成功地将模拟 DAO 替换为真实的 DAO?

静态使您的代码变得脆弱。尽可能避免它们。

于 2013-08-01T14:58:37.460 回答