Serait-il acceptable de mettre un calcul du «prix moyen», comme ci-dessous, dans la vue?Prix moyen - logique appropriée à effectuer dans la vue ou mieux dans le contrôleur?
Ou est-ce contre MVC et est-il préférable de le faire dans le contrôleur?
<p>Average price: <%= @seller.total_sales/@seller.num_sales %></p>
Aucune. Mettez-le dans le modèle. Ensuite, il devient facile de tester un appareil.
Demandez vous plusieurs choses:
Est-ce que ce prix moyen sera souvent affiché Est-ce une partie d'une vue (? Est-il utilisé pour afficher quelque chose) -t-il besoin de choses complexes pour obtenir/calcul/récupérer ou peu importe ?
Si vous pensez que c'est juste un indice pour votre utilisateur, il est utilisé une seule fois, alors vous pouvez le laisser à votre vue.
Mais si vous vous sentez unconfortable avec elle, ou vous devez faire des mathématiques plus complexes sur le prix, le mettre votre modèle ..
"Si vous pensez que c'est juste un indice pour votre utilisateur, il est utilisé une seule fois, alors vous pouvez le laisser à votre vue." - Et si la police de MVC m'attrape? – eggdrop
Mettez votre logique métier où il appartient dans le modèle:
<p>Average price: <%= @seller.get_average_price() %></p>
Il est également assez facile de tester dans le contrôleur, mais il est clair que cela devrait être une propriété du modèle, sinon vous devrez le calculer dans chaque action du contrôleur où cela est nécessaire. – tvanfosson
Je ne suis pas familier avec les tests unitaires, désolé. Quelle serait l'importance de tester ces unités? – eggdrop
En effet. La logique métier appartient au modèle. –