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
100Gyeon
requested review from
hyesungoh,
Dongkyuuuu,
gutenLEE,
metacode22,
Orchemi,
sangbooom,
Seojunhwan,
sjoleee and
sumi-0011
as code owners
hyesungoh
approved these changes
Aug 12, 2024
Comment on lines
+38
to
+44
| 얼랭과 같은 환경에서는 **프로그램이 실패하면 멈추도록 설계**되어 있는데, 이런 **실패는 슈퍼바이저가 관리**한다. | ||
| 슈퍼바이저 자체가 실패하면 어떻게 될까? 슈퍼바이저의 슈퍼바이저가 실패를 관리한다. | ||
| 이렇게 슈퍼바이저가 층층이 '**슈퍼바이저 트리**'를 이루는 설계로 이어진다. | ||
| 이 기법은 고가용성, 결함 감내 시스템에서 얼랭이나 엘릭서를 채택하는 요인이 된다. | ||
|
|
||
| > '슈퍼바이저도 실패하면?'을 생각하고 있었는데 바로 몇 줄 뒤에 슈퍼바이저 트리를 설명하고 있어서 웃겼네요 | ||
| > 이 부분 좀 흥미로웠습니다👍 |
Comment on lines
+55
to
+60
| 절대 그런 일이 일어나지 않을 거라는 생각이 든다면 이를 확인하는 코드를 작성하라. = **단정문**(assertion)을 사용해라. | ||
| 하지만 진짜 오류 처리가 필요한 곳에 단정을 대신 사용하지는 말라. | ||
| 단정은 결코 일어나면 안 되는 것들을 검사한다. | ||
|
|
||
| > 절대 일어나지 않을 일인데 확인하는 게 저만 비효율적으로 느껴지나요? | ||
| > 아직 저한테 와닿는 토픽은 아니네요🤔 |
There was a problem hiding this comment.
저는 뭔가 .. 절대 그런 일이 일어나지 않을 거라 장담해서 일어날 수 있다 .. 정도로 이해했어요 🤔 🤔
Comment on lines
+75
to
+76
| > 얼마 전에 기능 구현하면서 `MutationObserver`, `IntersectionObserver`를 사용했는데 | ||
| > `disconnect`를 통해 리소스를 해제했던 게 떠오르네요 |
Comment on lines
+71
to
+73
| > 자바스크립트를 사용하면서 메모리를 크게 신경 쓰지 않았던 것 같아 반성하는 마음으로 관련 아티클을 읽었습니다..^^ | ||
| > [자바스크립트 메모리 관리: 일반적인 메모리 누수를 방지하고 성능을 개선하는 방법](https://ykss.netlify.app/translation/javascript_memory_management) | ||
| > [교묘한 리액트 메모리 누수 - 리액트 컴파일러가 당신을 구하지 못하는 이유](https://emewjin.github.io/react-compiler-closure-performance) |
There was a problem hiding this comment.
https://www.hyesungoh.xyz/how-to-collect-garbage-in-js
아마 비슷한 이야기를 할 거 같긴 한데 ... 옛날에 적었던 아티클 공유드립니다 ..
There was a problem hiding this comment.
옛날부터 글을 잘 쓰셨군요 공유 감사합니다 ㅎㅎ
metacode22
reviewed
Aug 13, 2024
| 실용주의 프로그래머는 어느 누구도, 심지어는 자기 자신도 | ||
| 완벽한 코드를 작성할 수 없음을 알기에 실수에 대비한 방어책을 마련한다. | ||
|
|
||
| ## Topic 23 계약에 의한 설계 |
There was a problem hiding this comment.
오... 어려워서 넘어갔는데 고수분이 여기 계셨군요! 발표 기다리겠습니다 😀
gutenLEE
approved these changes
Aug 13, 2024
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.