Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CT-4] 차트게임 기능 domain 레이어 작성 (+ktlint 설정 변경) #13

Merged
merged 38 commits into from
May 17, 2024
Merged

[CT-4] 차트게임 기능 domain 레이어 작성 (+ktlint 설정 변경) #13

merged 38 commits into from
May 17, 2024

Conversation

nosorae
Copy link
Collaborator

@nosorae nosorae commented May 11, 2024 •
edited
Loading

close #4

Overview

  • 핵심 기능인 차트게임 기능부터 구현.
  • domain 레이어 작성.

의도

  • domain 레이어의 클래스/객체/변수/함수 등의 이름은 기술스택과 관련없는 기획자/사용자 입장에서 지음
  • Subscribe~UseCase 로 구독해놓으면 다른 (매수/매도/다음턴불러오기 같은)UseCase를 호출해서 변경된 최신 데이터에 반응할 수 있는 방식

고민되는 점 (TBD)

  • 일단 API에서는 달러로 줄텐데 어느 시점(레이어)에서 다른 화폐단위(일단 원으)로 환산할지?
    • 결정이 안나서 우선 Money 라는 클래스로 만들었습니다.
  • data class에 주생성자에 넣을 프로퍼티와 중괄호 안에 넣을 프로퍼티는 어떻게 구분해줄까?
    • ChartGame과 Trade 엔티티를 작성하다가 든 고민입니다.
    • 지금 생각으로는, 기준점을 "주생성자로부터 이끌어낼 수 있는 프로퍼티는 중괄호 안으로 이동시키는 것이 옳다"로 잡았습니다.
      • 객체 생성자 호출 코드를 더 단순화 시켜서 가독성을 높일 수 있기 때문입니다.

*이제부터 스쿼시 머지 하겠습니다! -> 생각보다 스쿼시 머지 할 만큼 커밋 양이 많진 않네요

nosorae added 23 commits May 9, 2024 00:36
@nosorae
[CT-1] 프로젝트 생성 및 gitignore 적용
ffc27e3
@nosorae
[CT-1] 프로젝트 생성 후 compileSdk와 라이브러리 버전 호환 에러 해소
5e89d3f
@nosorae
[CT-1] domain 모듈 추가
cf8960a
@nosorae
[CT-1] data 모듈 추가
d15d947
@nosorae
[CT-1] presentation 모듈 추가
7c5f0a7
@nosorae
[CT-1] 모듈간 의존성 설정
8a05cab
@nosorae
[CT-1-2] ktlint 설정 및 코드 스타일 적용 시작
c08d130
@nosorae
[CT-1-2] PR시 실행할 Github Action YML파일 추가
0c09aa0
@nosorae
[CT-1-2] hilt 의존성 추가 및 초기 앱모듈 세팅
600376a
@nosorae
[CT-1-2] buildSrc 추가 및 적용, 불필요 의존성 제거
8a3177e
@nosorae
[CT-1-2] ktlintFormat
d923eed
@nosorae
[CT-1-2] JDK 버전 호환 문제로 인한 yml 파일 수정 (action/job name 이름 수정)
6b386cd
@nosorae
[CT-1-2] 배포자 지정
cfd9d50
@nosorae
[CT-1-2] 배포자 수정, 순서 수정
3cc6ad4
@nosorae
[CT-1-2] 일단 Action 제거, 추후 테스트 완료하고 다시 추가 예정
e7eb99a
@nosorae
[CT-4] Entity 추가
00ccbe5
@nosorae
[CT-4] 커스텀 예외 추가
6006196
@nosorae
[CT-4] domain 레이어에 kotlinx-coroutines 의존성 추가
12a3a5d
@nosorae
[CT-4] 트레이드 게임에 필요한 Repository 추상 메소드 정의
c127680
@nosorae
[CT-4] 게임데이터 조회, 매매하기, 다음 턴 불러오기, 차트 변경 UseCase 추가
bf8e0e2
@nosorae
[CT-4] 코드 스타일 변경, 유저 수익 필드명 변경
0b1c8d1
@nosorae
Merge branch 'develop' into feature/ct-4-chart-game-domain
ff0dc95 
# Conflicts:
#	app/.gitignore
#	buildSrc/src/main/kotlin/Dependency.kt
#	domain/build.gradle.kts
@nosorae
[CT-4] 주석 수정
fed07fa
@nosorae nosorae requested a review from f-lab-nathan May 11, 2024 14:43
@nosorae nosorae self-assigned this May 11, 2024
f-lab-nathan
f-lab-nathan reviewed May 12, 2024
View reviewed changes
Copy link
Collaborator

@f-lab-nathan f-lab-nathan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

지금 생각으로는, 기준점을 "주생성자로부터 이끌어낼 수 있는 프로퍼티는 중괄호 안으로 이동시키는 것이 옳다"로 잡았습니다.

저도 동일하게 생각해요. ㅎㅎ

buildSrc/src/main/kotlin/Dependency.kt Show resolved Hide resolved
domain/src/main/java/com/yessorae/domain/entity/Chart.kt Outdated Show resolved Hide resolved
domain/src/main/java/com/yessorae/domain/entity/ChartGame.kt Outdated Show resolved Hide resolved
domain/src/main/java/com/yessorae/domain/entity/ChartGame.kt Outdated Show resolved Hide resolved
domain/src/main/java/com/yessorae/domain/entity/ChartGame.kt Outdated Show resolved Hide resolved
domain/src/main/java/com/yessorae/domain/entity/tick/TickUnit.kt Outdated Show resolved Hide resolved
domain/src/main/java/com/yessorae/domain/exception/ChartGameException.kt Show resolved Hide resolved
domain/src/main/java/com/yessorae/domain/usecase/QuitChartGameUseCase.kt Outdated Show resolved Hide resolved
@nosorae
Copy link
Collaborator Author

nosorae commented May 13, 2024

Tick의 TickUnit 프로퍼티를 Chart로 이동 은 리뷰 외의 수정입니다.

@nosorae nosorae requested a review from f-lab-nathan May 13, 2024 14:00
@nosorae nosorae changed the title [CT-4] 차트게임 기능 domain 레이어 작성 [CT-4] 차트게임 기능 domain 레이어 작성, ktlint 설정 변경 May 13, 2024
@nosorae nosorae changed the title [CT-4] 차트게임 기능 domain 레이어 작성, ktlint 설정 변경 [CT-4] 차트게임 기능 domain 레이어 작성 (+ktlint 설정 변경) May 13, 2024
This was referenced May 13, 2024
Closed
Closed
@nosorae nosorae requested a review from f-lab-nathan May 15, 2024 06:01
@f-lab-nathan
Copy link
Collaborator

코멘트와 변경사항이 많아서 머지하겠습니다. Reuslt 변경건은 다음 PR에 적용해주세요~!

@f-lab-nathan f-lab-nathan merged commit dbb45e8 into develop May 17, 2024
1 check passed
@nosorae nosorae mentioned this pull request May 18, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@f-lab-nathan f-lab-nathan Awaiting requested review from f-lab-nathan

Assignees

@nosorae nosorae

Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[CT-4] 차트게임 기능 - domain - 5h
2 participants
@nosorae @f-lab-nathan