Skip to content

[문서] C09 충돌 해결 및 미번역분 번역#1856

Open
zerohyun00 wants to merge 16 commits into
javascript-tutorial:2026-en-mergefrom
zerohyun00:zerohyun00/sync-c09
Open

[문서] C09 충돌 해결 및 미번역분 번역#1856
zerohyun00 wants to merge 16 commits into
javascript-tutorial:2026-en-mergefrom
zerohyun00:zerohyun00/sync-c09

Conversation

@zerohyun00
Copy link
Copy Markdown

@zerohyun00 zerohyun00 commented May 15, 2026

요약

  • 일감 배분 시트상 제가 맡은 부분 충돌해결 및 미번역분 번역했습니다.

번역 범위

  • 1-js/04-object-basics/04-object-methods — article.md
  • 1-js/04-object-basics/06-constructor-new — article.md, task.md 2개
  • 1-js/04-object-basics/07-optional-chaining — article.md
  • 1-js/04-object-basics/08-symbol — article.md
  • 1-js/04-object-basics/09-object-toprimitive — article.md
  • 1-js/05-data-types/01-primitives-methods — article.md, task.md 1개
  • 1-js/05-data-types/02-number — article.md, solution.md 1개

연관 이슈

(fix #일이삼)

Pull Request 체크리스트

TODO

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

zerohyun00 and others added 11 commits May 16, 2026 01:28
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…rean

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 15, 2026

CLA assistant check
All committers have signed the CLA.

@zerohyun00 zerohyun00 marked this pull request as ready for review May 15, 2026 16:49
@zerohyun00
Copy link
Copy Markdown
Author

zerohyun00 commented May 16, 2026

@hcy020817 리뷰 부탁드립니다 밑에 파일들 해주시면 됩니다!

1-js/04-object-basics/06-constructor-new/article.md
1-js/04-object-basics/07-optional-chaining/article.md
1-js/04-object-basics/08-symbol/article.md
1-js/04-object-basics/09-object-toprimitive/article.md
1-js/05-data-types/01-primitives-methods/1-string-new-property/task.md
1-js/05-data-types/01-primitives-methods/article.md
1-js/05-data-types/02-number/2-why-rounded-down/solution.md
1-js/05-data-types/02-number/article.md

Copy link
Copy Markdown

@hcy020817 hcy020817 left a comment

Choose a reason for hiding this comment

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

안녕하세요! 파일 수가 많고 초반 충돌 정리량도 꽤 많았는데 작업하시느라 정말 고생 많으셨습니다. 전체적으로 충돌 마커 정리나 문서 흐름은 잘 처리되어 있었습니다.

다만 리뷰하면서 아래와 같은 부분들이 몇 군데 보여 코멘트 남겼습니다.
-번역이 누락된 부분, 원문과 줄 바꿈이 달라진 부분, 원문에서는 삭제되었지만 한글 버전에는 남아 있는 내용

반복적으로 나타나는 유형이라 동일한 내용의 코멘트가 너무 많아지지 않도록, 처음 등장하는 부분들 위주로만 표시해두었습니다. 확인 부탁드립니다!

<<<<<<< HEAD
자바스크립트는 객체 프로퍼티 키로 오직 문자형과 심볼형만을 허용합니다. 숫자형, 불린형 모두 불가능하고 오직 문자형과 심볼형만 가능하죠.

지금까지는 프로퍼티 키가 문자형인 경우만 살펴보았습니다. 이번 챕터에선 프로퍼티 키로 심볼값을 사용해 보면서, 심볼형 키를 사용할 때의 이점에 대해 살펴보도록 하겠습니다.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

이 문장이 원문에서는 삭제된 문장인 것 같습니다. 확인 한번 부탁드립니다.

생성자의 의의는 바로 여기에 있습니다. 재사용할 수 있는 객체 생성 코드를 구현하는 것이죠.

<<<<<<< HEAD
잠깐! 모든 함수는 생성자 함수가 될 수 있다는 점을 잊지 마시기 바랍니다. `new`를 붙여 실행한다면 어떤 함수라도 위에 언급된 알고리즘이 실행됩니다. 이름의 '첫 글자가 대문자'인 함수는 `new`를 붙여 실행해야 한다는 점도 잊지 마세요. 공동의 약속이니까요.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

간결함을 위해 '시'를 붙이지 않고 "모든 함수는 생성자 함수가 될 수 있다는 점을 잊지 마세요."로 하는 것이 어떨까요?

```

It works, there's no error... But it's quite inelegant. As you can see, the `"user.address"` appears twice in the code.
에러 없이 잘 동작하네요. 하지만 코드가 꽤 볼품없습니다. 보시다시피 "user.address"가 코드에 두 번이나 등장합니다.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

"user.address" 에 백틱 추가해야 할 것 같아요.

>>>>>>> upstream/master
즉, `value?.prop`은 다음과 같이 평가됩니다.

* `value`가 존재하면 `value.prop`처럼 동작합니다.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

*를 - 로 바꿔서 원문과 동일하게 유지해 줘야 할 것 같아요.

Comment on lines -170 to -172
=======
E.g. in `user?.address.street.name` the `?.` allows `user` to safely be `null/undefined` (and returns `undefined` in that case), but that's only for `user`. Further properties are accessed in a regular way. If we want some of them to be optional, then we'll need to replace more `.` with `?.`.
>>>>>>> upstream/master
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

이 부분 번역 빠져있는 것 같아요. 확인 부탁드립니다

Comment on lines 236 to 238
`?.`은 `?.`왼쪽 평가대상이 없어도 괜찮은 경우에만 선택적으로 사용해야 합니다.

꼭 있어야 하는 값인데 없는 경우에 `?.`을 사용하면 프로그래밍 에러를 쉽게 찾을 수 없으므로 이런 상황을 만들지 말도록 합시다.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

원문과 줄 바꿈 동일하게 해주시면 좋을 것 같아요.

`obj1 + obj2` 처럼 객체끼리 더하는 연산을 하거나, `obj1 - obj2` 처럼 객체끼리 빼는 연산을 하면 어떤 일이 일어날까요? `alert(obj)`로 객체를 출력할 때는 무슨 일이 발생할까요?

<<<<<<< HEAD
이 모든 경우에 자동 형 변환이 일어납니다. 객체는 원시값으로 변환되고, 그 후 의도한 연산이 수행됩니다.
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 👻

1 similar comment
@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 👻

1 similar comment
@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

@jjyy0804 jjyy0804 left a comment

Choose a reason for hiding this comment

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

1-js/04-object-basics/06-constructor-new/1-two-functions-one-object/task.md
1-js/04-object-basics/06-constructor-new/2-calculator-constructor/task.md
1-js/04-object-basics/06-constructor-new/article.md

위 의 파일 확인 후 수정이 필요해보이는 부분 리뷰 코멘트 달아두었습니다!

위처럼 `function`을 생략해도 메서드를 정의할 수 있습니다.

<<<<<<< HEAD
일반적인 방법과 단축 구문을 사용한 방법이 완전히 동일하진 않습니다. 객체 상속과 관련된 미묘한 차이가 존재하는데 지금으로선 이 차이가 중요하지 않기 때문에 넘어가도록 하겠습니다.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

원문의 In almost all cases, the shorter syntax is preferred.에 해당하는 내용이 빠진 것 같습니다. “대부분의 경우에는 더 짧은 단축 구문이 선호됩니다” 정도를 마지막에 추가하면 원문 의미가 더 잘 반영될 것 같습니다.

user = null; // user를 null로 덮어씁니다.

<<<<<<< HEAD
admin.sayHi(); // sayHi()가 엉뚱한 객체를 참고하면서 에러가 발생했습니다.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

원문에서는 admin.sayHi() 줄을 강조하기 위해 *!* / */!* 마커가 사용되고 있는데, 번역본에서는 이 마커가 빠진 것 같습니다! 원문처럼 유지하면 좋을 것 같습니다!

또한 에러 메시지는 thisnull인 상태에서 name 프로퍼티를 읽으려 해 발생한 TypeError를 보여주는 부분이라, TypeError 메세지 그대로 남겨두는 건 어떨까요 ?

참고)

README.md
에러 내용은 번역하지 않습니다.
예시: let rabbit = new Rabbit("White Rabbit"); // Error: this is not defined에서 주석은 번역하지 않습니다.

아래와 같은 세 개의 메서드를 가진 생성자 함수, `Calculator`를 만들어보세요.

<<<<<<< HEAD
- `read()` -- `prompt` 함수를 이용해 사용자로부터 값 두 개를 받고, 이를 객체 프로퍼티에 저장합니다.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

원문에서는 함수명 뒤에 -- 없이 바로 설명이 이어지는데, 번역에서는 read() / sum() / mul() 뒤에 --가 추가된 것 같습니다 !

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.

5 participants