[디디] 체스 스프링 실습 1단계 제출합니다. #12
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
… 불러왔을 때 현재 Turn이 나오지 않는 문제 수정
- 방 제목 empty 에 대한 validation 추가 - default 요청시 바로 방 목록을 불러오도록 수정
chomily
approved these changes
Apr 23, 2020
Comment on lines
+16
to
+17
| protected Color color; | ||
| protected MoveStrategy moveStrategy; |
Comment on lines
+27
to
+49
| public static void createBlackPieces(Color color, Map<Position, Piece> pieces) { | ||
| initKing("e8", "K", color, pieces); | ||
| initQueen("d8", "Q", color, pieces); | ||
| initKnight("b8", "N", color, pieces); | ||
| initKnight("g8", "N", color, pieces); | ||
| initBishop("c8", "B", color, pieces); | ||
| initBishop("f8", "B", color, pieces); | ||
| initRook("a8", "R", color, pieces); | ||
| initRook("h8", "R", color, pieces); | ||
| initPawn("7", "P", color, pieces); | ||
| } | ||
|
|
||
| public static void createWhitePieces(Color color, Map<Position, Piece> pieces) { | ||
| initKing("e1", "k", color, pieces); | ||
| initQueen("d1", "q", color, pieces); | ||
| initKnight("b1", "n", color, pieces); | ||
| initKnight("g1", "n", color, pieces); | ||
| initBishop("c1", "b", color, pieces); | ||
| initBishop("f1", "b", color, pieces); | ||
| initRook("a1", "r", color, pieces); | ||
| initRook("h1", "r", color, pieces); | ||
| initPawn("2", "p", color, pieces); | ||
| } |
Comment on lines
+25
to
+28
| @Override | ||
| public boolean isKing() { | ||
| return true; | ||
| } |
There was a problem hiding this comment.
부모 클래스에서 구현된 메서드를 오버라이드해서 사용해도 될까요?
이렇게 사용하면 어떤 문제점이 있을까요? :)
Comment on lines
+36
to
+46
| public boolean isPawn() { | ||
| return false; | ||
| } | ||
|
|
||
| public boolean isKing() { | ||
| return false; | ||
| } | ||
|
|
||
| public boolean isBlank() { | ||
| return false; | ||
| } |
There was a problem hiding this comment.
Piece의 구현체는 모두 Pawn, King, Blank가 될 수 없는 건가요? :)
- 게임 초기화 기능 구현
gracefulBrown
approved these changes
Apr 23, 2020
fucct
added a commit
to fucct/jwp-chess
that referenced
this pull request
Apr 27, 2020
* 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]>
fucct
added a commit
to fucct/jwp-chess
that referenced
this pull request
Apr 27, 2020
* 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]>
woowahanCU
pushed a commit
that referenced
this pull request
Apr 27, 2020
* [디디] 체스 스프링 실습 1단계 제출합니다. (#12) * 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]> * [디디] 체스 - 스프링 실습 2단계 제출합니다. (#73) * 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]> Co-authored-by: DD <[email protected]>
woowahanCU
pushed a commit
that referenced
this pull request
May 3, 2020
* [디디] 체스 스프링 실습 1단계 제출합니다. (#12) * 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]> * [디디] 체스 - 스프링 실습 2단계 제출합니다. (#73) * 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]> * refactor : RoomDAO 를 spring data jdbc repository로 변경 * refactor : Spring data jdbc 에 맞도록 테이블, 엔티티, dto 설계 * refactor : 변경 사항에 맞게 Controller와 서비스 수정 * refactor : 공백 제거 * refactor : GameRequestDto 에 기본 생성자 추가 * refactor : HTTP 메서드에 맞게 Mapping 어노테이션 수정,@PathVariable 적용 * refactor : HTTP 메서드에 맞게 Mapping 어노테이션 수정,@PathVariable 적용 * refactor : getPath 이름을 path로 수정, del : GameRepository, PieceRepository Co-authored-by: DD <[email protected]> Co-authored-by: dd <[email protected]>
gracefulBrown
pushed a commit
that referenced
this pull request
May 4, 2020
* [디디] 체스 스프링 실습 1단계 제출합니다. (#12) * 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]> * [디디] 체스 - 스프링 실습 2단계 제출합니다. (#73) * 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]> * refactor : RoomDAO 를 spring data jdbc repository로 변경 * refactor : Spring data jdbc 에 맞도록 테이블, 엔티티, dto 설계 * refactor : 변경 사항에 맞게 Controller와 서비스 수정 * refactor : 공백 제거 * refactor : GameRequestDto 에 기본 생성자 추가 * refactor : HTTP 메서드에 맞게 Mapping 어노테이션 수정,@PathVariable 적용 * refactor : HTTP 메서드에 맞게 Mapping 어노테이션 수정,@PathVariable 적용 * refactor : getPath 이름을 path로 수정, del : GameRepository, PieceRepository * refactor : JDBCConnectorTest 삭제 및 build.gradle 수정 * refactor : Status라는 네이밍을 Score로 변경 * refactor : banner 변경 및 addRoom에 리턴을 RoomEntity로 하도록 수정 * feat : SpringDefaultControllerTest 추가 * feat : SpringRoomControllerTest 추가 * feat : SpringGameControllerTest 추가 Co-authored-by: DD <[email protected]> Co-authored-by: dd <[email protected]>
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.
잘부탁드립니다