나는 허브의 내일이란다 #12

greeng00se · 2023-03-26T12:55:14Z

No description provided.

* docs: 요구사항 정리 Co-authored-by: iamjooon2 <[email protected]> * docs: 요구사항 수정 Co-authored-by: iamjooon2 <[email protected]> * feat: 랭크 생성 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 파일 생성 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: given-when-then 주석 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: Position 생성 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: Color Enum 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 기물에 대한 추상 클래스 추가 Co-authored-by: iamjooon2 <[email protected]> * chore: 패키지 변경 Co-authored-by: iamjooon2 <[email protected]> * feat: 체스 기물 생성 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * refactor: 체스 기물 생성시 정적 팩터리 메서드를 사용하도록 변경 Co-authored-by: iamjooon2 <[email protected]> * feat: 빈 칸을 생성하는 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 보드 초기화 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 출력 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 체스 게임 생성 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * style: Board 클래스 포맷팅 변경 Co-authored-by: iamjooon2 <[email protected]> * feat: 입력 반복 제한 횟수 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: Command 추가 및 적용 Co-authored-by: iamjooon2 <[email protected]> * feat: 각 기물의 추상 메서드 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: Rank의 차이를 반환하는 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: File의 차이를 반환하는 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * test: 랭크 차이 테스트에 ParameterizedTest 적용 Co-authored-by: iamjooon2 <[email protected]> * fix: 랭크와 파일 의미 변경 Co-authored-by: iamjooon2 <[email protected]> * feat: 나이트 이동 확인 기능 구현 Co-authored-by: iamjooon2 <[email protected]> * feat: 룩 이동 확인 기능 구현 Co-authored-by: iamjooon2 <[email protected]> * feat: 비숍 이동 확인 기능 구현 Co-authored-by: iamjooon2 <[email protected]> * refactor: 나이트 상수명 수정 Co-authored-by: iamjooon2 <[email protected]> * refactor: 킹 이동 확인 기능 구현 Co-authored-by: iamjooon2 <[email protected]> * refactor: 퀸 이동 확인 기능 구현 Co-authored-by: iamjooon2 <[email protected]> * feat: 폰 이동 확인 기능 구현 Co-authored-by: iamjooon2 <[email protected]> * feat: 위치값으로 Rank랑 File 받는 메서드 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 랭크, 파일 사이의 값을 반환하는 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 시작과 끝 사이의 위치를 구하는 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 위치를 입력받아 position을 리턴하는 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 위치를 입력받아 position을 리턴하는 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 기물을 이동하는 기능 구현 Co-authored-by: iamjooon2 <[email protected]> * refactor: 예외 자세하게 출력하도록 수정 Co-authored-by: iamjooon2 <[email protected]> * feat: START 커맨드 검증 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: ChessGame의 기물을 움직이는 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 체스 게임 입력 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * feat: 이동 경로에 기물이 존재하는지 확인하는 기능 추가 Co-authored-by: iamjooon2 <[email protected]> * test: 루이로페즈 모든 슈타이니츠 바리에이션 테스트 추가 Co-authored-by: iamjooon2 <[email protected]> * test: 테스트 도와주는 메서드 추가 Co-authored-by: iamjooon2 <[email protected]> * test: 테스트 주석 수정 Co-authored-by: iamjooon2 <[email protected]> * refactor: final 빠진 부분 추가 Co-authored-by: iamjooon2 <[email protected]> * docs: 요구사항 수정 Co-authored-by: iamjooon2 <[email protected]> * feat: Lambda, Stream 이해하기 * refactor: 이동 로직 개선 * refactor: 폰의 초기 위치(랭크) 상수명 수정 * feat: 체스게임이 초기화되었는지 확인하는 기능 추가 * refactor: Controller 로직 개선 --------- Co-authored-by: iamjooon2 <[email protected]>

# Conflicts: # src/main/java/chess/view/input/RoomInputView.java

drunkenhw

당신 수료하세요

drunkenhw · 2023-03-30T09:29:18Z

README.md

+
+### 개요
+
+체스는 가로와 세로가 각각 8줄씩 64칸으로 격자로 배열 된 체스보드에서 두 명의 플레이어가 기물들을 규칙에 따라 움직여 싸우는 보드 게임이다.


나무위키 같아서 좋네요

어떻게 알았지?

drunkenhw · 2023-03-30T09:30:33Z

src/main/java/chess/controller/CommandMapper.java

