+3,596
−13
Conversation
This file contains hidden or 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
youngxpepp
approved these changes
Feb 26, 2025
|
|
||
| import java.util.NoSuchElementException; | ||
|
|
||
| @RestControllerAdvice |
There was a problem hiding this comment.
ControllerAdvice를 써주신 분들도 계시더라구요~?
RestControllerAdvice vs ControllerAdvice 두 개의 차이점을 한번 찾아보셨으면 좋겠습니다!
|
|
||
| @RequestMapping(method = RequestMethod.POST) | ||
| public ResponseEntity<User> createUser(@RequestBody UserCreateRequest request) { | ||
| User user = userService.create(request, Optional.empty()); |
There was a problem hiding this comment.
보통 파라미터엔 Optional을 쓰지 않아요. 우리가 제어할 수 없는 값이기 때문이죠.
다른 사람이 Optional을 넣지 않고 null을 넣을 수도 있는거잖아요?
| return ResponseEntity.ok(users); | ||
| } | ||
|
|
||
| @RequestMapping(value = "/{userId}/status", method = RequestMethod.PUT) |
There was a problem hiding this comment.
따로 userStatus api를 만들지 않고 user 내에서 해결하니까 좋네요! 잘하셨습니다👍
앞으로 리소스가 상하관계에 있다면 이 방법을 적극 활용해보세요!
| return ResponseEntity.noContent().build(); | ||
| } | ||
|
|
||
| @RequestMapping(value = "/channel/{channelId}", method = RequestMethod.GET) |
There was a problem hiding this comment.
채널과 메시지 어떤 것이 더 상위 계층에 있는 리소스일까요?
user와 userStatus 와의 관계를 한번 생각해보시구 url을 만들어보세요!
Comment on lines
+35
to
+36
| @RequestMapping(value = "/user/{userId}", method = RequestMethod.GET) | ||
| public ResponseEntity<List<ReadStatus>> getReadStatusForUser(@PathVariable UUID userId) { |
There was a problem hiding this comment.
user와 readStatus는 어떤 관계일까요? 여긴 상하 관계를 찾기가 좀 어려울거에요!
고민해보시면 좋겠습니다!
Comment on lines
+21
to
+26
| // POST: 바이너리 파일 업로드 | ||
| @RequestMapping(method = RequestMethod.POST) | ||
| public ResponseEntity<BinaryContent> createBinaryContent( BinaryContentCreateRequest request) { | ||
| BinaryContent binaryContent = binaryContentService.create(request); | ||
| return ResponseEntity.status(201).body(binaryContent); | ||
| } |
There was a problem hiding this comment.
보통 파일 업로드를 어떻게 구현할까요? 아래 내용 참고하시겠어요?
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.
기본 요구사항
컨트롤러 레이어 구현
DiscodeitApplication의 테스트 로직은 삭제하세요.
지금까지 구현한 서비스 로직을 활용해 웹 API를 구현하세요.
이때 @RequestMapping만 사용해 구현해보세요.
웹 API 요구사항
API 테스트
Postman API 테스트 결과를 다음과 같이 export하여 PR에 첨부해주세요.
스프린트 미션 4.postman_collection.json
웹 API 요구사항
사용자 관리
권한 관리
채널 관리
메시지 관리
메시지 수신 정보 관리
바이너리 파일 다운로드
[] 바이너리 파일을 1개 또는 여러 개 조회할 수 있다.
멘토에게
최대한 빠르게 수정해서 다시 업로드 하겠습니다.
미션 3를 완료하지 못하여 모법답안을 이용해 미션4를 진행하였고 심화는 아직 진행중입니다