-
Notifications
You must be signed in to change notification settings - Fork 22
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
base: master
Are you sure you want to change the base?
[Draft] Add Array.prototype.[flat, flatMap] to 2.9 array page #48
Conversation
댓글 달아놓은 것 말고는 모두 좋아보입니다! 고생 많으셨습니다 지원님! 🙇 |
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` 메소드는 중첩된 배열 내 요소를 바깥 배열에 이어붙인 **새로운 배열**을 만듭니다. |
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.
다차원 배열 섹션 전에 중첩 배열 내용이 나오는 문제 -
그러고보니 다차원 배열은 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` 메소드는 중첩된 배열 내 요소를 바깥 배열에 이어붙인 **새로운 배열**을 만듭니다. |
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.
용어 관련하여 -
MDN에서 concatenate 라는 용어를 쓰고 있네용. 저는 적절한 것 같습니다!
다만 '중첩된 배열 내 요소' 라는 말이 조금 어려운 것 같아서, 조금 쉽게 '중첩된 배열의 요소' 로 바꾸는건 어떨까요?
arr.flat(); // [1, 2, 3, 4] | ||
``` | ||
|
||
`flat` 메소드는 인자에 1이상의 숫자를 넘겨주는 경우 숫자 값에 따라 여러번 중첩된 배열 요소도 이어 붙일 수 있습니다. |
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.
아주 사소한 것이긴 하지만 ㅎㅎㅎ
'여러번' -> '여러 번'
'두번' -> '두 번'
등으로 수정해주시면 좋을 것 같습니다!
관련 이슈
#34
변경 내용
(초안)
Array.prototype.[flat, flatMap]
내용을 2.9 배열 페이지에 추가합니다.PR Point
flat
메소드를 설명할 때 단어 선택이 어려워서 '이어붙인다' 라고 했는데 더 적절한 표현이 있을까요...?초안은 큰 고민하지 말고 올려보라고 조언주셔서 올려봤습니다!
글을 쓰는게 어렵네요 ㅠㅠ 승하님 존경스럽습니다. ㅎㅎ..