Skip to content

[문서] C11 충돌 해결 및 번역 진행#1858

Open
HoonDongKang wants to merge 6 commits into
javascript-tutorial:2026-en-mergefrom
HoonDongKang:sync-c11
Open

[문서] C11 충돌 해결 및 번역 진행#1858
HoonDongKang wants to merge 6 commits into
javascript-tutorial:2026-en-mergefrom
HoonDongKang:sync-c11

Conversation

@HoonDongKang
Copy link
Copy Markdown

요약

  • en 저장소에서 업데이트된 내용을 ko 저장소로 업데이트하는 과정에서 발생한 충돌을 해결하였습니다.
  • 커밋된 히스토리를 타고 들어가며 수정된 내용에 대해 번역을 진행하였습니다.

충돌 해결 및 번역 범

  • 1-js/05-data-types/05-array-methods/article.md
  • 1-js/05-data-types/06-iterable/article.md
  • 1-js/05-data-types/07-map-set/03-iterable-keys/task.md
  • 1-js/05-data-types/07-map-set/article.md
  • 1-js/05-data-types/08-weakmap-weakset/01-recipients-read/solution.md
  • 1-js/05-data-types/08-weakmap-weakset/article.md
  • 1-js/05-data-types/09-keys-values-entries/article.md
  • 1-js/05-data-types/10-destructuring-assignment/article.md

연관 이슈

(fix #일이삼)

Pull Request 체크리스트

TODO

  • 번역 규칙을 확인하셨나요?
    • 줄 바꿈과 단락을 '원문과 동일하게' 유지하셨나요?
    • 맞춤법 검사기로 맞춤법을 확인하셨나요?
    • 마크다운 문법에 사용되는 공백(스페이스), 큰따옴표("), 작은따옴표('), 대시(-), 백틱(`) 등의 특수문자는 그대로 두셨나요?
  • 로컬 서버 세팅 후 최종 결과물을 확인해 보셨나요?
  • PR 하나엔 번역문 하나만 넣으셨나요?
  • 의미 있는 커밋 메시지를 작성하셨나요?
    • 예시
      • [프락시] 번역
      • [프락시] 과제 번역
      • [if문과 조건부 연산자 '?'] 리뷰
      • [주석] 2차 리뷰
      • [Date 객체와 날짜] 번역

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 15, 2026

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown

@DevJunz DevJunz left a comment

Choose a reason for hiding this comment

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

많은 분량인데 고생하셨습니다.
수정이 필요하다고 생각하는 부분 코멘트 남겨 놨습니다.
확인 후 다시 연락부탁드립니다!

- `arr.includes(item, from)` -- looks for `item` starting from index `from`, returns `true` if found.

Usually, these methods are used with only one argument: the `item` to search. By default, the search is from the beginning.
일반적으로 위 메서드는 검색할 `item` 하나만 인수로 전달해 호출하며, 기본적으로 검색은 배열의 0번 인덱스부터 시작합니다.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

KIGO에서 가급적 단수형 사용을 권장하지만 본 문장에서는
명확하게 위에 나오는 2개의 메서드를 지칭하고 있어서 복수형이 더 적절하다고 생각됩니다.

`thisArgs`에 `army`를 지정하지 않고 단순히 `users.filter(army.canJoin)`를 사용했다면 `army.canJoin`은 단독 함수처럼 취급되고, 함수 본문 내 `this`는 `undefined`가 되어 에러가 발생했을 겁니다.

<<<<<<< HEAD
`users.filter(user => army.canJoin(user))`를 사용하면 `users.filter(army.canJoin, army)`를 대체할 수 있긴 한데 `thisArg`를 사용하는 방식이 좀 더 이해하기 쉬우므로 더 자주 사용됩니다.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

해석이 잘못되어있는 것 같습니다.
'thisArg'를 사용하는 방식은 users.filter(army.canJoin, army) 를 의미합니다.
원문에서 'latter' 는 users.filter(user => army.canJoin(user))를 의미하고 이 방식이 더 자주 사용되고 더 이해하기 쉽다고 합니다.

하지만 번역에서는 'thisArg'를 사용하는 방식 즉, users.filter(army.canJoin, army) 가 더 쉽다고 되어 있어서 수정이 필요해보입니다.

이터러블을 사용한 예시는 다음과 같습니다.

<<<<<<< HEAD
```js
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

원문은 js run인데 번역에서 js 로 바뀐 것 같습니다

예시:

<<<<<<< HEAD
```js
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

여기도 js -> js run으로 바꿔야 할 것 같아요

@javascript-translate-bot
Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

3 similar comments
@javascript-translate-bot
Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@javascript-translate-bot
Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@javascript-translate-bot
Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants