ruby-on-rails - 有一个更好的方法吗？即更好地链接命名范围

Question

这段代码运行良好，但我正在查看它并认为它可能更清洁。也许有一种更惯用的 ruby​​ / rails 方式来做到这一点？顺序很重要，因为 member_of 范围必须在最后但在分页之前（返回集合而不是范围）

这样做的一个优点是很清楚发生了什么

@locations = Location.send(params[:type]) if type_sent_and_valid? #refine to a particular type if present
@locations = (@locations || Location.locatable).near(latlng_params) if latlng_sent? #refine to location

@locations = (@locations || Location).member_of(@interest_group.id).paginate(:page=>params[:page], :per_page=>20)

如果参数字符串是这样的：

?lat=50&lng=150&type=restaurant&page=1

然后它应该产生这个

Location.restaurant.near([50.0,150.0]).member_of(@interest_group).paginate(:page=>1, :per_page=>20)

Answer 1

清理它的一种方法是使用滑动范围机制，您可以使用相同的变量一次移动范围：

location_scope = Location

if (type_sent_and_valid?)
  location_scope = location_scope.send(params[:type])
end

if (latlng_sent?)
  location_scope = location_scope.locatable.near(latlng_params)
end

location_scope = location_scope.member_of(@interest_group.id)

@locations = location_scope.paginate(:page=>params[:page], :per_page=>20)

您可以根据需要添加其他条件。