Discussion :

[JFrancoisB]

Bonjour,

Dans l'exemple suivant, je souhaite conserver l'ordre des loisirs.

Code java :

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

1
2
3
4
5
6
7
8
9
10
@Entity
@Table(name = "personne")
public class Personne {
 
	@OneToMany(fetch = FetchType.LAZY)
	@JoinTable(name = "loisir_personne", joinColumns = {@JoinColumn(name = "personne", nullable = false, updatable = false)}, inverseJoinColumns = {@JoinColumn(name = "loisir", nullable = false, updatable = false)})
	@OrderColumn(name = "idx")
	private List<Loisir> loisirs = new ArrayList<Loisir>(0);
 
}

L'enregistrement et la récupération marche bien, mais si je change l'ordre de la liste je reçois un message de violation de clé primaire:

Code :

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

ERROR: duplicate key value violates unique constraint "pk_loisir_personne"  DétailÂ*: Key (personne, loisir)=(11900, 4) already exists.

Dans ce cas Hibernate essaye de faire un update:

Code :

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

Hibernate: update madb.loisir_personne set loisir=? where personne=? and idx=?

Alors que sans le @OrderColumn, il supprime puis fait un insert (dans ce cas je n'ai pas de message d'erreur évidemment)

Code java :

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

1
2
3
4
@OneToMany(fetch = FetchType.LAZY)
@JoinTable(name = "loisir_personne", joinColumns = {@JoinColumn(name = "personne", nullable = false, updatable = false)}, inverseJoinColumns = {@JoinColumn(name = "loisir", nullable = false, updatable = false)})
//@OrderColumn(name = "idx")
private List<Loisir> loisirs = new ArrayList<Loisir>(0);

Code :

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

1
2
3
4
Hibernate: delete from madb.loisir_personne where personne=?
Hibernate: insert into madb.loisir_personne (personne, loisir) values (?, ?)
Hibernate: insert into madb.loisir_personne (personne, loisir) values (?, ?)
Hibernate: insert into madb.loisir_personne (personne, loisir) values (?, ?)

Est-ce que quelqu'un aurait une solution?
Merci.


SQL:

Code sql :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
CREATE TABLE madb.loisir_personne
(
  personne numeric(10,0) NOT NULL,
  loisir numeric(5,0) NOT NULL,
  idx numeric(2,0) NOT NULL DEFAULT 0,
  CONSTRAINT pk_loisir_personne PRIMARY KEY (personne, loisir),
  CONSTRAINT fk_loispers_pers FOREIGN KEY (personne)
      REFERENCES madb.personne (id) MATCH SIMPLE
      ON UPDATE NO ACTION ON DELETE RESTRICT,
  CONSTRAINT fk_loispers_lois FOREIGN KEY (loisir)
      REFERENCES madb.loisir (id) MATCH SIMPLE
      ON UPDATE RESTRICT ON DELETE RESTRICT
)

-----------------------
DB: PostgreSQL v 9.2
Hibernate v 4.2.1.Final

[jeffray03]

salut
peux-tu nous montrer comment est ce que tu changes l´ordre, ainsi que le contenu avant le changement de l´ordre.

Eric

[JFrancoisB]

Bonjour jeffray03
Merci de t'intéresser à mon problème.

Voici le contenu avant changement:

personne	loisir	idx
11900	2	0
11900	4	1
11900	3	2

Sur l'IHM (gwt), l'utilisateur devrait pouvoir réordonner ses loisirs (en fonction de ses préférences) via Drag & Drop ou autre

Ce que je fais quand un utilisateur envoie une nouvelle list:

Code java :

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

1
2
3
4
5
6
7
8
9
10
11
 
