一个模型应该创建其他模型的实例吗？ - 导轨 MVC

Question

我正在与我的一位编程朋友讨论一个模型是否应该能够创建其他模型。我们目前正在尝试创建一个人们可以互相对战的网站。假设我们有一个硬币翻转。这是我的 CoinFlipController 的示例

class CoinFlipsController < ApplicationController
  def join
    coinflip = CoinFlip.find(params[:id])
    participant = coinflip.join(current_user)
  end
end

每个硬币翻转可以有 2 名玩家加入。当玩家加入投币时，我们创建投币参与者。我希望控制器在模型 coinflip.join(current_user) 上调用 join。

class CoinFlip < ApplicationRecord
def join(user)
  if self.coin_flip_participants.count < 2
    participant = CoinFlipParticipant.create(coin_flip: self, user: user, client_seed: user.client_seed, face: :t)
  else
    raise CoinFlipFull.new("Coin Flip Full")
  end
end

但是我的朋友希望我像这样将逻辑放在 CoinFlipController 中，并从模​​型中删除 join 方法。

class CoinFlipsController < ApplicationController
def join
  @coinflip = CoinFlip.find(params[:id])
  if @coinflip.coin_flip_participants.count < 2
    @participant = CoinFlipParticipant.create(coin_flip: @coinflip, user: current_user, client_seed: current_user.client_seed, face: :t)
  else
    raise CoinFlipFull.new("Coin Flip Full")
  end
end

这给出了相同的结果，但是如果我想让另一个用户从另一个控制器加入，我必须在创建参与者之前实施相同的检查。这些解决方案中的任何一个是正确的还是有不同的解决方案？

Answer 1

我认为问题在于您的操作称为联接，因此它应该以实际联接模型为中心。我的意思是，就个人而言，我会这样做：

@coinflip = CoinFlip.find(params[:id])

@participant = CoinFlipParticipant.join(current_user, @coinflip)
raise CoinFlipFull.new('Coin Flip Full') unless @participant

然后在 CoinFlipParticipant 定义一个方法

def self.join(user, coinflip)
  return if coinflip.coin_flip_participants.count < 2

  create(coin_flip: coinflip, user: user, client_seed: user.client_seed, face: :t)
end

现在，控制器实际上调用动作来加入您想要的两件事，并且与这两件事相关的实际模型负责。