ruby-on-rails – 为了瘦小的控制器,轨道型号是否应该关注其他型号？

我到处读到业务逻辑属于模型而不是控制器但是限制在哪里？
我正在玩个人会计应用程序.

Account
Entry
Operation

创建操作时,只有创建相应的条目并将其链接到帐户才有效,以便操作平衡,例如购买6件装：

o=Operation.new({:description=>"b33r",:user=>current_user,:date=>"2008/09/15"})
o.entries.build({:account_id=>1,:amount=>15})
o.valid? #=>false
o.entries.build({:account_id=>2,:amount=>-15})
o.valid? #=>true

现在,在基本操作的情况下向用户显示的表格被简化以隐藏条目细节,根据用户请求的操作类型(默认帐户 – >权益来计算,花费资产)从5中默认选择帐户 – >费用,赚取收入 – >资产,借入负债 – >资产,支付债务资产 – >负债……)我想要从默认值创建的条目.

我还希望能够创建更复杂的操作(超过2个条目).对于第二个用例,我将有一个不同的形式,其中暴露了额外的复杂性.第二个用例使我无法在操作中包含借方和贷方字段并删除Entry链接.

哪种形式最好？像我一样在SimpleOperationController中使用上面的代码,或者在Operation类上定义一个新方法,这样我就可以调用Operation.new_simple_operation(params [：operation])

是不是打破了关注点的分离,实际上是从Operation类创建和操作Entry对象？

我不是在寻找关于我扭曲的会计原则的建议:)

编辑 – 似乎我没有太清楚地表达自己.
我不太关心验证.我更关心创建逻辑代码应该去哪里：

假设控制器上的操作被称为花费,当使用花费时,params散列将包含：金额,日期,描述.借方和贷方账户将从被调用的操作中派生出来,但之后我必须创建所有对象.拥有它会更好吗？

#error and transaction handling is left out for the sake of clarity
def spend
  amount=params[:operation].delete(:amount)#remove non existent Operation attribute
  op=Operation.new(params[:operation])
  #select accounts in some way
  ...
  #build entries
  op.entries.build(...)
  op.entries.build(...)
  op.save
end

或者在操作上创建一个方法,使上面看起来像

def spend
  op=Operation.new_simple_operation(params)
  op.save
end

这肯定会给一个更薄的控制器和一个更胖的模型,但随后该模型将创建和存储其他模型的实例,这是我的问题所在.

解决方法

but then the model will create and store instances of other models which is where my problem is.

这有什么问题？

如果您的“业务逻辑”声明操作必须具有一组有效的条目,那么操作类肯定没有任何错误可以了解并处理您的Entry对象.

如果你把它拿得太远,你只会遇到问题,让你的模型操纵他们不需要知道的东西,比如EntryHtmlFormBuilder或其他:-)