2

Tenho uma aplicação que recebe varias observações de uma view e preciso inserir isso em uma tabela de observações.

Quero saber se nesse caso, o controller é quem faz o foreach e então chama o método no modelo para inserir um a um ou se o próprio modelo é quem tem de ser responsável por esse foreach.

Qual seria o ideal com MVC? E como pensar sempre ao utilizar um foreach em aplicações MVC, isto é, qual camada é responsável pelo quê?

Exemplo:

  • Em um cadastro de clientes, posso inserir varias observações;
  • O armazenamento dessas observações é feito numa tabela cliente_obs;
  • O processo para armazenamento hoje é:

    foreach($input['observacoes'] as $observacao){
        insert cliente_obs($observacao) //exemplo
    }
    
  • Esse processo se localiza no controller

9
  • Ok @Caffé. Atualizando a pergunta. Commented 4/12/2014 às 15:30
  • O controler é responsável por receber as entradas do usuário e integrá-las ao model. Não há nada que desabone o modo como você fez. Tem algo que você não goste nessa solução?
    – Caffé
    Commented 4/12/2014 às 15:40
  • Eu gosto dela. Só acordei com 'isso está errado' na mente e queria sanar a dúvida. Commented 4/12/2014 às 15:44
  • 1
    @BrunoAugusto "Fat model"? Não recomendo. Em um bom design, todo mundo tem que ser magrinho! :D
    – Caffé
    Commented 4/12/2014 às 17:18
  • 1
    Bom eu criei meu "ORM" (e bota aspas nisso) do jeito que eu sei. Não ficou ruim, mas as models não se relacionam automaticamente. Commented 4/12/2014 às 18:13

1 Resposta 1

1

Seu requisíto de negócio atual

  • Em um cadastro de clientes, posso inserir varias observações

Observe que pela descrição do texto, ele diz posso inserir varias observações. Como esse processo está envolvido em atualização de estado de um objeto, consequentemente ele estará envolvido em uma transação. Você deveria fazer o loop de inserts dentro da mesma transação e só no final do processo fazer commit. Assim, caso você tenha algum problema neste meio tempo e queira abortar a ação (você vai querer também apresentar uma mensagem de erro para o usuário [que será originada de uma Exception]), nenhum dos comentários será salvo no banco.

Sendo mais claro, seu loop de inserts não deveria estar em um Controller.

Veja mais sobre os princípios ACID

edição extra: Estou colocando também, uma explicação sobre DDD que vai exatamente de encontro com sua dúvida:

... DDD é uma coleção de padrões e princípios que ajudam em seus esforços para construir aplicações que refletem uma compreensão e a satisfação das exigências do seu negócio ...

Veja esta explicação detalhada sobre DDD, TDD e BDD.

4
  • Entao o controller nao faz 'nada'? Apenas pega a colecao e manda pro model? Commented 4/12/2014 às 16:40
  • Muito bom o ACID Commented 4/12/2014 às 16:46
  • 1
    ^ A missão do controller falando em alto nível, é receber o modelo de dados e delegar para um método que faça algo (geralmente o método de negócio quando se fala em sistemas web). No seu caso, seu negócio fala claramente que espera uma lista de comentários.
    – wryel
    Commented 4/12/2014 às 17:54
  • 1
    Não acho que sua pergunta é caso de "principalmente baseadas em opiniões", muito pelo contrário, até eu já me peguei com esta dúvida no passado e depois de pesquisar, ficou claro na minha cabeça que não é "questão de opinião". Votei para reabrir :)
    – wryel
    Commented 5/12/2014 às 22:28

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .