我的代码有什么问题（试图缩写）？

Question

我正在使用 irb/ruby1.9.1。

第一步

我写了下面的代码：

  def isUppercase  
    self>= ?A && self<= ?Z  
  end  

 class String  
    def abbreviate  
      abbr = ""  
      each_byte do |c|  
        if c.isUppercase  
          abbr += c.chr  
        end  
      end  
      abbr  
    end  
  end  

第二步

我正在评估下面的代码，我希望它是“UFO”。
“不明飞行物”.abbreviate

但是，出现了错误。 我该如何纠正它？ 错误就在这里。

irb(main):044:0> load("abbrevi.rb")
=> true
irb(main):045:0> "Unidentified Flyng Object".abbreviate ArgumentError: comparison of Fixxnum with String failed
from C:/Ruby192/lib/ruby/1.9.1/abbrevi.rb:4:in >=' from 
C:/Ruby192/lib/ruby/1.9.1/abbrevi.rb:4:in isUppercase' from 
C:/Ruby192/lib/ruby/1.9.1/abbrevi.rb:12:in block in abbreviate' from 
C:/Ruby192/lib/ruby/1.9.1/abbrevi.rb:11:in each_byte' from 
C:/Ruby192/lib/ruby/1.9.1/abbrevi.rb:11:in abbreviate' from (irb):45 from 
C:/Ruby192/bin/irb:12:in <main>

Answer 1

试试这个：

class Fixnum
  def isUppercase
    self.chr >= ?A && self.chr <= ?Z
    #note use of `chr` to avoid error that occurs when
    #comparing a Fixnum to a String
  end
end

class String
  def abbreviate
    abbr = ""
    each_byte do |c|
      if c.isUppercase
        abbr += c.chr.to_s #note this usage as well
      end
    end
    abbr
  end
end

请注意，您不能将字符串添加到数字或进行比较，因此以下内容将生成错误：

irb> 1 >= "A"
# => ArgumentError: comparison of Fixnum with String failed

更新： @coreyward 的答案是完成您正在做的事情的更好方法，但是我的回答只是指出如何修复您的代码以及错误的原因。更好的方法可能是使用正则表达式。

Answer 2

我不明白为什么您要检查字节是否为大写，而不是字符（存在多字节字符），但这完全回避了您的问题：

class String
  def abbreviate
    each_char.reduce('') do |abbr, c|
      abbr += c if ('A'..'Z').include?(c)
      abbr
    end
  end
end

不过，这仍然没有真正考虑到非 AZ 字母/字符。