## 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á-lo para poder seguir o SRP.

***

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

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.

***

### 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)
        {

        }

    }

    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 ela herdar, pois a herança utilizada de forma exagerada pode gerar algumas hierarquias confusas. 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 ele 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!