Rails:Bad Associations?[has_many,through]如何测试是否有效



我正在努力解决数据模型中的一个问题。我有以下型号:

class User < ActiveRecord::Base
...
  has_many :claims #user-claims
  has_many :claims, through: :rulings, as: :commissars
...
end
class Claim < ActiveRecord::Base
 ...
  belongs_to :user
  has_many :users, through: :rulings, as: :commissars
 ...
end
class Ruling < ActiveRecord::Base
    belongs_to :user
    belongs_to :claim
end

错误:

undefined method `commissars' for #<Claim:0xc5ac090>

型号说明:

用户可以编写索赔(索赔属于一个用户),用户可以扮演政委的角色来裁决索赔(每个索赔的最大政委数量=3)。

有什么办法可以解决这个问题或改善关系吗?

这个域模型需要一些非常复杂的关系,所以第一次尝试时没有得到它并不丢人。

让我们从用户和索赔开始:

class User < ActiveRecord::Base
  has_many :claims, foreign_key: 'claimant_id',
                    inverse_of: :claimant
end
class Claim < ActiveRecord::Base
  belongs_to :claimant, class_name: 'User',
                        inverse_of: :claims
end

这是一个非常基本的一对多关系,有一个转折点。由于User将有一组关系要声明,我们将该关系称为默认user以外的关系,以便定义关系的性质。

class_name: 'User'选项告诉ActiveRecord加载类User,并使用它来确定要查询的表以及返回结果的类。当类名称不能直接从关联的名称派生时,就需要它。这个选项应该是一个字符串,而不是一个常量,这是因为Rails延迟解析类依赖关系的方式。

现在让我们添加政委角色。我们将使用ruling作为联接表:

class Ruling < ActiveRecord::Base
  belongs_to :claim
  belongs_to :commissioner, class_name: 'User'
end

请注意,这里我们有一个与User的关系,为了清楚起见,我们称之为commissioner。现在我们将关系添加到Claim:

class Claim < ActiveRecord::Base
  belongs_to :claimant, class_name: 'User',
                        inverse_of: :claims
  has_many :rulings
  has_many :commissioners, through: :rulings
end

然后我们需要在用户端设置关系:

class User < ActiveRecord::Base
  has_many :claims, foreign_key: 'claimant_id',
                    inverse_of: :claimant
  # rulings as claimant
  has_many :rulings, through: :claims
  has_many :rulings_as_commissioner, class_name: 'Ruling',
                                     foreign_key: 'commissioner_id'
  has_many :claims_as_commissioner, through: :rulings_as_commissioner,
                                    source: :claim
end

请注意source: :claim选项,其中我们告诉ActiveRecord我们希望从联接表中选择哪一方。

当然,为了实现这一点,我们需要正确设置列和外键。这些迁移是从头开始创建表,但您可以很容易地重写它们来更改现有的表:

class CreateClaims < ActiveRecord::Migration
  def change
    create_table :claims do |t|
      t.belongs_to :claimant, index: true, foreign_key: false
      t.timestamps null: false
    end
    # we need to setup the fkey ourself since it is not conventional
    add_foreign_key :claims, :users, column: :claimant_id
  end
end
class CreateRulings < ActiveRecord::Migration
  def change
    create_table :rulings do |t|
      t.belongs_to :claim, index: true, foreign_key: true
      t.belongs_to :commissioner, index: true, foreign_key: false
      t.timestamps null: false
    end
    add_foreign_key :rulings, :users, column: :commissioner_id
    add_index :rulings, [:claim_id, :commissioner_id], unique: true
  end
end

最大政委人数=每次索赔 3人

这实际上不是关联的一部分,而是通过添加验证或关联回调来强制执行此规则。

class Ruling < ActiveRecord::Base
  # ...
  validate :only_three_rulings_per_claim
  private 
    def only_three_rulings_per_claim
      if claim.rulings.size >= 3
        errors.add(:claim, "already has the max number of commissars")
      end
    end
end

参见:

  • Rails指南:活动记录迁移
  • 导轨:has_many though:关系

首先,我建议您回去仔细阅读本指南,因为我相信您从根本上误解了很多东西。例如,as:选项并不表示角色,而是表示多态联接的存在。此外,不能在同一个模型上声明has_many :claims两次。不管怎样,去再读一遍。

但是,对于你的问题——一种功能性但有点不雅的方法可能看起来像:

class User < ActiveRecord::Base
  ...
  has_many :claims 
  has_many :claim_commissars, foreign_key: "commissar_id"
  has_many :commissar_claims, through: :claim_commissars, class_name: "Claim" 
  #                                                     ^^^^^^^^^^^^^^^^^^^^^
  #                                                     this bit may be wrong
  ...
end
class Claim < ActiveRecord::Base
  ...
  belongs_to :user
  has_one    :ruling
  has_many   :claim_commissars
  has_many   :commissars, through: :claim_commissars
  ...
end
class ClaimCommissar < ActiveRecord::Base
  ...
  belongs_to :claim
  belongs_to :commissar, class_name: "User"
  ...
end
class Ruling < ActiveRecord::Base
  ...
  belongs_to :claim
  belongs_to :commissar, class_name: "User"
  ...
end

你需要在代码中强制执行"最多3个政委"。

这还没有经过测试,你可能需要摆弄它才能让它继续运行。但是,希望它能让你朝着更好的方向前进。

祝你好运!

最新更新