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
}
}