Rubocop Style/StringConcatenation とか Style/FormatStringToken って重要ですか？

Question

10年前に作ったシステムをRails6化しています。ついでにrubocop free にすることを試みています。
ここで、
Rails.root + 'foo' + 'bar' が Style/StringConcatenation だと指摘されます。それはご無体な。。。

そもそも some_string_valiable + 'bar' をsome_string_valiable.concat('bar') にしろ、と言われるのが、はて、、と思っていたこともあり、
「コンパクトに書ける所、なぜわざわざ複雑にしなければならぬのか」改めて疑問になってきました。

Style/StringConcatenation　とか　Style/FormatStringToken　とか

コンパクトにに書けるところわざわざ複雑な書式にするように指摘しているのは何を防ぐためのものなのでしょうか

Answer 1

そもそも some_string_valiable + 'bar' をsome_string_valiable.concat('bar') にしろ

これ自体が誤解かと思います。あくまで、文字列連結と比較して変数展開を推奨するとのことです（Rubocop公式）。

Rails.root + 'foo' + 'bar'

この記法を知らなくて意図を誤解しそうになりました。Rails.root.join('foo', 'bar')などの形のほうが適当かと思います。

Comments

[maisumakun]

「some_string_valiable + 'bar'」でしたら「"#{some_string_variable}bar"」です。

[maisumakun]

それか、「Rails.root / 'foo' / 'bar'」のような書き方も考えられます。スッキリ書けるし、文字列連結と間違われる心配もありません。

[winterboum]

ありがとうございます
「some_string_valiable + 'bar'」でしたら「"#{some_string_variable}bar"」
で逃げられることは承知しています。
でもなぜ ruby がもともと持っている基本演算子を使うのが非推奨なのか、がわからないのです。
Rails.root / 'foo' / 'bar'　これは知りませんでした。結果入るのが "/"なのですから ＋ よりも感覚的に良いかも。

[maisumakun]

> でもなぜ ruby がもともと持っている基本演算子を使うのが非推奨なのか、がわからないのです。

文字列リテラルと連結すると、「リテラル」と「結果の文字列」の両方のインスタンスを生成する必要がありますが、変数展開であれば生成するのは結果の文字列だけ、というように効率的です。

[winterboum]

なるほど