Ruby on Rails中Booking与Destination表字段关联配置是否正确？

现有配置错误点 #

• 关联方向颠倒：你的业务逻辑为「1个目的地可对应多个预订，1个预订仅对应1个目的地」，因此Booking侧应使用belongs_to关联Destination，而非你当前配置的has_many :destinations，错误的关联方向会导致所有关联查询结果异常。
• 字段命名不符合Rails默认约定：Rails默认外键命名规则为关联模型名_id，你当前使用Booking表的location字段、Destination表的location_id字段做关联，虽然可通过手动指定参数兼容，但后续维护成本高，也容易出现拼写错误。

调整方案 #

方案1：不修改现有表字段，仅修正模型关联 #

直接修正Booking模型的关联配置即可，Destination侧的has_many配置无需调整：

class Booking < ApplicationRecord
  # 注释或删除原有错误的has_many配置
  # has_many :destinations, :class_name => 'Destination', :foreign_key=> 'location_id'
  belongs_to :destination, class_name: 'Destination', foreign_key: 'location', primary_key: 'location_id', optional: true
  validates :name, :start_date, :end_date, :email, presence: true
end

注：如果业务要求预订必须关联目的地，删除optional: true参数即可，Rails会自动校验关联存在性。

方案2：修改字段适配Rails约定（更推荐） #

如果没有特殊业务要求，建议调整字段名适配Rails默认规则，后续不需要手动指定各种关联参数，维护成本更低：

1. 生成调整字段的迁移文件：

rails g migration AdjustBookingDestinationForeignKey

2. 编写迁移逻辑：

class AdjustBookingDestinationForeignKey < ActiveRecord::Migration[6.1]
  def change
    # 新增符合Rails约定的外键字段，自动加索引和外键约束
    add_reference :bookings, :destination, foreign_key: true
    # 同步原有location字段的数据到新外键字段
    execute "UPDATE bookings SET destination_id = location"
    # 删除冗余的旧location字段
    remove_column :bookings, :location
    # 如果Destination表的location_id没有特殊业务用途，直接删除用默认主键id做关联即可
    remove_column :destinations, :location_id
  end
end

3. 简化模型关联配置：

class Booking < ApplicationRecord
  belongs_to :destination, optional: true
  validates :name, :start_date, :end_date, :email, presence: true
end

class Destination < ApplicationRecord
  has_many :bookings
end

额外注意事项 #

• 如果你选择保留Destination表的location_id作为关联键，一定要给该字段加唯一索引，避免出现重复值导致关联异常。
• 关联调整完成后可以在Rails控制台测试：Booking.last.destination、Destination.last.bookings是否能返回正确的关联数据。

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