如何用ruby重写每个循环？

Question

我有这个循环：

  stations = Station.where(...)
  stations.all.each do |s|
    if s.city_id == city.id
      show_stations << s
    end
  end

这很好用，但是由于循环所有数据，我认为这有点慢。 我尝试使用select重写它，如下所示：

show_stations << stations.select { |station| station.city_id == city.id}

但是与each版本相比，保存到show_stations数据量有所不同，因此数据的格式（数组/对象）不同。

有没有更好/更快的方法来重写循环版本？

Answer 1

最快的版本可能是用于找到关联对象的内置rails ActiveRecord方法。

因此，只要您的Station模型包含以下内容：

class Station < ActiveRecord::Base
  belongs_to :city

您的城市模型包含以下内容：

class City < ActiveRecord::Base
  has_many :stations

然后，rails自动生成方法city.stations ，该方法自动从数据库中获取包含该城市ID的站点。 它应该是相当优化的。
如果要使其更快，则可以在迁移中将add_index :stations, :city_id到表中，它将更快地检索。 请注意，这仅在您有大量要搜索的电台时才节省时间。

如果需要将其设置为数组，则可以在之后使用city.stations.to_a对其进行转换。 并且，如果您想进一步缩小范围，只需使用select方法并在Station.where(...)语句中添加之前要添加的条件。
（例如city.stations.to_a.select { |item| your_filter } ）

Answer 2

您还应该缓存查询结果，例如

    stations ||= Station.where("your where").where(:city_id => city.id)

Answer 3

Station似乎是一个活动记录模型。 如果是这种情况，并且不需要所有站点，则可以将city.id过滤器添加到where语句中。

现在遇到的问题是，您要将select返回的数组添加为show_stations的最后一项。 如果要让show_stations只包含与city.id匹配的电台， city.id使用show_stations = ...而不是show_stations << ... 如果希望show_stations包含已经包含的内容以及与city.id匹配的站点， city.id使用show_stations + stations.select { |station| station.city_id == city.id } show_stations + stations.select { |station| station.city_id == city.id } 。 （还有许多其他方法可以将两个数组加在一起。）

Answer 4

也许您需要在where子句中包含city参数：

stations = Station.where("your where").where(:city_id => city.id)

或相同

stations = Station.where("your where").where('city_id = ?', city.id)