我正在创建一个ruby程序,该程序将计算出现在文本中的每个字母的频率,并将其作为哈希返回。
下面是我的代码:
class LetterHistogram
attr_reader :letters
attr_accessor :text
def initialize(t = "Hello World!")
@text = t
end
def display
calculateFrequencies
("A".."Z").each {|x| puts "#{x}: " + "*" * letters[x]}
end
private
attr_writer :letters
def calculateFrequencies
calcuFreq = String.new(text)
calcuFreq.upcase!.gsub!(/\W+/, '')
letters.clear
letters.default = 0
calcuFreq.each_char {|char| letters[char] += 1}
end
end
但是运行显示方法时出现此错误 100 counts
错误是什么意思以及如何解决?
答案 0 :(得分:0)
主要问题是在calculateFrequencies
中,您使用的是未分配的变量:letters
。因此,当您在calculateFrequencies
中调用display
时,letters = nil
并在.clear
上调用nil
会返回错误。
这是使用snake_case(这是Ruby编写标准)的代码的修改版本。
class LetterHistogram
attr_accessor :text
def initialize(t = "Hello World!")
@text = t
end
def display
calculate_frequencies.each { |letter, freq| puts "#{letter}: #{freq}"}
end
private
def calculate_frequencies
freq = @text.upcase.gsub!(/\W+/, '').each_char.with_object(Hash.new(0)) { |letter, freq| freq[letter] += 1 }
freq.sort_by{ |letter, freq| freq }.reverse # just to sort
end
end
实例化对象并对其调用.display
senctence = LetterHistogram.new
senctence.display
#=> L: 3
#=> O: 2
#=> D: 1
#=> R: 1
#=> W: 1
#=> E: 1
#=> H: 1
工作方式
为了计算频率,我使用了一个由https://ruby-doc.org/core-2.5.1/Enumerable.html#method-i-each_with_object
填充的哈希从freq
中打印出calculate_frequencies
,您可以看到它:
#=> {"H"=>1, "E"=>1, "L"=>3, "O"=>2, "W"=>1, "R"=>1, "D"=>1}
freq
哈希值初始化为所有值均为0,然后更新哈希值,如下所示:
freq = ("A".."Z").each_with_object({}) { |letter, freq| freq[letter] = 0 }
"Hello World!".upcase.gsub!(/\W+/, '').each_char { |letter| freq[letter] += 1 }
#=> {"A"=>0, "B"=>0, "C"=>0, "D"=>1, "E"=>1, "F"=>0, "G"=>0, "H"=>1, "I"=>0, "J"=>0, "K"=>0, "L"=>3, "M"=>0, "N"=>0, "O"=>2, "P"=>0, "Q"=>0, "R"=>1, "S"=>0, "T"=>0, "U"=>0, "V"=>0, "W"=>1, "X"=>0, "Y"=>0, "Z"=>0}
最后,打印出直方图:
freq.each { |letter, freq| puts "#{letter}: " + "◼︎" * freq if freq > 0 }
#=> D: ◼︎
#=> E: ◼︎
#=> H: ◼︎
#=> L: ◼︎◼︎◼︎
#=> O: ◼︎◼︎
#=> R: ◼︎
#=> W: ◼︎