Ruby: 長くなったif文をコンパクトにまとめる方法を知りたい

Question

実現したいこと

長くなった後置if文を読みやい書き方にする方法について知りたい

困っていること
条件式が論理演算子で&&で横にひたすら繋げているので可読性が悪い
現状だと行数が長いためrubocopに怒られてしまいます。

処理内容
userテーブル内に住所に関するカラムがあり、未入力のユーザーに入力を促すロジックの処理です。

  def empty_address_info
    return default_link_color if f_postcode? && f_prefecture_code? && f_address_city? && f_address_street? && f_address_building? && facility?

    funspo_link_color
  end

Answer 1

def 条件を表す適切な命名
  f_postcode? && f_prefecture_code? &&
      f_address_city? && f_address_street? &&
       f_address_building? && facility?
end

# としておいて
def empty_address_info
    return default_link_color if 条件を表す適切な命名
    funspo_link_color
end

Comments

[nanase21]

ご教示いただきありがとうございます。

Answer 2

rubocp で警告が出ないようにしてみました。

Comments

[nanase21]

ご教示いただきありがとうございます。