Rails ActiveDecoratorでconcerns配下にあるmoduleをdecorateしたい

前提・実現したいこと

app/models/concerns配下にあるmodule（app/models/concerns/fruit.rb）が肥大化していたので、ActiveDecoratorを用いてviewの描画でしか使われていないようなメソッドをデコレーター層に切り離したいです。

イメージとしては下記のようにapp/decorators/concerns/fruit_decorator.rbを作成しfruit.rbからfruit_decorator.rbにメソッドを移し利用できるようにしたいです。

app/decorators/concerns/fruit_decorator.rb

module FruitDecorator
  extend ActiveSupport::Concern

  def full_name
    "#{first_name} #{last_name}"
  end
end

※ fruit.rbには他にもたくさんのメソッドがありますが一例としてviewの描画でしか使われていないfull_nameメソッドのみ定義してみました。

現状

module

app/models/concerns/fruit.rb

module Fruit
  extend ActiveSupport::Concern

  def full_name
    "#{first_name} #{last_name}"
  end
end

moduleをincludeしているモデル

app/models/apple.rb

class Apple < ApplicationRecord
  include Fruit
end

app/models/grape.rb

class Grape < ApplicationRecord
  include Fruit
end

app/models/banana.rb

class Banana < ApplicationRecord
  include Fruit
end

試したこと

app/decorators/concerns/fruit_decorator.rb

module FruitDecorator
  extend ActiveSupport::Concern

  def full_name
    "#{first_name} #{last_name}"
  end
end

上記、前提・実現したいことに記載のようにfruit_decorator.rbを作成しfruit.rbからfruit_decorator.rbにfull_nameメソッドを移しました。
その後、下記のようにviewでfull_nameメソッドを使用しましたが、エラーが出てしまいdecorator層に定義したメソッドを呼び出すことができませんでした。

app/views/apples/index.html.erb

<% @apples.each do |apple| %>
  <%= apple.full_name %><br>
<% end %>

エラー文

ActionView::Template::Error - undefined method `full_name'

補足情報（FW/ツールのバージョンなど）

Rails 6.0.3.5
gem "active_decorator"

退会済みユーザー

2021/04/22 05:49 編集

> 上記、前提・実現したいことに記載のfruit_decorator.rbですが、Fruitはclassではなくmoduleなので当然full_nameメソッドは呼ばれませんでした。この表現が意味わからないのですがどう言う事ですか？どういう風に呼んでどう失敗したのですか？ > イメージとしては下記のようにapp/decorators/concerns/fruit_decorator.rbを作成しfruit.rbからfruit_decorator.rbにメソッドを移し利用できるようにしたいです。現状ですとこの質問に対して、すれば良いじゃんという回答以外思い浮かばない感じです。何が出来ないのかがちょっとわからないです。

Homaresan

2021/04/22 06:26

ご指摘誠にありがとうございます! どういう風に呼んでどう失敗したのか記載しました！

行動規範の内容に同意します

回答2件

すみません、ActiveDecoratorを使ったことが無いので自信ないんですが、

まず、公式の説明読みましたか？(https://github.com/amatsuda/active_decorator)

僕もさっきパッと見ただけなので間違ってるかもしれませんが、公式のexmpleを読む限り、特にIncludeとかしてないので、多分命名で自動判別してるんだと思います。

例えばBookDecoraterであればBookモデルにincludeされる様な形で実装されてる気がします。
なので、FruitDecoraterのメソッドをAppleモデルで使える様にするのは多分無理か、なんか明示的に設定しないとダメな気がします。

ActiveDecoraterを使わなければ良いと思うのですが、使わなければいけない理由あるんでしょうか？

投稿2021/04/22 08:28

退会済みユーザー

総合スコア0

退会済みユーザー

2021/04/22 08:31

あ、すみません追記ですが、各モデルのDecoraterを別々に定義してFruitDecoraterをincludeなりなんなりすれば出来る気がします。

Homaresan

2021/04/22 10:41

> なので、FruitDecoraterのメソッドをAppleモデルで使える様にするのは多分無理か、なんか明示的に設定しないとダメな気がします。 > 各モデルのDecoraterを別々に定義してFruitDecoraterをincludeなりなんなりすれば出来る気がします。やはりそうするしかなさそうですね、、、絶対にActiveDecoraterを使わなければならない理由は特に無いのでもう少し検討してみます。ありがとうございました。

shinoharat

2021/04/27 04:05 編集

軽くソースコード読んだ感じ、１・コントローラーのインスタンス変数をチェック２・もし ActiveRecord::Base のインスタンスなら、クラス名に "Decorator" を付与したモジュールを探す（例: Apple → AppleDecorator）という実装でした。（モデルクラス名のみ命名規則に影響し、コントローラー名やインスタンス変数名は無関係） https://github.com/amatsuda/active_decorator/blob/825588367448f823e28f56da613055a76f1e63a3/lib/active_decorator/decorator.rb#L68 なので benza さんの推測の通りですね。オプション指定でどうにかなる感じでもないです。

shinoharat

2021/04/27 04:22 編集

対応方法ですが、私も > 各モデルのDecoraterを別々に定義してFruitDecoraterをinclude で良いんじゃないかと思います。しかし、まだ回答受付中ということは、上記の方法だと解決しない問題があるのでしょうか？例えば、「includeが1行あるだけのデコレータを大量に定義するのに心理的な抵抗がある。gem "active_decorator" にこだわらないので、もう少し記述量を抑えられる方法を教えてほしい」とか「実際にモデルごとのデコレータを作ったけどやっぱり undefined method `full_name' になった」とか問題点についてもう少し詳しく教えてもらえると答えやすいです。

行動規範の内容に同意します