user_helper.rbのメソッドをcareer_decorator.rbに移動する方法

Question

もともとuser_helper.rbに記述していたメソッドを新しく作成したcareer_decorator.rbに移動したいのですが、undefined method `term' for #Career:0x007feba8b82130のエラーがでます。
いくら考えて、実行しても改善できないので質問させていただきました。
以下は現状のコードです
career_decorator.rb

module CareerDecorator

  def term
    start_date = l(start_date, format: :term)

    if end_date.blank?
      "#{start_date} - #{t('date.formats.ongoing')}"
    else
      "#{start_date} - #{l(end_date, format: :term)}"
    end
  end

  def distance_of_term
    if end_date.blank?
      t('datetime.distance_in_words.ongoing')
    else
      distance_of_time_in_words(start_date, end_date)
    end
  end

end

app/views/users/user.html.slim

 - @user.careers.order('start_date desc, end_date desc').each do |career|
          tr
            td
              p = link_to career.term, edit_user_career_path(user_id: @user, id: career)
              p = career.distance_of_term
            td = career.title

user_helper.rb（termとdestance_of_termメソッドをcareer_decorator.rbに移動したい）

module UsersHelper

  def term(career)
    start_date = l(career.start_date, format: :term)

    if career.end_date.blank?
      "#{start_date} - #{t('date.formats.ongoing')}"
    else
      "#{start_date} - #{l(career.end_date, format: :term)}"
    end
  end

  def distance_of_term(career)
    if career.end_date.blank?
      t('datetime.distance_in_words.ongoing')
    else
      distance_of_time_in_words(career.start_date, career.end_date)
    end
  end

careerがtermメソッドを持っていないことはなんとなくわかっていますがどうしたら良いのかわかりません。

自分なりの考え

• helperからdecoratorに該当メソッドを移動する時にコピペだけだとだめっぽい=>decoratorのメソッドを変更するのか
• モデルのアソシエーション操作が必要なのか?
• decorator.rbは手打ちではなくコマンドで作成しないといけないのか？？=> rails g decorator careerとやったらgem周りでエラーが出ました
• careerオブジェクトがtermメソッドを持っていない
• users_controller.rbのshowにcareerのインスタンスを作成する必要がありそう

ヒントいただければ幸いです。

Answer 1

users/show.html.slimで

@user.careers.order('start_date desc, end_date desc').each do |career|
tr
td
p = link_to ActiveDecorator::Decorator.instance.decorator(career).term, edit_user_career_path(user_id: @user, id: career)
p = career.distance_of_term

のようにするだけでエラー回避できました。
追加したActiveDecorator::Decorator.instance.decorator(career).termはdecoratorを適用させますよ!という役割をしているそうです。
これでusers/showがcareerのdecoratorのメソッドを呼んでくれるようになりました。

Answer 2

ここで呼ばれているuser.careerは、モデルクラスではないかと。
(user has_many :careersと想像)
ヘルパモジュールに何を記述しても、モデルクラス自体には何の影響も及ぼさないのでは?

モデルのメソッドをくくり出したいのであれば、どこかにモジュールを書いて、それをモデルクラスでincludeする必要があるでしょう。

module CareerDecorator
 
  def term
    start_date = l(self.start_date, format: :term)

    if self.end_date.blank?
      "#{start_date} - #{t('date.formats.ongoing')}"
    else
      "#{start_date} - #{l(self.end_date, format: :term)}"
    end
  end

  def distance_of_term
    if self.end_date.blank?
      t('datetime.distance_in_words.ongoing')
    else
      distance_of_time_in_words(self.start_date, self.end_date)
    end
  end

end

class Carrier < ActiveRecord::Base
  include CareerDecorator
  # これでcareerインスタンスにtermメソッドとdistance_of_termメソッドが追加される
  # 以下略
end

ほとんど想像で書いているので、外していたらすみません。

Comments

[mikan1110]

回答いただきましてありがとうございます。そして返信遅れてしまい、申し訳ありませんでした。
さきに自分の解決方法を先に述べますと。
users/show.html.slimの
```
@user.careers.order('start_date desc, end_date desc').each do |career|
tr
td
p = link_to ActiveDecorator::Decorator.instance.decorator(career).term, edit_user_career_path(user_id: @user, id: career)
p = career.distance_of_term
```
のようにするだけでエラー回避できました。
追加したActiveDecorator::Decorator.instance.decorator(career).termはdecoratorを適用させますよ!という役割をしているそうです。
これでusers/showがcareerのdecoratorのメソッドを呼んでくれるようになりました。

回答頂いたCareerクラスでDecoratorをincludeすることによってcareerインスタンスにtermメソッドを追加する方法は想像していませんでしたので、勉強になりました。
ありがとうございました。