Skip to content

10-destructuring-assignment 리뷰#166

Merged
Violet-Bora-Lee merged 1 commit intojavascript-tutorial:masterfrom
CreatiCoding:master
Sep 27, 2019
Merged

10-destructuring-assignment 리뷰#166
Violet-Bora-Lee merged 1 commit intojavascript-tutorial:masterfrom
CreatiCoding:master

Conversation

@CreatiCoding
Copy link
Contributor

@CreatiCoding CreatiCoding commented Sep 21, 2019

  1. (오탈자 수정): 되없습니다 -> 되었습니다 : 오탈자

  2. (기타 어색한 표현): "수반" -> "의미" : mean을 수반하다라는 말보단 의미하다라는 말이 일반적일 것 같습니다.

  3. (기타 어색한 표현): "걸러졌지만" -> "생략되었지만" : 생략으로 바꾸면 skip이란 뜻이 좀 더 명확하게 전달 될 것 같습니다.

  4. (미번역): 'only extract title as a variable' -> title만 변수로 뽑아내기

  5. (미번역): 'now title="Menu", rest={height: 200, width: 100}' -> 이제 title은 "Menu", rest는 {height: 200, width: 100} 입니다

  6. (원문에서 삭제됨): // 분해의 대상이 아닌 기타 요소 -> 삭제

  7. (잘못된 의미): 분해할 -> 반복할, 원문에서의 반복의 의미가 빠져있습니다.

원문

We can use it with destructuring to loop over keys-and-values of an object:
#번역리뷰처음 #근데빡세네요 #ㅠㅠ #힘내죠 #화이팅

@CLAassistant
Copy link

CLAassistant commented Sep 21, 2019

CLA assistant check
All committers have signed the CLA.

@CreatiCoding
Copy link
Contributor Author

commit 하나로 합쳤습니다.

Copy link
Member

@Violet-Bora-Lee Violet-Bora-Lee left a comment

Choose a reason for hiding this comment

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

꼼꼼히 봐주셔서 제가 놓친 부분도 수정이 되었네요.
감사합니다 👍 💯

[Object.entries(obj)](mdn:js/Object/entries)는 이전 챕터에서 학습한 바 있습니다.

이 메서드와 구조 분해를 조합하면 객체의 키와 값을 변수로 분해할 수 있습니다.
이 메서드와 구조 분해를 조합하면 객체의 키와 값을 변수로 반복할 수 있습니다.
Copy link
Member

Choose a reason for hiding this comment

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

객체의 키를 특정 변수에, 값도 특정 변수에 할당할 수 있다는 걸로 이해하여 분해라는 단어를 썼는데, 혹시나 제가 원문을 잘못이해할수도 있어서 확인부탁드릴게요.

Copy link
Contributor Author

@CreatiCoding CreatiCoding Sep 26, 2019

Choose a reason for hiding this comment

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

image

원문입니다.
반복 혹은 순회라는 내용이 빠지면 loop이라는 의미가 전달되지 않을 것 같아 여쭙습니당.

loop over keys-and-values라는 내용이 아래 주석에서는 객체의 키와 값 순회하기로 번역되어있습니다.
같은 내용인 것 같아 통일시키는건 어떠신가요?

Suggested change
이 메서드와 구조 분해를 조합하면 객체의 키와 값을 변수로 반복할 수 있습니다.
이 메서드와 구조 분해를 조합하면 객체의 키와 값을 순회할 수 있습니다.

@Violet-Bora-Lee

Copy link
Member

@Violet-Bora-Lee Violet-Bora-Lee Sep 26, 2019

Choose a reason for hiding this comment

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

이문장은 어떨까요

Suggested change
이 메서드와 구조 분해를 조합하면 객체의 키와 값을 변수로 반복할 수 있습니다.
이 메서드와 구조 분해를 조합하면 객체의 키와 값을 순회해 변수로 분해 할당할 수 있습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

와 좋습니다! 바로 반영해서 올리도록 하겠습니다!

*/!*

// now title="Menu", rest={height: 200, width: 100}
// 이제 title은 "Menu", rest는 {height: 200, width: 100} 입니다
Copy link
Member

Choose a reason for hiding this comment

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

문장이므로 끝에 마침표를 붙이는게 좋을 것 같습니다.

*/!*

// now title="Menu", rest={height: 200, width: 100}
// 이제 title은 "Menu", rest는 {height: 200, width: 100} 입니다
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
// 이제 title은 "Menu", rest는 {height: 200, width: 100} 입니다
// 이제 title은 "Menu", rest는 {height: 200, width: 100} 입니다.

@javascript-translate-bot

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

@CreatiCoding
Copy link
Contributor Author

피드백 감사합니다 :)

@Violet-Bora-Lee
Copy link
Member

감사합니다. 머지 진행하겠습니다 👍

@Violet-Bora-Lee Violet-Bora-Lee merged commit 8a85cf4 into javascript-tutorial:master Sep 27, 2019
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