将逻辑转移到模型

Question

在我的 Rails 4 应用程序中，我开始将我的逻辑转移到模型中（根据 Fat Model 原则）。但是，我有点不确定如何最好地解决基础问题。

我的应用显示已登录用户的预订。在我的 BookingsController 中，我有一个显示已确认预订的操作：

def confirmed
  @bookings = Booking.where(:status => 3, :accommodation_id => current_user.accommodation.id).order('updated_at DESC')
end

但是，我尝试将逻辑移至控制器，以便 BookingsController 现在看起来像这样：

def confirmed
  @bookings = Booking.confirmed_bookings
end

Booking 模型如下所示：

def confirmed_bookings
  bookings = where(:status => 3, :accommodation_id => current_user.accommodation.id).order('updated_at DESC')
end

我收到 undefined method `confirmed_bookings' 错误，不确定自己做错了什么。对此问题的修复和一些简单的“入门”建议将不胜感激。

Answer 1

将你的作用域分成可重用的块：

scope :confirmed, ->{ where(:status => 3) } 
scope :for_user,  ->(user) { where(:accommodation_id => user.accommodation.id) }


def self.confirmed_bookings_for(user)
  confirmed.for_user(user).order('updated_at DESC')
end

注意，最好不要提及用户，因为这是为了住宿。

Answer 2

如果您将 confirm_bookings 方法更改为，这应该可以工作

self.confirmed_bookings
  ...
end

[编辑] 错过了当前用户的部分，该功能在模型中将不可用，一种解决方案是将其作为参数传递给确认预订功能：

class Booking < ActiveRecord::Base
  def self.confirmed_bookings_for (user)
    where(:status => 3, :accomodation_id => user.accomodation.id).order('updated_at DESC')
  end
  ...
end

然后在你的控制器中你可以写

@bookings = Booking.confirmed_bookings_for current_user