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

엔티티/서비스 변경 로직 논의 #49

Open
yaeoni opened this issue Jul 21, 2024 · 2 comments
Open

엔티티/서비스 변경 로직 논의 #49

yaeoni opened this issue Jul 21, 2024 · 2 comments
Assignees

Comments

@yaeoni
Copy link
Collaborator

yaeoni commented Jul 21, 2024

#44 (comment)

^ 위 코멘트에 달린 것 같이 논의해봐요~

@yaeoni yaeoni self-assigned this Jul 21, 2024
@yaeoni
Copy link
Collaborator Author

yaeoni commented Jul 21, 2024

코멘트에도 달아두었지만 저는 명시적 세이브 호출이 코드 이해하기 더 쉽다고 생각해요.
그 전에 준형님이 제시해주신 방향을 제 코드에 접목시켯을때 어떤 모습일지 알려주시면 컴케하기 더 쉬울 것 같습니당!

기존 제 코드

val id =
            memberRepository.save(
                member.update(
                    profileImageId = command.profileImageId
                ).toEntity()
            ).id

@xonmin
Copy link
Member

xonmin commented Jul 22, 2024

@toychip

저도 QueryDSL 안쓴지 꽤나 지나서 기억이 가물가물한데 예시코드나 작업내용 예시 등 보여주시면 좋을 것 같습니다! ( _ _ )

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

No branches or pull requests

2 participants