Step2 진짜 누누 사칭 아님 #10

be-student · 2023-04-19T08:47:22Z

No description provided.

* initial commit * docs: 기능 요구사항 작성 * feat: DB 테이블 스키마 구현 * chore: H2 의존성 추가 * docs: DAO 관련 기능 요구사항 작성 * test: ascii 경고 억제 * feat: dao 구현 * fix: db 관련 설정 변경 * test: dao 테스트 작성 * feat: repository 구현 * test: repository 테스트 작성 * feat: 서비스 구현 * test: 서비스 테스트 작성 * feat: 컨트롤러 작성 * test: controller 테스트 작성 * docs: 문서 최신화 * docs: DB 테이블, API 문서화 * chore: validation 의존성 추가 * feat: 검증 로직 및 예외 처리 로직 추가 * refactor: race 메서드 반환 타입 변경 * feat: 이름 입력 시 공백 관련 검증 로직 추가 * refactor: selectDao와 insertDao를 빈에서 제거 * feat: Cars에서 도메인을 반환하는 기능 구현 * style: 코드 포맷팅 변경 * feat: Entity 구현 * chore: final 추가 * refactor: Repository와 DAO 메소드 반환 타입이 일관적이도록 변경 * refactor: Repository 반환 타입 변경으로 인한 코드 변경 * test: 불필요한 테스트 삭제 * fix: test 에 id 가 1로 하드코딩 되어있던 부분 수정 * fix: dto 패키지 수정 * refactor: 패키지 분리 * chore: 메서드 saveAll 에서 insertAll 로 변경 * refactor: repository 에서 gameId 만을 반환하도록 변경 * feat: exception 처리 방식 변경 --------- Co-authored-by: apptie <[email protected]>

domain module 을 진짜 pojo로 변경

woo-chang

수고하셨습니다 누누 🫡

많이 배워갑니다 🔥

woo-chang · 2023-04-23T12:09:52Z

build.gradle

+    apply plugin: 'org.springframework.boot'
+    apply plugin: 'io.spring.dependency-management'


버전 명시안하면 현재 루트 버전과 달라질수도 있지 않나요? (진짜 궁금)

woo-chang · 2023-04-23T12:11:08Z

build.gradle

