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
metacode22
requested review from
hyesungoh,
sjoleee,
Seojunhwan,
sangbooom,
Orchemi,
sumi-0011,
100Gyeon and
WooWan
as code owners
100Gyeon
approved these changes
Oct 21, 2024
|
|
||
| 객체 간 커뮤니케이션을 개선하고 간소화한다. | ||
|
|
||
| > 제로초님이 요즘 유튜브 쇼츠로 디자인 패턴 영상을 올리던데 출퇴근길에 하나씩 보면 도움이 될 것 같네요..! |
There was a problem hiding this comment.
오 아예 쇼츠 모음집이 있네요 ㅋㅋㅋㅋ 책 읽으면서 같이 봐야겠어요 👀
hyesungoh
approved these changes
Oct 21, 2024
Comment on lines
+8
to
+9
|
|
||
| > 저의 경우, React에서 컴포넌트를 잘 나누지 않았던 것이 안티 패턴이었던 것 같아요. '그냥 한 파일에서 다 파악 가능하도록 하자'라는 생각이었는데요, 나중에 다시 보니까 그 컴포넌트가 어디까지 동작하는지 파악하기 힘들더라구요. 요즘엔 적절히... 나누고 있습니다. |
There was a problem hiding this comment.
적절히가 참 어렵더라고요 ..
저도 잘게 나눠진 파일보다 하나의 파일을 선호하는데 적당히가 어렵어요 어..
https://leerob.com/n/stack
리로빈슨도 그렇긴 하더라고요
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.
이번에도 짧아서 한 번에 해버렸읍니다.