[Spring 2.5] Problème de création d'un service Générique IService<T>

Version imprimable

bonjour,

Je me suis inspiré de cet article http://www.ibm.com/developerworks/ja...enericdao.html pour créer un DAO générique.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
public interface IDAO<T> {
 
	public T get(Long id);
 
	public T getReference(Long id);
 
	public void save(T t);
 
	public void update(T t);
 
	public void remove(T t);

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
 
@Repository
public abstract class AbstractDAOBean<T extends AbstractEntity> implements IDAO<T> {
 
	private static final Logger	log	= Logger.getLogger(AbstractDAOBean.class);
 
	@PersistenceContext(type = PersistenceContextType.EXTENDED)
	protected EntityManager		em;
 
	private final Class<T>		entityClass;
 
	/**
         * Default constructor.
         */
	@SuppressWarnings("unchecked")
	public AbstractDAOBean() {
		this.entityClass = (Class<T>) ((ParameterizedType) getClass().getGenericSuperclass()).getActualTypeArguments()[0];
	}
 
	/**
         * Returns the underlying Hibernate Session of the current EntityManager.
         * 
         * @return the Hibernate Session.
         */
	protected Session getSession() {
		return (Session) em.getDelegate();
	}
 
	protected Class<T> getEntityClass() {
		return entityClass;
	}
 
	public T get(Long id) {
		return em.find(entityClass, id);
	}
 
	public T getReference(Long id) {
		return em.getReference(entityClass, id);
	}
 
	public void save(T t) {
			if (t.getId() == null) {
				// new
				em.persist(t);
			}
			else {
				// update
				em.merge(t);
			}
		log.debug(entityClass.getSimpleName() + "Saves");
	}
 
	public void update(T t) {
		em.merge(t);
	}

Lorsque j'appelle toutes les classes fils de mon DAO générique depuis une seule classe service. Pas de problème tout marche parfaitement.

En revanche pour éviter de dupliquer les même fonctions de la couche services dans toutes les classes qui en hérite. J'ai voulu créer une classe Service Générique.

Cette interface << IService<T> >> liste toutes les fonctions CRUD dans mon IDAO.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public interface IService<T> {
 
	public T get(Long id);
 
	public T getReference(Long id);
 
	public void save(T t);
 
	public void update(T t);
 
	public void remove(T t);
}

Une classe AbstractService générique implémente l"interface IService<T>

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
@Service
@Transactional
public abstract class AbstractServiceBean<T extends AbstractEntity> implements IService<T> {
 
	// couche [dao]
	public IDAO<T>	dao;
 
	public IDAO<T> getDao() {
		return dao;
	}
 
	@Autowired
	public void setDao(IDAO<T> dao) {
		this.dao = dao;
	}
 
	private final Class<T>	entityClass;
 
	/**
         * Default constructor.
         */
	@SuppressWarnings("unchecked")
	public AbstractServiceBean() {
		this.entityClass = (Class<T>) ((ParameterizedType) getClass().getGenericSuperclass()).getActualTypeArguments()[0];
	}
 
	protected Class<T> getEntityClass() {
		return entityClass;
	}
 
	public T get(Long id) {
		return dao.get(id);
	}
 
	public T getReference(Long id) {
		return dao.getReference(id);
	}
 
	public void save(T t) {
		dao.save(t);
	}
 
	public void update(T t) {
		dao.update(t);
	}
 
	public void remove(T t) {
		dao.remove(t);
	}

Je me fait également injecter les différents DAO dans les différents services comme suit

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
@Service("RequestService")
@Transactional
public class RequestServiceBean extends AbstractServiceBean<Request> implements RequestService {
	private static final Logger	log	= Logger.getLogger(RequestServiceBean.class);
 
	// Inject couche [dao]
	private RequestDao			dao;
 
	public RequestDao getDao() {
		return dao;
	}
 
	@Autowired
	public void setDao(RequestDao dao) {
		this.dao = dao;
	}
   ....

Lorsque je veut me faire injecter tous mes services lors de l'initialisation de l'application :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
 
	ApplicationContext ctx = new ClassPathXmlApplicationContext("spring-config.xml");
 
       @Autowired
	private PDVService		pdvService;
 
	@Autowired
	private InfosService	infosService;
 
	@Autowired
	private RequestService	requestService;
 
	@Autowired
	private PageService		pageService;
 
	@Autowired
	private XmlFileService	xmlFileService;
 
