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!