Skip to content

[문서] C17 충돌 과제 해결#1863

Open
jhroom wants to merge 3 commits into
javascript-tutorial:2026-en-mergefrom
jhroom:fix/c17-conflict
Open

[문서] C17 충돌 과제 해결#1863
jhroom wants to merge 3 commits into
javascript-tutorial:2026-en-mergefrom
jhroom:fix/c17-conflict

Conversation

@jhroom
Copy link
Copy Markdown

@jhroom jhroom commented May 16, 2026

요약

c17 영문 문서 동기화 충돌을 해결하고 누락된 번역 추가하였습니다.

1-js/06-advanced-functions/03-closure/10-make-army/task.md
1-js/06-advanced-functions/03-closure/5-function-in-if/task.md
1-js/06-advanced-functions/03-closure/7-let-scope/solution.md
1-js/06-advanced-functions/03-closure/article.md
1-js/06-advanced-functions/04-var/article.md
1-js/06-advanced-functions/05-global-object/article.md
1-js/06-advanced-functions/06-function-object/5-sum-many-brackets/solution.md
1-js/06-advanced-functions/06-function-object/article.md

연관 이슈

Pull Request 체크리스트

TODO

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

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 16, 2026

CLA assistant check
All committers have signed the CLA.

@jhroom
Copy link
Copy Markdown
Author

jhroom commented May 16, 2026

@meanzzi 리뷰 부탁드립니다

Copy link
Copy Markdown

@meanzzi meanzzi left a comment

Choose a reason for hiding this comment

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

제가 submit review를 안눌렀었네요.. 죄송합니다 ㅠㅠ 여기 부분 확인부탁드릴게요!

*!*
// all shooters show 10 instead of their numbers 0, 1, 2, 3...
army[0](); // 10 from the shooter number 0
army[1](); // 10 from the shooter number 1
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

여기 예시가 누락된 것 같아요! army1, army2 부분이요!


위에서 살펴본 바와 같이, `var`는 `if`, `for` 등의 코드 블록을 관통합니다. 아주 오래전의 자바스크립트에선 블록 수준 렉시컬 환경이 만들어 지지 않았기 때문입니다. `var`는 구식 자바스크립트의 잔재이죠.
=======
alert(phrase); // ReferenceError: phrase is not defined
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

주석에 ReferenceError대신 Error가 사용된 것 같습니다!


```js run
// Tries to declare and immediately call a function
function() { // <-- SyntaxError: Function statements require a function name
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

SyntaxError대신 Error가 사용된 것 같습니다!

<<<<<<< HEAD
전역 객체의 이름을 `globalThis`로 표준화하자는 내용이 최근에 자바스크립트 명세에 추가되었기 때문에 모든 호스트 환경이 이를 따라야 하죠. Chromium 기반이 아닌 몇몇 브라우저는 아직 `globalThis`를 지원하진 않지만, 이에 대한 폴리필(polyfill)을 쉽게 만들 수 있습니다.
=======
Recently, `globalThis` was added to the language, as a standardized name for a global object, that should be supported across all environments. It's supported in all major browsers.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

번역이 잘못된 것 같습니다..!

- [모듈](info:modules)을 사용하고 있지 않다면, 브라우저에서 `var`로 선언한 전역 변수는 전역 객체의 프로퍼티가 됩니다.
- 이해하기 쉽고 요구사항 변경에 쉽게 대응할 수 있는 코드를 구현하려면, `window.x`처럼 전역 객체의 프로퍼티에 직접 접근합시다.
=======
...But more often is referred by "old-school" environment-specific names, such as `window` (browser) and `global` (Node.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.

번역이 잘못된 것 같습니다.

@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 👻

2 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 👻

Copy link
Copy Markdown

@meanzzi meanzzi left a comment

Choose a reason for hiding this comment

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

여기 부분 추가 수정 가능할까요..!?


sayHi();
<<<<<<< HEAD
alert(phrase); // Error: phrase is not defined
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

여기 부분 alert(phrase); // ReferenceError: phrase is not defined 로 되어야할것 같아요!


```js run
// 함수를 선언과 동시에 실행하려고 함
function() { // <-- Error: Function statements require a function name
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

여기도 function() { // <-- SyntaxError: Function statements require a function name 이렇게 해주셔야 할 것 같아요..!

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

앗 감사합니다 제가 리드미에서 에러주석은 한국어로 번역을 안한다는걸 다르게 이해를 했었네요..
반영했습니다. (5e5810)

Copy link
Copy Markdown

@meanzzi meanzzi left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@meanzzi
Copy link
Copy Markdown

meanzzi commented May 17, 2026

/done

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.

4 participants