[1, 2단계 - Spring 적용하기] 배럴(김나은) 미션 제출합니다. #245

knae11 · 2021-04-16T08:59:32Z

안녕하세요! 미립 :) 배럴입니다.

이번 스프링을 처음 접하게 되었습니다. 페어의 코드로 같이 스프링으로 옮기는 작업을 진행하였습니다 :) 페어가 조엘이었는데, 리뷰어가 미립이었다고 하더라구요 :) (미립에게 조금은 친숙한 코드나 UI일거라는 생각이 드네요..ㅎㅎ)

아직 레이어 구조나 Spring에 대한 이해가 많이 없어서ㅠㅠ 어떻게 공부해야하는지도 많이 모르는 상태입니다..ㅎㅎㅎ
많은 피드백을 줄수록 제가 학습할 수 있는 내용들이 많을 것 같아요!
잘 부탁드립니다! 😃

DB 테이블

CREATE DATABASE chess_db;
USE chess_db;
CREATE TABLE chess_game (
    current_turn_team VARCHAR(5) NOT NULL,
    is_playing boolean NOT NULL
);
CREATE TABLE team_info (
    team VARCHAR(5) NOT NULL,
    piece_info VARCHAR(400) NOT NULL,
    primary key (team)
);

seok-2-o

안녕하세요 배럴. 이번 미션을 함께할 미립입니다 :)

큰 변화 없이 스프링을 잘 적용해주신것 같아요.
다만, HTTP 메소드가 각각 어떤 상황에서 활용되는지 학습해보면 좋을 것 같아요.

미션 진행하면서 궁금하거나 어려운점 있으면 편하게 DM 주세요.

src/main/java/chess/SpringChessApplication.java

seok-2-o · 2021-04-18T08:45:44Z

src/main/java/chess/controller/SpringChessController.java

+        this.springChessService = springChessService;
+    }
+
+    @GetMapping(value = "/startNewGame", produces = MediaType.APPLICATION_JSON_VALUE)


https://javaplant.tistory.com/18

각각 상황에 맞는 HTTP 메소드를 활용해보는 것은 어떤가요 ?

produces 가 하는 역할을 설명해주실 수 있나요 ?

[Spring] GetMapping, PostMapping - 4

내용

@PostMapping(path="/home", consumes="application/json")

consumes 는 Content-Type 헤더에 기반한다.

@GetMapping(path = "/pets/{petId}", produces = "application/json")

produces 는 Accept 헤더에 기반한다.

Content-Type은 해당 바디의 타입이 무엇인지를 나타내며 Accept는 받을 수 있는 타입을 나타낸다.

명시하지 않으면 기본적으로 json 타입이 적용되는 듯 하다.

MediaType 클래스의 상수인 APPLICATION_JSON_VALUE, APPLICATION_XML_VALUE를 많이 사용한다.

링크

스프링 공식문서 ~Mapping

mdn 문서 Content-Type

mdn 문서 Accept

이렇게 이해하고 있는데 맞을까요?~!

http 관련해서는 DM으로 질문드릴게요 :)

consumes나 produces는 굳이 명시하진 않나요~~??

저는 개인적으로 명시할 이유가 없다면 생략하는 편입니다.

개발자마다 취향 차이가 있는것 같아요.

src/main/java/chess/controller/SpringChessController.java

src/main/java/chess/webdao/PiecePositionDAOConverter.java

seok-2-o · 2021-04-18T08:50:19Z

src/main/java/chess/webdao/SpringChessGameDao.java

+
+    private ChessGame generateChessGame(final Team blackTeam, final Team whiteTeam) {
+        final String chessGameQuery = "SELECT * FROM chess_game";
+        final ChessGameTableDto chessGameTableDto = this.jdbcTemplate.queryForObject(chessGameQuery, actorRowMapper);


검색 되는 게임이 없으면 어떻게 동작하나요 ?

해당 내용은 status 코드를 통해서 클라이언트단에서 alert 창을 띄워주게 만들었어요! 자바로직이나 Dao 측에서 처리해줄 수 있는 방법이 떠오르지 않아서요ㅠㅠ 혹시 방법이 있나요..?🙏

아 컨트롤러에서 DAO 에서 발생하는 버그를 직접 컨트롤하고 있나요?

네 지금SQL exception을 잡아서 예외처리해서 INTERNAL_SERVER_EXCEPTION status code를 반환해주고 있어요!:) 혹시 다른데서 처리하는 방안이 있나요??

seok-2-o · 2021-04-18T08:51:00Z

src/main/java/chess/webdto/ChessGameDTO.java

+    private final Map<String, Map<String, String>> piecePositionByTeam;
+    private final String currentTurnTeam;
+    private final Map<String, Double> teamScore;
+    private final boolean isPlaying;


맵을 활용하신 이유가 있으실까요 ?
ScoreDto 등을 활용하는것은 어떻게 생각하시나요 ?

적용해 보았어요!ㅎㅎㅎ

Map<String(Position),String(Piece)> 으로 여러개 들어가는 경우는 어떻게 해야하나요..? 2개만 있는 경우에는 하나씩 정의해주었는데.. 여러개가 들어가게 되면 List 형식으로 관리해야하는 건지 잘 모르겠습니다ㅠㅠ

설계자체가 너무 중복되게 설계되지 않았나 고민 해볼 필요가 있을 것 같아요
팀별 리스트로 각각가지고 있으면 어떤가요?
예를들어 List balckTeam
List whiteTeam 두가지를 가지는 경우요

수정해보려고 했는데.. 유지하면서 수정하는 방안은 잘 모르겠어요ㅠㅠㅠ 테이블 구조나 도메인과 연결하여 통신하는 방법에 대해서 다시 고민을 해볼게요!

몇시간 더 시도해보았는데.. 역시 안 되네요ㅠㅠ

knae11 · 2021-04-21T11:15:46Z

안녕하세요 미립!

테이블 구조를 변경해 보았어요ㅎㅎ 기존에 사용하던 테이블을 지우고 board 하나의 테이블로 관리하게 해보았는데 어떨까요? 이벤트소싱?! 방식이란걸 알게되서 적용해 보았습니다 😀

DDL

CREATE DATABASE chess_db;
USE chess_db;
CREATE TABLE board (
	board_id bigint NOT NULL AUTO_INCREMENT,
	room_id bigint NOT NULL,
	start char(2) NOT NULL,
	destination char(2) NOT NULL,
	date TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
	PRIMARY KEY(board_id)	
);

knae11 · 2021-04-22T05:01:56Z

학습로그 1-2

[Spring] `@Transactional` - 3

내용

트랜잭션의 원자성을 유지시켜준다.
기본은 (readOnly = false)로 셋팅되어 있다. (readOnly = true)의 경우, 읽기 이외의 경우 exception을 던진다.

[DB] 이벤트소싱 패턴 - 4

내용

이벤트를 저장하는 방식이다.
히스토리를 관리가 가능하며, 비즈니스 로직과 동기화할 필요가 없다.
히스토리가 많으면 비즈니스로직을 수행해야하는 오버헤드가 크다. 단점을 스냅샷을 활용하여 보완할 수 있다고 한다.

링크

마이크로소프트 이벤트소싱 관련 글

seok-2-o

피드백 반영 잘 해주셨네요 👍
저는 체스 미션에 이벤트소싱 방식이 잘 어울리는 구조인지 의문인데
배럴은 어떤 관점에서 이벤트소싱에 어울린다고 생각하는지 궁금해요.

해당 부분에 대한 답변을 다시 PR 혹은 DM 주시면, 해당 미션은 마무리 할 수 있도록 할께요.

knae11 · 2021-04-25T09:26:16Z

안녕하세요! 미립 드디어 다시 PR을 보내네요!
이번에도 DB 테이블을 변경해보았어요!ㅎㅎㅎ 아직 방만들기는 안해서 방만들기 고려해서 테이블을 다시 설계해 보았습니다. 일단 room_id = 1로 놓고 진행하였습니다!ㅎㅎㅎ

더불어 Dto도 정리한다고 정리해봤는데,,, 너무 어렵네요.ㅎㅎ dto 패키지 안에 converter 패키지를 만들고, 여기서 조립해주려고 해보았어요. Service 레이어에서는 만드는 역할이 아니라고 하는데..어디서 Dto를 만들어 줘야하는건지 잘 모르겠네요ㅠㅠ

Dto가 많아서 헷갈려서 Dto 안에 view, dao 등으로 패키지를 나눠서 관리해보았습니다.

피드백 부탁드릴게요. 🙏

DDL

CREATE TABLE room (
	room_id BIGINT NOT NULL,
	turn CHAR(5) NOT NULL,
	is_playing BOOLEAN NOT NULL,
	name VARCHAR(25) NOT NULL,
	password VARCHAR(16),
	create_at DATETIME DEFAULT CURRENT_TIMESTAMP,
	PRIMARY KEY(room_id)
)

CREATE TABLE board (
	board_id BIGINT NOT NULL AUTO_INCREMENT,
	team VARCHAR(10) NOT NULL,
	position VARCHAR(10) NOT NULL,
	piece VARCHAR(10) NOT NULL, 
	is_first_moved BOOLEAN NOT NULL,
	room_id BIGINT NOT NULL,
	PRIMARY KEY(board_id),
	FOREIGN KEY (room_id) REFERENCES room (room_id)
)

seok-2-o

이전 구조에 비해 다소 코드를 이해하기 어려워 진 것 같아요.
개인적인 생각이지만 소프트웨어는 멋진 구조를 가지기보단 쉬운 구조를 먼저 생각하는게 좋다고 생각해요.
불필요한 클래스가 없는지 살펴보고 해당 클래스들을 삭제해보면 좋을 것 같네요.

src/main/java/chess/webdao/MysqlChessDao.java

src/main/java/chess/webdto/converter/DaoToChessGame.java

knae11 · 2021-04-26T05:56:04Z

학습로그 1-3

[DB] 데이터 모델링 - 5

내용

[Architecture] Layered Architecture, Dto - 4

내용

layered architecture는 관심사를 분리하여 레이어로 구분한 것을 의미한다.
presentation layer: UI에 관련된 내용
application layer: 서비스들을 관리한다. 도메인에 포함되지 않지만 기능 요구사항들을 관리
domain layer: 핵심 비지니스 로직이 존재
infrastructure layer(persistence layer): DB나 자료를 영구저장하는 것과 관련된 레이어
관심사를 가지고 구분하였기 때문에 관심사별로 응집되어 있다. 레이어간 교체가능할 수 있기 때문에 유지보수에 좋을 수 있다.
레이어 간에 소통을 하므로 소통하기 위해서는 각 모든 레이어를 거쳐야 한다. 적당한 규모로 나누는 것이 어렵다.
레이어간 소통하기 위해 dto를 사용하기도 한다.

링크

layered architecture에 대한 참고자료 링크

학습로그 1-4

[Architecture] 추상화 - 4

내용

과한 추상화는 코드의 가독성을 해친다.
지나친 추상화 프로젝트 깃허브 참고자료 링크
현재 단계에서 굳이 interface를 사용하여 의존관계를 분리해주는 것은 지나칠 수 있다.
현재 단계의 프로젝트 규모에서 dto 조립기 까지 만들 필요 없이 dto에서 도메인 정보를 받아 만들어주는 로직정도는 가져도 괜찮다.
dto의 비즈니스 로직이 들어가면 안 된다는 것은 도메인의 로직을 의미하는 것이다.
도메인은 dto 정보를 모르게, dto는 일부 도메인 정보를 알고 적절한 값으로 사용가능하게 만드는 방식을 사용할 수 있다.

DDL

DROP DATABASE if exists chess_db;
CREATE DATABASE chess_db DEFAULT character set utf8 collate utf8_general_ci;

USE chess_db;

CREATE TABLE room (
	room_id BIGINT NOT NULL,
	turn VARCHAR(16) NOT NULL,
	is_playing BOOLEAN NOT NULL,
	name VARCHAR(16) NOT NULL,
	password VARCHAR(16),
	create_at DATETIME DEFAULT CURRENT_TIMESTAMP,
	PRIMARY KEY(room_id)
);

CREATE TABLE board (
	board_id BIGINT NOT NULL AUTO_INCREMENT,
	team VARCHAR(16) NOT NULL,
	position VARCHAR(16) NOT NULL,
	piece VARCHAR(16) NOT NULL, 
	is_first_moved BOOLEAN NOT NULL,
	room_id BIGINT NOT NULL,
	PRIMARY KEY(board_id),
	FOREIGN KEY (room_id) REFERENCES room (room_id)
);

seok-2-o

제가 다소 많은 피드백는데 잘 반영해주셨네요 ;-;
이번 미션은 여기서 마무리하도록 할게요.
이어서 다음 미션 진행해주세요.

joelonsw and others added 10 commits April 14, 2021 18:35

feat: SparkJava 체스 옮겨오기

39c55f4

feat: 기능 구현 목록 및 요구 사항 정리

29baa9c

feat: 초기 화면 및 체스 보드 판 띄우기

4eed99a

feat: Spring MVC 적용

45a3ef2

feat: Spring JDBC Create, Read 구현

759ccb2

feat: Spring JDBC Update 구현

60b59b2

feat: Spring JDBC Delete 구현

b2bdcdc

feat: Spring JDBC 적용

9a9a6bc

refactor: RowMapper를 위한 dto 분리

d5fbe56

refactor: Spring으로 옮기면서 사용하지 않는 클래스 삭제

15f6ab2

seok-2-o suggested changes Apr 18, 2021

View reviewed changes

knae11 added 19 commits April 19, 2021 13:37

docs: 1차 피드백 이후 해야할일 정리

88cbe84

docs: todo 구체적인 내용추가

e4a4713

feat: 레거시 테스트 코드 옮겨옴

331d239

fix: 움직임 안되는 내용 수정

600c468

refactor: 클래스 및 메소드 이름 수정

8ff5c2a

docs: 도메인 코드 구조 파악

2cd7328

docs: todos 내용추가

c958287

docs: 임시 체크포인트 내용추가

2e2e77e

refactor: 중복 dto 제거

6157e6a

docs: 레거시 코드분석 view 부분 및 DDL 추가

72b6072

refactor: driver 버전 변경

c41481b

refactor: Boolean.valueOf 적용

0b2bb01

refactor: 이름변경

f0d9f69

docs: dto 관련 내용 추가

acf8c75

refactor: SpringChessApplication에서 @RestController 어노테이션 제거

574dbe9

refactor: 메인에 @RestController 제거 (불필요한 어노테이션 제거)

f8d0125

style: format codes

0935123

docs: http 관련 학습로그 추가

6191a2b

docs: check todos

cc2530f

refactor: Controller, Service어노테이션 수정

96638ef

knae11 added 4 commits April 21, 2021 20:16

refactor: 테이블 구조 변경

04d797d

refactor: 안쓰는 클래스 제거

5db67c2

style: format codes

9c1a5b4

docs: 학습로그 추가

a12763a

seok-2-o suggested changes Apr 22, 2021

View reviewed changes

knae11 added 7 commits April 25, 2021 11:11

refactor: dto 만드는 로직 dto 안으로 넣기

5404d4d

refactor: 테이블 구조 수정

0a6a57a

refactor: dto 패키지 정리

3e10139

refactor: dao에 dto로 넘기기

97d3ce3

