## 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.](https://pt.stackoverflow.com/questions/333521/boas-praticas-para-classe-que-cresce-muito/333635#333635)

***

### 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!