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

[박상범] 챕터 7: 신뢰할 수 없는 코드를 쓰면서 불변성 지키기 #42

Merged
merged 1 commit into from
May 22, 2024

Conversation

sangbooom
Copy link
Contributor

No description provided.

Copy link
Member

@hyesungoh hyesungoh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

갓범님 고생하셨습니다 👍 👍

Comment on lines +12 to +28
tanstack/query 에서 구조적 공유(structural share)는 어떻게 동작될까?

- 해당 코드 : https://github.com/TanStack/query/blob/2aca521a882e54b01ee6a070cb4a38f38bfa4bcf/packages/query-core/src/utils.ts#L210-L253
- 코드 해석:

**입력으로 `a`와 `b` 두 매개변수를 받으며, 이 두 매개변수가 완전히 동일할 경우 `a`를 반환**

**만약, `a`와 `b`가 배열이거나 객체인 경우, 각 항목 또는 속성을 순회하며 다음을 수행:**

- **배열 또는 객체의 모든 항목(또는 속성)을 비교**
- **동일한 항목은 `a`에서 가져옴**
- **`a`와 `b` 사이에 동일하지 않은 항목이 있다면, 그 항목에 대해 재귀적으로 `replaceEqualDeep` 함수를 호출하여 깊은 비교를 수행**
- **만약 `a`와 `b`의 크기가 같고, 모든 항목이 동일하다면 원본 `a`를 반환**
- **그렇지 않다면, 변경된 사본을 반환**

**이 코드의 핵심은, 깊은 동일성(deep equality) 검사를 통해 불필요한 데이터 복사를 방지하고, 가능한 경우 원본 데이터 구조를 재사용하여 메모리 사용을 최적화하는 것.
⇒ 큰 JSON 구조에서 데이터를 효율적으로 다루기 위함일 듯, 최적화 굿**
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

우왕 이런 것이 있는지 어떻게 아셨나요?!?!?! 궁금합니다!!!

공유 감사드려요!! 👍 👍

Copy link
Contributor Author

@sangbooom sangbooom Apr 30, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tkdodo 블로그에 있었어요! 🙌🏻

rel: https://tkdodo.eu/blog/react-query-render-optimizations#structural-sharing

@hyesungoh hyesungoh merged commit 0172c09 into main May 22, 2024
1 check passed
@hyesungoh hyesungoh deleted the 챕터7/박상범 branch May 22, 2024 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants