0

我有一些使用 minmax 算法定位鸟类的简单代码。一切正常，但我发现我的编程不好，我相信有更好的解决方案。我在 RoR 方面没有那么有经验，但如果有人知道更好的方法来实现相同的解决方案，那么我很高兴 ;)。

我讨厌两个部分，我必须创建 4 个列表来确定不同组合的最大值或最小值（最小值-最大值算法的核心）和非常丑陋的 SQL hack。

谢谢！

 def index
# fetch all our birds
@birds = Bird.all
# Loop over the birds
@birds.each do |bird|
  @fixed = Node.where("d7type = 'f'")
  xminmax = []
  xmaxmin = []
  yminmax = []
  ymaxmin = []
  @fixed.each do |fixed|
    rss = Log.find_by_sql("SELECT logs.fixed_mac, AVG(logs.blinker_rss) AS avg_rss FROM logs
              WHERE logs.blinker_mac = '#{bird.d7_mac}' AND logs.fixed_mac = '#{fixed.d7_mac}' ORDER BY logs.id DESC LIMIT 30")
    converted_rss = calculate_distance_rss(rss[0].attributes["avg_rss"])
    xminmax.push(fixed.xpos + converted_rss)
    xmaxmin.push(fixed.xpos - converted_rss)
    yminmax.push(fixed.ypos + converted_rss)
    ymaxmin.push(fixed.ypos - converted_rss)
  end

  pos = {x: (xminmax.min + xmaxmin.max) / 2, y: (yminmax.min + ymaxmin.max) / 2}
  puts pos


end

结尾

4

1 回答 1

1

您可以做的两件事是（假设 Birds 可能是一张大桌子）将 Bird.all 更改为

Bird.find_each do |bird|
  ... code ...
end

这是循环许多表记录的更有效方式。

第二：@fixed = Node.where("d7type = 'f'")退出每个循环，因为它的查询不需要任何变量。将它放在循环之上，这样它就不会每次都执行。

第三（与其说是优化，不如说是更安全的代码）：您Log.find_by_sql看起来很简单，可以使用 active_record，您可以将其更改为：

Log.select('fixed_mac, AVG(logs.blinker_rss) AS avg_rss, blinker_mac').
    where(blinker_mac: bird.d7_mac, fixed_mac: fixed.d7_mac).
    order('id DESC').limit(30)

converted_rss = calculate_distance_rss(rss.first.avg_rss)

其他一切看起来都很好。

于 2013-07-22T16:03:02.760 回答