Rails中require_dependency已加载文件但调用方法报NoMethodError问题求助

• 你定义的initialise_municipal是UniversalMethods类的实例方法，UsersController和该类没有继承/混入关系，无法直接调用该方法
• 额外问题：initialise_municipal内部直接调用了控制器专属的params对象，单独的自定义类实例无法直接访问该对象，就算你实例化调用也会触发新的报错
• 额外冗余逻辑：方法已经接收了user_params传参，第一行又重新赋值user_params = params[:user]，会覆盖传入的参数，属于无效冗余代码

方案1：使用Rails原生推荐的控制器Concern（最推荐） #

1. 在app/controllers/concerns目录下新建universal_methods.rb，代码改为模块形式：

module UniversalMethods
  extend ActiveSupport::Concern

  def initialise_municipal(user_params)
    municipal_via_id = user_params[:municipal_id]
    # [...] 剩余逻辑
  end
end

2. 在UsersController中直接引入该模块，不需要手动require_dependency，Rails会自动加载concerns下的文件：

class UsersController < ApplicationController
  include UniversalMethods

  def admin_create
    set_user_login_name
    user_params = params[:user]
    initialise_municipal(user_params)
    # [...] 剩余逻辑
  end
end

方案2：修改原有类调用逻辑（仅适配你当前的文件放置方式） #

如果坚持把文件放在controllers目录下用require_dependency引入，有两种修改方式：

方式A：实例化类后调用 #

修改initialise_municipal方法，移除内部对params的直接引用：

class UniversalMethods
  def initialise_municipal(user_params)
    municipal_via_id = user_params[:municipal_id]
    # [...] 剩余逻辑
  end
end

控制器中调用时先实例化：

require_dependency 'universal_methods'

class UsersController < ApplicationController
  def admin_create
    set_user_login_name
    user_params = params[:user]
    UniversalMethods.new.initialise_municipal(user_params)
    # [...] 剩余逻辑
  end
end

方式B：改为类方法调用 #

修改UniversalMethods的方法为类方法：

class UniversalMethods
  def self.initialise_municipal(user_params)
    municipal_via_id = user_params[:municipal_id]
    # [...] 剩余逻辑
  end
end

控制器中直接调用类方法：

require_dependency 'universal_methods'

class UsersController < ApplicationController
  def admin_create
    set_user_login_name
    user_params = params[:user]
    UniversalMethods.initialise_municipal(user_params)
    # [...] 剩余逻辑
  end
end

内容的提问来源于stack exchange，提问作者Jerome