오잉또잉 체스게임 #6

hanueleee · 2023-03-17T09:09:09Z

No description provided.

Co-authored-by: hanueleee <[email protected]>

greeng00se

테스트 부분은 전부 given-when-then(준비-실행-검증)을 적용해보면 어떨까?
다른 사람이 보기에는 너무 어려운 것 같아
체스 미션한다고 고생했어!

greeng00se · 2023-03-18T04:14:08Z

README.md

+
+### 랭크 : 가로줄을 나타내는 enum
+
+### 파일 : 세로줄을 나타내느 enum


느 -> 는 오타입니다~

greeng00se · 2023-03-18T04:20:11Z

src/main/java/chess/controller/PieceMapper.java

+        String defaultPiece = "";
+        if (piece.getType() == PieceType.EMPTY) {
+            return ".";
+        }
+        if (piece.getType() == PieceType.PAWN) {
+            defaultPiece = "p";
+        }
+        if (piece.getType() == PieceType.BISHOP) {
+            defaultPiece = "b";
+        }
+        if (piece.getType() == PieceType.KNIGHT) {
+            defaultPiece = "n";
+        }
+        if (piece.getType() == PieceType.ROOK) {
+            defaultPiece = "r";
+        }
+        if (piece.getType() == PieceType.QUEEN) {
+            defaultPiece = "q";
+        }
+        if (piece.getType() == PieceType.KING) {
+            defaultPiece = "k";
+        }


다음과 같이 맵을 사용하면 많은 분기를 줄일 수 있을 것 같아 👍

private static filnal Map<PieceType, String> Mapper = Map.of( PieceType.PAWN, "p" )

아뭐야 똑같은말했네용

오 그렇네 감사링

친절했다 ㄷㄷ

greeng00se · 2023-03-18T04:20:35Z

src/main/java/chess/domain/Board.java

+    public static final int BLACK_GENERALS_RANK = 7;
+    private final Map<Position, Piece> board;


정적 필드랑 필드는 구분해주면 좋을 것 같아

greeng00se · 2023-03-18T04:25:49Z

src/main/java/chess/domain/Board.java

