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.

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

[#060] 종합 점수 등록 및 업데이트 #64

Merged
merged 16 commits into from
Apr 15, 2024

Conversation

koomin1227
Copy link
Member

이슈

체크리스트

  • 회원가입 시 user table 생성과 함께 total table도 생성
  • stat 에 대해 CUD 작업이 들어오면 total 점수도 함께 업데이트
  • stat 관련 모듈들을 stat 모듈에 합침

고민한 내용

순환종속성

  • TotalService 에서 githubService, gradeService, AlgorithmService 를 사용하고 각 모듈에 import를 하니까 순환 종속 문제가 발생한듯하다.
  • 그래서 흩어져 있던 stat 관련 모듈들을 하나의 모듈로 합쳐서 만들었다.

Toal 업데이트 시기

  • 다른 stat 들이 업데이트 되면 Total도 함께 업데이트 되어야하기 때문에 모든 Stat이 업데이트 되는 메서드 호출이후에 호출 하도록 하였다.

@koomin1227 koomin1227 requested a review from namewhat99 April 15, 2024 06:36
@koomin1227 koomin1227 self-assigned this Apr 15, 2024
@namewhat99 namewhat99 merged commit 38a5f94 into develop Apr 15, 2024
1 check passed
@koomin1227 koomin1227 deleted the feature/RegistTotal-#60 branch April 17, 2024 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants