search

danielvdg / auth-user

0 stars 0 forks source link

Improve Controller #1

Open pedrop3 opened 1 month ago

pedrop3 commented 1 month ago

Entendo que esta estudando um exemplo para auth, mas é sempre bom seguir um padrão :D

  1. private PersonaRepository personaRepository; entenda quando utlizar o final e o pq..
  2. Cuidado ao utlizar Repository dessa forma direito no controller, vc acaba eliminando a regra de negocio e aumentando a acomplação do seu código.
  3. Tenha o costume de trabalhar com DTO
  4. Isso faria sentido se tivesse utilizando DTO.. existem alternativas como modelMapper
persona.setName(persona.getName());
persona.setCpf(persona.getCpf());
persona.setEmail(persona.getEmail());

https://github.com/danielvdg/auth-user/blob/a958fa0159596393acbe325f96746ab3f64e6b42/src/main/java/com/auth/authuser/controller/core/PersonaController.java#L19

danielvdg commented 1 week ago

@pedrop3 corrigido , no entanto removi persona e deixei somente como user, pretendo criar um micro-serviço para utilizar os dados do persona