Discussion :

[PhiMau]

Bonjour à tous,

J'aurai souhaité avoir votre avis sur cette question.

Sur mon application certaines classes Repository sont un peu trop "chargées".
Par exemple ma classe DocumentRepository possède plus de 15 méthodes et environ 500 lignes. Certaines de ces requêtes n'ont rien avoir entre elles, bien qu'elles soient liées à la même table.

Pensez-vous que diviser cela en sous-classes soit une pratique acceptable ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
/**
 * @method Document|null find($id, $lockMode = null, $lockVersion = null)
 * @method Document|null findOneBy(array $criteria, array $orderBy = null)
 * @method Document[]    findAll()
 * @method Document[]    findBy(array $criteria, array $orderBy = null, $limit = null, $offset = null)
 */
class DocumentRepository extends ServiceEntityRepository
{
    public function __construct(ManagerRegistry $registry)
    {
        parent::__construct($registry, Document::class);
    }
}

Code :

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

1
2
3
4
 
class RapportRepository extends DocumentRepository {
 
}

Merci d'avance pour vos avis

[Alexandre T]

Bonjour,

Je pense que c'est une mauvaise pratique d'avoir plusieurs dépôts pour la même entité parce que :

• les autres développeurs ne sauront pas dans quelle classe se trouve la méthode de façon intuitive (mais cela peut se pallier par une bonne documentation)
• certaines classes du framework font automatiquement appel au dépôt en fonction du nom de l'entité (le nm de la classe est d'ailleurs le second argument du registre (RegistryManager).

Mais, tu as raison, des classes avec trop de méthodes sont souvent signe d'un défaut de conception. Pour ma part, dans un repository, je ne place que des méthodes qui n'affectent que l'entité en cours ou qui effectue des recherches très simples. Dès que plusieurs entités sont concernées, je crée une classe EntityModel qui prend en argument du constructeur, le manager d'entité. Dans ce constructeur, je charge des propriétés privées qui sont les différents dépôts affectés.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<?php
namespace App\Model;
 
class FooModel
{
    public function __construct(EntityManagerInterface $em)
    {
        $this->entityManager = $em;
        $this->repository = $em->getRepository(MonEntite::class);
        $this->otherRepository = $em->getRepository(SecondeEntite::class);
    }
}

Après c'est une question d'organisation, donc toute l'équipe doit en discuter et une fois que tout le monde est ok, on fait une petite note de synthèse (de préférence accessible depuis le code, un readme, un markdown, etc.) et on s'y tient

[PhiMau]

Bonjour,

Merci pour cette réponse qui a le mérite d'être claire.

Cela va m'aider à peser le pour et le contre, et aussi à nettoyer certains de mes repository.

Bonne journée

[Alexandre T]

De rien ! N'oublie pas le Tag "résolu" comme la réponse te convient.