Rubyのコード説明とRuboCopのエラーについて

Question

お世話になります。
[Rails 4.x] Viewでアクションごとの条件分岐を綺麗に書く小技に記載されていたコードがどのように展開されているのかがよくわかりませんでした。
何が起こっているのか順を追って説明いただけないでしょうか？

before_action :request_path
def request_path
    @path = controller_path + '#' + action_name
    def @path.is(*str)
        str.map{|s| self.include?(s)}.include?(true)　← ここで何が起こっているのかがわからない
    end
end

上記アクションを使うと下記のように展開できるようです。

<% if @path.is('pages#index') %>
           ↓
<% if controller.controller_name == 'pages' && controller.action_name == 'index' %>

またこのコードをRails6でそのまま使おうとすると、RuboCopのエラーが出てしまいます。
こちらの回避方法もあわせて教えていただけると助かります。

Answer 1

またこのコードをRails6でそのまま使おうとすると、RuboCopのエラーが出てしまいます。

こちらの回避方法もあわせて教えていただけると助かります。

当該のコードが使いやすいと思っているのでなければ、普通に判定するのが正解です。クラスやモジュール、特殊なDSLで使うのであればともかく、一般のオブジェクトに特異メソッドを立てるようなコードを使うものではありません。

Comments

[no1knows]

redirect_to edit_company_path if controller_name == 'companies' && action_name == 'new'
だとif以下が長くてもう少し短くできないかと思って調べ始めたのですが、上記で問題ないのですね。
ありがとうございます！

Answer 2

個人的には可読性も悪いし、良い方法とは思わないかなぁ。
とりあえず、何が起きているか考えるには、分解すれば良いのではないでしょうか？

def request_path
  @path = controller_path + '#' + action_name

  # strは文字列または文字列の配列を想定している
  def @path.is(*str)
    #str.map{|s| self.include?(s)}.include?(true)

    # ここでのselfは@path＝"コントローラ#アクション"の文字列を指します

    # str="pages#index"の場合、[true] or [false]
    # str=["pages#index", "pages#new"]など配列の場合は、[true, false]など
　  # 一致していればtrue, 一致してないならfalse
　　match_list = str.map{|s| self.include?(s) }

    # どれか１つでもtrueならtrueを返すという事　　
　　match_list.include?(true)
  end
end

Comments

[no1knows]

分解することをすっかり忘れていました。
自分で動かしてみると、当たり前のことをやっていました。
アドバイスいただきありがとうございます！