Eu vejo em vários exemplo de código isso, e até mesmo no projeto "padrão" que se cria no visual studio, eles deixam uma boa parte da regra no controller
É errado? Quando usar isso? Qual a vantagem e desvantagem?
Eu vejo em vários exemplo de código isso, e até mesmo no projeto "padrão" que se cria no visual studio, eles deixam uma boa parte da regra no controller
É errado? Quando usar isso? Qual a vantagem e desvantagem?
Depende.
Acho melhor categorizar por tipo de regra pra ficar mais claro.
Sim, é errado. Validação de dados é uma característica do tipo de dado, portanto, de responsabilidade do Model.
Não. É função do Controller verificar se o registro de outra tabela realmente existe antes de fazer qualquer atribuição. A responsabilidade do Controller é harmonizar os dados entre Models e também entre a camada de apresentação.
Depende. Se o tipo de dados a ser auditado segue um padrão de formato e dados, o melhor é usar uma biblioteca (como as que trabalham com aspectos) que interceptam o dado, ou então um ActionFilter
.
Caso contrário, não há problemas em usar isso no Controller, desde que seu Controller tenha um suporte adequado a transações.
Sim, muito errado. O Controller deve apenas fornecer os dados para as Views. Nunca montar HTML, por exemplo.
Isto não se aplica caso o resultado devolvido por um Controller seja um arquivo (por exemplo, uma imagem ou um PDF), ou algum formato de dados padronizado (um JSON, um XML, e assim por diante).
Há uma ressalva sobre PDFs. Existe um pacote chamado RazorPDF2 que monta PDFs a nível de View. Este pacote é da minha autoria, então qualquer dúvida ou se quiser reportar bugs, pode me contatar através dos meios disponíveis aqui no site (menção, chat, etc.).
Service
, sendo que o Controller
absorve bem esta função.
Commented
16/09/2014 às 20:57
De preferência evite fazer isso, claro que pode haver exceções.
Existem várias argumentações do por quê, se você tiver uma mesma regra que será utilizada por outro controller? Você irá duplicar? Então mover para uma outra classe que muitas vezes será um Service vai te facilitar o reuso.
Você está ferindo o princípio de responsabilidade exclusiva do SOLID. Basicamente você estará deixando seu controller fazer mais coisas do que ele deveria fazer.
Mas pra não parecer inflexível demais, se você estiver apenas fazendo uma agenda telefônica isso pode não ser um problema, mas quando sua agenda telefônica começar a crescer é hora de seguir boas práticas que vão te evitar dores de cabeças e códigos macarrônicos.