스프링부트 서비스레이어에서 어떤 방식이 더 좋을지... 취준하면서 포폴준비중입니다.

Question

일단 상황은관리자권한에서 유저의 role을 바꾸는 로직입니다.@PostMapping("/change-role") public ResponseEntity userRoleChanges( @RequestBody UserRoleChangeRequestDto requestDto ) { ResponseEntity response = adminService.userRoleChanges(requestDto); return response; }일단 컨트롤러의 코드는 동일하며1번 비지니스 로직:@Override public ResponseEntity userRoleChanges(UserRoleChangeRequestDto requestDto) { try { //유저ID 를 가져옴 User user = userRepository.findByUserId(requestDto.getChangeUserId()); if (user == null) { // 에러처리. return UserRoleChangeResponseDto.noExistUser(); } // user엔티티에 role을 set해주는 메서드 생성하고 user.setRole(requestDto.getRoleEnum()); // 저장 userRepository.save(user); return UserRoleChangeResponseDto.success(); } catch (Exception e) { return UserRoleChangeResponseDto.databaseError(); } }2번 비지니스 로직:@Transactional @Modifying @Query("UPDATE user u SET u.role = :role WHERE u.userId = :userId") int updateUserRole(@Param("userId") String userId, @Param("role") UserRoleEnum role); 위코드는 repository 코드 @Override public ResponseEntity userRoleChanges(UserRoleChangeRequestDto requestDto) { try { //이제 updatedRows 가 1이면 성공한 쿼리 int updatedRows = userRepository.updateUserRole(requestDto.getChangeUserId(), requestDto.getRoleEnum()); if (updatedRows == 0) { // 업데이트된 행이 없습니다. 이는 사용자를 찾을 수 없음을 나타냅니다. return UserRoleChangeResponseDto.noExistUser(); } return UserRoleChangeResponseDto.success(); } catch (Exception e) { e.printStackTrace(); return UserRoleChangeResponseDto.databaseError(); } } 이제 결과적으론 유저의 role을 변경하는 코드인데 1번 코드는 직관적으로 알수있음2번 코드는 JPQL을 사용해서 쿼리의 수를 줄였음어떤방식이 더좋은지..궁금합니당

kyuho · Answer

전 동료와의 의사소통과 팀의 코딩컨벤션, 유지보수의 용이성의 이유로 1번 방식을 주로 사용해요. 하지만 성능 이슈로 쿼리를 최적화 하는 경우 2번 방식도 사용해요.어떤 방식으로 작성하던 왜 그렇게 했는지 설명할 수 있어야 하는게 중요하다고 생각해요.

초무쿤 · Answer

ORM이니 1번처럼 쓰는게 의도에 맞을듯 합니다.

스프링부트 서비스레이어에서 어떤 방식이 더 좋을지... 취준하면서 포폴준비중입니다.

2개의 답변

기술 목록