Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

1-js/02-first-steps/04-variables/article.md 리뷰#217

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Violet-Bora-Lee merged 1 commit intojavascript-tutorial:masterfromJuYeong0413:master
Sep 29, 2019
Merged

Conversation

JuYeong0413
Copy link
Contributor

No description provided.

@@ -78,11 +78,11 @@ let user = 'John'
, message = 'Hello';
```

위에서 소개한 방식들에 기술적인 차이가 있지는 않습니다.개인 취향과 미적 감각에 따라 원하는 방식으로 코드를 작성하시길 바랍니다.
위에서 소개한 방식들에 기술적인 차이가 있지는 않습니다.개인의 취향과 미적 감각에 따라 원하는 방식으로 코드를 작성하시길 바랍니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

~시 사용을 제한적으로 사용하는건 어떤가요?

Suggested change
위에서 소개한 방식들에 기술적인 차이가 있지는 않습니다. 개인의 취향과 미적 감각에 따라 원하는 방식으로 코드를작성하시길 바랍니다.
위에서 소개한 방식들에 기술적인 차이가 있지는 않습니다. 개인의 취향과 미적 감각에 따라 원하는 방식으로 코드를작성하세요.

https://github.com/javascript-tutorial/ko.javascript.info/wiki/번역-모범-사례#시는-제한적으로-사용합니다

@@ -242,17 +242,17 @@ const myBirthday = '18.04.1982';
```js run
const myBirthday = '18.04.1982';

myBirthday = '01.01.2001'; //Error: Assignment to constant variable.
myBirthday = '01.01.2001'; //error, can't reassign the constant!
Copy link
Contributor

Choose a reason for hiding this comment

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

번역이 빠져있어요!

Suggested change
myBirthday = '01.01.2001'; //error, can't reassign the constant!
myBirthday = '01.01.2001'; //에러, 상수를 재할당 할수 없어요!

Copy link
Contributor

Choose a reason for hiding this comment

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

아니면 혹시 주석에 있는 건 번역하지 않는게 맞나요? 👀👀👀👀

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

실제 코드를 실행해보면Assignment to constant variable이라는 에러 문구가 뜨기 때문에 처음부터 번역을 하지 않은 상태로 둔 것이 아닐까 생각했습니다! (eval 번역 및 수정사항 반영 PR을 보니 얼럿창에 undefined라고 나와서 해당 부분이 주석에 표현된 것은 번역하지 않는 것으로 말씀해 주셔서요...!) 다만 현재 원문과 비교했을 때Error: Assignment to constant variable. 부분이error, can't reassign the constant! 로 바뀌어서 수정했습니다. 😃

CreatiCoding reacted with thumbs up emojiCreatiCoding reacted with laugh emoji

Choose a reason for hiding this comment

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

맞습니다~ 실제 실행해보고 나타나는 문구이면 주석에있는 문장도 번역하지 않고있습니다.

CreatiCoding reacted with thumbs up emoji
Copy link
Contributor

Choose a reason for hiding this comment

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

호호 그렇군요! 좋은 정보 감사합니다! 에러로 뜨는 문장이면 번역하지 않는게 맞군요!

```

변숫값이 절대 변경되지 않을것이라 확신하면,이를 보장하고,다른 개발자들에게 이 변수는상수라는것을 전달하기 위해 `const`를 사용해 변수를 선언하도록 합시다.
변숫값이 절대 변경되지 않을것을 확신하면, 다른 개발자들에게 이 변수는상수라는 것을 보장하고 이를 전달하기 위해 `const`를 사용해 변수를 선언하도록 합시다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
변숫값이 절대 변경되지 않을 것을 확신하면, 다른 개발자들에게 이 변수는 상수라는 것을 보장하고 이를 전달하기 위해`const`를 사용해 변수를선언하도록 합시다.
변숫값이 절대 변경되지 않을 것을 확신하면, 다른 개발자들에게 이 변수는 상수라는 것을 보장하고 이를 전달하기 위해`const`를 사용해 변수를선언합시다.

아래에서사용하도록 합시다.사용합시다.로 바꾸신 것 처럼선언합시다.로 바꾸는건 어떤가요?

@JuYeong0413
Copy link
ContributorAuthor

@CreatiCoding 리뷰를 하면서 '-시-'에 대한 부분을 확인한다고 보긴 했는데, 아직 많이 부족하네요 😢 코멘트 정말 감사합니다! 반영하도록 하겠습니다.

CreatiCoding reacted with thumbs up emoji

@@ -12,7 +12,7 @@

자바스크립트에선 `let` 키워드를 사용해 변수를 생성합니다.

아래문(statement)은 "message"라는 이름을 가진 변수를 생성(*선언(declare)* 또는 *정의(define)*)합니다.
아래구문은 "message"라는 이름을 가진 변수를 생성(*선언*)합니다.

Choose a reason for hiding this comment

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

여러 책이랑 번역물들을 뒤져봤는데 statement는 조금씩 다르게 번역되고 있더라구요.
그래서 저희는 문 혹은 구문으로 번역하기로 결정했습니다.
독자중 의아해 하실만한 분들도 있으실 것 같으니 병기 처리를 유지해 주시기 바랍니다.



````smart header="`let` 대신 `var`"
만들어진 지 오래된 스크립트에서 `let` 대신 `var`라는 키워드를발견하는 경우가 있습니다.
만들어진 지 오래된 스크립트에서 `let` 대신 `var`라는 키워드를볼 수 있습니다.

Choose a reason for hiding this comment

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

may found라는 원문의 특징을 살려서 번역해 보았는데볼 수로 번역하면 이를 살릴 수 있을지 모르겠네요.
의견주세요~

CreatiCoding reacted with thumbs up emoji

처음 봤을 땐 좀 이상해 보일 수 있지만, 함수형 언어는 중대한 개발에 상당히 적합한 언어입니다. 이런 제약이 장점으로 작용하는 병렬계산(parallel computation)과 같은 영역도 있죠. 당장은 사용할 계획이 없더라도 이런 언어를 공부하는 것은 시야를 넓히는 데 도움이 되므로, 학습을 권유 드립니다.
처음 봤을 땐 좀 이상해 보일 수 있지만, 함수형 언어는 중대한 개발에 상당히 적합한 언어입니다. 이런 제약이 장점으로 작용하는 병렬계산과 같은 영역도 있죠. 당장은 사용할 계획이 없더라도 이런 언어를 공부하는 것은 시야를 넓히는 데 도움이 되므로, 학습을 권유 드립니다.

Choose a reason for hiding this comment

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

parallel computation을 다양한 방식으로 번역 할 수 있을것 같아 병기하여 작성했는데, 많이 알법한 단어인가요?
자주 쓰이는 용어는 아닌것 같아 병기하는게 좋을 것 같습니다.
의견주세요

````

````warn header="예약어"
[예약어(reserved name) 목록](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Lexical_grammar#Keywords)에 있는 단어는 변수명으로 사용할 수 없습니다. 이 단어들은 자바스크립트 내부에서 이미 사용 중이기 때문입니다.

예약어 예시: `let`, `class`, `return`, `function`

아래 코드는 문법(syntax) 에러를 발생시킵니다.
아래 코드는 문법 에러를 발생시킵니다.

Choose a reason for hiding this comment

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

syntax는 구문, 문법 등으로 해석되어서 원문의 의미를 전달드리고자 병기하였습니다.
문법에러라고 많이들 알고계신건지... 의견주세요

@@ -242,17 +242,17 @@ const myBirthday = '18.04.1982';
```js run
const myBirthday = '18.04.1982';

myBirthday = '01.01.2001'; //Error: Assignment to constant variable.
myBirthday = '01.01.2001'; //error, can't reassign the constant!

Choose a reason for hiding this comment

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

맞습니다~ 실제 실행해보고 나타나는 문구이면 주석에있는 문장도 번역하지 않고있습니다.