// personneDto vient de la partie client (GWT)
@Override
@Transactional(rollbackFor = Exception.class, propagation = Propagation.REQUIRED)
public Boolean updatePersonne(PersonneDto personneDto) throws Exception {
      Personne personne = personneDao.findById(personneDto.getId());
      List<Loisir> loisirs = loisirDao.findByIds(personneDto.getLoisirs());
      personne.getLoisirs().clear();
      personne.getLoisirs().addAll(loisirs);
      personneDao.update(personne);
}

loisirDao.findByIds() étant:

Code java :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
@Override
public Collection<T> findByIds(List<Long> lIds) throws Exception {
	if (lIds==null||lIds.isEmpty()) return null;
	Collection<T> lst = findByIdsUnsorted(lIds);
	Map<Integer, T> map = new TreeMap<>();
	for (T val : lst){
		map.put(lIds.indexOf(val.getId()), val);
	}
	return new ArrayList<>(map.values());
}
 
private Collection<T> findByIdsUnsorted(Collection<Long> lIds) throws Exception {
	logger.debug("find By Ids {}", clazz.getName());
	if (lIds==null||lIds.isEmpty()) return null;
	try {
		DetachedCriteria criteria = DetachedCriteria.forClass(clazz);
		criteria.add(Restrictions.in("id", lIds));
			List<T> lst = findByCriteria(criteria);
		return lst;
	} catch (RuntimeException re) {
		logger.error("find by property name failed", re);
		throw re;
	}
}

personneDao.update() étant:

Code :

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

1
2
3
4
5
@Override
public T update(final T entity) {
	Preconditions.checkNotNull(entity);
	return getEntityManager().merge(entity);
}

Résultat désiré:

personne	loisir	idx
11900	2	0
11900	4	2
11900	3	1

[JFrancoisB]

En faisant un flush() entre clear() et addAll() ça marche:

Code java :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
// personneDto vient de la partie client (GWT)
@Override
@Transactional(rollbackFor = Exception.class, propagation = Propagation.REQUIRED)
public Boolean updatePersonne(PersonneDto personneDto) throws Exception {
      Personne personne = personneDao.findById(personneDto.getId());
      List<Loisir> loisirs = loisirDao.findByIds(personneDto.getLoisirs());
      personne.getLoisirs().clear();
 
      personneDao.getEntityManager().flush();
 
      personne.getLoisirs().addAll(loisirs);
      personneDao.update(personne);
}

Je n'ai pas l'impression que cela soit une "best practice" et je ne comprends pas le besoin d'utiliser flush() entre les 2 instructions.
PS: j'ai essayé cela par hasard (ou par désespoir) après avoir lu le post: https://forum.hibernate.org/viewtopi...48410#p2448410


Est-ce que quelqu'un peu m'éclairer sur ce point?

Merci

[jeffray03]

salut,
en faisant le flush(), le changement s´effectue au niveau de la base de données, c´est a dire une sorte de commit,
et c´est pour cela que la derniere operation marche, car elle n´est plus presente dans la base de données.


Eric

[JFrancoisB]

Salut Eric et merci pour ton explication.
Je vais donc utiliser flush.

[jeffray03]

salut, pour rappel,
si tu veux que cela marche comme tu veux?
il faudra creer ta table comme ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
CREATE TABLE madb.loisir_personne
(
  personne numeric(10,0) NOT NULL,
  loisir numeric(5,0) NOT NULL,
  idx numeric(2,0) NOT NULL DEFAULT 0,
  CONSTRAINT pk_loisir_personne PRIMARY KEY (personne, loisir, idx_numeric),
  CONSTRAINT fk_loispers_pers FOREIGN KEY (personne)
      REFERENCES madb.personne (id) MATCH SIMPLE
      ON UPDATE NO ACTION ON DELETE RESTRICT,
  CONSTRAINT fk_loispers_lois FOREIGN KEY (loisir)
      REFERENCES madb.loisir (id) MATCH SIMPLE
      ON UPDATE RESTRICT ON DELETE RESTRICT
)

et tu veras que ton premier code marchera sans souci.

Eric

[JFrancoisB]

Excellent, et si simple.
Merci Eric