Discussion :

[Tanebisse]

Bonjour,
J'utilise Spring MVC mais pas JPA/Hibernate, je fais donc mes requêtes à la main en SQL.
J'ai un objet Adherent qui contient un objet Personne, quand j'arrive dans ma classe DAO j'arrive avec un adherent et je fais en SQL un update de la table adherent puis juste après un UPDATE de ma table personne.
Sauf que Personne est aussi utilisé pour l'objet auteur qui contient un objet personne. Ça m'embête d'écrire de chacun des DAO (Adherent et Auteur) du code qui met à jour la table personne (avec les mêmes données).
Quelle est la bonne pratique ?
Dois-je créer un DAO personne ? Mais du coup ou l'appeler ? Dans le service ? Dans chacun des DAO Adherent et Auteur ?
Merci d'avance pour vos conseils.

[bilgetz]

Dupliquer du code, c'est une source de futur problème.
Donc oui, je créerai une DAO pour Personne et ferai l'update dans le service (Bien vérifier que le service est transactionnel!).
Si un jour la table personne change, tu n'a qu'un seul endroit ou changer ça, tu limite la responsabilité à une seul DAO, le service étant le responsable de la cohérence du tout.

[Tanebisse]

OK.
Par contre quand je suis dans mon DAO adherent comment récupérer le DAO de personne ? Aujourd'hui je fais un new DaoPersonne en lui passant en paramètre le Datasource car dans ma classe Personne si le DatatSource est en autowired ça ne fonctionne pas. Y-a-t-il une meilleure solution ?

[bilgetz]

new et Spring ne font pas bon ménage !

Jamais au grand jamais tu ne doit faire un new d'une DAO ou d'un Service (sauf dans le cas du Javaconfig mais uniquement dans le fichier @Configuration).

Quand tu fait un new toto(), Spring ne passe pas par dessus, donc rien ne sera autowired.

Normalement, tu devrai avoir 2 DAO et un service.

DAO Adherent

Code :

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
 
public class AdherentDAOImpl implements IAdherentDAO {
 
     private    DataSource datasource;
 
    @Autowired
    public AdherentDAOImpl(DataSource datasource) {
         this.datasource = datasource;
 
   }
 
   @override
  public  void save(Adherent adherent) {
          //Code du save
 
   }
}

DAO Personne

Code :

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
 
public class PersonnetDAOImpl implements IPersonneDAO {
 
     private DataSource datasource;
 
    @Autowired
    public PersonneDAOImpl(DataSource datasource) {
         this.datasource = datasource;
 
   }
 
   @override
  public  void save(Personne personne) {
          //Code du save
 
   }
}

Ton service

Code :

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
 
public classe MonSuperServiceImpl implements IMonSuperServiceImpl  {
 
   @Autowired
   private IPersonneDAO  personneDao;
 
  @Autowired
  private AdherentDAOImpl  adherentDao;
 
   @Override
   public void saveAdherent(mes parametre adherent et personne) {
 
     adherentDao.save(adherent);
     personneDao.save(personne);
 
   }
 
}

Le tous devrait être instancié par Spring

• @Bean sur une methode si Javaconfig
• <bean class="...."> si xml
• DAO et service annoté en @component / @Service si tu utilise le component scan

[Tanebisse]

OK je déplace mon code au niveau du service.
Par contre si j'ai à gérer une transaction j'utilise l'annotation @Transactionnal dans le DAO mais comment je fais si je suis dans le service ?

[Tanebisse]

Je viens de me rendre compte qu'il suffit de mettre l'annotation @transactionnal au niveau du service et non au niveau du DAO.