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

Feature 6/farmclub 기본 api #15

Merged
merged 14 commits into from
May 9, 2024
Merged

Conversation

MinchoGreenT
Copy link
Contributor

@MinchoGreenT MinchoGreenT commented May 6, 2024

#️⃣연관된 이슈

#6

📝작업 내용

�미션포스트 관련 api

💬리뷰 요구사항(선택)

까먹고 커밋을 한방에 해버려서 커밋이름이 좀 구립니다..

@Ryeolee
Copy link
Contributor

Ryeolee commented May 7, 2024

코드 확인 했습니다! 머지하셔도 좋습니다.
한 가지 궁금한 점이 있으면 vo의 역할이 궁금합니다!

@MinchoGreenT
Copy link
Contributor Author

코드 확인 했습니다! 머지하셔도 좋습니다. 한 가지 궁금한 점이 있으면 vo의 역할이 궁금합니다!

답변 늦게 드려 죄송합니다,,
vo의 경우, dto와 같이 데이터를 담는 객체다 라는 점이 비슷하고 실제로 명확히 구분하는 건 저도 아직 잘 못하지만,
개인적으로 dto는 데이터를 송/수신 하는 역할이라는 점에 중심을 두어 dto는 controller가 주고받는,
그리고 vo는 서비스단에서 데이터를 다룰 때, 대표적인 예로(사실 여기서밖에 안쓰지만) 레포지토리에서 데이터를 불러 올 때 비즈니스 로직 상 필요한 데이터만 담기 위해 생성 및 사용하는 편입니다.
정교한 앱 구조를 구성하고자 한다면 entity와 vo, dto 사이의 매핑을 담당하는 mapper와 이러한 관계를 단순화 할 수 있는 정교한 클래스 설계가 필요하지만, 지금은 우선 기능 구현에 초점을 맞추고 추후 있을 리팩토링 과정에서 이를 자세히 다룰 예정입니다!

@MinchoGreenT MinchoGreenT merged commit 61ba1d1 into dev May 9, 2024
1 check passed
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