+
+    private void placePieceAtPosition(final List<Piece> pieces, final Position position, int rank) {
+        if (position.isRank(rank)) {
+            this.board.put(position, pieces.get(position.getFile().getIndex()));


인스턴스 필드, 메서드에 접근할 때 일관성있게 this를 사용하고 있는데
이렇게 사용하는 이유는 페어와 컨벤션을 맞췄기 때문이겠지?
이 부분에 대해서 어떻게 생각해? 사용하지 않는 쪽과 사용하는 쪽 뭐가 더 좋을 것 같아?

사실 별생각없는데 인텔리제이에서 저장 키맵?에 this가 자동으로 붙도록 설정해두고 진행했어.
this를 붙여서 정확도?명확성?은 올라갔는데, 전체적으로 다시보니까 코드가 괜히 길어지는 것 같기도 하네.

greeng00se · 2023-03-18T04:28:45Z

src/main/java/chess/domain/Board.java

+        if (List.of(Direction.NE, Direction.SE, Direction.NW, Direction.SW).contains(direction)) {
+            this.checkDiagonalPiece(destination);
+            this.checkSameColor(destination, color);


�direction.isDiagnoal 과 같이 해당 Direction이 대각선인지 확인하는 메서드를 만들어보면 어떨까?

인정 Direction을 객체답게!!

greeng00se · 2023-03-18T04:49:29Z

src/main/java/chess/domain/position/Position.java

+        int sourceFIleIndex = this.getFile().getIndex();
+        int sourceRankIndex = this.getRank().getIndex();
+        int destinationFIleIndex = destination.getFile().getIndex();
+        int destinationRankIndex = destination.getRank().getIndex();
+
+        int fileGap = destinationFIleIndex - sourceFIleIndex;
+        int sourceGap = destinationRankIndex - sourceRankIndex;


이 부분은 File과 Rank에서 각각 같은 Enum을 받아 받을 수 있을 것 같아

음 enum으로 연산하라는 말?

greeng00se · 2023-03-18T04:50:40Z

src/main/java/chess/domain/position/Position.java

+        try {
+            return Direction.findByVector(fileGap, sourceGap);
+        } catch (Exception ignored) {
+        }


orElseGet(빈 방향)을 반환한다면 try-catch를 사용하지 않아도 될 것 같아

단위벡터를 사용해서 해결했습니다 ㅎㅎㅎ

greeng00se · 2023-03-18T04:56:14Z

src/main/java/chess/domain/position/Position.java

+        if (fileGap > 0) {
+            if (sourceGap > 0) {
+                return Direction.NE;
+            }
+            if (sourceGap < 0) {
+                return Direction.SE;
+            }
+            return Direction.E;
+        }
+        if (fileGap < 0) {
+            if (sourceGap > 0) {
+                return Direction.NW;
+            }
+            if (sourceGap < 0) {
+                return Direction.SW;
+            }
+            return Direction.W;
+        }
+
+        if (sourceGap > 0) {
+            return Direction.N;
+        }
+        return Direction.S;


Direction.findByVector가 아닌

static 필드로 나이트의 �방향 리스트(NNE, ENE, ESE, SSE, SSW, WSW, WNW, NNW); 과
그 외 방향을 필드로 두고

Direction.knightDirection(a, b)
Direction.of(a, b) 받아서 구할 수 있을 것 같아

그러면 반복 제거 할 수 있을 듯

greeng00se · 2023-03-18T04:57:13Z

src/main/java/chess/view/InputView.java

+
+    public List<String> readCommand() {
+        String input = scanner.nextLine();
+        return Arrays.stream(input.split(" ")).collect(Collectors.toList());


" " 매직넘버

greeng00se · 2023-03-18T04:57:59Z

src/test/java/chess/domain/BoardTest.java

+            List<Piece> pieces = List.of(Rook.create(Color.WHITE), Knight.create(Color.WHITE), Pawn.create(Color.BLACK),
+                    Rook.create(Color.BLACK));
+            for (int i = 0; i < 4; i++) {
+                assertThat(board).extracting("board").asInstanceOf(InstanceOfAssertFactories.MAP)


필드와 강결합(필드명을 알아야함)되는데 이렇게 사용할 이유가 있을까?

무슨말이느뇨 이해하지 못했느뇨

shin-mallang

1차

shin-mallang · 2023-03-18T06:15:19Z

src/main/java/chess/CommandLine.java

+    private static final String INVALID_COMMAND_ERROR_MESSAGE = "잘못된 명령어입니다.";
+    private static final String INVALID_ARGUMENT_ERROR_MESSAGE = "는 인자를 입력할 수 없습니다.";
+    private static final String INVALID_ARGUMENT_COUNT_ERROR_MESSAGE = "는 인자를 2개만 가질 수 있습니다.";
+    private static final String NO_ARGUMENT_ERROR_MESSAGE = "인자를 반환할 수 없는 명령입니다.";


굳이 메세지를 상수가 할 필요가 있었어?
오히려 더 파악하기 힘든 것 같아

동의. 수정하겠삼 ㅎㅎㅎ

shin-mallang · 2023-03-18T06:15:56Z

src/main/java/chess/controller/ChessController.java

+    private final InputView inputView;
+    private final OutputView outputView;
+
+    public ChessController(final InputView inputView, final OutputView outputView) {


굳이 주입해준 이유가 있을까?
static으로 사용해 보는건 어때?

말랑 근데 static으로 사용하는 거의 단점은 없나?
당장 InputView가 여러가지로 가능한게 아니니 필요 없으려나??

주입하기

주입안하고 그냥 생성자에서 알아서 초기화

static으로 사용

이 셋 중 아무거나 쓰는 편
말랑이 3을 선호하는 이유는?

@hanueleee
@eunkeeee
테스트할 것도 아니고 그냥 컨크롤러에서만 편하게 쓸건데 굳이 주입을 할 필요가 있어?
회면이 바뀔 것도 아니니까?

shin-mallang · 2023-03-18T06:17:36Z

src/main/java/chess/controller/PieceMapper.java

+
+public class PieceMapper {
+
+    public static String map(Piece piece) {


Map<Type, String>을 쓰면 분기를 줄일 수 있을 것 같아

shin-mallang · 2023-03-18T06:17:49Z

src/main/java/chess/controller/PieceMapper.java

+        String defaultPiece = "";
+        if (piece.getType() == PieceType.EMPTY) {
+            return ".";
+        }
+        if (piece.getType() == PieceType.PAWN) {
+            defaultPiece = "p";
+        }
+        if (piece.getType() == PieceType.BISHOP) {
+            defaultPiece = "b";
+        }
+        if (piece.getType() == PieceType.KNIGHT) {
+            defaultPiece = "n";
+        }
+        if (piece.getType() == PieceType.ROOK) {
+            defaultPiece = "r";
+        }
+        if (piece.getType() == PieceType.QUEEN) {
+            defaultPiece = "q";
+        }
+        if (piece.getType() == PieceType.KING) {
+            defaultPiece = "k";
+        }


아뭐야 똑같은말했네용

shin-mallang · 2023-03-18T06:18:30Z

src/main/java/chess/domain/Board.java

+    public static final int BLACK_GENERALS_RANK = 7;
+    private final Map<Position, Piece> board;


shin-mallang · 2023-03-18T06:22:56Z

src/main/java/chess/domain/Board.java

+        return this.board.get(position).isSameColor(color);
+    }
+
+    public List<Piece> getPiecesAt(Rank rank) {


findAllByRank도 좋을 듯?

ㅇㅇ get어쩌고를 쓰면 다른 사람 입장에서는 내부에 다른 로직이 존재할 것이라고 잘 생각 못하는 경우가 있어서 리얼 get만 딱 하는 곳에서 쓰는게 좋을듯

get은 진짜 get만 하는 곳에서 써라 굿

shin-mallang · 2023-03-18T06:23:19Z

src/main/java/chess/domain/Board.java

+        }
+    }
+
+    private void checkDiagonalPiece(final Position destination) {


왜 diagonal 체크인데 isEmpty를 하는거야?

폰은 대각선이 비어있으면 (다른 말을 잡을 수 없으면) 대각선으로 이동할 수 없으니까

shin-mallang · 2023-03-18T06:25:13Z

src/main/java/chess/domain/ChessGame.java

+
+    public ChessGame() {
+        this.status = GameStatus.START;
+        this.board = Board.create();


주입 받는것도 좋을 것 갘아

ChessGame에 `Board를 주입하라는? 왜죠?

shin-mallang · 2023-03-18T06:26:00Z

src/main/java/chess/domain/ChessGame.java

+    }
+
+    public void start() {
+        if (this.status != GameStatus.START) {


상태 패턴을 학습해 보는것도 도움이 될 거 같은걸?

shin-mallang · 2023-03-18T06:26:46Z

src/main/java/chess/domain/piece/Color.java

+    BLACK,
+    NONE;
+
+    public static Color reverse(Color color) {


static 떼고, color.reverse()도 좋을 듯?

반영했습니다. 굿

gitchannn

오잉아 나도 다녀간다

from. 깃짱코딩

gitchannn · 2023-03-20T06:32:55Z

README.md

@@ -1,7 +1,62 @@
 # java-chess


와 commit 보고 벌써 4단계 DB연결 다한줄 ㄷㄷ

킥 킥 어림도 없지

gitchannn · 2023-03-20T06:33:08Z

README.md

+    - [x] 룩
+    - [x] 비숍
+    - [x] 퀸
+    - [x] 킹


좀 구체적으로 적어주세요

왜여 >< 다 알자나

gitchannn · 2023-03-20T06:33:24Z

README.md

+- [x] 빈 보드를 생성
+- [x] 좌표를 입력으로 주었을때, 좌표의 피스를 반환(getPiece)
+    - [x] 좌표에 피스가 존재하는지 확인
+    - [x] 같은 색깔 피스 인지 확인.


gitchannn · 2023-03-20T06:33:58Z

src/main/java/chess/CommandLine.java

+    private static final String INVALID_COMMAND_ERROR_MESSAGE = "잘못된 명령어입니다.";
+    private static final String INVALID_ARGUMENT_ERROR_MESSAGE = "는 인자를 입력할 수 없습니다.";
+    private static final String INVALID_ARGUMENT_COUNT_ERROR_MESSAGE = "는 인자를 2개만 가질 수 있습니다.";
+    private static final String NO_ARGUMENT_ERROR_MESSAGE = "인자를 반환할 수 없는 명령입니다.";


gitchannn · 2023-03-20T06:35:44Z

src/main/java/chess/GameStatus.java

+public enum GameStatus {
+    START,
+    END,
+    MOVE


땀 안흘리고 끝나도 되나요?

엥 그러게 얘말고도 다른 enum도 땀 안붙였는데 잘돌아가네 왤까,, 일단 수정했슴당

오호 enum내에 따로 로직 없으면 땀 안붙여도 되는듯

gitchannn · 2023-03-20T06:45:32Z

src/main/java/chess/domain/piece/Color.java

+    BLACK,
+    NONE;
+
+    public static Color reverse(Color color) {


gitchannn · 2023-03-20T06:46:59Z

src/main/java/chess/domain/piece/Piece.java

+public abstract class Piece {
+
+    public static final String PIECE_CANNOT_MOVE_ERROR_MESSAGE = "이(가) 움직일 수 없는 위치입니다.";
+    public static final String OUT_OF_DISTANCE_ERROR_MESSAGE = "이(가) 움직일 수 없는 거리입니다.";


꼼꼼한 (가)얼?

어떤데 ㅋ

gitchannn · 2023-03-20T06:47:42Z

src/main/java/chess/domain/piece/PieceFactory.java

+    }
+
+    private static List<Piece> createGenerals(final Color color) {
+        List<Piece> generals = new ArrayList<>();


List.of(Rook.create(color), ~~~~ 해도 좋을듯 근데 똑같은듯

gitchannn · 2023-03-20T06:51:03Z

src/main/java/chess/domain/position/Direction.java

+    SSW(-1, -2),
+    WSW(-2, -1),
+    WNW(-2, 1),
+    NNW(-1, 2);


너무 축약이 심한듯?? 풀어서 써주면 더 좋을 것 같아

요거 나침판에 쓰이는 표기여서 그대로 가져와봤삼

gitchannn · 2023-03-20T06:51:43Z

src/main/java/chess/domain/position/File.java

+    public static final String INVALID_FILE_ERROR_MESSAGE = "잘못된 좌표입니다.";
+    private final String label;
+
+    private final int index;
+


끔찍한 혼종ㅋㅋㅋㅋ

woowahan-neo and others added 30 commits March 13, 2023 12:57

feat: remove web dependency

ff4f19d

feat: add example for functional programming api example

1bab9ff

chore: change package

98598c0

feat: Database 연결

abee5c9

Co-authored-by: hanueleee <[email protected]>

docs: 기능목록 작성

4bbccbe

Co-authored-by: hanueleee <[email protected]>

docs: 기능목록 수정

644808c

Co-authored-by: hanueleee <[email protected]>

feat: Position 구현

29390c8

Co-authored-by: hanueleee <[email protected]>

feat: Position 검증 기능 추가

80458aa

Co-authored-by: hanueleee <[email protected]>

feat: Piece 추상 클래스 구현

d902919

Co-authored-by: hanueleee <[email protected]>

feat: Piece 상속하는 King,Queen,Rook,Knight,Bishop,Pawn 추가

fa15be0

Co-authored-by: hanueleee <[email protected]>

feat: PieceFactory 기능 구현

2e1eca1

Co-authored-by: hanueleee <[email protected]>

feat: Board 생성 구현

14a6733

Co-authored-by: hanueleee <[email protected]>

feat: Board 초기화 구현

edaa59a

Co-authored-by: hanueleee <[email protected]>

refactor: 패키지 이동

f482756

Co-authored-by: hanueleee <[email protected]>

feat: InputView 구현

467627d

Co-authored-by: hanueleee <[email protected]>

feat: CommandLine 구현

c7a7db4

Co-authored-by: hanueleee <[email protected]>

fix: Board 초기화 오류 해결

232fe0a

Co-authored-by: hanueleee <[email protected]>

feat: Board에 Piece리스트를 반환하는 메서드 구현

cc72eb0

Co-authored-by: hanueleee <[email protected]>

feat: Piece에 타입 필드 추가

c3f340b

Co-authored-by: hanueleee <[email protected]>

feat: ChessGame 구현

e79765c

Co-authored-by: hanueleee <[email protected]>

feat: PieceMapper 구현

31c776d

Co-authored-by: hanueleee <[email protected]>

feat: BoardDto 구현

34b9b80

Co-authored-by: hanueleee <[email protected]>

feat: 체스보드 출력 구현

a5f3e86

Co-authored-by: hanueleee <[email protected]>

feat: ChessController 구현

d174cd9

Co-authored-by: hanueleee <[email protected]>

feat: Application 구현

8dae288

Co-authored-by: hanueleee <[email protected]>

feat: 해당 좌표의 피스를 구하는 기능 구현

83915ae

Co-authored-by: hanueleee <[email protected]>

feat: 좌표에서 또다른 좌표로 방향 구하는 로직 구현

f6e8145

Co-authored-by: hanueleee <[email protected]>

feat: 좌표에서 방향 더하는 연산 추가

6d814c7

Co-authored-by: hanueleee <[email protected]>

feat: 이동 경로 상에 다른 피스가 있는지 확인하는 기능 구현

a256899

Co-authored-by: hanueleee <[email protected]>

feat(Board): 목적지에 같은 색깔 피스가 있는지 확인하는 기능 구현

03d5ca7

Co-authored-by: hanueleee <[email protected]>

echo724 and others added 14 commits March 17, 2023 14:52

feat: 보드에서 말 이동 검증 구현

5ce5dc3

Co-authored-by: hanueleee <[email protected]>

feat(Piece): 피스들 구현

b6a8808

Co-authored-by: hanueleee <[email protected]>

feat(Position): 거리 계산하는 로직 구현

053604f

Co-authored-by: hanueleee <[email protected]>

feat(Game): 검증 후 이동하는 로직 구현

7c9622e

Co-authored-by: hanueleee <[email protected]>

feat(GameStatus): 게임 상태 관리 기능 구현

4cd77f2

Co-authored-by: hanueleee <[email protected]>

feat: 게임 전체 로직 조합

fc66c75

Co-authored-by: hanueleee <[email protected]>

style(View): 출력 문구 수정

a869608

Co-authored-by: hanueleee <[email protected]>

refactor(BoardDto): 게임 대신 보드를 주입하는 방식으로 수정

e83eb59

Co-authored-by: hanueleee <[email protected]>

feat(OutputView): 에러 메세지 출력 기능 구현

31e0fe7

Co-authored-by: hanueleee <[email protected]>

refactor(Controller): 메서드 정리 및 게임 턴 책임 분리

a0bc8fe

Co-authored-by: hanueleee <[email protected]>

refactor(EmptyPiece): 빈 피스 실행 예외 처리

c6f5f35

Co-authored-by: hanueleee <[email protected]>

refactor: 코드 정리

df8cd7a

Co-authored-by: hanueleee <[email protected]>

refactor(Board): 보드 리펙토링

ecfc8b6

Co-authored-by: hanueleee <[email protected]>

refactor(Game): 게임 리펙토링

d6a8ff7

Co-authored-by: hanueleee <[email protected]>

greeng00se reviewed Mar 18, 2023

View reviewed changes

shin-mallang reviewed Mar 18, 2023

View reviewed changes

hanueleee added 3 commits March 20, 2023 14:13

refactor: this 제거

8f8d542

refactor(PieceMapper): if 대신 Map을 사용하여 분기 줄이기

d7d310e

refactor(Piece): 모든 Piece 상속 클래스들의 생성자 로직을 정적 팩토리 메서드 내부로 이동(생성자 단순화)

2e37b07

gitchannn reviewed Mar 20, 2023

View reviewed changes

hanueleee added 10 commits March 20, 2023 18:01

refactor(Position): 위치 간의 방향을 계산하는 로직 수정

22384af

refactor: 에러 메시지 상수 해제

dfe166c

refactor(dto): BoardDto의 책임을 RankDto와 분배

14ddc61

refactor(view): 시작 메시지를 각각 단일 상수화

1b6daee

refactor: 패키지 정리

c7612a3

test: PieceMapper 테스트

bec77c8

refactor(Board): getPiecesAt -> findAllByRank 메소드명 수정 및 stream 제거

6481479

refactor(ChessController): 정확한 예외클래스 지정

d0f7612

refactor(Color): static메소드 해제

2ba065c

refactor: 메소드 순서 정리, 코드 중복 제거

7e54715


		### 랭크 : 가로줄을 나타내는 enum

		### 파일 : 세로줄을 나타내느 enum

		public static final int BLACK_GENERALS_RANK = 7;
		private final Map<Position, Piece> board;


		public class PieceMapper {

		public static String map(Piece piece) {

오잉또잉 체스게임 #6

Are you sure you want to change the base?

오잉또잉 체스게임 #6

Conversation

hanueleee commented Mar 17, 2023

greeng00se left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hanueleee Mar 20, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shin-mallang left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hanueleee Mar 20, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hanueleee Mar 21, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

gitchannn left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hanueleee Mar 20, 2023 •

edited

Loading

hanueleee Mar 20, 2023 •

edited

Loading

hanueleee Mar 21, 2023 •

edited

Loading