Skip to content
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

[Draft] Add Array.prototype.[flat, flatMap] to 2.9 array page #48

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jiwonkirn
Copy link
Collaborator

@jiwonkirn jiwonkirn commented Feb 4, 2021

관련 이슈

#34

변경 내용

(초안) Array.prototype.[flat, flatMap] 내용을 2.9 배열 페이지에 추가합니다.

PR Point

  • 지금은 master 밖에 없어서 master를 target branch로 했는데요. 중간 브랜치 필요 없을까요?
  • 중첩 배열 설명이 페이지 맨 하단에 있어서 순서가 좀 꼬인 것 같습니다... 중첩 배열 설명을 위로 올리면 어떨까 하는데 리뷰 부탁드립니다ㅎㅎ..
  • 문장을 좀 더 쉽고 가독성있게 하면 좋을 것 같은데 리뷰 부탁드립니다. :)
    • ex) flat 메소드를 설명할 때 단어 선택이 어려워서 '이어붙인다' 라고 했는데 더 적절한 표현이 있을까요...?

초안은 큰 고민하지 말고 올려보라고 조언주셔서 올려봤습니다!
글을 쓰는게 어렵네요 ㅠㅠ 승하님 존경스럽습니다. ㅎㅎ..

브랜치 명에 오타가 있네요 ^^;

@seungha-kim seungha-kim marked this pull request as draft February 28, 2021 02:50
@seungha-kim
Copy link
Collaborator

댓글 달아놓은 것 말고는 모두 좋아보입니다! 고생 많으셨습니다 지원님! 🙇

@seungha-kim
Copy link
Collaborator

Fixes #34

@@ -391,6 +391,48 @@ const arr = [1, 2];
arr.concat([3, 4], [5, 6], [7, 8]); // [1, 2, 3, 4, 5, 6, 7, 8]
```

### flat

`flat` 메소드는 중첩된 배열 내 요소를 바깥 배열에 이어붙인 **새로운 배열**을 만듭니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

다차원 배열 섹션 전에 중첩 배열 내용이 나오는 문제 -

그러고보니 다차원 배열은 2차원, 3차원 같이 공간에 대한 정보를 다루려고 쓰는데 반해, 여기에서 말하는 중첩된 배열 요소는 배열에 아무렇게 요소가 들어있어도 된다는 느낌이네용

그래서 다차원 배열을 위로 끌어올리는 것보다는, 두 번째 코드 예제를 수정하는 쪽이 좋아 보입니다!

const empty = []; // 빈 배열
const numbers = [1, 2, 3]; // 숫자가 들어있는 배열
const strings = ['one', 'two', 'three']; // 문자열이 들어있는 배열
const objects = [{prop: 1}, {prop: 2}, {prop: 3}]; // 객체가 들어있는 배열
const arrays = [[1], [2], [3]] // <- 추가: 배열이 들어있는 배열
const mixed = [1, 'one', {prop: 1}, [1, 2, 3], null]; // <- 수정: 아무거나(?) 들어있는 배열

이런 느낌이면 어떨까요?

@@ -391,6 +391,48 @@ const arr = [1, 2];
arr.concat([3, 4], [5, 6], [7, 8]); // [1, 2, 3, 4, 5, 6, 7, 8]
```

### flat

`flat` 메소드는 중첩된 배열 내 요소를 바깥 배열에 이어붙인 **새로운 배열**을 만듭니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

용어 관련하여 -

MDN에서 concatenate 라는 용어를 쓰고 있네용. 저는 적절한 것 같습니다!

다만 '중첩된 배열 내 요소' 라는 말이 조금 어려운 것 같아서, 조금 쉽게 '중첩된 배열의 요소' 로 바꾸는건 어떨까요?

arr.flat(); // [1, 2, 3, 4]
```

`flat` 메소드는 인자에 1이상의 숫자를 넘겨주는 경우 숫자 값에 따라 여러번 중첩된 배열 요소도 이어 붙일 수 있습니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

아주 사소한 것이긴 하지만 ㅎㅎㅎ

'여러번' -> '여러 번'
'두번' -> '두 번'

등으로 수정해주시면 좋을 것 같습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants