ネストオブジェクトの罠 RE: TypeScriptで「選択肢」の定義をEnum的な定数にまとめる

この記事は、静的解析とビルドサイズ面で興味深いテーマでした。記事として自分の考えを書きます。 注意。あくまでビルドパフォーマンス視点での最適化です。強い意図があって、自分のドメインモデリングの方法論ではこれが最適なんだ、というなら元コードの方法論を止めるつもりはありません。 元記事のコードを minify するとどうなるか 元コードを参考に、それにアクセスするサンプルコードを書いてみます。 const sortingOptions = { priceDesc: { id: "priceDesc", sort: "price", order: "desc", label: "価格が高い順", }, priceAsc: { id: "priceAsc", sort: "price", order: "asc", label: "価格が安い順", }, ratingDesc: { id: "ra

[yhoriz]

コードを書くときからオプティマイザの微妙な挙動によるパフォーマンス変化を意識する方がアンチパターンに思えるし、ましてやセキュリティの担保をオプティマイザに依存するのは「絶対アカン」でしょ

[mito15]

面白いけど、パフォーマンス落ちる例がコードジェネレーションでMBオーダーの定数オブジェクトがある場合だし、そのために$で区切ったenum書けってレビューついたら無視すると思う

[hase0510]

基本的に必要で使用されるから定義してるんだと思うので未使用時の挙動はほぼ影響ないんでは？ 何が使われるかわからないライブラリを作る時は意味ありそう。

[kiririmode]

“定数宣言はオブジェクトメンバでやらない”

[roshi]

コンパイラーが賢くなるか構文追加などで元の形から一気に最適化して欲しい。literal typeから取りうる値全部取り出せればもっといいんだけど。。

[jay-es]

ツリーシェイキング視点だとオブジェクトで定数定義するのは良くないという話

[mizchi]

書いた。基本的にネストオブジェクトに定数を詰めるべきではない。