-
Notifications
You must be signed in to change notification settings - Fork 1k
Translate tuples.md and Modify the 'num' of elements in tour docs #1403
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
can you suggest a reviewer? |
@byeonggukgong Could I ask you review? |
@wooruang I'll review it this weekend. :) |
@byeonggukgong Good:thumbsup: Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good to me, just two suggestion
_ko/tour/tuples.md
Outdated
## 엘리먼트 접근하기 | ||
|
||
튜플의 엘리먼트에 접근하기 위한 한가지 방법은 위치로 접근하는 것이다. 각 개별적인 엘리먼트들은 `_1`, `_2`, ... 이와 같은 방법으로 이름지어진다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When translating into Korean, it seems natural to use active type rather than passive type.
각 개별적인 엘리먼트들은 _1
, _2
, ... 이와 같은 방법으로 이름지어진다.
How about this?
각 개별적인 엘리먼트들은 _1
, _2
, ... 와 같은 이름을 갖는다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think 각 and 개별 is duplicate semantically.
각 개별적인 엘리먼트들은
How about this?
각 요소들은
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@horace-velmont @byeonggukgong Both are good suggestions!
_ko/tour/tuples.md
Outdated
## 튜플과 케이스 클래스 | ||
|
||
사용자들은 때때로 튜플과 케이스 클래스 사이의 선택의 기로에서 무엇을 선택해야할지 힘들어 할지도 모른다. 케이스 클래스는 이름이 있는 엘리먼트들을 갖는다. 그 엘리먼트의 이름들은 어떤 종류의 코드에서 가독성을 향상 시켜줄 수 있다. 위의 planet 예제에서 우리는 튜플을 사용하는 것보다 `case class Planet(name: String, distance: Double)`로 정의 하는 것이 더 나을 수도 있다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그 엘리먼트의 이름들은 어떤 종류의 코드에서 가독성을 향상 시켜줄 수 있다.
I suggest,
그 엘리먼트의 이름들은 어떤 종류의 코드에서 가독성을 높일 수 있다.
_ko/tour/tuples.md
Outdated
스칼라에서 하나의 튜플은 정해진 갯수의 각각 구별된 타입을 갖는 엘리먼트들을 포함하는 하나의 값이다. 튜플은 불변적이다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a good idea to translate people so that you can understand them better. And I translate as much as possible into Korean. I prefer a way to increase readability in parentheses if necessary.
스칼라에서 하나의 튜플은 정해진 갯수의 각각 구별된 타입을 갖는 엘리먼트들을 포함하는 하나의 값이다.
How about this?
스칼라에서 튜플은 정해진 요소(element)를 가지는 값으로 각 요소는 고유한 타입을 가진다.
_ko/tour/tuples.md
Outdated
## 튜플과 케이스 클래스 | ||
|
||
사용자들은 때때로 튜플과 케이스 클래스 사이의 선택의 기로에서 무엇을 선택해야할지 힘들어 할지도 모른다. 케이스 클래스는 이름이 있는 엘리먼트들을 갖는다. 그 엘리먼트의 이름들은 어떤 종류의 코드에서 가독성을 향상 시켜줄 수 있다. 위의 planet 예제에서 우리는 튜플을 사용하는 것보다 `case class Planet(name: String, distance: Double)`로 정의 하는 것이 더 나을 수도 있다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The target of these documents is the Scala beginner. So I don't think you need to use "사용자, 유저 등" explicitly.
사용자들은 때때로 튜플과 케이스 클래스 사이의 선택의 기로에서 무엇을 선택해야할지 힘들어 할지도 모른다.
How about this?
때로는 튜플과 케이스 클래스 중 무엇을 사용할지 고르는데 힘들 수 있다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
thank you! |
I translated tuples.md in Korean.