使用辅助方法路由到 Index 的不同实例

Question

我有一个名为 BookingsController 的控制器，带有 bookings#index 操作。在索引操作中，有 2 个实例变量，@pending_bookings 和 @approved_bookings，它们通过 status 查询 Booking 对象。

 def index
     @pending_bookings = Booking.where(host_id:@user.id, 
    status:'pending') 
     @approved_bookings = Booking.where(host_id:@user.id, 
    status:'approved')
 end

我想根据用户点击的 link 将用户路由到不同的索引实例。基本上 bookings_path(@pending_bookings) 应该将用户路由到显示所有 pending_bookings 的索引页面，相反，bookings_path(@approved_bookings) 应该将用户路由到显示所有 approved_bookings.

的索引页面

在我看来，我有 2 个 link 应该分别将用户定向到每个路径。

 <%= link_to 'Pending Reservations', bookings_path(@pending_bookings)%>
 <%= link_to 'Approved Reservations', bookings_path(@approved_bookings)%> `

index.html.erb 文件：

  <%= booking_index_helper_path %>

包含一个嵌入式辅助方法，该方法应识别用户点击的路径并呈现正确的 Booking 对象。

这是用于识别用户选择的路径并呈现必要对象的（有缺陷的）逻辑：

pages_helper.rb:

 def booking_index_helper_path
    if bookings_path(@pending_bookings)
      render @pending_bookings
    elsif bookings_path(@approved_bookings)
      render @approved_bookings
    else bookings_path(@total_bookings)
      @total_bookings
    end
 end

我在辅助方法中放置了一个 binding.pry 以确认它被命中（确实如此）。出于某种原因，当我单击 link 将我定向到正确的对象时，第一个条件总是得到满足。编写此条件以识别用户选择的路径的更好方法是什么？

Answer 1

看起来你正在以一种比你需要的更复杂的方式来解决这个问题。为什么不只是有一个像这样的索引：

 def index
   #Rails autoescapes this string so no fear of sql injection using user supplied strings
   @bookings = Booking.where(host_id:@user.id, status: "#{params[:status]}")
 end

然后使用 link 比如：

 <%= link_to 'Pending Reservations', bookings_path(status: 'pending')%>
 <%= link_to 'Approved Reservations', bookings_path(status: 'approved')%> `

现在您的视图可以只处理 @bookings 而不必关心 @bookings 的类型，因为这是由您的控制器中的逻辑完成的。这是最低限度，但您应该养成向控制器添加错误消息等的习惯，因此请考虑这样做：

 def index
   if params[:status].present?
     #Rails autoescapes this string so no fear of sql injection using user supplied strings
     @bookings = Booking.where(host_id:@user.id, status: "#{params[:status]}")
     flash[:success] = "#{params[:status].titleize} Bookings loaded."
     redirect_to whatever_path
   else
     flash[:error] = "Something went wrong"
     redirect_to some_path
   end
 end

使用辅助方法路由到 Index 的不同实例

Routing to different instances of Index using a Helper Method

ruby-on-rails

ruby

controller

helper

link-to