rails陷阱：has_many的build和create

这是我在实际中遇到的问题.
废话不多说，直接上代码。
在out里

 has_many :out_items
 def pick(os)
    update_attributes(os)
    out_items.each(&:change_placement)
 end
 def out_item_attributes=(out_item_attributes)
    out_item_attributes.each do |key,value|
      OutItem.find_by_id(key).update_attributes(value)
    end
  end

在out_item里

  has_many :out_placements
  def new_out_placement_attributes=(out_placement_attributes)
    out_placement_attributes.each do |out_placement_attribute|
      #out_placements.create(out_placement_attribute)
      op = out_placements.build(out_placement_attribute)
      op.save
    end
  end
  def change_placement
    self.out_placements.each(&:change_placement)
  end

在执行完out中update_attributes后，out_items.each(&:change_placement)，而这时debug代码到change_placement方法中，发现self.out_placements这里是empty array。但是此时，打开console，通过命令，你发现，out_placements不是空的，而且通过数据库也可以看到已经成功的将out_placement更新并且和out_item关联。out_placements明明有，为什么在这里却是empty array呢？

是哪里出问题？

经过一番折腾，猜想是不是在has_manyut_placements中，默认的out_placements使用了缓存？这里的out_placements用的缓存，而没有从数据库里重新查询。如果是这样，应该将缓存更新即可。通过分析代码。发现可能是这里出问题里。

      op = out_placements.build(out_placement_attribute)
      op.save

但是如果我将out_item里的代码改成（注释部分即改变部分）out_placements.create(out_placement_attribute)，发现debug到self.out_placements后，out_placements就不是空数组，运行正常了。

成功了。原来使用out_placements.create(out_placement_attribute)这样的操作，不但更新了数据库，而且更新了out_placements缓存，而直接的用op = out_placements.build(out_placement_attribute);op.save，虽然将out_placement保存到了数据库，但是此时再调用out_placements得到的还是之前缓存。

评论

6 楼 orcl_zhang 2010-03-12  

发现其实在rails里面，在表间关联中很多这样的例子。

self.my_steps.build(:name => name)
self.my_steps.each do |mstep|
   mstep.save(false)
end
ms = self.my_steps

如果这样写

self.my_steps.create(:name => name)
ms = self.my_steps

则返回的ms是不同的。

5 楼 orcl_zhang 2010-02-23  

out.out_items这个，第一句执行应该是要查询数据库的吧？第二次就不需要了。应该是rails的缓存。应该是类似

def current_user
  @current_user ||= User.find(session[:user_id])
end

这样的方式实现的。

引用

console中看到的东西不见得是真相哦。

这个还是真不知道。。
out.out_items; nil 这句确实没有查询。暂时还不知道原因。
有空了看下irb和inspect方法的文章。
挺晚了，睡觉。明天继续搞。

4 楼 darkbaby123 2010-02-22  

orcl_zhang 写道

引用

# 看你这种写法，可能是从前端用params批量传值过来，进行批量提交的，所以key可能是字符串，采用id.to_s比较

我很是不同意这点。就算key是字符串怎样？本来他也就是字符串。
你可以在console下试下，A.find('1')。

引用

     oi = out_items.detect {|o| o.id.to_s == key } 
     oi.update_attributes value 

至于这段代码，我觉得也没有必要这样写。

这点你误会了，使用find时，当然不管是字符串还是数字都可以。
我注明这点只是因为

o.id.to_s == key

这句可能看起来有点奇怪，所以对为什么要把o.id变成字符串来再比较，做的解释。

至于那样改写的原因，只是要利用out_items这个association找出来的对象，而不是OutItem.find，这也是我和你代码的不同，原因我在上面一个帖子里都写明了。

再说一句，console中看到的东西不见得是真相哦。我刚开始学习Rails被它欺骗了好一阵子。所以在console中测试不是最好的办法。
不信你可以分别试试下面两句，其中第一句是不会查询数据库的，但第二句会（其实真实运行环境中都是不会查数据库的，这是console的一点小把戏）：

out.out_items; nil
out.out_items

你可以在console中敲一句，然后在日志中看看生成了SQL语句没有。
最后，如果不明白，你可以找一下irb和inspect方法的文章。

3 楼 darkbaby123 2010-02-22  

<p>刚才仿造你的代码测试了一下，结果和我原先想的有些出入。</p>
<p>我的测试结果是，不管是create还是build，最后out_placements中都是空数组。</p>
<p>我下面会说明我的测试方法，并说明原因。</p>
<p>现在先说说LZ的结论，其实对Rails的association而言，不管是使用create还是build都会相应的改变association中的缓存。要是这点都没想到DHH也可以退休了。</p>
<p>LZ的错误还是在Out#out_item_attributes方法上</p>
<pre name="code" class="ruby">def out_item_attributes=(out_item_attributes) 
  out_item_attributes.each do |key,value| 
    # 这里不该用OutItem直接查。
    # 因为对这个OutItem查出来的对象（暂称oi）调用update_attributes的话，
    # 会间接调用oi的new_out_placement_attributes方法，虽然生成了OutPlacements，但更新的是oi.out_placements的缓存。
    # 而oi会在这个each循环结束后销毁，自然什么都记不住。实际上这个oi和最外层Out对象的out_items是没有任何关系的。
    OutItem.find_by_id(key).update_attributes(value) 
  end 
end</pre>
<p>所以对out对象调用如下方法：</p>
<pre name="code" class="ruby">out.pick ...</pre>
<p> 时，out.out_items的缓存根本就没动过，实际上在update_attributes的整个阶段，它都应该没有查询过数据库，缓存自然无从谈起，直到下面一句each。</p>
<p>这时的问题才刚出现，刚才pick方法的第一句update_attributes已经将out_placements写进数据库了（不管是create还是build），这时执行第二句out_items.each时，才进行数据库查询（association只在“需要”的时候才会执行数据库查询，这就不多说了），那查询出来的结果应该是正确的，绝不可能是空数组。</p>
<p><span style="color: #0000ff;">而LZ碰到的是空数组，我想是因为在调用out.pick之前，在其他地方遍历过out.out_items，从而提前查出了数据，放进了out_items的缓存，之后在update_attributes时，又没有更新缓存，才出现的问题。所以不管create还是build，应该都会报空数组的错误的。</span></p>
<p>要改过来很简单，就照我上一个帖子的代码改。</p>
<p> </p>
<p>下面附上本人的测试代码，和一些逻辑。我根据方法命名，猜测LZ做的这个功能是利用update_attributes来做批量数据提交。所以测试逻辑都是围绕这个来写的，如果错了，还请LZ指正。</p>
<pre name="code" class="ruby"># RAILS_ROOT/test/unit/out_test.rb
require 'test_helper'

class OutTest &lt; ActiveSupport::TestCase
 
  fixturesuts,ut_items

  def setup
    # 初始数据库环境如下：
    # outs表中有一条记录
    # out_items表中也有一条记录，从属于outs表中的那一条记录
    # out_placements表没有记录
    @out = outs(:o1)
    @out_attrs = {
     ut_item_attrs =&gt; {
        # 更新一个id为1的OutItem对象
        "1" =&gt; {
          :title =&gt; "out item 1",
          # 为out_placements写入两条记录
          :new_out_placement_attributes =&gt; [
            {:title =&gt; "placement 1"},
            {:title =&gt; "placement 2"}
          ]
        }
      }
    }
  end

  # 测试1，调用@out.pick，再看之后的数据库记录是不是增加了
  test "1" do
    # 直接执行pick，方法调用顺序如下：
    # Out#pick -&gt; Out#update_attributes -&gt; Out#out_item_attr= -&gt; OutItem#Update_attributes -&gt; OutItem#new_out_placement_attrs=
    @out.pick @out_attrs
    oi1 = @out.out_items[0]
    # 可以看到，写入了两条记录
    assert_equal oi1.out_placements.size, 2
  end

  # 测试2，和测试1唯一不同的就是在调用pick之前，先把@out.out_items的数据读出来，生成缓存了
  test "2" do
    # 这段代码没其他目的，就是提前把out_placements的数据读到内存中来
    # 此时out_placements是空数组
    @out.out_items.each { |oi| oi.out_placements.each {|op|} }
   
    @out.pick @out_attrs
    oi1 = @out.out_items[0]
    # 这里因为缓存原因，还是0条记录，其实数据已经写进去了
    assert_equal oi1.out_placements.size, 0
    # reload之后就显示真实的数字了
    assert_equal oi1.reload.out_placements.size, 2
  end
end</pre>
<p>然后是fixtures，很简单，就不解释了</p>
<pre name="code" class="yaml"># RAILS_ROOT/test/fixtures/outs.yml
# outs表，title属性是随意加的，要不要无所谓
o1:
  id: 1
  title: out

# RAILS_ROOT/test/fixtures/out_items.yml
# out_items表
oi1:
  id: 1
  title: out item 1
  out_id: 1</pre>
<p> 运行</p>
<pre name="code" class="console">ruby -I test ./test/unit/out_test.rb</pre>
 

2 楼 orcl_zhang 2010-02-22  

引用

# 看你这种写法，可能是从前端用params批量传值过来，进行批量提交的，所以key可能是字符串，采用id.to_s比较

我很是不同意这点。就算key是字符串怎样？本来他也就是字符串。
你可以在console下试下，A.find('1')。

引用

     oi = out_items.detect {|o| o.id.to_s == key } 
     oi.update_attributes value 

至于这段代码，我觉得也没有必要这样写。

引用

我是猜测你用的params批量传值，也不知道对不对……有空写写文字吧。个人觉得代码从来不是程序员的问题。

代码也就是实现这个功能。我在前面没有表述，让大家看着受累了。
多写写文字确实是不错，代码只是实现手段。
这点偶吸取教训，多写些文字。

1 楼 darkbaby123 2010-02-22  

我觉得，这确实是association的缓存原因，但你的写法也有点问题。
你把out文件的out_item_attributes=方法如下改一下试试：

# RAILS_ROOT/app/models/out.rb
def out_item_attributes=(out_item_attributes)  
  out_item_attributes.each do |key,value|  
    # OutItem.find_by_id(key).update_attributes(value)
    # 在association查出来的array中去找，而不是直接find_by_id
    # 看你这种写法，可能是从前端用params批量传值过来，进行批量提交的，所以key可能是字符串，采用id.to_s比较
    oi = out_items.detect {|o| o.id.to_s == key }
    oi.update_attributes value
  end
end

下面的代码还是用build，应该没问题的。
暂时想到这些，还没测试过，LZ自己试试吧
PS: 你这样直接上代码，而且只是一部分代码，很容易让人迷惑的。我是猜测你用的params批量传值，也不知道对不对……有空写写文字吧。个人觉得代码从来不是程序员的问题。