refactor: 예외처리 컨트롤러 분리

f789aae

refactor: converter 분리

3ddf90a

style: move 메소드 라인 구분

8129de5

knae11 added 2 commits April 25, 2021 18:32

style: dao 라인 정리

f78515e

style: 안쓰는 import 정리

2efc0a4

seok-2-o suggested changes Apr 25, 2021

View reviewed changes

knae11 added 6 commits April 26, 2021 13:11

docs: todo 정리

6583ab7

docs: 학습로그 추가

c0cf380

docs: todos 정리

b89dbe1

refactor: dao 인터페이스 제거

ba9ed4c

refactor: dto 관련 클래스 좀 더 단순하게 정리

973a75f

refactor: 미션 프로그래밍 요구사항 위반 내용 수정

9446750

docs: 학습로그 내용 추가

a7348f9

seok-2-o approved these changes Apr 27, 2021

View reviewed changes

seok-2-o merged commit 088d0af into woowacourse:knae11 Apr 27, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[1, 2단계 - Spring 적용하기] 배럴(김나은) 미션 제출합니다. #245

[1, 2단계 - Spring 적용하기] 배럴(김나은) 미션 제출합니다. #245

knae11 commented Apr 16, 2021 •

edited

Loading

seok-2-o left a comment

seok-2-o Apr 18, 2021

knae11 Apr 19, 2021

knae11 Apr 19, 2021 •

edited

Loading

knae11 Apr 20, 2021

seok-2-o Apr 27, 2021

seok-2-o Apr 18, 2021

knae11 Apr 20, 2021

seok-2-o Apr 21, 2021

knae11 Apr 21, 2021

seok-2-o Apr 18, 2021

knae11 Apr 20, 2021 •

edited

Loading

knae11 Apr 20, 2021

seok-2-o Apr 21, 2021

knae11 Apr 21, 2021

knae11 Apr 21, 2021

knae11 commented Apr 21, 2021

knae11 commented Apr 22, 2021 •

edited

Loading

seok-2-o left a comment

knae11 commented Apr 25, 2021 •

edited

Loading

seok-2-o left a comment

knae11 commented Apr 26, 2021 •

edited

Loading

knae11 commented Apr 26, 2021

seok-2-o left a comment

[1, 2단계 - Spring 적용하기] 배럴(김나은) 미션 제출합니다. #245

[1, 2단계 - Spring 적용하기] 배럴(김나은) 미션 제출합니다. #245

Conversation

knae11 commented Apr 16, 2021 • edited Loading

DB 테이블

seok-2-o left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

[Spring] GetMapping, PostMapping - 4

내용

링크

knae11 Apr 19, 2021 • 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

knae11 Apr 20, 2021 • 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

knae11 commented Apr 21, 2021

DDL

knae11 commented Apr 22, 2021 • edited Loading

학습로그 1-2

[Spring] @Transactional - 3

내용

[DB] 이벤트소싱 패턴 - 4

내용

링크

seok-2-o left a comment

Choose a reason for hiding this comment

knae11 commented Apr 25, 2021 • edited Loading

DDL

seok-2-o left a comment

Choose a reason for hiding this comment

knae11 commented Apr 26, 2021 • edited Loading

학습로그 1-3

[DB] 데이터 모델링 - 5

내용

태그

[Architecture] Layered Architecture, Dto - 4

내용

링크

태그

학습로그 1-4

[Architecture] 추상화 - 4

내용

태그

knae11 commented Apr 26, 2021

DDL

seok-2-o left a comment

Choose a reason for hiding this comment

knae11 commented Apr 16, 2021 •

edited

Loading

knae11 Apr 19, 2021 •

edited

Loading

knae11 Apr 20, 2021 •

edited

Loading

knae11 commented Apr 22, 2021 •

edited

Loading

[Spring] `@Transactional` - 3

knae11 commented Apr 25, 2021 •

edited

Loading

knae11 commented Apr 26, 2021 •

edited

Loading