forked from woowacourse/java-chess
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Step2 진짜 어렵네요 #10
Open
be-student
wants to merge
80
commits into
woowacourse-code-review-study:main
Choose a base branch
from
be-student:step2
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Step2 진짜 어렵네요 #10
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
* study : jdbc 실습 * docs: 기능 목록 장성 * feat: Piece 구현 * docs : 기능 목록 수정 * feat: piece 의 기본 생성자 구현 * feat: space 구현 * refactor: 도메인 구조 변경 및 기능목록 최신화 * feat: position 생성 * feat: Board 구현 * feat: 체스판 출력 기능 구현 * feat: 시작, 종료 입력 구현 * docs : 추가 기능 작성 * feat: pawnState 구현 * feat: knightState 구현 * feat: bishopState 구현 * feat: rookState 구현 * feat: queenState 구현 * feat: kingState 구현 * feat: position rank, file 차이 반환 기능 구현 * feat: color 에서 방향 결정하는 기능 구현 * feat: position 의 직선 경로 반환 기능 구현 * feat: board 의 말 이동 기능 구현 * feat: controller 커맨드 기능 구현 * feat: 출력 양식 변경 * feat: controller 작성 및 2단계 요구사항 완료 * refactor: file, rank 경로 생성 메서드 개선 * refactor: 안 쓰는 메서드 삭제 * refactor: state 메서드 분리 * refactor: state 들을 캐싱한다. * chore: 잡는 예외 변경 및 출력 양식 변경 * refactor: state 메서드 가독성 개선 * refactor: 보드 빈 칸에 빈 피스를 넣어둔다 * refactor: file 과 rank 의 필드명 변경 * chore: 메서드 명 통일 * refactor: 기물 순서 반환 메서드 분리 * refactor: 메서드 명 변경 및 순서 정리 * refactor: 컨트롤러 메서드 개선 * fix: board 이동 버그 수정 * feat: stream 람다 미니 미션 구현 * feat: default 메서드로 공통 로직 분리 * chore: 줄바꿈 수정 * test: board 테스트 케이스 추가 * chore: public 키워드들 제거 * refactor: controller 에서 도메인 객체를 response 로 변경하도록 수정 * refactor: Position 에서 직선 경로가 아니면 빈 List 를 반환한다 * fix: 필요하지 않은 테스트 disabled * refactor: rankSize 를 rank 에서 관리하도록 변경 * fix: 오타 수정 * style: 줄바꿈 변경 * feat: pieceType 의 필드명 name 으로 수정 * typo: 오타 수정 * refactor: x,y 변수명 fileDifference, rankDifference 로 변경 * chore: ArrayList 타입 List 형태로 교체 * chore: 안 쓰이는 생성자 제거 * refactor: rank, file 에 char 형으로 갖도록 변경 * refactor: requestInfo 클래스를 view 로 옮김 * refactor: piece 와, game 을 패키지로 분리 * refactor: piece 움직임에 대한 예외를 piece 패키지로 이동 * feat: game 추상화를 수준에 맞는 예외 던지기 * refactor: 에러 메시지에 나오는 예외 클래스명 제외 * feat: chessGame 에 state 패턴과, command 패턴 적용 * refactor: board 에서 catch 하는 부분 game 에서 catch 하도록 변경 * fix: 안 쓰는 파일 제거 * feat: 체스 위치와 관련된 상수 클래스 추가 * refactor: indent 2 제거 * chore: 안 쓰이는 코드 제거 * chore: 변수명 변경 * chore: 메서드 네이밍 변경 * fix: 깨진 테스트 수정 * refactor: 조건문에 느낌표 부분 제거 * fix: 캐싱 때문에, 상태가 공유되던 문제 해결 * feat: requestType 을 controller 로 이동 * fix: file, rank 에서도 custom 예외를 사용하도록 변경 * feat: position 을 String 으로 생성하는 기능 추가 * feat: isRunning 제거하고, 반복 가능하게 변경 * feat: command 를 controller 쪽으로 변경 * chore: 접근 제어자 변경 * chore: 사용되지 않는 클래스 제거 * feat: view 에서 사용되는 start, move, end 에 대해 컨트롤러 부분 제거 기존에는 controller 패키지에 가서, start, move, end 를 변경했어야 하지만 지금은 view 패키지에서 직접 변경할 수 있습니다 * feat: pieceMapper 추가 * feat: input 과 output 을 구분해서 의존성 방향을 설계 * feat: RequestType 이 바로 생성 불가능하게 변경 * refactor: 안 쓰는 코드 제거 * feat: 하위 레이어의 예외 번역 * refactor: pieceResponse 패키지 변경 * feat: 도메인에 있던 piece 이름들 controller 에서 mapper 로 변경 * feat: piece를 불변 객체로 변경 * refactor: 위치가 있는 상수 클래스 enum 으로 변경 * feat: 입력 재입력 과정 추가 * refactor: 출력하는 부분을 request type 쪽으로 옮김 * chore: 줄바꿈 변경 * fix: 정규 표현식에 ^$ 추가 * refactor: split 하는 부분 request 안쪽으로 이동 --------- Co-authored-by: BGuga <[email protected]>
This reverts commit f2bc421.
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.
No description provided.