Rails中地理距离计算方法调用及报错问题求助

首先来看你遇到的两个核心问题：undefined method 'dst_line' for [#<Poi id: nil>]:Array 和疑惑 @poi 为何是nil，我们一步步拆解解决：

1. 为什么会出现undefined method 'dst_line' for Array？ #

find_by_sql方法的返回值始终是一个ActiveRecord对象数组，哪怕你的SQL只返回一条结果。你现在直接调用.dst_line在数组上，自然会报错。

另外，你的SQL查询并没有返回Poi的任何字段（只查了dst_line），所以返回的Poi对象所有属性都是nil——这也是报错里#<Poi id: nil>的原因，这个Poi对象是“空”的，只有你自定义的dst_line属性。

修复这个错误的快速方法 #

把模型里的方法修改为取数组的第一个元素，再获取dst_line：

def distance_along_track_to(poi2, track)
  distance_sql = <<-SQL
    SELECT ST_Distance(tr.path::geography, pta.lonlat::geography) + 
           ST_Distance(tr.path::geography, ptb.lonlat::geography) + 
           ST_Length(ST_LineSubstring( 
             tr.path, 
             least(ST_Line_Locate_Point(tr.path, pta.lonlat::geometry), ST_Line_Locate_Point(tr.path, ptb.lonlat::geometry)), 
             greatest(ST_Line_Locate_Point(tr.path, pta.lonlat::geometry), ST_Line_Locate_Point(tr.path, ptb.lonlat::geometry))
           )::geography) AS dst_line 
    FROM tracks tr, pois pta, pois ptb 
    WHERE tr.id = #{track.id} 
      AND pta.id = #{self.id} 
      AND ptb.id = #{poi2.id}
  SQL
  # 取数组第一个元素的dst_line，没有结果则返回nil
  result = Poi.find_by_sql(distance_sql).first
  result&.dst_line
end

2. 为什么@poi可能是nil？ #

看你控制器里的代码：

@poi = Poi.find_by id: 1

find_by在找不到对应记录时会返回nil，如果你的数据库里没有id=1的Poi，@poi就会是nil。你可以改成find（找不到会抛出异常，方便你排查），或者检查数据库里是否存在这条记录：

# 改成find，找不到会主动报错，便于调试
@poi = Poi.find(1)

3. 优化你的实现（避免SQL注入+更合理的查询方式） #

你当前的SQL直接拼接了track.id、self.id、poi2.id，这存在SQL注入风险，而且返回Poi对象也不符合逻辑（因为你查的是距离值，不是Poi数据）。我们可以用参数化查询+select_value来优化：

优化后的Poi模型方法 #

def distance_along_track_to(poi2, track)
  distance_sql = <<-SQL
    SELECT ST_Distance(tr.path::geography, pta.lonlat::geography) + 
           ST_Distance(tr.path::geography, ptb.lonlat::geography) + 
           ST_Length(ST_LineSubstring( 
             tr.path, 
             least(ST_Line_Locate_Point(tr.path, pta.lonlat::geometry), ST_Line_Locate_Point(tr.path, ptb.lonlat::geometry)), 
             greatest(ST_Line_Locate_Point(tr.path, pta.lonlat::geometry), ST_Line_Locate_Point(tr.path, ptb.lonlat::geometry))
           )::geography) AS dst_line 
    FROM tracks tr
    JOIN pois pta ON pta.id = ?
    JOIN pois ptb ON ptb.id = ?
    WHERE tr.id = ?
  SQL
  # 用select_value直接获取数值，返回单个值而不是数组/对象
  result = ActiveRecord::Base.connection.select_value(
    distance_sql,
    nil,
    [self.id, poi2.id, track.id]
  )
  result&.to_f # 转成浮点数方便使用
end

这里用select_value直接返回查询结果的单个值，比find_by_sql更高效，也避免了返回无用的Poi对象；同时用参数占位符?和参数数组来传递值，彻底避免SQL注入风险。

4. 视图调用的注意事项 #

在index视图里，你要确保@poi和@track都存在，才调用方法：

<% @pois.each do |poi| %>
  <div>
    <%= poi.name %>
    <%= poi.track_id %>
    <% if @poi.present? && @track.present? %>
      <%= @poi.distance_along_track_to(poi, @track) %>
    <% else %>
      无法计算距离：缺少起点或轨迹数据
    <% end %>
  </div>
<% end %>

5. 关于跨视图调用方法的初始需求 #

回到你最开始的需求：在poi.show计算起点A到当前poi的距离，在track.index计算每个poi的距离。你可以基于上面的模型方法进一步调整：

• 在poi.show控制器里，获取传入的起点A（比如从params里解析成Point），然后调用类似的方法计算距离
• 在track.index里，遍历每个track的pois，计算它们之间的距离

比如，如果你需要计算任意起点（不是Poi）到某个Poi的沿轨迹距离，可以在Track模型里加一个方法：

class Track < ApplicationRecord
  has_many :pois

  def distance_from_point_to_poi(start_point, poi)
    # start_point是ST_Point类型，比如从params里解析的坐标
    distance_sql = <<-SQL
      SELECT ST_Distance(path::geography, ?::geography) + 
             ST_Distance(path::geography, ?::geography) + 
             ST_Length(ST_LineSubstring( 
               path, 
               least(ST_Line_Locate_Point(path, ?::geometry), ST_Line_Locate_Point(path, ?::geometry)), 
               greatest(ST_Line_Locate_Point(path, ?::geometry), ST_Line_Locate_Point(path, ?::geometry))
             )::geography) AS dst_line 
    SQL
    start_point_geo = "SRID=4326;POINT(#{start_point.x} #{start_point.y})"
    poi_geo = "SRID=4326;POINT(#{poi.lonlat.x} #{poi.lonlat.y})"
    result = ActiveRecord::Base.connection.select_value(
      distance_sql,
      nil,
      [start_point_geo, poi_geo, start_point_geo, poi_geo, start_point_geo, poi_geo]
    )
    result&.to_f
  end
end

这样在poi.show里就可以调用@track.distance_from_point_to_poi(start_point, @poi)来计算距离了。

内容的提问来源于stack exchange，提问作者Ben