JPA сохраняется, а не обновляется

Question

Я столкнулся с какой-то странной проблемой: всякий раз, когда я пытаюсь редактировать регистр в своем приложении, а не обновлять состояние объекта, он сохраняется в новой. Вот идет мой update метод

public void update(Object obj) { 
    EntityManagerFactory fac = Persistence.createEntityManagerFactory("crud"); 
    EntityManager em = fac.createEntityManager(); 
    em.merge(obj); 
    em.getTransaction().begin(); 
    em.getTransaction().commit(); 
    em.close(); 
    fac.close(); 
} 

Я думаю, когда я отправить форму редактирования и сеттеры называются Id каким-то образом установить на 0. Возможно ли это произойти? Форма выглядит следующим образом:

<h:form> 
       <p:panelGrid columns="2" style="margin: 0 auto;"> 
        <f:facet name="header"> 
         Edit form 
        </f:facet> 
        Name: 
        <h:inputText id="nome" value="#{editUserBean.userToEdit.name}"/> 
        Age: 
        <h:inputText value="#{editUserBean.userToEdit.age}" converter="javax.faces.Integer"/> 
        Pass: 
        <h:inputText value="#{editUserBean.userToEdit.password}"/> 
        Gender: 
        <h:selectOneRadio value="#{editUserBean.userToEdit.gender}" style="font-size: 12px;"> 
         <f:selectItem itemValue="Masc" itemLabel="Masc" /> 
         <f:selectItem itemValue="Fem" itemLabel="Fem" /> 
        </h:selectOneRadio> 
        <f:facet name="footer"> 
         <div align="center"> 
          <h:commandButton value="save" actionListener="#{editUserBean.save}" icon="ui-icon-check"/> 
         </div> 
        </f:facet> 
       </p:panelGrid> 
</h:form> 

Hibernate запрашивает это:

Hibernate: 
    select 
     hibernate_sequence.nextval 
    from 
     dual 
Hibernate: 
    insert 
    into 
     users 
     (age, gender, name, password, id) 
    values 
     (?, ?, ?, ?, ?) 

Answer 1

Реальная проблема заключается в слое JSF.

Все данные указывают, что editUserBean является запрос областью. Это означает, что вы получите новый компонент по запросу.

Итак, мы работаем с новымиuserToEdit bean на каждый запрос. Его свойства будут либо null, либо значением по умолчанию для примитивов (например, 0 для int и long), если вы явно не объявили значение по умолчанию для свойства внутри компонента. Это включает в себя id, который в вашем случае равен 0.

Когда вы объединяете свой bean-компонент, нет сущности с идентификатором 0 в контексте персистентности или соответствующей записи в базе данных, поэтому merge будет сохранять новый объект. Если вы генерируете значения для первичного ключа, то вероятность того, что у вас никогда не будет 0 как id, так что уровень JPA будет продолжать сохранять новые объекты.

Одно простого решение этой проблемы заключается в хранении информации о пользователе в бобе с broader scope, скажем сеанса область действия или определенный пользователь разговора области действия:

@ConversationScoped 
class UserToEdit extends User { 
    // ... or if you want, create a HAVE-A relationship 
} 

Затем вы можете придать свой компонент в качестве обычный, а сохраненный id сохранится между запросами (не забудьте очистить, правильно разграничить границы разговора и/или очистить сеансовый компонент).

---

Другой обходной путь - если вы абсолютно необходимо сохранить данные пользователя в область запроса - использует скрытое поле для хранения значения id (а также любые нередактируемую свойства, которые вы хотите) , Таким образом, пользователь id также будет отправлен по следующему запросу, и модель будет правильно обновлена.

<h:inputHidden value="#{editUserBean.userToEdit.id}" /> 

Предупреждение: Хотя этот способ является довольно популярным, только сделать это, если текущий пользователь имеет возможность обновлять информацию любого другого пользователя. Зачем? Злоумышленник может легко манипулировать DOM, чтобы изменить user.id и/или отправить поддельный запрос, содержащий любые id, чтобы злоупотреблять системой.

Answer 2

Я не знаю, если вы уже пробовали это, но проверьте конфигурационный файл спящего режима ... и затем может быть один тега содержит значение create. Измените его для обновления. Надеюсь, что это помогает

Answer 3

пару вещей ... Во-первых впрыснуть ваши PersistenceContext как это в вашем EJB:

@PersistenceContext 
private EntityManager em; 

Далее, отметьте вашу функцию, или весь EJB с правильной автоматической обработки транзакций:

@Stateless 
@TransactionAttribute(TransactionAttributeType.REQUIRED) 
public class MyServiceEJBThingy { 
    @PersistenceContext 
    private EntityManager em; 

В своем методе не делайте ничего, кроме своей бизнес-логики. sematics транзакций теперь обрабатываются для вас автоматически:

@Stateless 
@TransactionAttribute(TransactionAttributeType.REQUIRED) 
public class MyServiceEJBThingy { 
    @PersistenceContext 
    private EntityManager em; 

    public void update(Object obj) { 
     //GUI independent business logic here 
     em.merge(obj); 
    } 
} 

Наконец, теперь вводит свое EJB в контроллере представления (при условии, что вы используете шаблон MVVM):

@RequestScoped (or something) 
public class EditUserBean { 
    @Inject 
    private MyServiceEJBThingy myServiceEJBThingy; 


    public void update(Object obj) { 
     // GUI logic here 
     myServiceEJBThingy.update(obj); 
    } 
} 

Намного проще да? :) Java может быть очень простой и продуктивной. Я не уверен, какой контейнер вы используете, но я бы предложил использовать Apache TomEE. Проверьте их огромную библиотеку современных примеров here. Удачи!!!