feat: problem 도메인 1차 구현 #49

hun-ca · 2024-06-17T16:04:51Z

🎫 연관 이슈

resolved #48

💁‍♂️ PR 내용

🙏 작업

아래 2개 API 신규 구현

🙈 PR 참고 사항

📸 스크린샷

🤖 테스트 체크리스트

체크 미완료
체크 완료

…#48_hunca

api/src/main/kotlin/com/few/api/web/usecase/problem/out/ReadProblemUseCaseOut.kt

api/src/main/kotlin/com/few/api/web/usecase/problem/CheckProblemUseCase.kt

belljun3395 · 2024-06-20T14:03:33Z

api/src/main/kotlin/com/few/api/web/usecase/problem/CheckProblemUseCase.kt

+        val submitAns = useCaseIn.sub
+
+        val record = problemDao.selectProblemAnswer(SelectProblemAnswerQuery(problemId))
+        val isSolved = submitAns.equals(record.answer)


record.answer.equals(submitAns)

위와 같이 순서를 변경하거나

submitAns는 모델을 만들어 처리하는게 어떨까요?

아,, 이런거 생각하면 JPA가 좋긴 하구나...

api/src/main/kotlin/com/few/api/web/usecase/problem/CheckProblemUseCase.kt

belljun3395 · 2024-06-20T14:04:50Z

api/src/main/kotlin/com/few/api/web/usecase/problem/ReadProblemUseCase.kt

+
+        val record = problemDao.selectProblemContents(SelectProblemQuery(problemId))
+
+        val contents: List<ReadProblemContentsUseCaseOut> = objectMapper.readValue(record.contents)


요거는 제가 contents 관련 mapper와 객체를 만들었는데 같이 사용하는 것은 어떨까요?

json string <-> kotlin class 매퍼인가요?

5ac7303 요런거요

이거 근데 uc 단에서도 한번 바꿔주고 컨트롤러단에서도 response로 또 한번 바꿔줘야겠네요...

api-repo/src/main/kotlin/com/few/api/repo/dao/problem/ProblemDao.kt

belljun3395 · 2024-06-20T14:08:43Z

api-repo/src/main/kotlin/com/few/api/repo/dao/problem/ProblemDao.kt

+            .where(Problem.PROBLEM.ID.eq(query.problemId))
+            .fetchOne()
+
+        result ?: throw RuntimeException("Problem with ID ${query.problemId} not found") // TODO: 에러 표준화


저는 잘못된 인자로 조회했어! 라는 의미로 IllegalArgumentException를 주로 사용해 오기는 했는데 어떤 예외를 상속에 사용하는게 좋을까요?

저도 주석으로 에러 표준화라고 적어두긴 했는데 자바꺼 말고 저희단에서 예외 클래스 만들어서 쓰는건 어떨까요

api-repo/src/main/resources/db/migration/entity/V1.00.0.0__draft_table_design.sql

api/src/main/kotlin/com/few/api/web/controller/problem/ProblemController.kt

api-repo/src/main/kotlin/com/few/api/repo/dao/problem/ProblemDao.kt

…UseCaseOutDetail

hun-ca · 2024-06-22T10:44:14Z

@belljun3395 종준님 컨트롤러 테스트 시 포트를 띄워야 하기 때문에 WAS가 올라가야 하는데 이때 데이터소스 같은 불필요한 클래스는 컨테이너에 로딩되지 않도록 설정필요합니다. 코파일럿의 힘을 빌려 함 해주세요

belljun3395

9c23cae
Problem 도메인에서 ProblemId로 조회하는 경우에는 ByProblemId를 안붙이고
추후 Title이나 다른 것으로 조회하는 경우에만 ByTitle 이런 식으로 붙이는게 어떨까요?

belljun3395 · 2024-06-22T12:15:44Z

api/src/test/kotlin/com/few/api/web/controller/problem/ProblemControllerTest.kt

+                ReadProblemContentsUseCaseOutDetail(4L, "투명성")
+            )
+        )
+        Mockito.`when`(readProblemUseCase.execute(ArgumentMatchers.any()))


저는 요렇게 any 처리가 NPE 뜨면서 안되던데 혹시 테스트 성공하나요?

val problemId = 1L Mockito.`when`(readProblemUseCase.execute(ReadProblemUseCaseIn(problemId))) .thenReturn(useCaseOut)

요렇게 테스트 코드 수정해주시면 될 것 같아요

belljun3395

테스트 코드만 동작하도록 수정하고 머지하면 될 것 같아요!
수고하셨습니다~!

belljun3395 · 2024-06-23T00:33:52Z

api/src/test/kotlin/com/few/api/web/controller/problem/ProblemControllerTest.kt

+                ReadProblemContentsUseCaseOutDetail(4L, "투명성")
+            )
+        )
+        Mockito.`when`(readProblemUseCase.execute(ArgumentMatchers.any()))


val problemId = 1L Mockito.`when`(readProblemUseCase.execute(ReadProblemUseCaseIn(problemId))) .thenReturn(useCaseOut)

요렇게 테스트 코드 수정해주시면 될 것 같아요

