这行抛出错误:
charLine.each_index { |i| charLine[i] = (charLine[i] + shift)}
编辑:
charLine[i]
是一个数字(ASCII码),并且shift
是一个数字我知道这与尝试将变量shift中的数字添加到charLine数组索引中的字节ascii代码有关。但是我什至根本不处理字符串,这就是为什么我对这个错误感到困惑……这是带有错误的方法:
def caesarCipher(string)
puts "Original:\n #{string}"
charLine = string.chars
charLine.each_index { |i| charLine[i]=charLine[i].bytes }
charLine.flatten!
puts charLine.inspect
shift = 1
alphabet = ('A'..'Z').to_a
while shift <= alphabet.size
#moving up in the alphabet using ascii code
charLine.each_index { |i| charLine[i] = (charLine[i] + shift)}
#converting back to letters
charLine.each_index { |x| charLine[x] = charLine[x].chr }
puts "Shifted:\n #{charLine.inspect}"
puts "With a letter shift of #{shift}"
shift = shift + 1
@@shiftyArray.push(charLine.flatten)
end
end
最佳答案
我设法重现了该问题,并通过一些调试输出可以看到第一次迭代运行良好,并且该问题仅出现在第二次迭代中(shift = 2)。
在第一次迭代中, charLine 是整数数组,因此integer + integer可以很好地解决以下问题:charLine.each_index { |i| charLine[i] = (charLine[i] + shift)}
但是,然后在循环的下一行将charLine转换为字符串数组:
#converting back to letters
charLine.each_index { |x| charLine[x] = charLine[x].chr }
因此,在第二次迭代开始时,charLine现在是一个Strings数组,因为它没有被转换回去。然后,在.each_line上,它尝试将字符串+整数加起来,然后炸毁。
解决方案是在每次迭代开始时将字符串数组重新映射为整数。
charLine.map!(&:ord)
另一种选择是不修改数组并将结果保存到temp变量中,这是一个有效的示例:
def caesar_cipher(string)
shiftyArray = []
charLine = string.split(//)
charLine.map!(&:ord)
shift = 1
alphabet_size = 26
while shift <= alphabet_size
shifted_array = charLine.map { |c| (c + shift) < 122 ? (c + shift) : (c + shift) - 26 }
shifted_array.map!(&:chr)
p shifted_array
shift += 1
shiftyArray.push(shifted_array.join)
end
end
caesar_cipher("testing")