2016-04-18 36 views
0

我失败了很多时间试着解决这个问题,但我在同一个地方。我怀疑我把CDI和EJB混合在一起。WFLYJPA0060:执行此操作需要事务(使用事务或扩展持久化上下文)

问题是持续存在,删除只是不起作用。

Caused by: javax.persistence.TransactionRequiredException: WFLYJPA0060: Transaction is required to perform this operation (either use a transaction or extended persistence context) 
    at org.jboss.as.jpa.container.AbstractEntityManager.transactionIsRequired(AbstractEntityManager.java:866) 
    at org.jboss.as.jpa.container.AbstractEntityManager.persist(AbstractEntityManager.java:579) 
    at com.oki.scope.console.model.dao.GenericDAO.save(GenericDAO.java:29) 
    at com.oki.scope.console.model.dao.GenericConsoleDAO.save(GenericConsoleDAO.java:12) 
    at com.oki.scope.console.service.ServidorServiceImp.salvar(ServidorServiceImp.java:27) 
    at com.oki.scope.console.service.ServidorServiceImp$Proxy$_$$_WeldClientProxy.salvar(Unknown Source) 
    at com.oki.scope.console.managedBean.consulta.ServidorMB.salvar(ServidorMB.java:65) 
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
    at java.lang.reflect.Method.invoke(Method.java:497) 
    at com.sun.el.parser.AstValue.invoke(AstValue.java:292) 
    at com.sun.el.MethodExpressionImpl.invoke(MethodExpressionImpl.java:304) 
    at org.jboss.weld.util.el.ForwardingMethodExpression.invoke(ForwardingMethodExpression.java:40) 
    at org.jboss.weld.el.WeldMethodExpression.invoke(WeldMethodExpression.java:50) 
    at org.jboss.weld.util.el.ForwardingMethodExpression.invoke(ForwardingMethodExpression.java:40) 
    at org.jboss.weld.el.WeldMethodExpression.invoke(WeldMethodExpression.java:50) 
    at com.sun.faces.facelets.el.TagMethodExpression.invoke(TagMethodExpression.java:105) 
    at javax.faces.component.MethodBindingMethodExpressionAdapter.invoke(MethodBindingMethodExpressionAdapter.java:87) 
    ... 40 more 

吾道

public class GenericDAO<T, K> { 

    protected EntityManager em; 
    private Class<T> entityClass; 

    public GenericDAO(Class<T> entityClass, EntityManager em) { 
     this.entityClass = entityClass; 
     this.em = em; 
    } 

    @Transactional 
    protected void save(T entity) { 
     em.persist(entity); 
    } 

通用DAO:

import javax.persistence.EntityManager; 

public abstract class GenericConsoleDAO<T, K> extends GenericDAO<T, K> { 

    public GenericConsoleDAO(Class<T> entityClass, EntityManager em) { 
     super(entityClass, em); 
    } 

    public void save(T t){ 
     super.save(t); 
    } 

} 

DAO厂:

package com.oki.scope.console.model.dao; 

import java.lang.reflect.InvocationTargetException; 
import java.util.Collections; 
import java.util.HashMap; 
import java.util.Map; 

import javax.ejb.Singleton; 
import javax.ejb.TransactionManagement; 
import javax.ejb.TransactionManagementType; 
import javax.enterprise.inject.Produces; 
import javax.persistence.EntityManager; 
import javax.persistence.PersistenceContext; 

@Singleton 
@TransactionManagement(TransactionManagementType.CONTAINER) 
public class DAOConsoleFactory { 
    private final static String UNIT_NAME = "scope-console"; 

    private static Map<String, Object> mapa = Collections.synchronizedMap(new HashMap<String, Object>()); 

    @PersistenceContext(unitName = UNIT_NAME) 
    private EntityManager entityManager; 

    @Produces public ServidorDAO   criaServidorDAO(){   return getDAO(ServidorDAO.class); } 
    @Produces public ConexaobdDAO   criaConexaoDAO(){   return getDAO(ConexaobdDAO.class);  } 
    @Produces public ContratoDAO   criaContratoDAO(){   return getDAO(ContratoDAO.class); } 
    @Produces public EmpresaDAO    criaEmpresaDAO(){   return getDAO(EmpresaDAO.class); } 
    @Produces public LojaDAO    criaLojaDAO(){    return getDAO(LojaDAO.class); } 
    //@Produces public RedeAutorizadoraDAO criaRedeAutorizadoraDAO(){ return getDAO(RedeAutorizadoraDAO.class); } 
    @Produces public RedeDAO    criaRedeDAO(){    return getDAO(RedeDAO.class); } 
    @Produces public RoteadorDAO   criaRoteadorDAO(){   return getDAO(RoteadorDAO.class); } 
    @Produces public TerminalDAO   criaTerminalDAO(){   return getDAO(TerminalDAO.class); } 
    @Produces public TipoHeaderDAO   criaTipoHeaderDAO(){  return getDAO(TipoHeaderDAO.class);  } 

    @SuppressWarnings("unchecked") 
    public <E> E getDAO(Class<E> classe){ 
     String key = classe.getSimpleName(); 
     if (!mapa.containsKey(key)) 
     { 
      try { 
       mapa.put(key, classe.getDeclaredConstructor(EntityManager.class).newInstance(entityManager)); 
      } catch (InstantiationException | IllegalAccessException | IllegalArgumentException 
        | InvocationTargetException | NoSuchMethodException | SecurityException e) { 
       System.out.println("Classe "+ key +" não possui um construtor que tenha EntityManager como parametro."); 
      } 
     } 
     return (E)mapa.get(key); 

    } 
} 

我的类:

@Named 
@ApplicationScoped 
public class ServidorServiceImp implements ServidorService { 

    @Inject private ServidorDAO dao; 


    @Override 
    public List<Servidor> getLista() { 
     return dao.getLista(); 
    } 

    @Override 
    public void salvar(Servidor servidor) { 
     if (servidor.getId()==0){ 
      dao.save(servidor); 
     } 
     else 
     { 
      dao.update(servidor); 
     } 
    } 

    @Override 
    public void remover(Servidor servidor) { 
     dao.delete(servidor); 

    } 

} 

回答

2

在试图提高性能时,你已经规避了容器应该为你做什么,这是在事务内实例化一个bean。

我想说的是从DAOConsoleFactory中删除@Singleton@TransactionManagement(TransactionManagementType.CONTAINER),并允许EJB事务由使用DAO的EJB bean处理。

UPDATE:另外,@ApplicationScoped不是一个EJB注解类ServidorServiceImp需要有一个EJB豆所以,它应该@Stateless或许@Statefull进行注释,并删除@ApplicationScoped。它看起来像一个无状态的EJB bean,所以不需要使其成为应用程序范围。

同样,在我看来,如果没有很好地理解EJB应该如何在容器中工作,您就会过分关注如何优化性能。我建议让所有的东西都可以工作,并遵循建筑最佳实践,特别是在“会议外观”概念中。其中一些帖子可能有所帮助:What is the point of a Facade in Java EE?Why use Facade pattern for EJB session bean

+0

我删除了这两个注释。这证明@TransactionManagement未被使用。问题保持不变。 –

+0

已添加更新以解答问题。 –

1

解决:2天之前患病。问题出在我的班上。 @Named @ApplicationScoped在哪里,我改为@Statefull。

+1

嗨,哥们儿,这听起来很粗鲁,但是,你的实现非常糟糕。只要看看这本书(http://realworldpatterns.com/),你就会明白所有的错误模式和样板代码,我向你推荐与DAO,SessionBean和Facade模式相关的章节。 –