Skip to content

Curring moved#111

Closed
singularity-code wants to merge 5 commits intojavascript-tutorial:masterfrom
singularity-code:#74_curring_moved
Closed

Curring moved#111
singularity-code wants to merge 5 commits intojavascript-tutorial:masterfrom
singularity-code:#74_curring_moved

Conversation

@singularity-code
Copy link
Contributor

다시 작업했어요. 그대로 사용한 문장이 많이 있었구요.
더 간결해졌었네요. 저도 올리고 리뷰한번더 하겠습니다.

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.

튜토리얼 원 작성자가 대대적인 수정을 하고 있어서 번역에 기여해 주시는 분들도 이를 follow-up하기가 난해한 상황입니다. 그럼에도 불구하고 맡으신 부분을 끝까지 책임져 주심에 감사드립니다.
오역은 많이 없어서 한국어 문장에 대한 코멘트를 많이 달아보았습니다. PR전에 꼭 로컬 서버 돌리셔서 확인 부탁드립니다. 그리고 영어원문 없이 본인이 번역하신 한글 번역본을 읽었을 때 이해되는지 확인 후 PR해주시기 바랍니다.

먼저 예제를 통해서 커링이 무엇인지 이해하고 그다음에 실용적인 적용법을 알아보겠습니다.

We'll create a helper function `curry(f)` that performs currying for a two-argument `f`. In other words, `curry(f)` for two-argument `f(a, b)` translates it into a function that runs as `f(a)(b)`:
`f`의 두 개의 인수를 커링하는 헬퍼 함수 `curry(f)`를 생성해 보겠습니다. 다르게 표현하면, `curry(f)`를 만들어서 두 개의 인수를 가진 `f(a, b)``f(a)(b)`로 변환하는 것입니다.
Copy link
Member

Choose a reason for hiding this comment

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

  • 다르게 표현하면, A를 B로 변환하는 것입니다. 라는 문장 구조가 어색합니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

끙..

Copy link
Member

Choose a reason for hiding this comment

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

한국어 호응관계가 어렵죠.. ㅠㅠ In other words다르게 표현하면으로 해석하셨기 때문에 그 뒤 문장은 이에 호응되게 해주셔야합니다.

구글에 "다르게 표현하면"으로 검색하시면 많은 예시문장을 볼수가있어요.

@javascript-translate-bot

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

@singularity-code singularity-code changed the title #74 curring moved Curring moved Sep 2, 2019
singularity-code added a commit to singularity-code/ko.javascript.info that referenced this pull request Sep 2, 2019
@singularity-code
Copy link
Contributor Author

지난번 리뷰를 해주셧던걸 바탕으로 했는데 대충(?!) 한 감이 없지 않아 있어, 실수가 많았네요;;;
꼼꼼히 리뷰해주셔서 감사합니다. 간단히 solve 된것은 닫고 나머지는 남겨두었는데 의견주세요. 저도 다시 읽어보겠습니다.

singularity-code added a commit to singularity-code/ko.javascript.info that referenced this pull request Sep 3, 2019
@Violet-Bora-Lee
Copy link
Member

커링 정의 부분만 수정되면 머지 진행하겠습니다 :)

@singularity-code
Copy link
Contributor Author

Line 10 인가요? 주어 누락된 부분이라면 수정했습니다.

@Violet-Bora-Lee
Copy link
Member

Line 10이 여전히 그대로입니다.

image

image

force push쓰셔서 푸시해보신다음에 반영되나 확인해주세요

singularity-code added a commit to singularity-code/ko.javascript.info that referenced this pull request Sep 12, 2019
@singularity-code
Copy link
Contributor Author

브랜치를 새로 했었는데 커밋만 한다고 되는게 아니었군요.
이건 닫고 다시 PR 날렸습니다. 수고하셨어요!

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.

3 participants