Discussion :

[speedster]

J'ai une classe utilitaire qui comporte des méthodes pour le calcul d'un classement d'équipes (avec plus ou moins de critères partage).

Une des méthodes

Code :

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

private static final void calculerButsExt(List<Classement> equipesEgalite)

permet de calculer les buts marqués à l'extérieur entre plusieurs équipes à égalité.
Dans le corps de cette méthode, actuellement, je fais un appel dans une boucle

Code :

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

classementService.findCriteresParticuliersExterieurByTournoi(tournoi, uPs, uP1)

qui elle-même fait appel à

Code :

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

classementDAO.findCriteresParticuliersDomicileByTournoi(tournoi, uP1, uPs);

.

Si je comprends bien, actuellement à chaque itération dans la boucle, spring ouvre puis ferme une nouvelle session.

Ne vaudrait-il mieux pas déplacer la méthode

Code :

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

private static final void calculerButsExt(List<Classement> equipesEgalite)

dans la couche services (ClassementService) et remplacer le code

Code :

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

classementService.findCriteresParticuliersExterieurByTournoi(tournoi, uPs, uP1)

par

Code :

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

classementDAO.findCriteresParticuliersDomicileByTournoi(tournoi, uP1, uPs);

.

Est-ce bien le rôle de la couche services ?

[trungsi]

C'est bien ça. Il faudrait éviter les méthodes statiques si possible et les méthodes "business" dans les classes utilitaires. Une autre remarque, "private" et "final" en même temps, c'est redondant

[fr1man]

En quoi l'utilisation conjointe de private et final est-elle redondante ?

La couche métier est la couche qui gère la "logique métier", donc toutes les règles de ton application.

[MangeurVasqué]

En quoi l'utilisation conjointe de private et final est-elle redondante ?

final empêche de redéfinir une méthode dans une sous classe.
Or, une méthode private n'est visible dans une sous-classe donc on ne peut pas la redéfinir de toute manière.

[fr1man]

Ok, je suis d'accord la dessus, mais l'attribut private, évoque la notion
de visibilité, ce que ne fait pas final.
Car une métode "public final" ou "private final" ce n'est pas du tout la même chose.

[MangeurVasqué]

Ok, je suis d'accord la dessus, mais l'attribut private, évoque la notion
de visibilité, ce que ne fait pas final.
Car une métode "public final" ou "private final" ce n'est pas du tout la même chose.

En effet, final est juste superflu (plutôt que redondant) dans le cas d'une méthode private. Et comme il est superflu, il est de coutume de ne pas l'indiquer.

Ou alors l'auteur ne sais pas que c'est superflu et dans ce cas final traduit une intention fausse.

[trungsi]

Désolé, j'étais pas très clair dans mes commentaires.

Merci MangeurVasqué.

[fr1man]

Ceci dit, une méthode private peut très bien être redéfinie dans une classe fille, ce qui n'est pas le cas de la private final.

[trungsi]

Qu'est ce que tu veux dire par "redéfinir une méthode privée" ???

[fr1man]

Oubliez mon dernier message, c'est une anerie