Discussion :

[*alexandre*]

J'ai un comportement extrêmement bizzard lorsque j'ajoute un élément à un Set dans une relation manyToMany

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
	/** The adresse of the personne.  */
	@ManyToMany(fetch = FetchType.EAGER)
	@Cascade((org.hibernate.annotations.CascadeType.SAVE_UPDATE))
	private Set<Adresse> adresses;

j'ai mon getter / setter sur mon set et j'ai également une méthode qui me permet d'ajouter à la collection une nouvelle entité

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
public void addAdresse(Adresse adresse) {
		Iterator<Adresse> iterator = this.getAdresses().iterator();
		Set<Adresse> newSet = new HashSet<Adresse>();
		newSet.add(adresse);
		while (iterator.hasNext()) {
			if (!newSet.contains(iterator.next())) {
				newSet.add(iterator.next());
			}
		}		
		this.setAdresses(newSet);
	}

Le problème que je rencontre est lorsque je fais le getAdresses() le set récupéré ne contient que la nouvelle adresse. Je ne fais aucun autre appel en modification à ce Set.

Une idée ?

[tchize_]

Ici:
tu fais deux fois appel à iterator.next()

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
			if (!newSet.contains(iterator.next())) {
				newSet.add(iterator.next());
			}

Donc ce code reviens à dire en pseudo code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
si newSet ne contient pas l'élément x
  alors y stocker l'élément x+1

de plus l'ensemble du code de la fonction pourrait avantageusement etre remplacé par

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
public void addAdresse(Adresse adresse) {
		this.getAdresses().add(adresse);
	}

[*alexandre*]

C'est ce que j'avais fait initialement mais j'ai toujours le même problème. La méthode selectedBeneficiaire.getAdresses(); me renvoye un set qui contient déjà les valeurs modifier avant même que j'y ajoute ma nouvelle adresse

Dans ce cas si j'ai bien un Set qui contient deux Adresse mais celles-ci sont identiques.

[tchize_]

Est-ce que ta classe Adresse implemente correctement equals et hashcode? Un set permet justement d'éviter les doublons.

Et j'ai pas trop compris ta phrase là:

La méthode selectedBeneficiaire.getAdresses(); me renvoye un set qui contient déjà les valeurs modifier avant même que j'y ajoute ma nouvelle adresse

de quelle valeur tu parle, comment tu crée ta nouvelle adresse avant de l'ajouter?