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

JavaScript: Map.prototype.get(), has() 추가 #11171

Merged
merged 2 commits into from Jan 31, 2023
Merged

Conversation

wisedog
Copy link
Contributor

@wisedog wisedog commented Jan 20, 2023

  • Map.prototype.get() 추가
  • Map.prototype.has() 추가

- Map.prototype.get() 추가
- Map.prototype.has() 추가
@wisedog wisedog requested a review from a team as a code owner January 20, 2023 07:32
@wisedog wisedog requested review from yechoi42 and removed request for a team January 20, 2023 07:32
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Jan 20, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jan 20, 2023

Preview URLs

Flaws (2)

URL: /ko/docs/Web/JavaScript/Reference/Global_Objects/Map/get
Title: Map.prototype.get()
Flaw count: 1

  • macros:
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Map/set does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Map/set

URL: /ko/docs/Web/JavaScript/Reference/Global_Objects/Map/has
Title: Map.prototype.has()
Flaw count: 1

  • macros:
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Map/set does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Map/set

(comment last updated: 2023-01-29 23:38:48)

Comment on lines 54 to 56
맵이 원본 객체에 대한 참조만 보유하고 있다는 것은 해당 객체는 예측하지 못한 메모리 문제가 발생할 소지가 있는 가비지 수집이 되지 않음을
의미합니다. 만약 맵에 저장되어 있는 객체가 원본 객체와 동일한 수명을 가지게 하려면 {{jsxref("WeakMap")}}을 고려하시기
바랍니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
맵이 원본 객체에 대한 참조만 보유하고 있다는 것은 해당 객체는 예측하지 못한 메모리 문제가 발생할 소지가 있는 가비지 수집이 되지 않음을
의미합니다. 만약 맵에 저장되어 있는 객체가 원본 객체와 동일한 수명을 가지게 하려면 {{jsxref("WeakMap")}}을 고려하시기
바랍니다.
맵이 원본 객체에 대한 참조만 보유하고 있다는 것은 오브젝트가 가비지 콜렉팅되지 않을 수 있으며 이로 인해예측하지 못한 메모리 이슈가 일어날 수 있음을 의미합니다. 만약 맵에 저장되어 있는 객체가 원본 객체와 동일한 수명을 가지게 하려면 {{jsxref("WeakMap")}}을 고려하시기 바랍니다.

안녕하세요! @wisedog 님. 다음 문장에 대한 번역 제안을 드립니다.

Note that the map holding a reference to the original object effectively means the object cannot be garbage-collected, which may lead to unexpected memory issues.

Copy link
Contributor Author

@wisedog wisedog Jan 29, 2023

Choose a reason for hiding this comment

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

중간에 끊은건 좋은 제안입니다.

다만 제안주신 내용 중 아래 내용을 한 번 살펴보시면 어떨까 합니다.

(제안주신 내용) -> (다시 제안드리는 내용)
오브젝트 -> 해당 객체
가비지 콜렉팅 -> 가비지 콜렉션 혹은 쓰레기 수집
메모리 이슈 -> 메모리 문제

독자가 이해하는데 크게 문제 없으면 영어보다는 한글로 적고자 합니다만, 가비지 콜렉팅은... 현업에서는 전자를 주로 쓰긴하던데 쓰려면 가비지 콜렉션으로 하던가 쓰레기 수집이 좋지 않을까 싶습니다.

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

Choose a reason for hiding this comment

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

넵 알겠습니다. 좋은 리뷰 감사합니다!

@yechoi42 yechoi42 merged commit dd4b73e into mdn:main Jan 31, 2023
@wisedog wisedog deleted the js/map-has-get branch January 31, 2023 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ko Issues related to Korean content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants