moduleからfind_or_initialize_byを使うとNoMethodError になる

Question

rails6.1.0

ほとんどのコントローラで共通に実行したい処理が発生したため、
初めてmoduleというものを使ってみました。

ちなみにその処理は、リクエスト度にuserのアクセス時間を記録するというものです。

module側

module UserAccess
  extend ActiveSupport::Concern

  included do
    helper_method :user_access_time
  end

  # アクセス時間を保存
  def user_access_time 
    if logged_in?
      user_access = UserAccess.find_or_initialize_by(user_id: current_user.id)
      user_access.update! user_id: current_user.id, remote_ip: request.remote_ip, updated_at: Time.now 
    end
  end

end

moduleを使うコントローラ側

class CategoriesController < ApplicationController
  include UserAccess
  before_action :user_access_time

  def index
    @categories = Category.all
  end

end
  

UserAccessモデル

class UserAccess < ApplicationRecord
end

これでCategoriesControllerのindexアクションを呼ぶと以下のエラーになりました。

NoMethodError in CategoriesController#index
undefined method `find_or_initialize_by' for UserAccess:Module

moduleについて一応調べて書いたつもりでしたが、理解できてないようです。
このような場合、どう記述するのでしょうか？

Answer 1

user_access = UserAccess.find_or_initialize_by(user_id: current_user.id)

この行は何をしたかったのでしょうか？

UserAccessは今実行しているこのモジュールですので、自分で実装しなければエラーメッセージの通りfind_or_initialize_byというメソッドは存在しません。

Comments

[maisumakun]

もしかして、モデル名もUserAccessだったのでしょうか。

その場合は同名でモジュールを作っても衝突して正常に動かないだけですので、モジュールの名前を変える必要があります。

[pecchan]

有難う御座います。
UserAccessは、UserAccessモデルです。

UserAccessテーブルに、すでにuser_idがあれば更新、
なければ新規追加をしたかったのです。

[pecchan]

あ、なるほど！仰る通りModule名と同じでした！

モジュール側の名前を変えたところ、エラーメッセージが変わりました。
NameError
uninitialized constant

[pecchan]

エラーメッセージで調べてみます。

[pecchan]

ファイル名も変えなきゃダメでした！
解消しました！有難う御座いました！！