-
-
Notifications
You must be signed in to change notification settings - Fork 224
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
ES2024: Object.groupBy/Map.groupBy #1716
Comments
@koralle 配列の章とMap/Setの章は別々のPRとして進めるのが良いと思います。(Assignするためにmentionしちゃいましたが、なぜかAssignはできなかった…) |
作業としては毎回こういうイメージのことをやっています。 ちょっと考えるところとしては、Web互換性のためにStatic Methodになってるというのを説明するべきかどうかという点ぐらいなのかなとは思います。(多分冗長になるので不要になりそうな気はしています) 参考
|
@azu |
配列の方を追加しました。 |
Mapの方もiterateの流れで入れるのが正しそう。 |
Mapを読んでいてショッピングカートの例がちょっと冗長な感じがしたので別のissueを作りました |
それぞれ追加したのでcloseします。 ES2025でSet MethodsとIterator Helpersがくるので、そこでMap/Setは大きく書き直しが必要になるかもしれません |
tc39/proposal-array-grouping: A proposal to make grouping of array items easier
Web互換性のために
Object.groupBy
になった。Map.groupBy(array, fn)
もある。グループ化は頻出ではあるので含めるのは妥当そうだけど、配置が難しい
Array.prototype.group
→Object.groupBy(array, cb)
Array.prototype.groupToMap
→Map.groupBy(array, cb)
という経緯。
どちらも配列を引数として受け取るようになってるので、配列関係。
データの変換的に見れば
Object.groupBy
は 配列をオブジェクトにするメソッドMap.groupBy
は配列をMapにするメソッドとして捉えられる。
Object.groupBy
は、 配列 to objectなので、Array.prototype.reduce の近くにあるのが良さそう。Map.groupBy
は、Map/Setにあるのが良さそう。Map.groupBy
はちょっと必要かは怪しい。TODO
Object.groupBy
: 配列の章に追加するMap.groupBy
は Map/Setの章に追加する?Originally posted by @azu in #1706 (comment)
The text was updated successfully, but these errors were encountered: