Отношения @OneToMany в JPA не работают должным образом

Привет, у меня следующие отношения:

@Entity
public class Athlete implements Serializable {

    private static final long serialVersionUID = 1L;

    @Id
    @GeneratedValue(strategy=GenerationType.AUTO)
    private Integer athleteId;

    @ManyToOne
    @JoinColumn(name = "closetWaiting")
    private Closet closetWaiting; 
....

 @Entity
public class Closet implements Serializable {

    private static final long serialVersionUID = 1L;

    @Id
    @GeneratedValue(strategy=GenerationType.AUTO)
    private Integer closetId;

    @OneToMany(mappedBy = "closetWaiting")
    private List<Athlete> waitingList;
 ....

и в моем классе Bean у меня есть этот код:

  public void addAthleteIntoWaitingList(String closetId, Athlete singleAthlete ) {
        Closet singleCloset=entityManager.find(Closet.class,Integer.parseInt(closetId));

        List<Athlete> waitingList=singleCloset.getWaitingList();        
        waitingList.add(singleAthlete);     
        singleCloset.setWaitingList(waitingList);

        singleAthlete.setClosetWaiting(singleCloset);
    }

который не работает, так как после того, как я вызываю метод addAthleteIntoWaitingList, столбец CLOSETWAITING в таблице Athlete всегда равен нулю.

Я делаю что-то не так здесь? Любое предложение для решения?

Обновлено:

Чтобы понять, что происходит за кодом, я активировал свойства регистрации eclipse в файле persistence.xml. Итак, вот что я получаю, когда вызываю метод:

2019-03-26T10:37:16.862+0100|Fein: SELECT ATHLETEID, FIRSTNAME, GENDER, LASTNAME, PHONENUMBER, closetWaiting FROM ATHLETE WHERE (ATHLETEID = ?)
    bind => [12]
2019-03-26T10:37:16.871+0100|Fein: SELECT CLOSETID, CLOSETRANK, GENDER, ISBROKEN, ISOCCUPIED, RESERVATIONENDED, RESERVATIONSTARTED, athleteId FROM CLOSET WHERE (CLOSETID = ?)
    bind => [5]
2019-03-26T10:37:16.876+0100|Fein: SELECT CLOSETID, CLOSETRANK, GENDER, ISBROKEN, ISOCCUPIED, RESERVATIONENDED, RESERVATIONSTARTED, athleteId FROM CLOSET

Это означает, что JPA получает Athlete с Id=12, а затем находит шкаф с Id=5. Но затем он полностью игнорирует остальную часть кода, где спортсмен добавляется в список ожидания, и не сохраняет его в базе данных.

Покажите теперь реальный вопрос, почему это происходит?

РЕШЕНИЕ:

За исключением решения @Peter, которое отлично сработало, я внес следующие изменения без добавления какого-либо каскадного типа и, что более важно, без вызова метода слияния:

Я изменил способ, которым мой метод получает объект Athlete. Вместо того, чтобы передавать его в качестве параметра, я получаю его через EntityManager.find() внутри моего EJB Bean. Вот как это выглядит:

public void addAthleteIntoWaitingList(String closetId, String athleteId ) {
    Closet singleCloset=entityManager.find(Closet.class, Integer.parseInt(closetId));

    Athlete singleAthlete=entityManager.find(Athlete.class, Integer.parseInt(athleteId));

    List<Athlete> waitingList=singleCloset.getWaitingList();        
    waitingList.add(singleAthlete);     
    singleCloset.setWaitingList(waitingList);

    singleAthlete.setClosetWaiting(singleCloset);
}

Может ли кто-нибудь объяснить мне, почему это сработало? Я думаю, потому что я манипулирую объектом Ahtlete внутри транзакции или нет?

Вероятно, вы не запустили этот код внутри транзакции или забыли зафиксировать транзакцию. Код, который вы разместили, можно было бы улучшить, но он правильный.

JB Nizet 25.03.2019 12:01

Класс бобов? Используете ли вы транзакции EJB или как сохраняется шкаф? Откуда вызывается метод?

Peter Šály 25.03.2019 14:34

@PeterŠály Я вызываю метод addAthleteIntoWaitingList из EJB-компонента LocalBean без сохранения состояния.

Ilias Mertzanidis 26.03.2019 09:33

Вы уже вызываете слияние (туалет)?

Peter Šály 26.03.2019 10:59
Стоит ли изучать PHP в 2026-2027 годах?
Стоит ли изучать PHP в 2026-2027 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Приемы CSS-макетирования - floats и Flexbox
Приемы CSS-макетирования - floats и Flexbox
Здравствуйте, друзья-студенты! Готовы совершенствовать свои навыки веб-дизайна? Сегодня в нашем путешествии мы рассмотрим приемы CSS-верстки - в...
Тестирование функциональных ngrx-эффектов в Angular 16 с помощью Jest
В системе управления состояниями ngrx, совместимой с Angular 16, появились функциональные эффекты. Это здорово и делает код определенно легче для...
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Пользовательский скаляр GraphQL
Пользовательский скаляр GraphQL
Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...
1
4
49
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

Ответ принят как подходящий

Без @OneToMany(cascade=CascadeType.ALL) коллекция не будет сохранена при сохранении родительского объекта.

Если вы изменяете свойство объекта в транзакции, нет необходимости явно вызывать слияние. Если объект выходит за пределы транзакции, он становится DETACHED. Отдельная сущность — это singleAthlete. Слияние необходимо. Транзакция автоматически фиксируется контейнером, выходящим из последнего метода EJB.

Явно вызовите entityManager.merge(closet).

Эй, спасибо за предложение, но это не работает для меня. В то время, когда я добавляю объект singleAthlete, он уже существует. Таким образом, этот каскадный тип вызывает исключение, связанное с нарушением первичного ключа, потому что JPA пытается вставить этот объект еще раз.

Ilias Mertzanidis 26.03.2019 10:21

Явно вызовите entityManager.merge(closet) с приведенным выше каскадированием

Peter Šály 26.03.2019 10:39

Эй, Питер, это сработало. Большое спасибо. Хотя я до сих пор не понимаю, что происходит с контекстом JPA. Я думал, что каждое изменение, внесенное в свойство, сохраняется без явного вызова любого метода обновления или слияния, не так ли?

Ilias Mertzanidis 26.03.2019 17:26

Подробнее о жизненном цикле объекта см. здесь dzone.com/articles/jpa-entity-lifecycle.

Peter Šály 26.03.2019 17:29

Я думаю, прежде чем добавлять "singleAthlete" в список, вы должны сохранить его в базе данных, поэтому перед waitList.add(singleAthlete) попробуйте: entityManager.persist(один спортсмен); Конечно, не забудьте объединить singleCloset и зафиксировать результаты: так что вы, наконец, вызываете: entityManager.merge (один шкаф); entityManager.getTransaction().commit();

Другие вопросы по теме

Похожие вопросы

Какова структура имени метода репозитория JPA для массива типов столбцов?
«MultipleBagFetchException: невозможно одновременно получить несколько сумок» при объединении трех таблиц глубины
Как создать TypedQuery в JPQL, который запрашивает две таблицы, используя неявное соединение?
Выберите количество вхождений каждого статуса в другой таблице, используя критерии jpa
Hibernate.ddl.auto=обновление создает ошибку DDL CommandAcceptanceException
Когда мы используем отношение One To Many с двух сторон, а когда мы используем его с одной стороны
Spring boot API, как предотвратить создание дубликатов данных
Предоставлено не менее 2 параметров, но в запросе присутствует только 1 параметр (параметры) с использованием спецификации и сортировки
JPQL и список кортежей в качестве параметра для операторов SELECT IN
JPA ManyToMany — всегда создавать новое значение в таблице, даже если значение существует