JPA remplacer un élément dans une collection

Question

J'utilise les données de printemps jpa. J'ai relation un à plusieurs défini comme ceci:

@OneToMany(mappedBy = "participant", cascade = CascadeType.ALL, orphanRemoval = true) 
    public List<SurveyResponse> getSurveyResponses() { 
     return surveyResponses; 
    } 

J'ai une méthode qui prend une collection de surveyResponses et remplace repsonses existants qui ont une clé correspondante. Malheureusement, lorsque je l'utilise, Hibernate supprime toutes les réponses autres que celles que j'ajoute. Pourquoi cela se passe-t-il?

Je ne sauvegarde jamais les réponses directement, je les mets juste dans la collection et laisse cascade faire le reste. Voici la méthode qui fait le remplacement:

public void updateResponses(List<SurveyResponse> responses) { 
     for (SurveyResponse response : responses) { 
      response.setParticipant(this); 
     } 
     if (this.surveyResponses != null) { 
      Set<String> questionKeys = responses.stream() 
        .map(SurveyResponse::getQuestionKey) 
        .collect(Collectors.toSet()); 
      this.surveyResponses.removeIf(
        surveyResponse -> questionKeys.contains(surveyResponse.getQuestionKey())); 
      this.surveyResponses.addAll(responses); 
     } else this.surveyResponses = responses; 
    } 

Après l'addAll appeler le PersistentBag a un mélange d'ancien (géré) et nouveau (pas encore persisté) réponses. Lorsque j'appelle save (méthode jpa crudRepository de données de printemps) sur l'entité participant, toutes les anciennes réponses sont supprimées. Pourquoi?

Answer 1

Cette question devrait être supprimée, car c'était un bug ailleurs dans mon code. J'ai fait un setter trop compliqué qui causait le problème. Supprimer des questions avec des réponses est déconseillé, alors je réponds à ma propre question ici. Pour la curiosité de quelqu'un, ce fut le mauvais compositeur:

public void setSurveyResponses(List<SurveyResponse> surveyResponses) { 
    if (this.surveyResponses == null) { 
     this.surveyResponses = surveyResponses; 
    } else { 
     this.surveyResponses.clear(); 
     if (surveyResponses != null) { 
      this.surveyResponses.addAll(surveyResponses); 
     } 
    } 
    if (this.surveyResponses != null) { 
     for (SurveyResponse response : this.surveyResponses) { 
      response.setParticipant(this); 
     } 
    } 
} 

Note à moi, setters sont pour mise en veille prolongée à utiliser, non pas pour votre propre code. Si vous avez besoin d'un fantaisiste, faites une nouvelle méthode.

Answer 2

Cela pourrait être la cause de votre problème: http://assarconsulting.blogspot.com/2009/08/why-hibernate-does-delete-all-then-re.html

Je suggère de remplacer le type de collecte de List à Set et voir comment cela se comporte.