[빙봉] 체스 스프링 3단계 미션 제출합니다. #142
Merged
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
* initial commit * fix : build.gradle mysql 의존성 추가 * refactor : 왕이 죽었을 때 게임이 종료 안되는 버그 수정 * refactor : 버튼으로 방에 입장할 수 있도록 함 * refactor : 방을 삭제하고 나서 새로고침해야 적용되던 문제 수정 * feat : 방에 입장하자마자 게임을 바로 불러오도록 하고 못 불러왔을 시 게임을 초기화하는 기능 추가, refactor : 불러왔을 때 현재 Turn이 나오지 않는 문제 수정 * refactor, style : tab 공백을 space로 변경, 익명 객체 람다식으로 수정 * feat : Spring room Controller 구현 - 방 제목 empty 에 대한 validation 추가 - default 요청시 바로 방 목록을 불러오도록 수정 * feat, refactor : Dto 생성 방식 수정, 게임 컨트롤러 구현 - 게임 초기화 기능 구현 Co-authored-by: aegis <[email protected]>
* refactor : 사용하지 않는 클래스 삭제 * refactor : 디폴트 생성자 접근제어자 수정 * refactor : 디폴트 생성자 접근제어자 수정 * refactor : 디폴트 생성자 접근제어자 수정 * refactor : Object 객체 타입 파라미터 반영하도록 수정 * refactor : 필드간 공백 추가 * refactor : 사용하지 않는 메서드 삭제 * refactor : 필드 접근제어자 추가 * style : 공백 추가 * feat : gameController init, movablePosition 구현 - todo : move시 에러 * feat : move 예외 처리 작업중 * refactor : spring MVC 구조에 맞게 refactor - GameController 예외 알맞게 처리 - DAO 클래스 Repository로 설정 - Service의 DAO를 bean으로 주입 * docs : README.md 추가 * test : domain test 메서드 추가 * refactor : 사용하지 않은 메서드 삭제 및 공백 수정 * refactor, fix : Spark Application과 Spring Application 이 모두 동작하도록 수정, 점수가 제대로 나오지 않는 버그 수정 * style : 공백 삭제 * refactor : move 메서드가 Post방식의 Json으로 요청을 받도록 수정 - front의 request uri 수정 - MoveManagerDTO 클래스로 request 매핑 * refactor : 불필요한 attribute 삭제 * refactor : move 메서드 리턴 타입 수정, 클래스명 수정 * refactor : 서비스 메서드 불필요한 동작 수정 Co-authored-by: aegis <[email protected]>
lowoon
approved these changes
Apr 30, 2020
| return new GameStatusDTO(gameService.getScore(roomId, Color.WHITE), gameService.getScore(roomId, Color.BLACK)); | ||
| @PostMapping("/getPath") | ||
| public List<String> getMovablePositions(@RequestBody MoveRequestDto moveRequestDto) { | ||
| System.out.println(moveRequestDto); |
There was a problem hiding this comment.
움직일 수 있는 Position을 가지고 오도록 하는 건데 좀 더 읽기 쉽게 수정해보도록 하겠습니다!
| private GameDAO gameDAO; | ||
| private RoomRepository roomRepository; | ||
|
|
||
| @Transactional |
There was a problem hiding this comment.
메서드에 roomRepository에 접근하는 로직이 두 번 들어가서 하나의 트랜잭션으로 묶기 위해@Transactional을 사용했습니다 :)
| @@ -0,0 +1,9 @@ | |||
| package wooteco.chess.repository; | |||
There was a problem hiding this comment.
gameRepository, pieceRepository가 안쓰이고 있는거 같아요!
woowahanCU
approved these changes
May 3, 2020
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.
안녕하세요 로운! JCF 테코톡 잘 봤습니다 👍 👍
DB의 테이블이 3개로 늘어남과 동시에 엔티티와 Repostiory를 작성했는데 잘 짰는지 확신이 서지 않네요 ㅎㅎ
아직 백엔드와 프론트엔드 모두 리펙토링이 필요하고 부족한 부분이 많습니다. 점진적으로 개선하도록 할게요.
저번에 피드백 주신대로 sql 파일을 넣어놨습니다! DB 만들어보시고 실행해보셔도 좋을 것 같아요 :)
연휴 잘 보내세요!