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

[4주차] 과제 제출 develop -> master #24

Merged
merged 1 commit into from
Oct 1, 2023
Merged

[4주차] 과제 제출 develop -> master #24

merged 1 commit into from
Oct 1, 2023

Conversation

kssumin
Copy link
Contributor

@kssumin kssumin commented Oct 1, 2023

어떤 내용에 대한 PR인가요?

4주차 과제 제출합니다!

4주차 작업 완료 내역

프로젝트 환경설정
프로젝트 공통 기능 생성
디렉터리 위치 변경
auditing 문제 해결
투표 생성
댓글 CRUD
회원가입
로그인

참고사항

  1. 인가 처리 부분이 완료되지 않아 우선 투표, 댓글 부분의 user정보 가져오는 부분은 하드코딩으로 해결했습니다.
  2. github actions로 build, lint를 검사하도록 하고자 했으나 BUG를 아직 수정 못 했습니다! (로컬 환경에서 돌아가는 것은 확인했습니다!)
    workflow 실패
  3. repository, entity 부분을 처음에는 외부에 의존(RDBMS)하기 때문에 infra라고 디렉터리 명을 결정하였으나 persistence라는 디렉터리 명이 더 어울릴 것 같다. 라는 판단에 디렉터리 명 변경을 고려하고 있습니다.
  4. 의미상 web에 dto가 존재해야 할 것 같지만 실제로 개발을 하다보니 패키지 간 양방향 의존성 때문에 web이 아닌 domain으로 변경하는 것을 고려하고 있습니다.

궁금한 점

패키지 구조

현재 저희 팀은 도메인을 기준으로 디렉터리를 나누고 그 안에서 controller, domain, infra(entity, repository)로 나뉘어 집니다.
하지만 도메인을 나누는 기준에 대해 고민이 있습니다.

  1. 첫 번째
  • vote를 생성할 때 option을 생성하므로 같은 패키지에 들어있어야 하는 게 아닌가?
    - member
    - vote
        - option
        - decision
        - hot(service단만 분리)
    - comment
  1. 두번째
  • vote를 생성할 때 option을 생성하는 건 맞지만, vote는 수정이 가능하지만 option은 수정이 가능하지 않음(11조 팀 프로젝트의 제약사항)
    즉 생명주기가 일치하지 않기 때문에 따로 분리하는 게 맞을 것 같다.
    - member
    - vote
    - option
        - decision
        - hot(service단만 분리)
    - comment
  1. 세번째
  • 생명주기로 본다면 오히려 decision을 optino과 같이 두는 것이 아니라 따로 빼야할 것 같음.
    - member
    - vote
        - option
    - decision
        - hot(service단만 분리)
    - comment

이런 식으로 우선 기준을 도메인의 생명 기준으로 잡았지만 이 또한 올바른 방식인지 모르겠습니다.
그래서 실제로 멘토님께서 디렉터리 구조를 짜실 때 어떠한 기준으로 도메인을 나누시는 지 궁금합니다!

Co-authored-by: kssumin <[email protected]>
Co-authored-by: 박현수 <[email protected]>
Co-authored-by: jinwooseok <[email protected]>
Copy link
Contributor Author

@kssumin kssumin left a comment

Choose a reason for hiding this comment

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

4주차 개발 중 궁금한 점을 남깁니다!!

return this.category;
}

public static Optional<Category> findCategory(String category) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

client가 요청한 category가 실제로 존재하는 Category인지 확인하기 위해
Category Enum class에 해당 메서드를 작성하였습니다.

하지만 예상과는 다르게 category를 찾지 못 하는 오류가 발생합니다.
혹시 코드 상의 문제가 있나요?
아니면 이와 같은 요구사항이 있을 때 해결할 수 있는 다른 방법이 있는지 궁금합니다!

@PHS00

Copy link
Contributor Author

Choose a reason for hiding this comment

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

해결중..

@kssumin kssumin changed the base branch from review to master October 1, 2023 14:49
@kssumin kssumin changed the title [4주차] 과제 제출 [4주차] 과제 제출 develop -> master Oct 1, 2023
@kssumin kssumin merged commit 2ff9f27 into master Oct 1, 2023
1 check failed
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.

1 participant