Estou estudando alguns conceitos de OOP e vejo bastante sobre a questão de SRP, "separar ações em classes etc". a nível de testes, decidi criar um pequeno sistema para cadastro de usuários afim de testar os conhecimentos, só que me deparei com algumas questões
1 - Devo separar a lógica de validação e a parte do CRUD em classes separadas?
2 - É de responsabilidade do usuário validar a si próprio?
2.1 - A classe User pode ser responsável por "acumular" os erros da validação?
3 - O usuário é responsável por salvar a si mesmo?
4 - Como seria este exemplo com milhares de atributos? usar um loop nos métodos setters?
O que concluí, com base nos estudos:
Devo usar 3 classes, User, UserValidator e UserCRUD, pois no meu entendimento são "ações" distintas
A classe User seria a mais "abstrata", relacionando a validação e a integração com o banco de dados.
Devo criar uma segunda classe chamada UserValidator que seria responsável pelas regras de validação, podendo ser acoplada à alguma biblioteca de validação? (Respect/rakit)
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
Exemplo de código:
class User {
private $name;
private $age;
function __construct () {
$this->userValidator = new userValidator;
$this->userCRUD = new userCRUD;
}
function setName($name) {
if($this->userValidator->validateName($name))
$this->name = $name;
}
function setAge($age) {
if($this->userValidator->validateAge($age))
$this->age = $age;
}
function hasErrors () {
return $this->userValidator->hasErrors();
}
function Save () {
$user_data['name'] = $this->name;
$user_data['age'] = $this->age;
return $this->userCRUD->create($user_data);
// retorna true para salvo e falso para erro
}
}
-
class UserValidator {
private $error_bag;
public function validateName ($name) {
if(strlen($name) > 5)
return 1;
else
$this->setValidationError('name', 'Nome deve ser maior que 5');
}
public function validateAge ($age) {
if(is_numeric($age))
return 1;
else
$this->setValidationError('age', 'Idade inválida');
}
private function setValidationError ($error, $error_msg) {
$this->error_bag[$error] = $error_msg;
}
private function getValidationErrors() {
return $this->error_bag;
}
public function hasErrors () {
if( empty($this->getValidationErrors()) )
return false;
else
return $this->getValidationErrors();
}
}
Chamada do código:
$User = new User;
$User->setName('Fulano de Tal');
$User->setAge(50);
if($User->hasErrors()) {
// exibe os erros na tela
print_r($User->hasErrors());
} else {
// salve o usuario
$saveUser = $User->Save();
var_dump($saveUser);
}