Conversation
1. 커밋 테스트
1. 게임 등록 기능(기본) 2. 게임 조회(다건/단건) 기능(기본) 3. 게임 수정 기능(기본) 4. 게임 삭제 기능(기본)
1. 게임 검색 기능
1. 리뷰 생성 기능(기본) 2. 리뷰 수정 기능(기본) 3. 리뷰 삭제 기능(기본)
1. 게임 단건 조회시 리뷰 리스트 페이징으로 불러오기 구현 2. 게임 다건(검샘포함) 평균별점, 리뷰 수 불러오기 구현
1. 게임등록 요청 2. 게임등록 요청 삭제 3. 게임등록 요청 수정 4. 게임등록 요청 조회
1. 도메인.game 패키지 추가로 인한 파일 이동
1. 도메인.review 패키지 추가로 인한 파일 이동
1. 게임 전체 조회 , 검색 코드 통합 /games 로 통합하여 사용
1. 게임 등록시 이미지 첨부 파일 기능 구현 2. 게임 삭제시 이미지 데이터 삭제 기능 구현
1. PR 전 간단한 코드 정리
1. 게임 등록 첨부 파일 등록 기능
1. 컨트롤러 메서드에서 파라미터 줄바꿀 때 ( 뒤에서 엔터!
# Conflicts: # src/main/java/com/example/gamemate/global/constant/ErrorCode.java
1. final 생성자는 @requiredargsconstructor 설정
| public class GameController { | ||
| private final GameService gameService; | ||
|
|
||
| @Autowired |
There was a problem hiding this comment.
@ AutoWired 말고 @ RequiredArgsConstructor 로 통일하면 좋을 것 같습니다.
| } | ||
|
|
||
| GameCreateResponseDto responseDto = gameService.createGame(requestDto, file); | ||
| return ResponseEntity.status(HttpStatus.CREATED).body(responseDto); |
There was a problem hiding this comment.
return 값을 new ResponseEntity<>(); 로 통일하기로 코드 컨벤션에 적혀있어서 통일하면 좋을 것 같습니다.
| * @return | ||
| */ | ||
| @PatchMapping("/{id}") | ||
| public ResponseEntity<GameUpdateResponseDto> updateGame(@PathVariable Long id, |
There was a problem hiding this comment.
반환 타입 void 로 고치면 좋을 것 같습니다.
| public class GameEnrollRequestController { | ||
| private final GameEnrollRequestService gameEnrollRequestService; | ||
|
|
||
| @Autowired |
There was a problem hiding this comment.
@ AutoWired 말고 @ RequiredArgsConstructor 로 통일하면 좋을 것 같습니다.
| * @return | ||
| */ | ||
| @PatchMapping("/{id}") | ||
| public ResponseEntity<GameEnrollRequestResponseDto> updateGameEnroll(@PathVariable Long id, @RequestBody GameEnrollRequestUpdateRequestDto requestDto) { |
There was a problem hiding this comment.
반환타입 void 로 고치면 좋을것같습니다.
| this.gameImageRepository=gameImageRepository; | ||
| } | ||
|
|
||
| public GameCreateResponseDto createGame(GameCreateRequestDto gameCreateRequestDto , MultipartFile file) { |
There was a problem hiding this comment.
트랜잭션 처리해주시면 좋을것 같습니다.
| * @param requestDto | ||
| * @return | ||
| */ | ||
| @PatchMapping("/{id}") |
There was a problem hiding this comment.
반환타입 void로 바꾸면 좋을것같습니다.
| @Getter | ||
| @NoArgsConstructor | ||
| @Table(name = "review") | ||
| public class Review extends com.example.gamemate.global.common.BaseEntity { |
There was a problem hiding this comment.
경로 설정으로 되어있어서 깔끔하게 써주시면 좋을것같습니다.
| this.gameRepository = gameRepository; | ||
| } | ||
|
|
||
| public ReviewCreateResponseDto createReview(Long gameId, ReviewCreateRequestDto requestDto) { |
| return new ReviewCreateResponseDto(saveReview); | ||
| } | ||
|
|
||
| public ReviewUpdateResponseDto updateReview(Long gameId, Long id, ReviewUpdateRequestDto requestDto) { |
| * @return | ||
| */ | ||
| @PostMapping(consumes = MediaType.MULTIPART_FORM_DATA_VALUE) | ||
| @ResponseStatus(HttpStatus.CREATED) |
There was a problem hiding this comment.
아래 return에서 명시되어있어서 빼도 될 것 같아요~
| * 게임 전체 조회 | ||
| * | ||
| * @param page | ||
| * @param szie |
There was a problem hiding this comment.
여기 오타가 있네용
szie -> size 로 수정 부탁드립니다~!
혹시 @param을 일부만 적으신 이유가 있을까요?
There was a problem hiding this comment.
아직 자바독을 완성하지 못했습니다! ㅠㅠ
| * 게임 정보 수정 | ||
| * | ||
| * @param id | ||
| * @param requestDto |
| public ResponseEntity<Page<GameEnrollRequestResponseDto>> findAllGameEnrollRequest() { | ||
|
|
||
| Page<GameEnrollRequestResponseDto> gameEnrollRequestAll = gameEnrollRequestService.findAllGameEnrollRequest(); | ||
| return ResponseEntity.ok(gameEnrollRequestAll); |
There was a problem hiding this comment.
return 값을 new ResponseEntity<>(); 로 수정 부탁드립니다~!
| @PathVariable Long id) { | ||
|
|
||
| GameEnrollRequestResponseDto gameEnrollRequestById = gameEnrollRequestService.findGameEnrollRequestById(id); | ||
| return ResponseEntity.ok(gameEnrollRequestById); |
There was a problem hiding this comment.
return 값을 new ResponseEntity<>(); 로 수정 부탁드립니다
| private String description; | ||
|
|
||
|
|
||
| public GameUpdateRequestDto(String title, String genre, String platform , String description) { |
There was a problem hiding this comment.
public GameUpdateRequestDto(String title, String genre, String platform, String description) {
띄어쓰기 수정 부탁드립니다
| @Entity | ||
| @Getter | ||
| @Table(name = "game") | ||
| @AllArgsConstructor |
There was a problem hiding this comment.
@AllArgsConstructor를 제한없이 사용하는 것에 대해 같이 고민해보면 좋을 것 같아요
(access = AccessLevel.PRIVATE)
| @Entity | ||
| @Getter | ||
| @Table(name = "game_enroll_request") | ||
| @AllArgsConstructor |
There was a problem hiding this comment.
@AllArgsConstructor를 제한없이 사용하는 것에 대해 같이 고민해보면 좋을 것 같아요
(access = AccessLevel.PRIVATE)
|
|
||
| private final ReviewService reviewService; | ||
|
|
||
| @Autowired |
There was a problem hiding this comment.
@Autowired 대신 @requiredargsconstructor 로 통일하면 좋을 것 같습니다.
| @RequestBody ReviewCreateRequestDto requestDto) { | ||
|
|
||
| ReviewCreateResponseDto responseDto = reviewService.createReview(gameId, requestDto); | ||
| return ResponseEntity.status(HttpStatus.CREATED).body(responseDto); |
There was a problem hiding this comment.
return 값을 new ResponseEntity<>(); 로 수정 부탁드립니다
1. @Autowired 말고 @ RequiredArgsConstructor 로 통일 2. 오타 및 줄간격 경로설정 수정 3. return 값을 new ResponseEntity<>(); 로 통일 4. 반환 타입 void 로 수정 5. 트랜잭션 처리 6. @AllArgsConstructor를 제한없이 사용하는 것 > 삭제
| } | ||
|
|
||
|
|
||
| return ResponseEntity.ok(games); |
There was a problem hiding this comment.
이 부분도return new ResponseEntity<>(games, HttpStatus.OK);
이렇게 해야될 것 같아요!
| @PathVariable Long id) { | ||
|
|
||
| GameFindByIdResponseDto gameById = gameService.findGameById(id); | ||
| return ResponseEntity.ok(gameById); |
There was a problem hiding this comment.
이 부분도 return new ResponseEntity<>(gameById, HttpStatus.OK);
이렇게 해야될 것 같아요!
| @RequestBody GameEnrollRequestCreateRequestDto requestDto) { | ||
|
|
||
| GameEnrollRequestResponseDto responseDto = gameEnrollRequestService.createGameEnrollRequest(requestDto); | ||
| return ResponseEntity.status(HttpStatus.CREATED).body(responseDto); |
There was a problem hiding this comment.
이 부분도 return new ResponseEntity<>( ) 로 수정되어야할 것 같습니다
1. return 값을 new ResponseEntity<>(); 로 통일
1. @Autowired 말고 @ RequiredArgsConstructor 로 통일 2. 오타 및 줄간격 경로설정 수정 3. return 값을 new ResponseEntity<>(); 로 통일 4. 반환 타입 void 로 수정 5. 트랜잭션 처리 6. @AllArgsConstructor를 제한없이 사용하는 것 > 삭제
1. return 값을 new ResponseEntity<>(); 로 통일
Feature/game feat : 게임, 리뷰 기능 구현
4 participants
#️⃣연관된 이슈
📝작업 내용
스크린샷 (선택)
💬리뷰 요구사항(선택)