Skip to main content
10 de 12
adicionou 5 caracteres ao conteúdo
raphael
  • 2,3mil
  • 9
  • 31

Em busca do SRP (Single Responsibility Principle)

Segue as respostas da suas perguntas.

1 - Devo separar a lógica de validação e a parte do CRUD em classes separadas?

Sim. É melhor separar a validação da parte que faz as operações de CRUD, pois são operações distintas.


2 - É de responsabilidade do usuário validar a si próprio?

Não necessariamente. Você pode criar um componente para validar um usuário, já que isso iria de encontro com o SRP.

2.1 - A classe User pode ser responsável por "acumular" os erros da validação?

Complementando a resposta anterior: o componente que faz validação seria um lugar mais apropriado para essas informações.


3 - O usuário é responsável por salvar a si mesmo?

É melhor criar uma classe para salvar um usuário, pois tudo fica mais separado. Quando um usuário salva ele mesmo o SRP está sendo violado, pois o objeto tem duas responsabilidades: conter as informações do usuário e persistir ele no banco de dados. Esse tipo abordagem se chama Active Record e ela acaba misturando as coisas, portanto é melhor evitá-la para poder seguir o SRP.


4 - Como seria este exemplo com milhares de atributos? usar um loop nos métodos setters?

Pelo que entendi, a sua dúvida nessa questão é nos casos de muitos dados. Caso o seu objeto User tenha vários dados, você pode passar um array com os dados para evitar uma quantidade imensa de getters e setters. Esta pergunta está respondida de forma mais detalhada aqui.


Seu exemplo da classe User

Percebi que seu código contém algumas dependências no objeto User:

function __construct () {

    $this->userValidator = new userValidator;
    $this->userCRUD      = new userCRUD;

}

Essa implementação poderia melhorar se você passar o objeto User para os objetos e não fazer ele recebê-los, por exemplo:

class UserValidator
{

    public function validate(User $user)
    {
        // Pegar os dados do objeto user e validá-los aqui
    }

}

class UserCrud
{

    public function save(User $user)
    {
         // Salva e edita o usuário
    }

    public function delete(User $user)
    {
         // Deleta o usuário
    }

}

Ainda sobre a classe UserCrud, você disse:

E por fim, criaria uma última classe chamada UserCRUD, que herdaria os métodos de outra classe chamada "crudModel", responsável pela integração com o banco de dados

Eu aconselharia você injetar a classe crudModel em vez de fazer UserCrud herdar dela, pois a herança utilizada de forma exagerada pode gerar algumas hierarquias confusas. Neste caso, a composição poderia ser uma alternativa e em vez de herdar você poderia fazer o seguinte:

class UserCrud
{

    private $crudModel;

    public function __construct(crudModel $crudModel)
    {
          $this->crudModel = $crudModel;
    }

}

Conclusão

Ao ler sua pergunta vi que você está buscando usar o SRP (um dos princípios SOLID), o que é muito positivo, já que esse principio busca incentivar a separação das coisas em responsabilidades deixando o código mais simples e bem definido. Procure evitar classes gigantes e refatore o código para separar as coisas, pois isso ajuda muito nesse processo para entender o principio.

Espero ter ajudado, Valeu!

raphael
  • 2,3mil
  • 9
  • 31