+    }
+
+    dependencies {
+        implementation 'org.springframework.boot:spring-boot-starter-web'


이렇게하면 콘솔에도 웹 의존성이 들어가지 않나요? (진짜 궁금)

woo-chang · 2023-04-23T12:16:26Z

domain-h2/src/main/java/racingcar/Main.java

+public class Main {
+
+    public static void main(final String[] args) {
+        //스프링이 시작하는 위치를 알려주기 위한 클래스


무슨 의미인지 설명해주세요

woo-chang · 2023-04-23T12:21:47Z

domain-h2/src/main/java/racingcar/dao/entity/CarEntity.java

+
+public class CarEntity {
+
+    private final CarId carId;


해당 엔티티의 식별을 위해서는 eqauls, hashcode를 재정의해줘야하지 않을까용?

CarId에는 정의해두었는데, 여기는 없어서!

woo-chang · 2023-04-23T12:22:19Z

domain-h2/src/main/java/racingcar/dao/entity/GameEntity.java

+
+public class GameEntity {
+
+    private final GameId gameId;


요기도 equals, hashcode 있으면 더 좋겠네요 :)

woo-chang · 2023-04-23T12:22:33Z

domain-h2/src/main/java/racingcar/dao/entity/WinnerEntity.java

+
+public class WinnerEntity {
+
+    private final WinnerId winnerId;


woo-chang · 2023-04-23T12:25:46Z

domain/src/main/java/racingcar/service/RaceFindService.java

+        this.racingGameRepository = racingGameRepository;
+    }
+
+    @Transactional(readOnly = true)


Command와 Query 분리했으면 클래스 레벨에 해당 어노테이션 붙이는건 어떨까요?

woo-chang · 2023-04-23T12:27:48Z

web/src/main/java/racingcar/dto/PlayGameRequest.java

+public class PlayGameRequest {
+
+    @NotEmpty(message = "배열을 입력해주세요")
+    private final List<@NotBlank(message = "공백은 입력할 수 없습니다. 입력 값 : ${validatedValue}") String> names;


이거 좋은데요 👍

아 이렇게 쓰는거구나 나한테 왜 안알려줌 개고생하다 포기했네

woo-chang · 2023-04-23T12:28:31Z

web/src/test/java/racingcar/controller/RacingCarControllerTest.java

+    }
+
+    @DisplayName("게임을 진행하고, 결과를 조회한다")
+    @TestFactory


그때 에러의 원인이 무엇이였나요? 🤔

drunkenhw

서울대를 가기 싫어서 서강대에 간 눈우 코드 잘 봣습니다.
역시 누누네요 시비를 걸고 싶은데 몰라서 못거네요
다음부터 이런식으로 하지마세요

drunkenhw · 2023-04-23T10:22:39Z

console/src/main/java/racingcar/ConsoleRunner.java

+    }
+
+    @Bean
+    public CommandLineRunner commandLineRunner() {


요 클래스는 태어나서 처음보네요 뭐죠?
제가 찾아볼 수는 있지만 눈우의 메타인지를 위해 설명해주세요

drunkenhw · 2023-04-23T10:24:44Z

domain/src/main/java/racingcar/service/RaceAddService.java

@@ -0,0 +1,28 @@
+package racingcar.service;


제가 잘 몰라서 그런데요 멀티모듈은 어렵네요 근데 domain이라는 모듈에 서비스와 레포지토리가 있는 이유가 뭐에요? 진짜 모르니깐 진짜 설명해주세요

drunkenhw · 2023-04-24T05:23:30Z

domain-h2/src/main/java/racingcar/dao/entity/WinnerEntity.java

+    private final GameId gameId;
+    private final CarId carId;


id만 포장해주신 이유가 뭐에요?>

drunkenhw · 2023-04-24T05:24:16Z

domain-h2/src/main/java/racingcar/dao/WinnerDao.java

+public class WinnerDao {
+
+    private final InsertWinnerDao insertWinnerDao;
+    private final SelectWinnerDao selectWinnerDao;
+
+    public WinnerDao(final JdbcTemplate jdbcTemplate) {


이런 식으로다가 다 갖다 나누니깐 어때요 좋아요?> 싫어요?

drunkenhw · 2023-04-24T05:25:12Z

domain-h2/src/test/java/racingcar/dao/CarDaoTest.java

+                () -> Assertions.assertThat(result).hasSize(1),
+                () -> assertThat(result.get(0).getName()).isEqualTo("토미"),
+                () -> assertThat(result.get(0).getPosition()).isSameAs(9),
+                () -> assertThat(result.get(0).getCarId().getValue()).isNotNull()


허걱 얘만 스태틱임포트 안함

drunkenhw · 2023-04-24T05:25:54Z

domain-h2/src/test/java/racingcar/dao/RepositoryFactory.java

+
+import org.springframework.jdbc.core.JdbcTemplate;
+
+public final class RepositoryFactory {


팩토리를 만든 이유는>?

drunkenhw · 2023-04-24T05:26:33Z

domain-h2/src/test/java/racingcar/repositoryImpl/RacingGameMapperTest.java

+                () -> assertThat(result.get(1).getCars().get(1).getCarName()).isEqualTo("asdf"),
+                () -> assertThat(result.get(1).getCars().get(1).getPosition()).isEqualTo(4),
+                () -> assertThat(result.get(1).getCars().get(1).getCarId().getValue()).isEqualTo(4)
+        );


아우 징그러워요

drunkenhw · 2023-04-24T05:28:11Z

web/src/main/java/racingcar/dto/PlayGameRequest.java

+public class PlayGameRequest {
+
+    @NotEmpty(message = "배열을 입력해주세요")
+    private final List<@NotBlank(message = "공백은 입력할 수 없습니다. 입력 값 : ${validatedValue}") String> names;


아 이렇게 쓰는거구나 나한테 왜 안알려줌 개고생하다 포기했네

be-student and others added 29 commits April 18, 2023 03:33

feat: domain 에 id 필드 추가

51c1a5a

feat: entity 를 테이블과 1:1 매핑이 되도록 변경

933e02e

refactor: dao 는 entity 를 통해서 소통하고, repository 는 domain 을 통해서 처리하도록 변경

85e7fac

refactor: domain 생성 방식 id 도 고려하도록 변경

1e23155

chore: 컨트롤러 responseEntity 반환시 체이닝으로 변경

4bf370a

fix: test 단계에서 rollback 되도록 transactional 개념 추가

8addd8f

fix: scannerClose 부분 제거

9a02622

feat: findAll repository 에서 기능 추가

67f7d15

feat: findAll api 추가

b4723d0

feat: console 과의 중복 로직 제거

7c81776

refactor: service 에서 도메인을 반환하도록 변경

9ef86e6

refactor: service 를 useCase 별로 분리

b36198f

docs: 기능 목록 작성

1d146ad

test: findAllService 테스트 추가

5c67368

chore: 사용하지 않는 주석 제거

3d7aa7f

feat: consoleApplication을 spring으로 실행시키도록 변경

4c5e606

feat: 모듈 생성

cb7a0b3

feat: 모듈에 맞게 코드 위치 변경

7a8d57a

fix: 깨진 테스트 수정

f197614

fix: root 경로에서 build 가능하도록 변경

f7688f3

chore: 안 쓰이는 코드 제거

3d4123c

chore: 메서드 네이밍 변경

c8aec85

feat: 출력 형식 변경

fa79391

chore: 사용하지 않는 주석 제거

cf4b01c

chore: 주석 제거

9476e21

feat: console 에서 service 를 호출하도록 변경

15e2189

test: outputView 테스트 작성

79fa2f1

test: mapperTest 추가

e25246b

be-student force-pushed the step2 branch from 363d8d8 to e25246b Compare April 20, 2023 12:43

feat: controller slice test 작성

f1cc1cc

be-student force-pushed the step2 branch from 6ce8092 to f1cc1cc Compare April 21, 2023 05:35

be-student added 6 commits April 21, 2023 14:40

test: controller 잘못된 요청에 400 검증 추가

e7a1951

chore: 이름 변경

40a329e

feat: domain 의 game 을 gameId 를 통해서 관리

c37e302

fix: 깨진 테스트 수정

266f0ce

chore: 테스트 코드 정리

6533917

feat: post 생성 후 201 반환하도록 변경

54b1c9b

be-student force-pushed the step2 branch from c03a82a to 54b1c9b Compare April 21, 2023 06:21

be-student added 3 commits April 21, 2023 15:34

test: 패키지에 맞게 test 파일 재배치 및 dao 테스트 작

aa24bba

test: 도메인 테스트 작성

2a681ad

chore: 네이밍 변경

b4a8d4f

be-student changed the title ~~Step2~~ Step2 진짜 누누 사칭 아님 Apr 23, 2023

be-student added 11 commits April 23, 2023 19:00

feat: application module 생성

8c20bc6

domain module 을 진짜 pojo로 변경

feat: racingGame 의 findWinner 에서 Winners를 반환하도록 변경

0a055fd

chore: 변수명 변경

86fa264

feat: winners 객체를 바탕으로 저장하는 기능 추가

24c889a

chore: 메서드 네이밍 변경

7c5a732

feat: gameEntity 에 createdAt 추가

d263e95

fix: application.yml 위치 변경

77c6e69

chore: 안 쓰이는 data.sql, application.yml 제거

985ca7a

chore: 쓸모 없는 메인 파일 제거

4b1a66e

chore: 오타 수정

d20a060

chore: 안 쓰이는 변수 제거

bd0f76d

woo-chang approved these changes Apr 23, 2023

View reviewed changes

be-student added 3 commits April 23, 2023 21:55

fix: log 에서 빼먹었던 부분 제거

e6ad7c7

fix: 깨지는 테스트 수정

cf774d6

feat: h2 일때만 truncate 가 작동하도록 변경

2511690

drunkenhw reviewed Apr 24, 2023

View reviewed changes

be-student force-pushed the step2 branch from 7eb192a to 2511690 Compare April 24, 2023 09:21

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Step2 진짜 누누 사칭 아님 #10

Step2 진짜 누누 사칭 아님 #10

be-student commented Apr 19, 2023

woo-chang left a comment

woo-chang Apr 23, 2023

woo-chang Apr 23, 2023

woo-chang Apr 23, 2023

woo-chang Apr 23, 2023

woo-chang Apr 23, 2023

woo-chang Apr 23, 2023

woo-chang Apr 23, 2023

woo-chang Apr 23, 2023

drunkenhw Apr 24, 2023

woo-chang Apr 23, 2023

drunkenhw left a comment

drunkenhw Apr 23, 2023

drunkenhw Apr 23, 2023

drunkenhw Apr 24, 2023

drunkenhw Apr 24, 2023

drunkenhw Apr 24, 2023

drunkenhw Apr 24, 2023

drunkenhw Apr 24, 2023

drunkenhw Apr 24, 2023

		apply plugin: 'org.springframework.boot'
		apply plugin: 'io.spring.dependency-management'


		import org.springframework.jdbc.core.JdbcTemplate;

		public final class RepositoryFactory {

Step2 진짜 누누 사칭 아님 #10

Are you sure you want to change the base?

Step2 진짜 누누 사칭 아님 #10

Conversation

be-student commented Apr 19, 2023

woo-chang 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

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