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

세미나/ 공지사항 게시물 생성 , 수정 ,삭제 , 조회 , 페이징 조회 도메인 로직 리팩터링 #127

Closed
wants to merge 13 commits into from

Conversation

kang20
Copy link
Member

@kang20 kang20 commented Feb 26, 2025

🌱 작업 사항

1️⃣ boardService 전체 로직 리팩터링

boardService 로직을 전체 리팩터링 했습니다.
리팩터링 내용은 도메인 계층에 다른 계층의 의존성을 분리하는 것 입니다.
아직 외부 리팩터링이 존재한다면 다른 도메인이거나 다른 리팩터링이 안된 계층에서 코드를 사용하여 리팩터링을 안했습니다

2️⃣ boardService 의 도구 계층 boardWriter boardReader boardValidator viewCounter boardCategoryWriter 리팩터링

service 메서드 안에 코드만 보고도 해당 비즈니스 로직이 어떻게 되는지 한번에 이해하기 쉽도록 네이밍과 예외 핸들링에 신경썻습니다
도구 계층의 각 컴포넌트에서 도메인 계층 외에 의존을 분리했습니다.

3️⃣ 생성 , 수정 ,삭제 , 조회 통합 테스트 작성

리팩터링 시 기존 코드의 로직이 변질되어 작동의 유무를 판단하기 위해 test 코드를 작성했습니다

4️⃣ 테스트 패키지에 builderSupport와 TestBuilder 의 패키지 위치를 변경했습니다.

패키지 변경 위치 숙지해주세요

❓ 리뷰 포인트

  • 도메인 코드와 외부 로직을 분리하도록 노력했습니다. 그럼에도 분리안되었거나 이상한 로직 리뷰 부탁드립니다
  • 패키지 이동때문에 파일 변경이 많은데 위에 언급한 컴포넌트만 봐도 됩니다. 컴포넌트 보실때 팁은 import 문을 봐서 domain 계층 외에 의존이 생기는지 확인해주세요

🦄 관련 이슈

resolves #이슈번호

@kang20 kang20 closed this Feb 26, 2025
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