Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gutenLEE
requested review from
Dongkyuuuu,
hyesungoh,
oyeon-kwon,
sensecodevalue,
Seojunhwan,
sangbooom and
noy3928
as code owners
hyesungoh
approved these changes
Apr 23, 2024
Comment on lines
+81
to
+83
| > update 함수를 void로 선언하면 테스트하기 어려웠다. void에서 리턴 타입이 있게 변경하면 테스트하기 쉬워진다.<br> | ||
| > 근데 리턴 타입을 도메인 자체로 하면 객체가 오래 떠돌아 다닐 위험성이 있어 사이드 이펙트가 발생할듯?<br> | ||
| > 업데이트한 속성의 결과 값을 원시형으로 리턴하거나 래퍼 또는 vo를 생성해서 리턴하는게 나을까?<br> |
There was a problem hiding this comment.
Value Object라고 데이터 나타낼때 많이 들었던 단어였던 것 같아요! 관련 블로그 글 있어서 첨부드려용
https://tecoble.techcourse.co.kr/post/2020-06-11-value-object/
Comment on lines
+86
to
+89
| > 생각해보기 | ||
| > 동작을 유지하면서 코드를 바꾸는 것은 어떤 장점이 있을까?<br> | ||
| > 동작을 유지하면서 코드를 바꾸면 코드 변경의 사이드 이펙트 없이 수정하는 것이라 위험 부담이 줄어든다.<br> | ||
| > 리팩토링한 코드가 기존 동작대로 동작하는게 검증되면 구조를 바꾸거나 패턴으로 추상화 하면서 점진적 개선이 가능하지 않을까나 |
Dongkyuuuu
approved these changes
Apr 23, 2024
Comment on lines
+81
to
+83
| > update 함수를 void로 선언하면 테스트하기 어려웠다. void에서 리턴 타입이 있게 변경하면 테스트하기 쉬워진다.<br> | ||
| > 근데 리턴 타입을 도메인 자체로 하면 객체가 오래 떠돌아 다닐 위험성이 있어 사이드 이펙트가 발생할듯?<br> | ||
| > 업데이트한 속성의 결과 값을 원시형으로 리턴하거나 래퍼 또는 vo를 생성해서 리턴하는게 나을까?<br> |
There was a problem hiding this comment.
Value Object라고 데이터 나타낼때 많이 들었던 단어였던 것 같아요! 관련 블로그 글 있어서 첨부드려용
https://tecoble.techcourse.co.kr/post/2020-06-11-value-object/
| - 전역변수가 없어야 하며 의존하지 않아야 한다 | ||
| - 함수가 결괏값을 리턴해야 한다 | ||
|
|
||
| > update 함수를 void로 선언하면 테스트하기 어려웠다. void에서 리턴 타입이 있게 변경하면 테스트하기 쉬워진다.<br> |
There was a problem hiding this comment.
update 함수를 void로 했을 때 어떤부분이 어려우셨는지 경험이 궁금하네요!!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.