[BE] 유저 생성 및 조회 기능 구현 #8

KoKimSS · 2024-08-09T09:03:44Z

resolves 유저를 생성한다. #6

🚀 어떤 기능을 구현했나요 ?

유저 생성 및 조회 기능 개발
유저 생성시 루트 폴더 생성

🔥 어떻게 해결했나요 ?

📝 어떤 부분에 집중해서 리뷰해야 할까요?

📚 참고 자료, 할 말

- 폴더 메타데이터 스키마 수정 - 유저 스키마 수정 - 유저 생성시 루트 폴더 생성

kariskan

수고하셨습니다!

kariskan · 2024-08-09T15:45:11Z

src/main/java/com/woowacamp/storage/domain/user/controller/UserController.java

+	private final UserService userService;
+
+	@GetMapping
+	@RequestMapping("/{userId}")


GetMapping에 uri 명시하면 RequestMapping은 필요 없지 않나요?

이 부분은 수정하겠습니다.

kariskan · 2024-08-09T15:46:05Z

src/main/java/com/woowacamp/storage/domain/user/controller/UserController.java

+	@GetMapping
+	@RequestMapping("/{userId}")
+	public ResponseEntity<UserDto> getUser(@PathVariable Long userId) {
+		return ResponseEntity.ok(userService.findById(userId));


ResponseEntity로 래핑해서 보내는 이유가 있을까요? RestController 붙이면 UserDto나 ResponseEntity나 같은 형태로 응답된다고 알고 있어서요

평소에 이유를 크게 생각하지 않고 ResponseEntity로 반환을 해주었는데요.
찾아보니 저희 프로젝트에는 Dto 자체를 반환해주는 것이 맞는 것 같다는 생각이 드네요.

-> 저희는 GlobalExceptionHandler 가 예외 발생 상황을 책임지기 때문에 controller는 dto를 반환하는 로직만 책임지면 될 것 같습니다. ResponseEntity를 사용하여 Header나 HttpStatusCode 등을 동적으로 변경할 일이 없습니다.

-> Response dto 가 정해져있기 때문에 굳이 ResponseEntity<? extends T>를 사용하여 dto를 상속받는 모든 객체를 반환할 수 있게 허용할 가능성을 열어 둘 필요가 없을 것 같습니다.

참조

https://yeonyeon.tistory.com/257

https://joojimin.tistory.com/54

ErrorResponse를 ResponseEntity로 감싸는 구조 cheese10yun/spring-guide#33

kariskan · 2024-08-09T15:46:53Z

src/main/java/com/woowacamp/storage/domain/user/controller/UserController.java

+
+	@PostMapping
+	@RequestMapping
+	public ResponseEntity<UserDto> createUser(@RequestBody CreateUserReqDto req) {


@ResponseStatus(value = HttpStatus.CREATED)
이거 붙이면 좋을 것 같습니다.

변경하겠습니다.

kariskan · 2024-08-09T15:48:44Z

src/main/java/com/woowacamp/storage/domain/user/entity/dto/UserDto.java

+
+@Getter
+@NoArgsConstructor
+public class UserDto {


dto는 필드 별로 없어서 record로 만들어도 좋을 것 같은데 어떻게 생각하시나요?

좋은 것 같습니다 ! record 로 바꾸겠습니다

kariskan · 2024-08-09T15:51:04Z

src/main/java/com/woowacamp/storage/domain/user/service/UserService.java

+
+@Service
+@RequiredArgsConstructor
+@Transactional(readOnly = true)


@transactional 어노테이션은 메서드마다 작성하는게 어떤가요? 가독성도 떨어지고, 밑에처럼 어노테이션 중복으로 재사용하면 혼동될 거 같아요.

공통적으로 @transactional(readOnly = true) 를 사용하여 불필요한 더티 체킹이 일어나지 않게 막고(실수 방지) 필요한 메소드에는 따로 @transactional(readOnly = false) 를 사용해주는 방식으로 사용했었습니다.

가독성 vs 실수방지 의 문제인 것 같네요 ! 댓글 다시 주시면 감사하겠습니다.

지금은 readonly 밖에 없지만 나중에 rollback, propagation 이런 것도 사용하게 되면 오히려 더 헷갈리지 않을까요? + 이렇게 사용할 때마다 클래스 맨 위에 보면서 확인해야 하는것도 불편할 것 같아요

저도 중진님과 같은 이유로 Transactional 어노테이션은 메소드마다 작성하는 것이 좋다고 생각합니다. 트랜잭션이라는 의미로만 봤을 때도 메소드마다 작성하는 것이 더 좋은 것 같아요.

kariskan · 2024-08-09T15:51:32Z

src/main/java/com/woowacamp/storage/domain/user/service/UserService.java

+
+	public UserDto findById(Long userId) {
+		User user = userRepository.findById(userId)
+			.orElseThrow(() -> new CustomException(HttpStatus.BAD_REQUEST, "잘못된 id 입니다"));


404 아닌가요?

맞네요 수정하겠습니다 !

kariskan · 2024-08-09T15:52:36Z

src/main/java/com/woowacamp/storage/domain/user/service/UserService.java

+	public UserDto save(CreateUserReqDto req) {
+		LocalDateTime now = LocalDateTime.now();
+
+		FolderMetadata rootFolder = folderMetadataRepository.save(FolderMetadata.builder()


builder가 서비스 코드에 있어서 별 역할 하지 않는데도 길어보이는데 FolderMetadataFactory 같은 클래스를 만들고 static 메서드로 빼는게 어떨까요

추가적인 변경사항이 없다고 생각하는 경우 factoryMethod를 만드는 것이 좋다고 생각합니다 ! 아직 개발 초기이기 때문에 builder()로 변경 가능성을 열어 두는 것은 어떨까요 ? -> 나중에 리팩토링을 하는 형식으로

아 빌더를 안쓰는게 아니라 엔티티 생성하는 부분을 따로 빼는게 어떤지 말씀드리는 거였습니다

- User, FolderMetadataFactory 정적 팩토리 메소드 추가

- User, FolderMetadataFactory 생성 팩토리 메소드로 변경

- ResponseEntity -> dto 로 변경

KoKimSS · 2024-08-10T12:30:37Z

리뷰해 주신 부분 수정했습니다

kariskan

요 두개만 봐주시면 될 듯 합니다!

kariskan · 2024-08-10T17:11:27Z

src/main/java/com/woowacamp/storage/domain/user/dto/request/CreateUserReqDto.java

+
+@NoArgsConstructor
+@Getter
+public class CreateUserReqDto {


이것도 record로 만들면 되겠네요.

kariskan · 2024-08-10T17:11:49Z

src/main/java/com/woowacamp/storage/domain/user/dto/UserDto.java

@@ -0,0 +1,4 @@
+package com.woowacamp.storage.domain.user.dto;
+
+public record UserDto(Long id, Long rootFolderId, String userName) {


request든지 response든지 dto에서는 not null인게 보장되면 래퍼타입 쓰지않고 원시 타입 써도 될 듯 합니다.

- id, rootFolderId 는 무조건 null 이 아님

KoKimSS · 2024-08-11T11:37:04Z

요 두개만 봐주시면 될 듯 합니다!

두개 고쳤습니다 !

feat: 유저 생성 조회 API 개발

fdebe8c

- 폴더 메타데이터 스키마 수정 - 유저 스키마 수정 - 유저 생성시 루트 폴더 생성

KoKimSS added the ✅기능 개발 Further information is requested label Aug 9, 2024

KoKimSS added this to the 파일 조회 및 다운로드 milestone Aug 9, 2024

KoKimSS assigned i960107 and KoKimSS Aug 9, 2024

KoKimSS linked an issue Aug 9, 2024 that may be closed by this pull request

유저를 생성한다. #6

Closed

2 tasks

kariskan requested changes Aug 9, 2024

View reviewed changes

KoKimSS added 5 commits August 10, 2024 21:21

feat : 정적 팩토리 메소드 추가

6f6dadb

- User, FolderMetadataFactory 정적 팩토리 메소드 추가

refactor : 객체 생성 팩토리 메소드로 변경

e0d7fbc

- User, FolderMetadataFactory 생성 팩토리 메소드로 변경

refactor : UserDto record 로 변경

e2b8296

refactor : 폴더 구조 변경

d8589d7

refactor : 컨트롤러 반환 객체 변경

f78ef07

- ResponseEntity -> dto 로 변경

KoKimSS requested review from kariskan and seungh1024 August 10, 2024 12:28

kariskan approved these changes Aug 10, 2024

View reviewed changes

KoKimSS added 3 commits August 11, 2024 20:33

refactor : CreateUserReqDto record 로 변경

b8c1063

refactor : UserDto referType -> primitiveType 으로 변경

9eba65d

- id, rootFolderId 는 무조건 null 이 아님

refactor : createUser 파라미터 변경

fd0539d

KoKimSS requested a review from kariskan August 11, 2024 11:37

seungh1024 approved these changes Aug 11, 2024

View reviewed changes

kariskan merged commit d704793 into main Aug 11, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BE] 유저 생성 및 조회 기능 구현 #8

[BE] 유저 생성 및 조회 기능 구현 #8

KoKimSS commented Aug 9, 2024

kariskan left a comment

kariskan Aug 9, 2024

KoKimSS Aug 10, 2024 •

edited

Loading

kariskan Aug 9, 2024

KoKimSS Aug 10, 2024

kariskan Aug 9, 2024

KoKimSS Aug 10, 2024

kariskan Aug 9, 2024

KoKimSS Aug 10, 2024

kariskan Aug 9, 2024

KoKimSS Aug 10, 2024

kariskan Aug 10, 2024 •

edited

Loading

seungh1024 Aug 10, 2024

kariskan Aug 9, 2024

KoKimSS Aug 10, 2024

kariskan Aug 9, 2024

KoKimSS Aug 10, 2024

kariskan Aug 10, 2024

KoKimSS commented Aug 10, 2024

kariskan left a comment

kariskan Aug 10, 2024

kariskan Aug 10, 2024

KoKimSS commented Aug 11, 2024

		@@ -0,0 +1,4 @@
		package com.woowacamp.storage.domain.user.dto;

		public record UserDto(Long id, Long rootFolderId, String userName) {

[BE] 유저 생성 및 조회 기능 구현 #8

[BE] 유저 생성 및 조회 기능 구현 #8

Conversation

KoKimSS commented Aug 9, 2024

🚀 어떤 기능을 구현했나요 ?

🔥 어떻게 해결했나요 ?

📝 어떤 부분에 집중해서 리뷰해야 할까요?

📚 참고 자료, 할 말

kariskan left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KoKimSS Aug 10, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kariskan Aug 10, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KoKimSS commented Aug 10, 2024

kariskan left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KoKimSS commented Aug 11, 2024

KoKimSS Aug 10, 2024 •

edited

Loading

kariskan Aug 10, 2024 •

edited

Loading