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
Member
hyesungoh
commented
Apr 20, 2024
hyesungoh
commented
hyesungoh
requested review from
Dongkyuuuu,
oyeon-kwon,
sensecodevalue,
Seojunhwan,
sangbooom,
noy3928 and
gutenLEE
as code owners
gutenLEE
approved these changes
Apr 21, 2024
Comment on lines
+68
to
+71
|
|
||
| > 이런 유틸 함수? 좋다 이거야... | ||
| > 근데 이런 함수를 만들어서 어디에 두며, 같이 일하는 사람들이 어떻게 쉽게 인지하고 사용할 수 있게 할 수 있을까나요 ... :thinking_face: | ||
|
|
There was a problem hiding this comment.
재사용도 고려한 유틸 함수는 저도 감이 잘 안오네요!
객체지향에서는 일급컬렉션에 적용하기 좋겠다 생각했습니다
There was a problem hiding this comment.
개인적으로는 과한 분리가 아닐까 라는 생각도 많이 들었어요 ㅋㅋㅋㅋ 재사용이 많지 않은 이상 굳이 저런 함수를 따로 만들어야 하나? 라는 생각이 많이 들었습니다.
Dongkyuuuu
approved these changes
Apr 23, 2024
|
|
||
| > 약 2년전에 정리한 내용이긴 한데 .. | ||
| > V8 동작은 크게 변하지 않았을 거 같아서 공유 드립니다.. | ||
| > https://www.hyesungoh.xyz/how-to-collect-garbage-in-js |
|
|
||
| ## 암묵적 입력과 출력은 적을수록 좋다 | ||
|
|
||
| > 책에서는 과한 코드는 분리된 함수에서 한 함수로 넣기도 하는데, 어차피 액션이라 그런가? |
There was a problem hiding this comment.
앞장인지는 모르겠지만 서브루틴 추출하기 라는 개념으로 계속 분리 하더라고요.. 분리해서 계산으로 만들고 액션을 최대한 줄이는게 포인트지 않을까... 라는 개인적인 생각입니다.
Comment on lines
+68
to
+71
|
|
||
| > 이런 유틸 함수? 좋다 이거야... | ||
| > 근데 이런 함수를 만들어서 어디에 두며, 같이 일하는 사람들이 어떻게 쉽게 인지하고 사용할 수 있게 할 수 있을까나요 ... :thinking_face: | ||
|
|
There was a problem hiding this comment.
개인적으로는 과한 분리가 아닐까 라는 생각도 많이 들었어요 ㅋㅋㅋㅋ 재사용이 많지 않은 이상 굳이 저런 함수를 따로 만들어야 하나? 라는 생각이 많이 들었습니다.
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.