Ruby字符串比较报错：ArgumentError（无效范围值）求助

错误原因 #

你的代码里，for i in str[i..l]这个循环会把i重新赋值为数组str里的元素（比如输入"hello"的话，第一次循环i会变成"h"）。当第二次循环时，str[i..l]就试图用字符串作为数组切片的起始索引，而数组切片要求索引是整数，这就触发了bad value for range的参数错误。内层循环的j也存在同样的变量覆盖问题，导致逻辑混乱。

修复原始逻辑的写法 #

如果想保留双重循环的思路，需要用索引遍历，避免循环变量覆盖索引变量：

puts "Enter the string"
str = gets.chomp.split("")
length = str.length
counts = {}

(0...length).each do |i|
  char = str[i]
  next if counts.key?(char) # 跳过已统计过的字符
  count = 1
  (i+1...length).each do |j|
    count +=1 if str[j] == char
  end
  counts[char] = count
end

counts.each do |char, num|
  puts "#{char}: #{num}"
end

Ruby更简洁的实现方式 #

Ruby本身提供了高效的方法统计元素出现次数，无需手动写双重循环：

方法1：使用tally方法（Ruby 2.7及以上版本）

tally是内置数组方法，可直接返回元素到出现次数的哈希：

puts "Enter the string"
str = gets.chomp
counts = str.chars.tally

counts.each do |char, num|
  puts "#{char}: #{num}"
end

方法2：手动用Hash统计（兼容旧版本Ruby）

puts "Enter the string"
str = gets.chomp.chars
counts = Hash.new(0) # 默认值为0的哈希

str.each do |char|
  counts[char] +=1
end

counts.each do |char, num|
  puts "#{char}: #{num}"
end

这两种方法比手动双重循环更简洁高效，也避免了变量覆盖导致的错误。

内容的提问来源于stack exchange，提问作者Ravi Shekhar Thakur