+import java.util.HashMap;
+import java.util.Map;
+
+public class CommandMapper<K, V> {


나만의 Map 쿨하네요

drunkenhw · 2023-03-30T09:31:38Z

src/main/java/chess/controller/game/GameCommand.java

+        return Arrays.stream(values())
+                .filter(command -> command != EMPTY)
+                .filter(command -> command.name().equalsIgnoreCase(commands.get(COMMAND_INDEX)))
+                .findFirst()


findAny() 도 있는데 first를 쓰신 이유 궁금

둘다 똑같다고 생각합니다

drunkenhw · 2023-03-30T09:32:55Z

src/main/java/chess/controller/game/GameCommand.java

+
+    public static GameCommand from(final List<String> commands) {
+        return Arrays.stream(values())
+                .filter(command -> command != EMPTY)


empty를 거르는 모습 정말 꼼꼼하네요

drunkenhw · 2023-03-30T09:36:02Z

src/main/java/chess/controller/game/GameController.java

+    }
+
+    private void move(final List<String> commands) {
+        final MoveDto moveDto = new MoveDto(commands.get(MOVE_SOURCE_INDEX), commands.get(MOVE_TARGET_INDEX));


index는 view가 알아야 할 정보아닐까요?

잘못된 부분일수도 있지만 일단 이게 가장 간단...

drunkenhw · 2023-03-30T09:43:55Z

src/main/java/chess/service/GameService.java

+        final List<MoveDto> moves = gameDao.findAllByRoomId(roomId);
+        for (MoveDto move : moves) {
+            final Position source = Position.from(move.getSource());
+            final Position target = Position.from(move.getTarget());
+            board.move(source, target);
+        }


db에서 나오는 순서대로 move를 시켜주는데 db에서 나오는 순서가 무조건 보장되나요?

굳~ 지금도 잘 돌아가지만 확실하게 하려면 orderBy를 해줘야겠군요

drunkenhw · 2023-03-30T09:44:49Z

src/main/java/chess/service/GameService.java

+        final Board board = boards.get(roomId);
+        if (board == null) {
+            throw new IllegalArgumentException("게임을 찾을 수 없습니다.");
+        }


이 부분이 계속 반복되는데 메소드로 분리하는건 어떸가요

그것도 굿~

drunkenhw · 2023-03-30T09:46:56Z

src/main/resources/chess.sql

+CREATE TABLE User
+(
+    id   int PRIMARY KEY AUTO_INCREMENT,
+    name varchar(255)
+);
+
+CREATE TABLE Room
+(
+    id       int PRIMARY KEY AUTO_INCREMENT,
+    name     varchar(255),
+    user_id  int
+);
+
+CREATE TABLE Move
+(
+    id      int PRIMARY KEY AUTO_INCREMENT,
+    source  varchar(2),
+    target  varchar(2),
+    room_id int
+);


null이 들어가도 되나요? not null 옵션은 안되나요?

곰곰하게 하려면 not null 굳

drunkenhw · 2023-03-30T09:47:40Z

src/test/java/chess/domain/board/GameResultTest.java

+
+@DisplayNameGeneration(DisplayNameGenerator.ReplaceUnderscores.class)
+@SuppressWarnings("NonAsciiCharacters")
+public class GameResultTest {


테스트가 왤케 꼼꼼한가요

drunkenhw · 2023-03-30T09:48:37Z

src/test/java/chess/repository/ChessJdbcDaoTest.java

+    static void beforeAll() {
+        connectionPool = new TestConnectionPool();
+        jdbcTemplate = new JdbcTemplate(connectionPool);
+        gameDao = new GameJdbcDao(jdbcTemplate);
+        final String query = "CREATE TABLE IF NOT EXISTS move ("
+                + "    id     INT        NOT NULL AUTO_INCREMENT PRIMARY KEY,"
+                + "    source VARCHAR(2) NOT NULL,"
+                + "    target VARCHAR(2) NOT NULL,"
+                + "    room_id INT NOT NULL"
+                + ");";
+        jdbcTemplate.executeUpdate(query);
+    }


뭐해주는 놈이죠 알려주세요

아 테스트 테이블을 만들었다 지웠다 하는 놈인강ㅅ?

예압! 테스트용 테이블 생성

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

refactor: 스트림 미니 미션 테스트 Disable

e22fcb3

refactor: 보드 생성 책임을 담당하는 BoardGenerator 추가

8a78c7b

test: 체스판 검증 기능 가독성 개선

4e41311

refactor: static 제거

8659523

refactor: 사용하지 않는 메서드 제거

85e3615

refactor: EmptyPiece의 정적 팩터리 메서드 네이밍 변경

547058d

refactor: board 검증부 부정연산자 제거

d878f8b

refactor: 기물 메서드명 일관성있게 변경

b904300

refactor: 줄바꿈 일관성있게 변경

319563d

refactor: Board를 문자열로 변경해주는 Converter 클래스 추가

5d450b8

refactor: Position 패키지 분리

0441806

docs: 3단계 요구사항 정리

d9522c7

feat: 같은 생상인지 확인하는 메서드 추가

0119bd3

feat: 기물의 점수를 계산하는 기능 추가

de53b5d

refactor: 보드 상태패턴 적용

b1b6758

feat: 왕이 잡히면 게임이 종료되는 기능 추가

65739a2

feat: 결과를 반환하는 기능 추가

0ad4ac9

feat: 결과를 계산하는 클래스 추가

2e408fc

feat: 상태 출력하는 기능 추가

505143e

refactor: 커맨드 길이 검증 개선

392be91

docs: 요구사항 정리

a87dc9d

refactor: 상태 클래스명 변경

abceac6

refactor: Board의 Move가 Position을 받도록 수정

e986881

feat: 데이터베이스 연동기능 추가

7130470

feat: 체스게임 초기화 기능 추가

01fe9d9

refactor: ChessGame에서 MoveDto 사용하도록 수정

4e8d1c2

greeng00se added 2 commits March 26, 2023 21:47

refactor: UI 수정

b9d5966

feat: ThreadLocal 값 제거 로직 추가

0ed5a87

greeng00se self-assigned this Mar 26, 2023

greeng00se added 12 commits March 26, 2023 22:53

refactor: 게임 진행 기능 수정

ecd5232

Merge remote-tracking branch 'origin/step2' into step2

e025164

# Conflicts: # src/main/java/chess/view/input/RoomInputView.java

refactor: nameDto 제거

1ffb77c

refactor: Action 하나만 사용하도록 수정

ef1ce69

refactor: static import 설정

1149eae

remove: 사용하지 않는 namedto 제거

682dd16

docs: 요구사항 정리

fb10477

refactor: 방 finished 필드 제거

9717a43

refactor: 방 미선택시 선택한 방 이름 none으로 보이도록 수정

515fbd1

remove: 사용하지 않는 메서드 제거

995fd19

docs: 다이어그램 추가

89d8395

fear: GameController에서 나가면 service에 올라간 board 제거

41e1643

greeng00se changed the title ~~지속적으로 코딩중.. 아직 미완성 문단속 잘하고 코딩해야지~~ 나는 허브의 내일이란다 Mar 26, 2023

greeng00se added 13 commits March 27, 2023 00:48

refactor: 메서드 위치 변경

172151a

remove: 사용하지 않는 생성자 제거

ba1baf4

remove: 사용하지 않는 변수 제거

8751e86

refactor: MainController Controller 인터페이스 구현하는 부분 제거

0d5f686

refactor: 인터페이스명 SubController로 변경

a8adb91

docs: 외래키 추가

1a13e1f

refactor: DAO optional 적용

e971899

refactor: rowMapper 재사용 가능하도록 필드로 설정

f9f953a

refactor: 각각의 DaoStub 이너 클래스로 생성

e24049a

refactor: InputView 중복 제거

602c24f

remove: 사용하지 않는 클래스 제거

9c8b418

refactor: jdbctemplate 중복 제거

80a91e0

refactor: 사용하지 않는 메서드 제거

a538ee0

drunkenhw reviewed Mar 30, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

나는 허브의 내일이란다 #12

나는 허브의 내일이란다 #12

greeng00se commented Mar 26, 2023

drunkenhw left a comment

drunkenhw Mar 30, 2023

greeng00se Mar 30, 2023

drunkenhw Mar 30, 2023

drunkenhw Mar 30, 2023

greeng00se Mar 30, 2023

drunkenhw Mar 30, 2023

drunkenhw Mar 30, 2023

greeng00se Mar 30, 2023

drunkenhw Mar 30, 2023

greeng00se Mar 30, 2023

drunkenhw Mar 30, 2023

greeng00se Mar 30, 2023

drunkenhw Mar 30, 2023

greeng00se Mar 30, 2023

drunkenhw Mar 30, 2023

drunkenhw Mar 30, 2023

drunkenhw Mar 30, 2023

greeng00se Mar 30, 2023


		### 개요

		체스는 가로와 세로가 각각 8줄씩 64칸으로 격자로 배열 된 체스보드에서 두 명의 플레이어가 기물들을 규칙에 따라 움직여 싸우는 보드 게임이다.

나는 허브의 내일이란다 #12

Are you sure you want to change the base?

나는 허브의 내일이란다 #12

Conversation

greeng00se commented Mar 26, 2023

drunkenhw 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment