2

Em um controller chamado atividades, tenho um metodo chamado lembrete.
Esse metodo, recebe um id, procura aquela atividade com base nesse id e envia um email pro moderador daquela atividade com dados da mesma.
Esse metodo esta com 'responsabilidades demais'?

Estou enviando dois metodos para já unir outra duvida. Por designer de código, qual deles está correto? O que envia separa a $client em uma variavel (reminder_a), ou o que sempre utiliza $activity para se referenciar aos clientes (reminder_b)?

Segue metodo:

/**
 * Send the remember email with a resource
 * @param  int $id
 */
public function reminder_a($id)
{

    $activity   = $this->activities->getById($id);
    $client     = $activity->client;

    $title = 'Reminder of activity ' . $activity->name;
    $view  = 'emails.activity.reminder';
    $data  = ['activity' => $activity, 'cliente' => $client];
    CustomMail::sendBasicMail($view, $title, $activity->client->email, $data);

}


/**
 * Send the remember email with a resource
 * @param  int $id
 */
public function reminder_b($id)
{

    $activity = $this->activities->getById($id);

    $title = 'Reminder of activity ' . $activity->name;
    $view  = 'emails.activity.reminder';
    CustomMail::sendBasicMail($view, $title, $activity->client->email, $activity);

}   

1 Resposta 1

3

Poder, pode. Mas não é recomendado.

Comparando os dois métodos, me parece que a diferença é só passar o client separado da activity, o que pra mim não haveria necessidade já que um é atributo do outro. Sendo assim, me parece que o método reminder_b é mais correto.

Também não vejo nenhum erro na aplicação do princípio Single Responsibility Principle (o "S" de SOLID) pois cada método do controlador possui apenas uma responsabilidade, que é a de enviar o email.

A única coisa que eu mudaria (além de optar pelo método reminder_b) seria encapsular o envio de email, talvez colocá-lo dentro de um serviço. O controlador deve ser visto apenas como um ponto de entrada da sua aplicação (por exemplo, você poderia enviar o email também por meio da linha de comando), executar uma única tarefa e retornar uma resposta correspondente.

Espero ter sido claro. :)

8
  • Perfeito @Rodrigo. Sobre encapsular email, como eu iria passar informações como: a view a ser utilizada, titulo customizado e etc, se eu não chamar isso no metodo onde eu fiz a consulta de atividade? Commented 6/12/2014 às 22:08
  • Se você observar, posso utilizar o sendBasicMail pra qualquer tipo de email. Commented 6/12/2014 às 22:08
  • Se o SendBasicMail já encapsula bem a tarefa de enviar e-mails, talvez colocar mais uma camada de abstração (por exemplo, um serviço) não seja tão necessária. Mas o que eu faria, por exemplo, seria criar um método sendActivityEmail, para o qual você passaria a atividade ou, simplesmente, o $id dela. (nesse caso seria necessário injetar o repositório de atividades como uma dependência do serviço de envio de emails). Commented 6/12/2014 às 22:12
  • 1
    Claro que não, você agrupa de acordo com os parâmetros dos métodos até um estado aceitável :) Commented 6/12/2014 às 22:21
  • 1
    Obrigado @Rodrigo. A discussão foi bastante produtiva e sanou minha dúvida. Commented 6/12/2014 às 22:23

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 .