ruby - '未定义方法 `>' 为 nil :NilClass in ruby

Question

这是我将用户输入的字符串转换为凯撒密码的代码。

puts "text?"
text = gets.chomp

puts "key?"
key = gets.chomp.to_i 

plainTex = Array.new 
ciphTex = Array.new

j = 0 

text.each_byte do |i|
  plainTex[j] = i
  j += 1
end

j = 0 

plainTex.each_entry do |i|

  if ( i == 32 )
    ciphTex[j] = plainTex[j]
    j += 1
    end

  if( plainTex[j] > 64) and (plainTex[j] < 91 )
    if( (plainTex[j] + key) > 91)
      ciphTex[j] = (plainTex[j] + key ) - 90
      j += 1
    else 
      ciphTex[j] = plainTex[j] + key 
      j += 1 
    end
  end

  if( plainTex[j] > 94) and (plainTex[j] < 123)
    if( (plainTex [j] + key) > 122)
      ciphTex [j] = (plainTex[j] + key) - 122
      j += 1 
    else 
      ciphTex[j] = plainTex[j] + key
      j += 1
    end
  end
 end 

ciphTex.each_entry do |i|
  puts i
end

现在，我收到错误: nil:NilClass (NoMethodError) 的未定义方法“>”

网络搜索让我得出结论，plainTex 可能为零，正如错误消息所说(这不应该是这种情况，因为 plainTex 是预先输入数据的)。

旁注:当输入字符串全是小写且没有空格时程序运行正常。我不知道为什么。

那么，我做错了什么？

Answer 1

问题解释

当您访问索引超出范围的数组时，您不会收到任何错误，但结果是 nil .

irb --simple-prompt
>> a = [1]
=> [1]
>> a[0]
=> 1
>> a[1]
=> nil
>> a[200]
=> nil

调试代码

一些 puts 语句揭示了这个问题:

text.each_byte do |i|
  plainTex[j] = i
  j += 1
end

j = 0 

puts "Length: #{plainTex.length}"

plainTex.each_entry do |i|

  if ( i == 32 )
    ciphTex[j] = plainTex[j]
    j += 1
  end
  puts "j: #{j}"

这是输出

text?
Testing Test
key?
32
Length: 12
j: 0
j: 2
j: 3
j: 4
j: 5
j: 6
j: 7
j: 8
j: 10
j: 11
j: 12
test.rb:30:in `block in <main>': undefined method `>' for nil:NilClass (NoMethodError)
    from test.rb:23:in `each'
    from test.rb:23:in `each_entry'
    from test.rb:23:in `<main>

在这种情况下，数组的最后一个索引是 11，而您访问了索引 12。你得到了结果 nil然后您尝试调用方法 >无。

专业调试

您可以安装 gem pry-byebug并使用短别名 s , n进入，下一步等

https://github.com/deivid-rodriguez/pry-byebug

发现错误

使用 pry-byebug立即清楚错误是什么，即使我并不真正理解您的代码，因为这些数字对我来说毫无意义。

对于第一个字母，它递增 j在第二个条件 if( plainTex[j] > 64) and (plainTex[j] < 91 ) .然后增加 j它移动到 if( plainTex[j] > 94) and (plainTex[j] < 123)它再次递增。

你应该使用 elsif对于所有这些或 next

代码清理

Ruby 惯例是使用蛇形命名变量，我将更改这段代码:

if( plainTex[j] > 64) and (plainTex[j] < 91 )
  if( (plainTex[j] + key) > 91)
    ciphTex[j] = (plainTex[j] + key ) - 90
    j += 1
  else 
    ciphTex[j] = plainTex[j] + key 
    j += 1 
  end
end

为此:

if plain_text[j].between?(65, 90)
  ciph_text[j] = plain_text[j] + key
  ciph_text[j] -= 90 if ciph_text[j] > 91
  j += 1
end

看起来你在每个条件下都重复 j += 1 ，所以为什么不把它移到 if/else block 之外并将它放在循环之前的最后一行 end ？

我也是人，我不想记住“A”是 65，“Z”是 90。为什么不在代码中这样说呢？

if plain_text[j].between?("A".ord, "Z".ord)

我从来没有在 ruby​​ 中弄乱过 ASCII，但我相信你可以进一步改进它，可能在一个简单的循环中用 10 行或更少的代码完成整个密码。

Ruby 非常擅长处理数组，所以请使用它。我会做这样的事情:

plain_text = text.codepoints
cyph_text = plain_text.map do |code|
  if code == ' '.ord
    ' '.ord
  elsif
    # return cypher code
  end
end

查看 map 函数的工作原理。