Opinem sobre meu DAO GENERICO

JavaChild Membro desde: 08/03/2007 18:12:44 Mensagens: 100 Offline

E ai pessoal,

Estou criando uma aplicacao aqui e ao invés de criar um DAO para cada entitdade eu fiz um DAO que dá pra utilizar com qualquer tabela! Mas queria umas opiniões se é legal se ficou bom e tudo!

aguardo as críticas

import br.com.sysaguas.entidade.DAO.EntityInterface;
import br.com.sysaguas.entidade.Tipoquarto;
import javax.persistence.EntityManager;
import javax.persistence.EntityManagerFactory;
import javax.persistence.NamedQuery;
import javax.persistence.Persistence;
import javax.persistence.Query;

public class GenericDAO<T> implements GenericRepository<T> {
    
    private EntityManagerFactory emf = null;
    private EntityManager em = null;
    
    public GenericDAO() {
        if(emf == null){
            this.emf = Persistence.createEntityManagerFactory("SysAguasPU");
        }
    }
    
    public void persist(T instance) {
        em = emf.createEntityManager();
        try{
            em.getTransaction().begin();
            em.persist(instance);
            em.getTransaction().commit();
        } finally {
            em.close();
        }
    }
    public void update(T instance) {
        em = emf.createEntityManager();
        try{
            em.getTransaction().begin();
            em.merge(instance);
            em.getTransaction().commit();
        } finally {
            em.close();
        }
    }
    public void delete(Integer codigo, String tipoObjeto) {
        //  if(!em.isOpen()){
        em = emf.createEntityManager();
        //   }
        try{
            em.getTransaction().begin();
            em.remove(findToRU(codigo,tipoObjeto));
            em.getTransaction().commit();
        } finally {
            em.close();
        }
    }
    private T findToRU(Integer id,String tipoObjeto) throws IllegalStateException{
        String sql = ("SELECT t FROM "+ tipoObjeto +" t WHERE t.codigo = :codigo");
        Query e = em.createQuery(sql);
        e.setParameter("codigo",id);
        return (T) e.getSingleResult();
    }
    
    public T findById(Integer id,String tipoObjeto) throws IllegalStateException{
        //    if(!em.isOpen()){
        em = emf.createEntityManager();
        //     }
        try{
            String sql = ("SELECT t FROM "+ tipoObjeto +" t WHERE t.codigo = :codigo");
            Query e = em.createQuery(sql);
            e.setParameter("codigo",id);
            return (T) e.getSingleResult();
        }finally {
            em.close();
        }
    }
}

JavaGuru Membro desde: 08/09/2004 14:40:16 Mensagens: 219 Offline

Um DAO reutilizável. Parabéns,

JavaChild Membro desde: 08/03/2007 18:12:44 Mensagens: 100 Offline

christielencc wrote:Um DAO reutilizável. Parabéns,

vlw! mas podem criticar! se ele me gerar algum problema futuro no desenvolvimento da aplicação! só quero mesmo aprender os pros e contras e se tem uma forma melhor ! mas vlw hehehe!

JavaGuru Membro desde: 27/05/2003 10:10:10 Mensagens: 241 Offline

Obs: Uma ótima pratica seria você colocar as transactions na sua camada de serviço, deixando o acesso para o DAO...

os metodos de consulta podem ser melhorados.
de uma olhada nesses links:
http://www.hibernate.org/328.html
http://www.ibm.com/developerworks/java/library/j-genericdao.html

mesma ideia, so usando hibernate
bons estudos.

[]'s

normalemnte a transação fica fora do DAO, embnora eu não use assim, mas as boas práticas dizem que uma transação não é responsabilidade do DAO...e sim da classe de negócios...

se vc inicia uma transação e vai salvando vários objetos diferentes aí no último save dá um erro...vc simplesmente não faz commit e dá um rollback...

do jeito que vc fez já foi feito o commit e já era....de resto esta bacana.

jonimca wrote:Obs: Uma ótima pratica seria você colocar as transactions na sua camada de serviço, deixando o acesso para o DAO...

eu usaria o Spring para controlar minhas transacoes, nao escreveria mais essas mesmas linhas de codigo varias vezes e me preocuparia somente com o business da minha aplicação.

[]'s

JavaChild Membro desde: 08/03/2007 18:12:44 Mensagens: 100 Offline

jonimca wrote:Obs: Uma ótima pratica seria você colocar as transactions na sua camada de serviço, deixando o acesso para o DAO...

explica melhor hehehehe!

JavaChild Membro desde: 08/03/2007 18:12:44 Mensagens: 100 Offline

Giulliano wrote:normalemnte a transação fica fora do DAO, embnora eu não use assim, mas as boas práticas dizem que uma transação não é responsabilidade do DAO...e sim da classe de negócios...

se vc inicia uma transação e vai salvando vários objetos diferentes aí no último save dá um erro...vc simplesmente não faz commit e dá um rollback...

do jeito que vc fez já foi feito o commit e já era....de resto esta bacana.

Me da um exemplo só pra mim visualizar melhor!

eu entendi que a parte de começar uma transação não deveria estar no dao não é isso??

como seria então mais ou menos?

Olha se estou pensando certo::

Dentro da camada de negocios teria um Factory que me criaria um entitymanager e abriria uma transação e devolveria esse entitymanager para mim!

ai todo metodo do DAO teria que tbém receber o entitymanager que tá com a transação em aberto para fazer as operações no banco????[b]

Acho desnecessário a utilização de um DAO quando você utiliza JPA, eu constumo colocar as consultas com @NamedQuery na Entidade quando referencia apenas uma entidade e no Pacote quando referencia várias entidades. O EntityManager pode ser injetado nas classes de serviço e ele genérico o suficiente para realizar todas as operações de persistência.

só a minha opnião.

eu utilizo em alguns DAO generico q faço um cara chamado Param que contém um Map<String, Serializable> onde coloco atributos para preencher uma query dinâmica, fica muito bom e prático! Fico bom o seu DAO

Titôsca wrote:

Giulliano wrote:normalemnte a transação fica fora do DAO, embnora eu não use assim, mas as boas práticas dizem que uma transação não é responsabilidade do DAO...e sim da classe de negócios...

se vc inicia uma transação e vai salvando vários objetos diferentes aí no último save dá um erro...vc simplesmente não faz commit e dá um rollback...

do jeito que vc fez já foi feito o commit e já era....de resto esta bacana.

Me da um exemplo só pra mim visualizar melhor!

eu entendi que a parte de começar uma transação não deveria estar no dao não é isso??

como seria então mais ou menos?

Olha se estou pensando certo::

Dentro da camada de negocios teria um Factory que me criaria um entitymanager e abriria uma transação e devolveria esse entitymanager para mim!

ai todo metodo do DAO teria que tbém receber o entitymanager que tá com a transação em aberto para fazer as operações no banco????[b]

Pesquisa sobre o pattern "open session in view"

jgbt wrote:

jonimca wrote:Obs: Uma ótima pratica seria você colocar as transactions na sua camada de serviço, deixando o acesso para o DAO...

eu usaria o Spring para controlar minhas transacoes, nao escreveria mais essas mesmas linhas de codigo varias vezes e me preocuparia somente com o business da minha aplicação.

[]'s

Tambem concordo que "tecnicamente" o controle transacional deve ficar fora dos DAOs, mas sempre vi coisas desse tipo sem muito problemas. O que eu faço, normalmente é colocar um método no DAO pra controlar isso e criar uma anotação colocada no método de negócio informando o tipo de transação, ou melhor, se o commit deve ser feito ao fim ou não. Se eu quizer no meio do método comitar, eu chamo o método do DAO. A abertura
da transação fica, no meu caso, normalmente em um interceptor.

heitor.rapcinski wrote:Acho desnecessário a utilização de um DAO quando você utiliza JPA, eu constumo colocar as consultas com @NamedQuery na Entidade quando referencia apenas uma entidade e no Pacote quando referencia várias entidades. O EntityManager pode ser injetado nas classes de serviço e ele genérico o suficiente para realizar todas as operações de persistência.

só a minha opnião.

Estive pensando nisso esses dias. Esse é o primeiro projeto grande que eu utilizo JPA. São 600 entidades + ou -... já imaginaram a zona que iriamos ter no fim do projeto?!?!?

Mas Heitor, como vc imagina fazer este controle? Dentro dos objetos de negocio mesmo ou dentro das entidades?

cassio wrote:Pesquisa sobre o pattern "open session in view"

Intercepting Filter

JavaChild Membro desde: 08/03/2007 18:12:44 Mensagens: 100 Offline

Certo gente.... agora uma dúvida que estou tendo...

geralmente fazemos a implementação de um DAO para cada ENTIDADE né... o certo é isso mesmo ter um DAO pra cada entidade??

Não se utiliza um DAO central que trabalha com todas as entidades nao? E se for necessario algo mais especifico em uma entidade extender do dao padrao e implementar da forma necessaria nesse caso???

Sobre a questão de transação:

Se o dao não ficarai responsavel por isso logo ele não iria criar o entitymanager certo???

Então meu down receberia como parametro um entitymanager que é criado pela minha regra de negocio?

Vlw!

Notícias, artigos e o maior fórum brasileiro sobre Java