2013-02-22 34 views
9

我有一个字典类,希望能够通过'add'方法将键(作为关键字)和值(作为定义)推送到空的哈希中。我不明白如何在语法上编写它。我也包含了一个RSPEC文件。如何将键和值推入空的哈希w/Ruby?

红宝石:

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 

Rspec的:

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 

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

更新: 谢谢戴夫牛顿的回复。我用你的代码,并得到这个错误:

错误:

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

我得到一个不同的错误,当我使用@hash [word.to_sym] =确定指标

转换“字”成符号
*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"* 

回答

18

实例化Dictionaryinitialize你的哈希:

class Dictionary 
    def initialize 
    @hash = {} 
    end 

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

现在你没有散列,直到你拨打entries,你没有。

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

keywords应该返回散列的密钥。

您不需要访问器keyworddefinition。这样的单项在词典课上没有意义。你可能想要lookup(word)这样的返回的一个定义。

另外,您将单词转换为符号,但我不知道为什么–特别是因为您在规范中使用了字符串键。挑一个,虽然我不相信这是符号增加价值的情况。

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

3

看着你的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 

不要在add方法使用key.to_sym,只是key

为了让灵活地添加方法,我可以自我对象返回continuesly补充。

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

所以,我现在可以做到这一点。

@d.add("a" => "apple").add("b" => "bat")