0

我正在做一个 Ruby Kata(罗马数字转换器),我试图将我的代码分开,所以它不是一个巨大的块。

这是我到目前为止所拥有的:

def roman s
  total = convert s[0,1]

  (0..s.length).each do |i|
    case convert (s[i+1,1]).to_i >= convert (s[i,1])
      when true : total += convert s[i+1,1]
      when false: total -= convert s[i+1,1]
    end
  end

  total
end

def convert s

  case s
    when 'I' :  1
    when 'V' :  5
    when 'X' :  10
    when 'L' :  50
    when 'C' :  100
    when 'D' :  500
    when 'M' : 1000
    else 0
 end

我想在罗马方法中移动 case 语句,所以它现在看起来像这样:

case preceding_number_is_lower_than_next_number s,i

除了添加方法:

def preceding_number_is_lower_than_next_number s,i
  convert (s[i+1,1]).to_i >= convert (s[i,1])
end

但是,当我这样做并给它输入“XXXI”时,它会从给出正确的输出 29 变为不正确的输出 10。

由于它是完全相同的代码运行,只是重构,我无法想象为什么行为会发生变化。

有任何想法吗?

4

0 回答 0