[문서] C09 충돌 해결 및 미번역분 번역#1856
Conversation
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>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
|
@hcy020817 리뷰 부탁드립니다 밑에 파일들 해주시면 됩니다!
|
hcy020817
left a comment
There was a problem hiding this comment.
안녕하세요! 파일 수가 많고 초반 충돌 정리량도 꽤 많았는데 작업하시느라 정말 고생 많으셨습니다. 전체적으로 충돌 마커 정리나 문서 흐름은 잘 처리되어 있었습니다.
다만 리뷰하면서 아래와 같은 부분들이 몇 군데 보여 코멘트 남겼습니다.
-번역이 누락된 부분, 원문과 줄 바꿈이 달라진 부분, 원문에서는 삭제되었지만 한글 버전에는 남아 있는 내용
반복적으로 나타나는 유형이라 동일한 내용의 코멘트가 너무 많아지지 않도록, 처음 등장하는 부분들 위주로만 표시해두었습니다. 확인 부탁드립니다!
| <<<<<<< HEAD | ||
| 자바스크립트는 객체 프로퍼티 키로 오직 문자형과 심볼형만을 허용합니다. 숫자형, 불린형 모두 불가능하고 오직 문자형과 심볼형만 가능하죠. | ||
|
|
||
| 지금까지는 프로퍼티 키가 문자형인 경우만 살펴보았습니다. 이번 챕터에선 프로퍼티 키로 심볼값을 사용해 보면서, 심볼형 키를 사용할 때의 이점에 대해 살펴보도록 하겠습니다. |
There was a problem hiding this comment.
이 문장이 원문에서는 삭제된 문장인 것 같습니다. 확인 한번 부탁드립니다.
| 생성자의 의의는 바로 여기에 있습니다. 재사용할 수 있는 객체 생성 코드를 구현하는 것이죠. | ||
|
|
||
| <<<<<<< HEAD | ||
| 잠깐! 모든 함수는 생성자 함수가 될 수 있다는 점을 잊지 마시기 바랍니다. `new`를 붙여 실행한다면 어떤 함수라도 위에 언급된 알고리즘이 실행됩니다. 이름의 '첫 글자가 대문자'인 함수는 `new`를 붙여 실행해야 한다는 점도 잊지 마세요. 공동의 약속이니까요. |
There was a problem hiding this comment.
간결함을 위해 '시'를 붙이지 않고 "모든 함수는 생성자 함수가 될 수 있다는 점을 잊지 마세요."로 하는 것이 어떨까요?
| ``` | ||
|
|
||
| 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"가 코드에 두 번이나 등장합니다. |
| >>>>>>> upstream/master | ||
| 즉, `value?.prop`은 다음과 같이 평가됩니다. | ||
|
|
||
| * `value`가 존재하면 `value.prop`처럼 동작합니다. |
| ======= | ||
| 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 |
| `?.`은 `?.`왼쪽 평가대상이 없어도 괜찮은 경우에만 선택적으로 사용해야 합니다. | ||
|
|
||
| 꼭 있어야 하는 값인데 없는 경우에 `?.`을 사용하면 프로그래밍 에러를 쉽게 찾을 수 없으므로 이런 상황을 만들지 말도록 합시다. |
| `obj1 + obj2` 처럼 객체끼리 더하는 연산을 하거나, `obj1 - obj2` 처럼 객체끼리 빼는 연산을 하면 어떤 일이 일어날까요? `alert(obj)`로 객체를 출력할 때는 무슨 일이 발생할까요? | ||
|
|
||
| <<<<<<< HEAD | ||
| 이 모든 경우에 자동 형 변환이 일어납니다. 객체는 원시값으로 변환되고, 그 후 의도한 연산이 수행됩니다. |
There was a problem hiding this comment.
원문에 대한 번역된 문장들을 아래에 추가하셨으니까 이 문장은 삭제해도 될 것 같습니다.
|
Please make the requested changes. After it, add a comment "/done". |
1 similar comment
|
Please make the requested changes. After it, add a comment "/done". |
|
Please make the requested changes. After it, add a comment "/done". |
1 similar comment
|
Please make the requested changes. After it, add a comment "/done". |
jjyy0804
left a comment
There was a problem hiding this comment.
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 | ||
| 일반적인 방법과 단축 구문을 사용한 방법이 완전히 동일하진 않습니다. 객체 상속과 관련된 미묘한 차이가 존재하는데 지금으로선 이 차이가 중요하지 않기 때문에 넘어가도록 하겠습니다. |
There was a problem hiding this comment.
원문의 In almost all cases, the shorter syntax is preferred.에 해당하는 내용이 빠진 것 같습니다. “대부분의 경우에는 더 짧은 단축 구문이 선호됩니다” 정도를 마지막에 추가하면 원문 의미가 더 잘 반영될 것 같습니다.
| user = null; // user를 null로 덮어씁니다. | ||
|
|
||
| <<<<<<< HEAD | ||
| admin.sayHi(); // sayHi()가 엉뚱한 객체를 참고하면서 에러가 발생했습니다. |
There was a problem hiding this comment.
원문에서는 admin.sayHi() 줄을 강조하기 위해 *!* / */!* 마커가 사용되고 있는데, 번역본에서는 이 마커가 빠진 것 같습니다! 원문처럼 유지하면 좋을 것 같습니다!
또한 에러 메시지는 this가 null인 상태에서 name 프로퍼티를 읽으려 해 발생한 TypeError를 보여주는 부분이라, TypeError 메세지 그대로 남겨두는 건 어떨까요 ?
참고)
README.md
에러 내용은 번역하지 않습니다.
예시: let rabbit = new Rabbit("White Rabbit"); // Error: this is not defined에서 주석은 번역하지 않습니다.
| 아래와 같은 세 개의 메서드를 가진 생성자 함수, `Calculator`를 만들어보세요. | ||
|
|
||
| <<<<<<< HEAD | ||
| - `read()` -- `prompt` 함수를 이용해 사용자로부터 값 두 개를 받고, 이를 객체 프로퍼티에 저장합니다. |
There was a problem hiding this comment.
원문에서는 함수명 뒤에 -- 없이 바로 설명이 이어지는데, 번역에서는 read() / sum() / mul() 뒤에 --가 추가된 것 같습니다 !
요약
번역 범위
1-js/04-object-basics/04-object-methods— article.md1-js/04-object-basics/06-constructor-new— article.md, task.md 2개1-js/04-object-basics/07-optional-chaining— article.md1-js/04-object-basics/08-symbol— article.md1-js/04-object-basics/09-object-toprimitive— article.md1-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