我在尝试设置这些实例变量时做错了什么?

Question

当我调用Quote#scrape_quote时，我在尝试在我的Quote实例中设置和/或访问这些变量时似乎遇到了一些基本错误。我可以看到这些值被抓取并保存到scraped_values哈希中，当我调用quote.lives时，我只是无法访问变量，例如我得到nil.

我在这里做错了什么？

quote.rb

class Quote < ApplicationRecord 
require 'watir'
attr_accessor :lives, :benefit, :payment
def scrape_quote
rows = @browser.trs
quote_rows = rows[1..8]
scraped_values = {}
quote_rows.each do |row|
scraped_values[row.tds[0].text] = row.tds[1].text
end
@lives      = scraped_values[0]
@benefit    = scraped_values[1]
@payment    = scraped_values[2]
puts scraped_values
end
end

Answer 1

scraped_values是一个哈希而不是数组。您正在尝试像访问数组一样访问它。

使用row.tds[0].text中的任何内容来引用哈希：

h = {a:1,b:2}
h[:a]
=> 1 

Answer 2

一般情况

如果只想将哈希值按顺序分配给成员变量，则可以使用hash#values返回值的并行赋值，如下所示：

2.4.1 :001 > h = {}
=> {} 
2.4.1 :002 > h[:one] = 1
=> 1 
2.4.1 :003 > h[:two] = 2
=> 2 
2.4.1 :004 > h[:three] = 3
=> 3 
2.4.1 :005 > @one, @two, @three = h.values
=> [1, 2, 3] 
2.4.1 :006 > @one
=> 1 
2.4.1 :007 > @two
=> 2 
2.4.1 :008 > @three
=> 3 
2.4.1 :009 >

---

具体应用

您案例中的特定代码将变成：

class Quote < ApplicationRecord 
require 'watir'
attr_accessor :lives, :benefit, :payment
def scrape_quote
rows = @browser.trs
quote_rows = rows[1..8]
scraped_values = {}
quote_rows.each do |row|
scraped_values[row.tds[0].text] = row.tds[1].text
end
@lives, @benefit, @payment = scraped_values.values
puts scraped_values
end
end

Answer 3

构建哈希的惯用方法是使用map而不是each，并且不使用局部变量的预先声明。

scraped_values = quote_rows.map do |row|
[row.tds[0].text, row.tds[1].text]
end.to_h

Answer 4

而不是scraped_values[0]你需要这样的东西：scraped_values[scraped_values.keys[0]]，因为scraped_values不是数组，而 0、1、2 就像任何其他缺失的键一样，所以哈希返回nil。