ActiveSupportの`presence`を使うときれいに書けるコードを検出するCop

rubocop / rubocop-jp

A place for RuboCop discussions in Japanese

55 stars 2 forks source link

Closed pocke closed 6 years ago

pocke commented 7 years ago

a.present? ? a : nil
a.blank? ? nil : a

上記のようなコードは、単にa.presenceで置き換えることが出来ます。また

a.present? ? a : b

はa.presence || bで置き換えることができます。

このようなコードをpresenceを使うようにうながすCopを作りたいです

hkdnet commented 7 years ago

タイトルのメソッド名が違いそうです？

pocke commented 7 years ago

タイトルのメソッド名が違いそうです？

ありがとうございます、修正しました 👍

wata727 commented 6 years ago

書いてみました。レビューお願いします 🙏 https://github.com/bbatsov/rubocop/pull/5208

pocke commented 6 years ago

It was merged and released. Thanks @wata727 !