	@Autowired
	private TestService		testService;

Spring me génère cette erreur :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
Exception in thread "main" org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'InfosService': Autowiring of methods failed; nested exception is org.springframework.beans.factory.BeanCreationException: Could not autowire method: public void fr.stime.service.AbstractServiceBean.setDao(fr.stime.dao.IDAO); nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No unique bean of type [fr.stime.dao.IDAO] is defined: expected single matching bean but found 6: [infosDaoBean, pageDaoBean, PDVDaoBean, requestDaoBean, testDaoBean, xmlFileDaoBean]
	at org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor.postProcessPropertyValues(AutowiredAnnotationBeanPostProcessor.java:256)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.populateBean(AbstractAutowireCapableBeanFactory.java:998)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:472)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory$1.run(AbstractAutowireCapableBeanFactory.java:409)
	at java.security.AccessController.doPrivileged(Native Method)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:380)
	at org.springframework.beans.factory.support.AbstractBeanFactory$1.getObject(AbstractBeanFactory.java:264)
	at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:222)
	at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:261)
	at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:185)
	at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:164)
	at org.springframework.beans.factory.support.DefaultListableBeanFactory.preInstantiateSingletons(DefaultListableBeanFactory.java:429)
	at org.springframework.context.support.AbstractApplicationContext.finishBeanFactoryInitialization(AbstractApplicationContext.java:728)
	at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:380)
	at org.springframework.context.support.ClassPathXmlApplicationContext.<init>(ClassPathXmlApplicationContext.java:139)
	at org.springframework.context.support.ClassPathXmlApplicationContext.<init>(ClassPathXmlApplicationContext.java:93)
	at fr.stime.manager.Injector.init(Injector.java:10)
	at fr.stime.test.InitDB.main(InitDB.java:49)
Caused by: org.springframework.beans.factory.BeanCreationException: Could not autowire method: public void fr.stime.service.AbstractServiceBean.setDao(fr.stime.dao.IDAO); nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No unique bean of type [fr.stime.dao.IDAO] is defined: expected single matching bean but found 6: [infosDaoBean, pageDaoBean, PDVDaoBean, requestDaoBean, testDaoBean, xmlFileDaoBean]
	at org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor$AutowiredMethodElement.inject(AutowiredAnnotationBeanPostProcessor.java:543)
	at org.springframework.beans.factory.annotation.InjectionMetadata.injectMethods(InjectionMetadata.java:117)
	at org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor.postProcessPropertyValues(AutowiredAnnotationBeanPostProcessor.java:253)
	... 17 more
Caused by: org.springframework.beans.factory.NoSuchBeanDefinitionException: No unique bean of type [fr.stime.dao.IDAO] is defined: expected single matching bean but found 6: [infosDaoBean, pageDaoBean, PDVDaoBean, requestDaoBean, testDaoBean, xmlFileDaoBean]
	at org.springframework.beans.factory.support.DefaultListableBeanFactory.resolveDependency(DefaultListableBeanFactory.java:621)
	at org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor$AutowiredMethodElement.inject(AutowiredAnnotationBeanPostProcessor.java:499)
	... 19 more

Il me dit que IDAO n'est pas un Bean unique ! je dois avoir une @Autowired en plus ou en moins dans un mauvais endroit !

Avez-vous une idée de quoi ce peut prevenir et comment résoudre ce problème ?

Merci par avance de votre aide.

30/12/2008, 07h41
sewatech

Effectivement, tu as trop de @Autowired. Tu ne peux pas injecter automatiquement tes dao dans ta classe de service. Il faut donc que tu supprimes cette annotation de AbstractServiceBean et que tu injectes explicitement tes dao dans les sous-classes : @Qualifier ou @Resource si tu as le JDK6.
30/12/2008, 09h06
Tommy31

Tu dois mettre l'annotation @Autowired dans le setter de la classe RequestServiceBean (et pour toutes les filles) et non dans la classe abstraite.

Erreur Service Générique

Bonjour,

J'ai déjà testé ça. Lorsque j'enlève @Autowired de la classe abstraite AbstractServiceBean et j'essaye d'appeler la fonction save(T t) depuis un service de la classe fille :

Code:

1
2
3
4
5
6
7
 
@Autowired
private PDVService		pdvService;
....
PDV t = new PDV();
t.setNAme("toto");
pdvService.save(t);

j'ai cette erreur :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
Exception in thread "main" java.lang.NullPointerException
	at fr.stime.service.AbstractServiceBean.save(AbstractServiceBean.java:64)
	at fr.stime.service.AbstractServiceBean.save(AbstractServiceBean.java:1)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:307)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:182)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:149)
	at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:106)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:171)
	at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:204)
	at $Proxy40.save(Unknown Source)
	at fr.stime.test.InitDB.main(InitDB.java:58)

Comme il n'arrive pas à injecter le IDAO<T> dans le service abstrait, j'ai java.lang.NullPointerException sur la fonction save ==> la variable dao est à null :cry:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 
@Service
@Transactional
public abstract class AbstractServiceBean<T extends AbstractEntity>
		implements
			IService<T> {
 
	// couche [dao]
	public IDAO<T> dao;
 
	public IDAO<T> getDao() {
		return dao;
	}
 
	// @Autowired
	public void setDao(IDAO<T> dao) {
		this.dao = dao;
	}
 
	private final Class<T> entityClass;
 
	/**
         * Default constructor.
         */
	@SuppressWarnings("unchecked")
	public AbstractServiceBean() {
		this.entityClass = (Class<T>) ((ParameterizedType) getClass()
				.getGenericSuperclass()).getActualTypeArguments()[0];
	}
 
	protected Class<T> getEntityClass() {
		return entityClass;
	}
 
	public T get(Long id) {
		return dao.get(id);
	}
 
	public T getReference(Long id) {
		return dao.getReference(id);
	}
 
	public void save(T t) {
		dao.save(t);
	}

30/12/2008, 10h42
sewatech
En enlevant tout les @Autowired, tu n'injectes plus du tout de dao. Il faut que tu fasses une injection explicite dans la sous-classe. Les façons de faire avec annotations sont :
- Redéfinir setDao, et l'annoter
- Faire de l'injection par constructeur

Je n'ai pas compris ta réponse :cry:

Peux-tu me donne plus d'explication STP ?

Je n'ai pas enlevé toutes les @Autowired de toutes les classes filles ! Je l'ai enlevé juste de la classe abstraite et je l'ai gardé dans toutes les classes filles

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
public abstract class AbstractServiceBean<T extends AbstractEntity>
		implements
			IService<T> {
 
	// couche [dao]
	public IDAO<T> dao;
 
	public IDAO<T> getDao() {
		return dao;
	}
 
	// @Autowired
	public void setDao(IDAO<T> dao) {
		this.dao = dao;
	}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
@Service("PDVService")
@Transactional
public class PDVServiceBean extends AbstractServiceBean<PDV> implements PDVService {
	private static final Logger	log	= Logger.getLogger(PDVServiceBean.class);
 
	// Inject couche [dao]
	private PDVDao				dao;
 
	public PDVDao getDao() {
		return dao;
	}
 
	@Autowired
	public void setDao(PDVDao dao) {
		this.dao = dao;
	}

Merci

Désolé, j'avais mal regardé.

Ce qui ne va pas dans ton code, c'est que tu redéfinis ton champs dao. Laisse-le dans la classe abstraite uniquement. Ensuite, dans la sous-classe, injecte explicitement, en précisant son nom, le bon DAO. C'est ici que mon message précédent redevient pertinent : par setter ou par constructeur.

Tu peux jeter un coup d'oeil ici pour la configuration du @Qualifier ou du @Resource.

Moyennant quelques améliorations de généricité, cela devrait ressembler à ceci :
Code:

1 2 3 4 5 6 7 8 9 10 @Service("PDVService") @Transactional public class PDVServiceBean extends AbstractServiceBean<PDV> implements PDVService { private static final Logger log = Logger.getLogger(PDVServiceBean.class); @Autowired @Qualifier(PDVDao) public void setDao(PDVDao dao) { super.setDao(dao); } }
Si cela ne marche pas, essaie par constructeur.

Génial !

Merci beaucoup sewatech !
Ca marche parfaitement. Je n'ai pas pensé à injecter le Dao dans la classe abstraite en passant par le super :oops:

Pour ceux qui sont intéressé par la solution, voici le code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 @Service("TestService") @Transactional public class TestServiceBean extends AbstractServiceBean<Test> implements TestService { private static final Logger log = Logger.getLogger(TestServiceBean.class); // Inject couche [dao] private TestDao testdao; public TestDao getTestdao() { return testdao; } @Autowired public void setDao(TestDao dao) { super.setDao(dao); this.dao = dao; }

30/12/2008, 11h56
sewatech

A quoi sert le champ testdao, dans ton exemple ?

La classe TestDao définit une autre fonction non déclarée dans l'interface générique IDAO<T>
Code:

1 2 3 4 5 public interface TestDao extends IDAO<Test> { public Test getTestByStep(ProjectStepType step); }
Donc, pour pouvoir invoquer cette fonction qui n'est pas visible au niveau de la classe service abstraite, je suis obligé de définir le champ testdao.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 // Inject du [Testdao] private TestDao testdao; @Autowired public void setDao(TestDao dao) { super.setDao(dao); } public Test getTestByStep(ProjectStepType step) { log.debug(TestServiceBean.class.getSimpleName() + " Call service"); return testdao.getTestByStep(step); }
D'ailleurs, tj dans la même problématique, aurais-tu une solution à me proposer concernant mon problème de Rollback ?
http://www.developpez.net/forums/d66...k/#post3897509
Merci !