0

我目前有一个涉及很多新类实例的系统，所以我不得不使用数组来分配它们，正如这里所建议的：创建和初始化具有顺序名称的类的实例

但是，每当出现新实例时，我都必须不断添加新实例，而不会覆盖现有实例。我现有代码的一些验证和修改版本可能是最佳选择吗？

这是我的代码，目前每次运行时，都会覆盖现有数据。如果状态发生更改，我希望状态被覆盖，但我也希望能够在其中永久存储一两个变量。

E2A：忽略全局变量，它们只是用于测试。

$allids = []
$position = 0 ## Set position for each iteration

    $ids.each do |x| ## For each ID, do
        $allids = ($ids.length).times.collect { MyClass.new(x)} ## For each ID, make a new class instance, as part of an array

        $browser.goto("http://www.foo.com/#{x}") ## Visit next details page

        thestatus = Nokogiri::HTML.parse($browser.html).at_xpath("html/body/div[2]/div[3]/div[2]/div[3]/b/text()").to_s ## Grab the ID's status

        theamount = Nokogiri::HTML.parse($browser.html).at_xpath("html/body/div[2]/div[3]/div[2]/p[1]/b[2]/text()").to_s ## Grab a number attached to the ID

        $allids[$position].getdetails(thestatus, theamount) ## Passes the status to getdetails

        $position += 1 ## increment position for next iteration
    end

E2A2：从我的评论中粘贴这个：

嗯，我只是在想，我首先将以前的值转储到另一个变量中，然后另一个变量获取新值，并遍历它们以查看是否有任何匹配以前的值。这是一种相当混乱的方法，我在想，self.create 会用 ||= 工作吗？– 乔 7 分钟前

4

1 回答 1

1

如果我理解正确，您需要为每个 ID 存储状态和金额，对吗？如果是这样，那么这样的事情会帮助你：

# I'll store nested hash with class instance, status and amount for each id in processed_ids var
$processed_ids = {}

$ids.each do |id|
  processed_ids[id] ||= {} #
  processed_ids[id][:instance] ||= MyClass.new(id)
  processed_ids[id][:status] = get_status # Nokogiri method
  processed_ids[id][:amount] = get_amount # Nokogiri method
end

这段代码做了什么：它只为每个 id 创建一次你的类的实例，但总是更新它的状态和数量。

于 2013-09-01T14:07:30.173 回答