在我的Rails应用程序中,创建业务时我有一个包含以下字段的表单：
<%= check_Box_tag(:default_company) %> <%= label_tag(:default_company,"Set Company as Default") %>
基本上当我创建一个企业时,如果他们选中此框,我需要它来运行类似下面的代码：
def set_default_company(company,user)
exists = DefaultCompany.find(user.id)
if exists
exists.update_attributes(company: company)
else
DefaultCompany.create(company: company,user: user)
end
end
在学习的过程中,我通常会在我的控制器中执行这些操作,但我正在尝试遵循最佳实践并使用胖模型,瘦的控制器,所以我想要使用这样的逻辑：
def create
@company = Company.new(params[:company])
if @company.save
if params[:default_company]
Company.set_default_company(@company.id,current_user.id,)
end
flash[:notice] = "Company was successfully created."
redirect_to @company
else
redirect_to new_company_path
end
end
这是我对是否使用类方法或实例方法,调用set_default_company感到困惑的地方.他们似乎都会工作,我看不到其中任何一方的好处.
除了给我任何关于使用哪种方法的信息之外,如果有人可以向我展示一个简短的写作实现,作为一个类方法与实例方法,它可以让我更好地理解为什么.
这是我写它们的方式：
def self.set_default_company(company,user) # Logic here end def set_default_company(company,user) # Logic here end
以这种方式写它我也没有看到任何好处.
解决方法
顾名思义,模型上的实例方法应该用于与用户的特定实例(调用该方法的实例)相关的逻辑/操作.因此,您可以考虑将用户的默认公司设置为用户的实例方法.类方法适用于不在模型的单个实例上运行的事物,或者您没有可用实例的情况.例如您可能有一个类方法来整理您的数据库,例如User.purge_expired_users,它不适用于单个用户对象.
例如
class User
def set_default_company(company)
exists = DefaultCompany.find(self.id)
if exists
exists.update_attributes(company: company)
else
DefaultCompany.create(company: company,user: self)
end
end
end
然后你的控制器方法看起来像：
def create
@company = Company.new(params[:company])
if @company.save
if params[:default_company]
current_user.set_default_company @company
end
flash[:notice] = "Company was successfully created."
redirect_to @company
else
redirect_to new_company_path
end
end
或者,您可以从另一个角度考虑关系,并将实例方法放在公司上,例如company.set_as_default_for(用户).