听起来像 ActiveRecord 在保存时弄乱了对象属性

Question

我在 locations 表中有一堆记录：

...
*************************** 8. row ***************************
        id: 8
      feed: http://feeds.bbc.co.uk/weather/feeds/rss/5day/world/4564.xml
...
*************************** 11. row ***************************
        id: 11
      feed: http://feeds.bbc.co.uk/weather/feeds/rss/5day/world/5007.xml
...

在这次迁移的帮助下：

  def self.up
    add_column :locations, :old_feed, :string
    Location.all.each do |l|
      l.old_feed = l.feed
      l.feed.sub!(/^.*?(\d+)\.xml$/, 'http://newsrss.bbc.co.uk/weather/forecast/\1/Next3DaysRSS.xml') # l.feed = l.feed.sub(...) does not make it any better
      l.save!
    end
  end

我想将它们变成

...
*************************** 8. row ***************************
        id: 8
      feed: http://newsrss.bbc.co.uk/weather/forecast/4564/Next3DaysRSS.xml
  old_feed: http://feeds.bbc.co.uk/weather/feeds/rss/5day/world/4564.xml
...
*************************** 11. row ***************************
        id: 11
      feed: http://newsrss.bbc.co.uk/weather/forecast/5007/Next3DaysRSS.xml
  old_feed: http://feeds.bbc.co.uk/weather/feeds/rss/5day/world/5007.xml
...

我得到的，

...
*************************** 8. row ***************************
        id: 8
      feed: http://feeds.bbc.co.uk/weather/feeds/rss/5day/world/4564.xml
  old_feed: http://newsrss.bbc.co.uk/weather/forecast/4564/Next3DaysRSS.xml
...
*************************** 11. row ***************************
        id: 11
      feed: http://feeds.bbc.co.uk/weather/feeds/rss/5day/world/5007.xml
  old_feed: http://newsrss.bbc.co.uk/weather/forecast/5007/Next3DaysRSS.xml
...

看起来像 feed 和 old_feed 不知何故被交换了，我不明白为什么。

Rails 2.3.4、MySQL 5.0。

附注
我敢打赌这是我在这里错过的一些愚蠢而明显的东西。

Answer 1

1. 您正在修改同一个对象（已经提到过）。
2. feed 属性未分配任何新内容。您正在就地修改返回值。

这些组合起来看起来就像你正在颠倒琴弦。有趣的是，结果实际上。 :-)

试试这个：

def self.up
  add_column :locations, :old_feed, :string
  Location.all.each do |l|
    l.old_feed = l.feed
    # Reassign the new feed and drop the ! from sub!() to return a new string.
    l.feed = l.feed.sub(/^.*?(\d+)\.xml$/,
      'http://newsrss.bbc.co.uk/weather/forecast/\1/Next3DaysRSS.xml')
    l.save!
  end
end

Answer 2

您正在此处修改同一对象：

l.old_feed = l.feed
l.feed.sub!(/^.*?(\d+)\.xml$/, 'http://newsrss.bbc.co.uk/weather/forecast/\1/Next3DaysRSS.xml') # l.feed = l.feed.sub(...) does not make it any better

它应该是：

l.old_feed = l.feed.clone
l.feed.sub!(/^.*?(\d+)\.xml$/, 'http://newsrss.bbc.co.uk/weather/forecast/\1/Next3DaysRSS.xml') # l.feed = l.feed.sub(...) does not make it any better