Objectに対してaget/asetを使うな、という話

Posted at 2017-07-04

ここ数日ClojureScript界隈でオブジェクトに対してaget/asetを使うな!
という話題が散見されており、何故そうなのかを調べてみた。

要するに?

aget/asetは元々配列に対する操作であり、オブジェクトに対しても動作するのは偶然
- jvmの方のClojureではこのような使い方は出来ない
Closure Libraryにオブジェクトを操作するための関数があるのでそちらを利用するべき
- goog.object/get
- goog.object/set
aget/asetでオブジェクトを操作しようとすると警告が出るようにするオプションがコンパイラに追加された
- aget/asetが動かなくなるわけでは無い
パフォーマンスクリティカルなケースに対応するための 'unsafe-get' マクロが追加された

扱うオブジェクトが':optimizations :advanced'(いわゆるアドバンスドコンパイル)の際に、
リネームの対象になる場合は以下の用にしてプロパティにアクセスする。
.-propertyはアドバンスコンパイルの際に適切にリネームされる。

(.-property an-object)

一方、扱うオブジェクトが外部ライブラリ等でアクセスするキーが予め定まっている場合、
'.-' でプロパティにアクセスを行うとアドバンスコンパイル時に意図していたキーとは異なる値にリネームされてしまう。
このようなケースにexternファイルをわざわざ書きたくない人は多数おり、

事実

(aget an-object "property")

は

(cljs.user.an-object["property"])

のようにコンパイルされ、動く。

agetの代わりに利用を推奨されているgoog.object/getは以下のように使う

(require '[goog.object :as o])
(o/get #js {:foo "bar} "foo")
=> "bar"