Skip to content

Conversation

@Gyuhyeok99
Copy link
Contributor

관련 이슈

작업 내용

기존에는 board, comment, post 패키지가 각각 분리되어 있었어서 이를 community 패키지로 통합하여 관리하도록 수정하였습니다.

리뷰 요구사항 (선택)

한 곳에서 관리하는 게 무조건 좋다고 생각했었는데 패키지가 너무 나눠진 거 같다는 생각도 들긴하네요. 컨트롤러를 예시로 community -> controller -> post -> postController로 가는데 괜찮은가요? 저는 유지보수나 작업을 생각했을 때 계층이 더 나눠지긴 해도 중간에 post가 들어가는 게 더 좋다고 생각했습니다.

@Gyuhyeok99 Gyuhyeok99 self-assigned this Feb 7, 2025
@Gyuhyeok99 Gyuhyeok99 linked an issue Feb 7, 2025 that may be closed by this pull request
2 tasks
Copy link
Collaborator

@nayonsoso nayonsoso left a comment

Choose a reason for hiding this comment

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

패지키 구조 리팩터링 좋네요😊

한 곳에서 관리하는 게 무조건 좋다고 생각했었는데 패키지가 너무 나눠진 거 같다는 생각도 들긴하네요

이 말씀에 제가 구현한 oauth 패키지의 구조도 다시 생각을 해보게 되네요😅

말씀하신 것처럼 community -> controller -> post -> postController은 다른 패키지들과 다른 구성이긴 하네요,
도메인별로 나눴다가, 레이어별로 나눴다가, 다시 도메인 별로 나눈 것 같기도 하고요.

차라리 community.post.controller, community.post.service …
처럼 도메인으로 먼저 나누고 그 다음 레이어별로 나누는 방식은 어떤가요?

@Gyuhyeok99
Copy link
Contributor Author

Gyuhyeok99 commented Feb 7, 2025

아하 그게 더 좋을 거 같네요! 수정하겠습니다!

반영했습니다!

Copy link
Member

@wibaek wibaek left a comment

Choose a reason for hiding this comment

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

좋습니다👍

Copy link
Collaborator

@nayonsoso nayonsoso left a comment

Choose a reason for hiding this comment

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

변경한 것 확인했습니다😊

@Gyuhyeok99 Gyuhyeok99 merged commit 2c8b200 into main Feb 8, 2025
@Gyuhyeok99 Gyuhyeok99 deleted the refactor/179-merge-community-packages branch February 11, 2025 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: 게시판 관련 패키지 구조 개선

4 participants