CreatiCoding reacted with thumbs up emoji
```

변숫값이 절대 변경되지 않을것이라 확신하면,이를 보장하고,다른 개발자들에게 이 변수는상수라는것을전달하기 위해 `const`를 사용해 변수를선언하도록 합시다.
변숫값이 절대 변경되지 않을것을 확신하면, 다른 개발자들에게 이 변수는상수라는 것을 보장하고 이를전달하기 위해 `const`를 사용해 변수를선언합시다.

Choose a reason for hiding this comment

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

  • 않을 것을이라고 하면 을이 두번 나와서 좀 이상한 것 같아요.

```

변숫값이 절대 변경되지 않을것이라 확신하면,이를 보장하고,다른 개발자들에게 이 변수는상수라는것을전달하기 위해 `const`를 사용해 변수를선언하도록 합시다.
변숫값이 절대 변경되지 않을것을 확신하면, 다른 개발자들에게 이 변수는상수라는 것을 보장하고 이를전달하기 위해 `const`를 사용해 변수를선언합시다.

Choose a reason for hiding this comment

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

Suggested change
변숫값이 절대 변경되지 않을것을 확신하면, 다른 개발자들에게 이 변수는 상수라는 것을보장하고 이를 전달하기위해`const`를 사용해 변수를선언합시다.
변숫값이 절대 변경되지 않을것이라 확신하면,값이 변경되는 것을 방지하면서다른 개발자들에게 이 변수는 상수라는 것을알리기위해`const`를 사용해 변수를선언하도록 합시다.


재사용된 변수는 과거에 붙여진 스티커를 떼지 않은 채 물건만 바뀐 상자와 같습니다. 상자 안에는 무엇이 들어 있을까요? 내용물에 대한 정보를 알고 있는 사람은 누구인가요? 이를 알기 위해선 상자에 가까이 다가가 확인해야만 합니다.

변수를 재사용하면 변수 선언에 쏟는 노력을 좀 덜 순 있겠지만, 디버깅에 열 배 더 많은 시간을 쏟아야 합니다.

변수를 추가하는 것은 악습이 아닙니다. 좋은 습관입니다.

모던 자바스크립트압축기(minifier)와 브라우저는 코드 최적화를 잘해줍니다. 변수를 추가한다고 해서 성능 이슈가 생기지 않죠. 값이 다른 경우, 변수를 다르게 선언해 주면 코드 최적화에 도움이 될 수도 있습니다.
모던 자바스크립트압축기와 브라우저는 코드 최적화를 잘해줍니다. 변수를 추가한다고 해서 성능 이슈가 생기지 않죠. 값이 다른 경우, 변수를 다르게 선언해 주면 코드 최적화에 도움이 될 수도 있습니다.

Choose a reason for hiding this comment

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

압축기는 잘 사용하는 용어가 아닌데, 일단 저희는 최대한 번역하는 쪽으로 하고있어서 병기표시하였습니다.
minifier가 많이 사용되는 용어이므로 병기를 유지해주시기 바랍니다.

@javascript-translate-bot

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

@JuYeong0413
Copy link
ContributorAuthor

주제에 관련되어 새롭게 등장하는 키워드에 대해서만 병기 처리를 하는 것인 줄 알고 상당수의 병기 처리를 지워버렸었네요... 유의하도록 하겠습니다. 말씀해주신 부분들 수정해서 다시 커밋합니다.
/done

@Violet-Bora-Lee
Copy link
Member

기여 감사합니다.
훨씬 좋은 문장이 되었네요.
나중에 배포되면 학습하시는 분들께 큰 도움이 될 것 같습니다 👍 👍 👍

@Violet-Bora-LeeViolet-Bora-Lee merged commit73e7200 intojavascript-tutorial:masterSep 29, 2019
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@CreatiCodingCreatiCodingCreatiCoding left review comments

@Violet-Bora-LeeViolet-Bora-LeeViolet-Bora-Lee requested changes

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@JuYeong0413@javascript-translate-bot@Violet-Bora-Lee@CreatiCoding

[8]ページ先頭

©2009-2025 Movatter.jp