rubyで正規表現にマッチしたら代入を簡潔に書きたい

Question

いつもこうやって書いてますが一行でかけるのでしょうか？

price = nil
if m = "1,000円".match(/([0-9,]+)円$/)
  price = m[1].gsub(/,/,'')
end

宜しくお願いします

Answer 1

String#matchおよびRagexp#matchにブロックを渡すと、マッチしたときだけ評価して最後のブロックの値を返します。
参考: instance method Regexp#match - Ruby リファレンスマニュアル
この動作を利用することで、次のように書けます。

price = "1,000円".match(/([0-9,]+)円$/) { |m| m[1].gsub(/,/,'') }

他にもRuby 2.3以上からは、&.演算子を使って次のようにも書けます。(2.2以下の場合でもActiveSupportのtry!を使って似たようなことが可能です)

price = "1,000円".match(/([0-9,]+)円$/)&.[](1)&.gsub(/,/,'')

Comments

[退会済みユーザー]

ありがとうございました！&.演算子初めて知りました。

Answer 2

(m = "1,000円".match(/([0-9,]+)円$/)) && (price = m[1].gsub(/,/,''))

すぐに思いつく単純な方法で、
なるべく元のコードをそのままにして書きました。

一行にはなりましたが、可読性が悪いので、
ここまで短縮する必要はないと私は思います。

『リーダブルコード』でも、
行数より理解にかかる時間を短くせよ、
という意味のことを言っていたと思います。

Comments

[退会済みユーザー]

ありがとうございました！