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

[ko] String.prototype[@@iterator]() 신규 번역 #20363

Merged
merged 2 commits into from
May 19, 2024

Conversation

wisedog
Copy link
Contributor

@wisedog wisedog commented May 13, 2024

- String.prototype[@@iterator]()  신규 번역
@wisedog wisedog requested a review from a team as a code owner May 13, 2024 23:14
@wisedog wisedog requested review from 1ilsang and removed request for a team May 13, 2024 23:14
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label May 13, 2024
Copy link
Contributor

github-actions bot commented May 13, 2024

Preview URLs

External URLs (1)

URL: /ko/docs/Web/JavaScript/Reference/Global_Objects/String/@@iterator
Title: String.prototype[@@iterator]()

(comment last updated: 2024-05-19 06:01:23)

Copy link
Member

@1ilsang 1ilsang left a comment

Choose a reason for hiding this comment

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

@wisedog 님 안녕하세요. 기여해주셔서 감사합니다! 💯

몇 가지 코멘트 남겨두었습니다.

확인하시고 다시 리뷰 요청 부탁드리겠습니다. 🙇

질문이나 도움이 필요하시다면 언제든 멘션 부탁드리겠습니다.


### for...of 반복를 사용한 순회

이 메서드를 직접 호출할 필요는 거의 없습니다. `@@iterator` 메서드가 존재하기 때문에 문자열 [iterable](/ko/docs/Web/JavaScript/Reference/Iteration_proocols#the_iterable_proocol)과 `for...of` 같은 반복 구문은 자동으로 이 메서드를 호출하여 반복할 반복자를 얻습니다.
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
이 메서드를 직접 호출할 필요는 거의 없습니다. `@@iterator` 메서드가 존재하기 때문에 문자열 [iterable](/ko/docs/Web/JavaScript/Reference/Iteration_proocols#the_iterable_proocol)`for...of` 같은 반복 구문은 자동으로 이 메서드를 호출하여 반복할 반복자를 얻습니다.
이 메서드를 직접 호출할 필요는 거의 없습니다. `@@iterator` 메서드가 존재하기 때문에 문자열 [iterable](/ko/docs/Web/JavaScript/Reference/Iteration_protocols#the_iterable_protocol)`for...of` 같은 반복 구문은 자동으로 이 메서드를 호출하여 반복할 반복자를 얻습니다.

링크가 잘못되었네요.

// "\uD835\uDC6A"
```

### 반복기 수동으로 돌리기
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
### 반복기 수동으로 돌리기
### 반복자 수동으로 돌리기

iterator는 "반복자"로 규정하고 있습니다.


### 반복기 수동으로 돌리기

반환된 반복기 객체의 `next()` 메서드를 수동으로 호출하여 반복 프로세스를 최대한 제어할 수 있습니다.
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
반환된 반복기 객체의 `next()` 메서드를 수동으로 호출하여 반복 프로세스를 최대한 제어할 수 있습니다.
반환된 반복자 객체의 `next()` 메서드를 수동으로 호출하여 반복 프로세스를 최대한 제어할 수 있습니다.


## 같이 보기

- [Polyfill of `String.prototype[@@iterator]` in `core-js`](https://github.com/zloirock/core-js#ecmascript-string-and-regexp)
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
- [Polyfill of `String.prototype[@@iterator]` in `core-js`](https://github.com/zloirock/core-js#ecmascript-string-and-regexp)
- [`core-js`에서 `String.prototype[@@iterator]`의 폴리필](https://github.com/zloirock/core-js#ecmascript-string-and-regexp)

@1ilsang 1ilsang self-requested a review May 18, 2024 14:02
Copy link
Member

@1ilsang 1ilsang left a comment

Choose a reason for hiding this comment

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

위에서 잘못 approve하여 request change로 남깁니다.

@wisedog
Copy link
Contributor Author

wisedog commented May 19, 2024

언제나 꼼꼼한 리뷰 감사합니다. 수정사항은 모두 반영했고, 다시 리뷰를 요청드립니다.

@wisedog wisedog requested a review from 1ilsang May 19, 2024 06:08
Copy link
Member

@1ilsang 1ilsang left a comment

Choose a reason for hiding this comment

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

LGTM 고생하셨습니다

@1ilsang 1ilsang merged commit 97479f7 into mdn:main May 19, 2024
7 checks passed
@wisedog wisedog deleted the js/string-iterator branch May 26, 2024 22:36
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