这段代码的验证错误怎么可能?

问题描述 投票:1回答:2

错误是ActiveRecord::RecordInvalid: Validation failed: Route must exist

这是代码:

new_route = Route.create!(new_route_data)

new_points.each_with_index do |point, index|
  new_point_data = { route_id: new_route.id,
                     latitude: point[0],
                     longitude: point[1],
                     timestamp: point[2] }
  new_point = Point.create!(new_point_data)
end

正在报道new_point = Point.create!(new_point_data)线。

相关细节:

  • 正如您在上面看到的那样,此代码在单个Sidekiq工作程序中运行(因此,不在一个工作程序中创建Route,而在另一个工作程序中创建Points - 这是全部内联的)
  • routes表几乎有3M记录
  • points表有大约2.5B的记录
  • Point模型包含belongs_to :route, counter_cache: true
  • Route模型没有验证
  • 如果它是相关的,Route模型确实包含belongs_to :user, counter_cache: true
  • users表中只有大约5k的记录

软件版本:

  • Rails 5.1.5
  • Ruby 2.5.0
  • PostgreSQL 9.6.7
ruby-on-rails ruby activerecord
2个回答
0
投票

首先,您的代码没有意义。您正在迭代new_point并将new_point分配给块中的某个值。所以我假设你的意思是迭代一些名为data_points的集合

试试这个。

在路线模型中

has_many :points

然后:

new_route = ...
data_points.each do |point|
  point_data = {latitude: point[0], ...}   # do not include route_id
  new_route.points.create(point_data) # idiomatic
end

`你不需要索引,所以不要使用each_with_index。


0
投票

如果没有看到Point模型中有哪种类型的验证,很难说出问题所在。

我的猜测是你在point.rb中的验证是:

validates :route, presence: true

使用ActiveRecord关系,您可以使用此快捷方式来避免显式分配route_id:

new_point_data = { latitude: point[0],longitude: point[1], timestamp: point[2] }
new_route.points.create!(new_point_data)

其中new_point数据没有route_id。

您还应该重命名要在块中分配的new_point,因为您正在编写正在迭代的数组。

© www.soinside.com 2019 - 2024. All rights reserved.