api/src/test/kotlin/com/few/api/web/controller/problem/ProblemControllerTest.kt

hun-ca · 2024-06-23T03:08:31Z

9c23cae Problem 도메인에서 ProblemId로 조회하는 경우에는 ByProblemId를 안붙이고 추후 Title이나 다른 것으로 조회하는 경우에만 ByTitle 이런 식으로 붙이는게 어떨까요?

@belljun3395 네넵 좋습니다 ID로 조회하는 것만 네이밍에서 제외할게요

feat: problem 도메인 임시 구현 (미완, 임시 커밋)

860b067

hun-ca added the feature 새로운 기능을 만들 때 사용됩니다 label Jun 17, 2024

hun-ca self-assigned this Jun 17, 2024

hun-ca requested a review from belljun3395 as a code owner June 17, 2024 16:04

belljun3395 and others added 7 commits June 18, 2024 21:15

refactor: data-jdbc 의존성을 api 방식으로 변경

9f36423

feat: api 모듈에 api-repo 모듈 의존성 추가

1097703

feat: ApiConfig 구현

7c8a6db

feat: api 모듈에 api-repo 모듈 빈 임포트

f523d59

feat: application.yml 추가

a10e990

Merge remote-tracking branch 'origin/feat/#50_belljun3395' into feat/…

67bb5a4

…#48_hunca

fix: 문제 컨텐츠(json string) to Object 변환 과정 추가

33c69bc

hun-ca changed the title ~~feat: problem 도메인 임시 구현 (미완, 임시 커밋)~~ feat: problem 도메인 임시 구현 Jun 19, 2024

github-actions bot added the config 설정 파일과 관련된 내용을 다룰 때 사용됩니다 label Jun 19, 2024

feat: 문제 정답 확인 API 1차 구현

77a9ab9

hun-ca changed the title ~~feat: problem 도메인 임시 구현~~ feat: problem 도메인 1차 구현 Jun 19, 2024

hun-ca added 2 commits June 19, 2024 23:19

refactor: problem 도메인 리펙토링

699991e

refactor: problem 도메인 api-repo 모듈 리펙토링

a673c63

belljun3395 requested changes Jun 20, 2024

View reviewed changes

belljun3395 reviewed Jun 20, 2024

View reviewed changes

api-repo/src/main/kotlin/com/few/api/repo/dao/problem/ProblemDao.kt Show resolved Hide resolved

hun-ca added 7 commits June 22, 2024 16:30

refactor: rename ReadProblemContentsUseCaseOut -> ReadProblemContents…

ef9895d

…UseCaseOutDetail

fix: remove duplicated application resource file

3e0be56

fix: remove DB schema migration file

26717cb

chore: DB 스키마 마이그레이션 파일 gitignore 등록

0e3bd80

refactor: jooq select 결과를 바로 class로 매핑하도록 수정

77654e4

refactor: DAO 메소드 이름 수정 (~~By~~)

9c23cae

test: usecase mocking 적용

f4d1da7

chore: rename test method name

e0f0f30

belljun3395 reviewed Jun 22, 2024

View reviewed changes

belljun3395 approved these changes Jun 23, 2024

View reviewed changes

fix: SQL에 deleted_at is null 조건 추가

9c7ae3e

hun-ca added 3 commits June 23, 2024 12:09

test: 모킹 인풋 객체 추가

89655f3

refactor: DAO 메소드 명 수정

72795d8

chore: merge from main (resolve conflict)

20141ce

hun-ca merged commit d09e7f3 into main Jun 23, 2024
3 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: problem 도메인 1차 구현 #49

feat: problem 도메인 1차 구현 #49

hun-ca commented Jun 17, 2024 •

edited

Loading

belljun3395 Jun 20, 2024

hun-ca Jun 22, 2024

belljun3395 Jun 20, 2024

hun-ca Jun 22, 2024

belljun3395 Jun 22, 2024

hun-ca Jun 22, 2024

belljun3395 Jun 20, 2024

hun-ca Jun 22, 2024

hun-ca commented Jun 22, 2024

belljun3395 left a comment •

edited

Loading

belljun3395 Jun 22, 2024

belljun3395 Jun 23, 2024

belljun3395 left a comment

belljun3395 Jun 23, 2024

hun-ca commented Jun 23, 2024


		val record = problemDao.selectProblemContents(SelectProblemQuery(problemId))

		val contents: List<ReadProblemContentsUseCaseOut> = objectMapper.readValue(record.contents)

feat: problem 도메인 1차 구현 #49

feat: problem 도메인 1차 구현 #49

Conversation

hun-ca commented Jun 17, 2024 • edited Loading

🎫 연관 이슈

💁‍♂️ PR 내용

🙏 작업

🙈 PR 참고 사항

📸 스크린샷

🤖 테스트 체크리스트

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

hun-ca commented Jun 22, 2024

belljun3395 left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

belljun3395 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hun-ca commented Jun 23, 2024

hun-ca commented Jun 17, 2024 •

edited

Loading

belljun3395 left a comment •

edited

Loading