Skip to content

[배열] 과제 번역#355

Closed
SimEunJu wants to merge 1 commit intojavascript-tutorial:masterfrom
SimEunJu:235
Closed

[배열] 과제 번역#355
SimEunJu wants to merge 1 commit intojavascript-tutorial:masterfrom
SimEunJu:235

Conversation

@SimEunJu
Copy link

@SimEunJu SimEunJu commented Nov 4, 2019

Pull Request 체크리스트

TODO

  • 번역 규칙을 확인하셨나요?
  • 줄 바꿈과 단락을 '원문과 동일하게' 유지하셨나요?
  • 맞춤법 검사기로 맞춤법을 확인하셨나요?
  • 공백(스페이스), 큰따옴표("), 작은따옴표('), 대시(-), 백틱(`)을 비롯한 모든 특수문자는 그대로 두셨나요?
  • 로컬 서버 세팅 후 최종 결과물을 확인해 보셨나요?
  • PR 하나엔 번역문 하나만 넣으셨나요?
  • 의미 있는 커밋 메시지를 작성하셨나요?
    • 예시
      • [프락시] 번역
      • [프락시] 과제 번역
      • [if문과 조건부 연산자 '?'] 리뷰
      • [주석] 2차 리뷰
      • [Date 객체와 날짜] 번역

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
// fruits에 어떤 값들이 들어 있을까요?
// fruits에 어떤 값이 들어 있을까요?

는 어떤가 제안 드립니다.

Copy link
Author

Choose a reason for hiding this comment

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

리뷰 주셔서 감사합니다! 말씀해주신 사항들은 반영하여 다시 pull request 보내도록 하겠습니다:smile:

Copy link
Contributor

Choose a reason for hiding this comment

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

[Maximum subarray problem] 부분도 번역해 주시면 좋을 것 같아요. 😃

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
`arr[2]` 함수 호출을 객체의 메소드처럼 할 수 있습니다. 따라서 `arr[2]``arr`을 참조하는 `this`를 받아 배열을 출력합니다.
`arr[2]` 함수 호출을 객체의 메서드처럼 할 수 있습니다. 따라서 `arr[2]``arr`을 참조하는 `this`를 받아 배열을 출력합니다.

method메서드로 번역합니다. 용어집

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
# 숫자 입력값들의
# 입력한 숫자의

는 어떤지 제안 드립니다.

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
- 배열 항목들의 합을 계산하고 리턴합니다.
- 배열 항목의 합을 계산하고 리턴합니다.

단수형을 사용해도 괜찮아보여서 제안 드립니다. Wiki - 어색하지 않다면 단수형을 사용합시다.

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
참고. 숫자 `0`은 유효한 숫자이므로, 사용자에게 질문을 멈추는 신호로 처리하지 마세요.
참고: 숫자 `0`은 유효한 숫자이므로, 사용자에게 질문을 멈추는 신호로 처리하지 마세요.

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.

독자의 이해를 돕기 위해 문장을 다듬어보았습니다.
의견 남겨주세요.

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
// 복사한 변수에 새로운 값을 push 함수로 추가합니다.
// 배열을 "복사"한 후, push 메서드를 이용해 새로운 값을 추가합니다.

Copy link
Member

Choose a reason for hiding this comment

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

-9옆에 쉼표가 들어가야 합니다.

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
입력은 `arr = [1, -2, 3, 4, -9. 6]` 같은 숫자로 이루어진 배열입니다.
입력값은 `arr = [1, -2, 3, 4, -9, 6]` 같이 숫자로만 구성된 배열이라고 가정해봅시다.

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
과제는 `arr`에서 요소들의 합이 최대인 부분배열을 찾아내는 것입니다.
우리가 해야 할 일은 요소의 총합이 최대인 `arr`의 부분 배열을 찾는 것입니다.

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
# 최대합 부분배열
# 최대합 부분 배열

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
let value = prompt("A number please?", 0);
let value = prompt("숫자를 입력해 주세요.", 0);

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
// 정지를 해야 할까요?
// 입력받는 것을 정지해야 하는 경우

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
- 숫자가 아닌 값, 빈 문자열, "Cancel"을 사용자가 입력하면 질문을 멈춥니다.
- 숫자가 아닌 값, 혹은 빈 문자열을 입력하거나 'Cancel' 버튼을 누르면 질문을 멈춥니다.

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
참고: 숫자 `0`은 유효한 숫자이므로, 사용자에게 질문을 멈추는 신호로 처리하지 마세요.
참고: 숫자 `0`은 유효한 숫자이므로, 사용자가 `0`을 입력하더라도 질문을 멈추지 마세요.

@javascript-translate-bot

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

@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Nov 22, 2019

작업 완료되면 #235 이슈 닫아주세요.

@SimEunJu
Copy link
Author

자세하게 리뷰해 주셔서 감사합니다!!
다음 번역에는 말씀하신 사항 유의하며 진행하도록 하겠습니다 😄
#235 이슈는 제가 만든 게 아니라서 어떻게 닫아야 할지 잘 모르겠습니다ㅠㅠ
방법 말씀해 주시면 바로 닫도록 하겠습니다!

@SimEunJu
Copy link
Author

/done

@Violet-Bora-Lee
Copy link
Member

제가 열어서 제가 닫아야하나봐요 ㅎㅎ
컴터 앞에 앉으면 바로 머지 진행하고 이슈도 닫겠습니다
기여해주셔서 감사합니다 👩

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.

몇가지 한국어 표기 관련 리뷰 달아보았습니다.

최종적으로 맞춤법 검사기 돌려주시길 바랍니다.

```

The code is actually a nested loop: the external loop over array elements, and the internal counts subsums starting with the current element.
위와 같은 알고리즘 사용하려면 중첩 반복문이 필요합니다. 외부 반복문에선 배열의 각 요소를 순회하고, 내부 반복문에선 각 요소부터 시작하는 부분 배열의 합을 계산하게 됩니다.
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
위와 같은 알고리즘 사용하려면 중첩 반복문이 필요합니다. 외부 반복문에선 배열의 각 요소를 순회하고, 내부 반복문에선 각 요소부터 시작하는 부분 배열의 합을 계산하게 됩니다.
위와 같은 알고리즘을 사용하려면 중첩 반복문이 필요합니다. 외부 반복문에선 배열의 각 요소를 순회하고, 내부 반복문에선 각 요소부터 시작하는 부분 배열의 합을 계산하게 됩니다.

let fruits = ["Apples", "Pear", "Orange"];

// push a new value into the "copy"
// 배열을 "복사"한 후, push 메서드를 이용해 새로운 값을 추가합니다.
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
// 배열을 "복사"한 후, push 메서드를 이용해 새로운 값을 추가합니다.
// 배열을 '복사'한 후, push 메서드를 이용해 새로운 값을 추가합니다.

if (partialSum < 0) partialSum = 0; // zero if negative
for (let item of arr) { // 배열의 각 요소을 위해
partialSum += item; // partialSum에 요소을 더합니다.
maxSum = Math.max(maxSum, partialSum); // 최대값을 기억합니다.
Copy link
Member

Choose a reason for hiding this comment

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

@javascript-translate-bot

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

@Violet-Bora-Lee
Copy link
Member

오랫동안 적체된 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.

4 participants

Comments