rubocop -a の freeze 対策

Question

rubocop -a すると定数にfreeze がつけまくられます。
それ自体はかまわないのですが次の様なcaseで困っています。

Media = { a: '朝日', b: '毎日', c: '読売'}
Media.defulat = 'その他'

これが次のように修正されます

Media = { a: '朝日', b: '毎日', c: '読売'}.freeze
Media.defulat = 'その他'

すると2行目でエラーになります。

rubocop に freeze されてもエラーにならないようにする記述方法はありますでしょうか。

Answer 1

複雑な定数を作りたい場合、.tapを使う手があります（るりま）。

Media = { a: '朝日', b: '毎日', c: '読売'}.tap do |hash|
  hash.default = 'その他'
  hash.freeze
end

Comments

[winterboum]

なるほど、こういう時に tap を使うのですね。

[maisumakun]

なお、このようなシンプルなリテラル以外の代入にはStyle/MutableConstantは効きません。

Answer 2

Style/MutableConstantをdisableにすれば良いのではないでしょうか。

# rubocop:disable Style/MutableConstant
MEDIA = { a: '朝日', b: '毎日', c: '読売' }
# rubocop:enable Style/MutableConstant
MEDIA.defulat = 'その他'

Comments

[winterboum]

それをしてしまうと、みなつかなくなるので、それもどうかな、と思っていたのです。
ありがとうございます。

[otn]

いや、該当の1行だけdisableにすればいいかと。
MEDIA = { a: '朝日', b: '毎日', c: '読売' } # rubocop:disable Style/MutableConstant
と右側に書くことも出来ます。この場合はその左だけに適用。ちょっと見にくい気がしますが。

[winterboum]

そういう手もありましたか。
ありがとう御座います