13

我有一个字典类，希望能够使用“添加”方法将键（作为关键字）和值（作为定义）推送到空哈希中。我不明白如何从语法上写出来。我也包含了一个 RSPEC 文件。

红宝石：

 class Dictionary
     attr_accessor :keyword, :definition
     def entries
          @hash = {}
     end
     def add(options)
          options.map do |keyword, definition|
               @hash[keyword.to_sym] = definition
          end
     end
 end

规格：

 require 'dictionary'

   describe Dictionary do
   before do
       @d = Dictionary.new
   end
   it 'can add whole entries with keyword and definition' do
       @d.add('fish' => 'aquatic animal')
       @d.entries.should == {'fish' => 'aquatic animal'}
       @d.keywords.should == ['fish']
   end

任何帮助表示赞赏。谢谢！

更新： 感谢 Dave Newton 的回复。我使用了您的代码并收到此错误：

错误：

 *Failure/Error: @d.keywords.should == ['fish']
 NoMethodError:
 undefined method `keywords' for #<Dictionary:0x007fb0c31bd458 
 @hash={"fish"=>"aquatic animal"}>*

当我使用 @hash[word.to_sym] = defintion 将 'word' 转换为符号时出现不同的错误

 *Failure/Error: @d.entries.should == {'fish' => 'aquatic animal'}
   expected: {"fish"=>"aquatic animal"}
        got: {:fish=>"aquatic animal"} (using ==)
   Diff:
   @@ -1,2 +1,2 @@
   -"fish" => "aquatic animal"
   +:fish => "aquatic animal"*
4

2 回答 2

24

Dictionary在's中实例化您的哈希initialize：

class Dictionary
  def initialize
    @hash = {}
  end

  def add(defs)
    defs.each do |word, definition|
      @hash[word] = definition
    end
  end
end

现在你没有哈希，直到你打电话entries，你没有。

entries应该返回现有的哈希，而不是创建一个新的。

keywords应该返回哈希的键。

keyword和不需要访问器definition。这样的单数项在字典类中是没有意义的。您可能想要这样的东西lookup(word)返回一个定义。

此外，您将单词转换为符号，但我不知道为什么——特别是因为您在规范中使用了字符串键。选择一个，尽管我不相信这是符号增加价值的情况。

请仔细考虑变量命名以提供尽可能多的上下文。

于 2013-02-22T22:31:59.207 回答
8

查看您的 Rspec，您似乎需要此设置。

class Dictionary
  def initialize
    @hash = {}
  end

  def add(key_value)
    key_value.each do |key, value|
      @hash[key] = value
    end
  end

  def entries
     @hash
  end

  def keywords
     @hash.keys
  end

end

不要key.to_sym在 add 方法中使用，只需key

为了灵活地添加方法，我可以返回 self 对象以继续添加。

def add(key_value)
  key_value.each do |key, value|
    @hash[key] = value
  end
  self
end

所以，我现在可以做到了。

@d.add("a" => "apple").add("b" => "bat")
于 2013-02-23T00:12:54.477 回答