1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
@Service
class AdminUserGoalService {
private final GoalRepository goalRepository;
private final DictionaryService dictionaryService;
AdminUserGoalService(GoalRepository goalRepository,
DictionaryService dictionaryService) {
this.goalRepository = goalRepository;
this.dictionaryService = dictionaryService;
}
мне кажется просто поля помечать автоваерами было бы проще т.к. инжектов может быть и не один. будете конструктор на 10 полей создавать?
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
UserResponseDTO create(UserRequestDTO userRequestDTO) {
final InUserType inUserTypeDb = userRequestDTO.getType().getId() == null ? null
: inUserTypeRepository.findOne(userRequestDTO.getType().getId());
final InUser inUser = new InUser();
final InUserEmail inUserEmail = new InUserEmail();
final MapBindingResult errors = new MapBindingResult(new HashMap<>(), String.class.getName());
emailValidator.validate(userRequestDTO.getEmail(), errors);
if (errors.hasErrors()) {
throw new UnauthorizedException(errors.getAllErrors().get(0).getDefaultMessage());
}
inUserEmail.setLogin(userRequestDTO.getEmail());
inUserEmail.setUser_name(userRequestDTO.getName());
inUserEmail.setPassword(passwordEncoder.encode("Qwerty+1"));
inUser.setInUserType(inUserTypeDb);
inUser.setInUserEmails(Arrays.asList(inUserEmail));
inUser.setD_sex(userRequestDTO.getGender());
inUser.setD_level(userRequestDTO.getLevel() == null ? null : "" + userRequestDTO.getLevel());
inUser.setWeight(userRequestDTO.getWeight() == null ? null : userRequestDTO.getWeight().floatValue());
setupGoals(userRequestDTO, inUser);
final InUser savedInUser = inUserRepository.save(inUser);
inUserEmail.setInUser(savedInUser);
inUserEmailRepository.save(inUserEmail);
return inUserToDto(adminProgramAssignService.assign(savedInUser));
}
просто первый попавшийся копипаст:
если креейт то креейт зачем скрещивать с апдейтом? да еще и так?
1) почему ин-юзер?
2) почему сущность называется inUserEmail а там стопитцот полей? включая ин_юзер? и емейл?
3) что за именование?
String user_name;
String device_id;
камелкейс - не?
4) юзайте маппер называйте поля одинаково будет всем хорошо.
5) зачем писать реально модифицирующий данные код в СУЩНОСТИ?? (назад в п.2)
6) inUser.setD_level(userRequestDTO.getLevel() == null ? null : "" + userRequestDTO.getLevel()); -- Што это штооо?
6) вы слышали что нибудь о книжке клин код?
7) вы слышали что нибудь об аббревиатуре SOLID?
8) вы слышали что нибудь о книге Java Head First?
