Yannick Francois
30 lines
1.8 KiB
Markdown
30 lines
1.8 KiB
Markdown
---
|
||
layout: post
|
||
title: Skinny controllers and fat models
|
||
---
|
||
|
||
Hier, j’ai posé une question à la terre entière (ou presque):
|
||
|
||
>
|
||
> MyModel.paginate(blablabla) or just call MyModel.give_data and call paginate in MyModel ?
|
||
>
|
||
|
||
J’ai mis en place une pagination sur une liste de commande. Cela à plusieurs avantages.
|
||
|
||
* Graphique d’abord. On ne peut vraiment pas proposer une interface claire avec une liste de 150 commandes
|
||
* Performances ensuite. Aller cherche 150 commandes d’un coup, c’est faire attendre le client qui finalement n’en fera que 10.
|
||
|
||
Ceci étant dit, sur divers exemple que j’ai observé à droite et à gauche, l’appel au modèle pour récupérer la liste de commandes dans le contrôleur ce fait sous la forme:
|
||
|
||
Je ne détail pas les conditions ici, mais pourtant ce sont elles qui me gène dans ce code. L'appel à `paginate` (ou une autre méthode) dans le contrôleur ne me gène pas, de toute façon, il faut bien qu'il travail un peu, même s'il est _maigre_ ! Mais par contre j'ai beaucoup de mal avec le fait que le contrôleur connaisse les conditions pour la récupération de la liste. D'autant que j'ai dans ces conditions quelque valeur par défaut qui dépende d'un contexte non présent dans les paramètres aujourd'hui.
|
||
|
||
Je vais donc opter pour le déplacement de l'appel à `paginate` dans mon modèle. J'ai donc maintenant dans mon contrôleur quelque chose de plus clair, et plus simple:
|
||
|
||
>
|
||
> Et dans mon model
|
||
>
|
||
|
||
C'est bien beau, mais je me retrouve maintenant avec des éléments de présentation dans mon modèle. *C'est mal*.
|
||
|
||
Je vais pour l'instant passer la page demandé et la quantité par page en paramètre. Ce n’est pas top non plus, mais je préfère cela pour l’instant. Si quelqu’un a une idée, je suis prêt à l’entendre.
|