[무늬] 체스 스프링 4단계 미션 제출합니다. #186
Conversation
* feat : 기존 Chess 코드 추가 * refactor : 기존 Chess 코드의 Application과 Controller 명명 수정 * refactor : JdbcTemplate 명명 수정 및 Spring Bean 등록 - 기존 JdbcTemplate 라이브러리와 충돌로 인한 명명 수정 - Dao, Service, ConnectionManager를 Spring Bean 등록 * feat : SpringChessController 추가 * feat : SpringChessController 구현 * refactor : controller 피드백 리팩토링 * refactor : database package 리팩토링 - ConnectionManager를 DataSource로 명명 수정. - CustomJdbcTemplate을 JdbcTemplate으로 명명 수정, 별칭 지정. - MySqlConnectionManager를 MySqlDataSource로 명명 수정, 별칭 지정. * refactor : 접근 지정자 수정 및 상수 추출 * refactor : BlackPawnStrategy와 WhitePawnStrategy를 PawnStrategy로 추상화
- Spring Data JDBC 적용을 위한 GameRoomRepository 구현 - schema.sql 구현
… step4 # Conflicts: # src/main/java/wooteco/chess/controller/SpringChessController.java # src/main/java/wooteco/chess/database/GameRoomRepository.java # src/main/java/wooteco/chess/entity/GameHistory.java # src/main/java/wooteco/chess/entity/GameRoom.java # src/main/java/wooteco/chess/service/ChessService.java # src/main/java/wooteco/chess/service/dto/ChessGameDto.java # src/main/java/wooteco/chess/web/PieceNameConverter.java # src/main/resources/templates/chess.hbs # src/main/resources/templates/index.hbs # src/main/resources/templates/result.hbs # src/test/java/wooteco/chess/database/GameRoomRepositoryTest.java # src/test/java/wooteco/chess/service/dto/ChessBoardDtoTest.java
There was a problem hiding this comment.
안녕하세요 무늬! 벌써 마지막 리뷰네요.
이번까지 4번이나 리뷰의 기회가 있었는데, 제가 그동안 제대로된 리뷰를 못드린 것 같아 마음이 아픕니다..ㅠㅠ
이번 4단계의 대부분이 3단계 리뷰할 때 이미 구현된 상태였어서 제가 지금 크게 리뷰할 게 없는 것 같네요...
또 가슴이 아픕니다ㅠㅠ 제 존재의의는 정말 뭘까요??
그런 의미에서 3단계 내용까지 다시 한번 훑어보기로 했습니다.
몇개 없지만... link
추가적으로 드리고 싶은 리뷰가 있습니다.
이번 4단계 미션의 요구사항 중 테스트를 구현할 수 있는 로직에 대해서는 테스트를 만들어 본다.가 있었는데요.
웹 환경에 대해서 controller나 service 역시 테스트를 할 수 있습니다.
기회가 된다면 mockMvc나 RestAssured를 통해 기능테스트를 진행해보는건 어떨까요?
처음 체스를 시작한 순간부터 생각해보면,
거의 두달에 가까운 시간을 체스 코드와 씨름했네요! 와우!
고생하셨어요 무늬! 제 리뷰가 작게나마 도움이 되었길 바랍니다ㅎㅎ 제 존재의의를 위해...
다음 미션도 화이팅이에요!
|
|
||
| import java.util.List; | ||
|
|
||
| public class ConsoleChessApplication { |
There was a problem hiding this comment.
엇! 브랜치를 병합하는 과정에서 다시 추가된 것 같습니다. 콘솔프로그램 관련 클래스 모두 삭제했습니다~ 😄
|
|
||
| return new ChessGameDto(id, chessBoardDto, pieceColorDto, chessStatusDtos, isEndStatus, isKingCaught); | ||
| } | ||
| // NOTE: 2020/04/28 DTO가 관리해야하는 필드의 종류 |
There was a problem hiding this comment.
공부해야할 부분에 대한 주석인데 미처 제거하지 못했네요! 🤣 삭제했습니다~
|
MockMvc를 활용해 컨트롤러를 일부 테스트하였으나... |
안녕하세요, 우!
3단계 피드백에서 말씀해주신 프론트 부분이 반영된 풀리퀘스트입니다!
코드리뷰에 혼동을 드려서 죄송합니다 😢
혹시 보시다가 의문점이 생기시면 언제든지 연락주세요!
항상 멋진 리뷰 감사합니다 😄