정유빈(B) - Spring Project with JPA and H2 Database #3
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
노션 링크입니다! |
coke98
reviewed
Jul 12, 2024
| } | ||
|
|
||
| @PutMapping("/{courseId}") | ||
| public ResponseEntity<Course> updateCourse(@PathVariable Long courseId, @RequestBody Course courseDetails) { |
There was a problem hiding this comment.
body에서 클래스를 바로 바인딩해준 점은 좋았어요. 다만 이게 엔티티를 직접 바인딩하는 방법보다는 DTO라는 중간 클래스를 두고 변환하는걸 추천합니다. 예시로 API 마다 모든 엔티티의 필드를 필요하는게 아니라 특정 정보만을 받는게 필요할 수도 있겠죠.
API 문서대로 받는 값들을 커스텀해야하는 경우가 생길 수 있어서 보통은 CourseUpdateRequest와 같은 DTO 클래스를 따로 두곤합니다! 채현님이랑 이 부분 관련 이야기를 서로 해보면 좋을 것 같네요.
- 자바 버전에 따라 DTO의 경우 record를 쓰는 것도 선호되기도 합니다
| @Autowired | ||
| private EnrollmentService enrollmentService; | ||
|
|
||
| @PostMapping("/add") |
There was a problem hiding this comment.
URL에서 add를 넣지 않아도 충분한 의미를 갖지 않을까 싶네요. 왜 그럴까요?
| @Autowired | ||
| private CourseRepository courseRepository; | ||
|
|
||
| public Enrollment addEnrollment(Long studentId, Long courseId) { |
There was a problem hiding this comment.
저장소에 접근 하는 코드를 한 메서드로 모은 경우 트랜잭션 어노테이션을 쓰면 좋을 것 같아요. 그럼 트랜잭션을 어떻게 사용할 수 있고, 이 코드들에 적용했을때 어떤 실질적인 이점이 있을까요?
| @@ -0,0 +1,23 @@ | |||
| package xxubin04.DB_spring.controller; | |||
|
|
|||
| public class AddEnrollmentRequest { | |||
There was a problem hiding this comment.
오 다른 리뷰에서 DTO 관련 글을 썼었는데, 여기서는 활용 해주셨었네요. 레코드 형식을 써보는것도 좋고, 네이밍을 저는 EnrollmentAddRequest로 쓰는걸 더 선호하는 편이긴해요. 동사형이 먼저 나오면, 메서드네이밍(행위) 처럼 보이더라구요
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1. H2 Database Test
2. Postman
3. H2 Database Check
많이 부족하지만 잘 부탁드립니다!🙇🏻♀️