Question

如果我在 irb 中调用下面的函数anagrams，我会按预期获得一个非空的哈希容器。但是如果您注释掉print "No Key\n"行，则返回的哈希容器现在为空。事实上，对于列表中的所有元素，elsif分支中的代码似乎都在执行。要么我疯了，要么这里有一个讨厌的错误：

def anagrams(list = ['cars', 'for', 'potatoes', 'racs', 'four','scar', 'creams', 'scream'])
        aHash = Hash.new()
        list.each { |el|
            aKey = el.downcase.chars.sort.to_a.hash
            if aHash.key?(aKey)
                # print "Has Key\n"
                aHash[aKey] << el
            elsif
                # print "No Key\n"
                aHash[aKey] = [el]
            end
        }

        return aHash
end

我安装了以下版本的 ruby 和 irb ：

ruby 1.9.2p290 (2011-07-09 revision 32553) [x86_64-linux]
irb 0.9.6(09/06/30)

Answer 1

您的问题是，您使用elsif表示else。这样：

elsif
    print "No Key\n"
    aHash[aKey] = [el]

是误导性的格式，它实际上被解释为更像这样：

elsif(print "No Key\n")
    aHash[aKey] = [el]

但print返回nil所以逻辑是这样的：

elsif(nil)
    aHash[aKey] = [el]

布尔上下文中的

和nil为false，因此aHash[aKey] = [el]永远不会发生。如果您删除了print，那么您最终会得到：

elsif(aHash[aKey] = [el])

并且分配发生;在布尔上下文中也是如此（因为数组是），但在这种情况下，真实性是无关紧要的。

您想在此处使用else：

if aHash.key?(aKey)
    aHash[aKey] << el
else
    aHash[aKey] = [el]
end

更好的方法是使用带有数组的Hash（通过块）作为其默认值：

aHash = Hash.new { |h, k| h[k] = [ ] }

然后根本不需要if，你可以这样做：

list.each do |el|
    aKey = el.downcase.chars.sort.to_a.hash
    aHash[aKey] << el
end

你可以使用任何东西作为Ruby Hash中的键，所以你甚至不需要.to_a.hash，你可以简单地使用数组本身作为键;此外，sort会为您提供一个数组，因此您甚至不需要to_a：

list.each { |el| aHash[el.downcase.chars.sort] << el }

有人可能会在你的方法结束时抱怨return，所以我会这样做：你的方法结束时不需要return，只需说{{1}它将是方法的返回值：

aHash

您还可以使用each_with_object进一步压缩它：

def anagrams(list = ['cars', 'for', 'potatoes', 'racs', 'four','scar', 'creams', 'scream'])
    aHash = Hash.new { |h, k| h[k] = [ ] }
    list.each { |el| aHash[el.downcase.chars.sort] << el }
    aHash
end

但我可能会这样做以减少噪音：

def anagrams(list = ['cars', 'for', 'potatoes', 'racs', 'four','scar', 'creams', 'scream'])
    list.each_with_object(Hash.new { |h, k| h[k] = [ ] }) do |el, h|
        h[el.downcase.chars.sort] << el
    end
end

只有在array.each中使用块内的print语句时才会出现Ruby错误

1 个答案: