'undefined method `>' for 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 可能是 nil，正如错误消息所说（这不应该是这种情况，因为 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 然后你尝试在 nil 上调用方法 >。

专业调试

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

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

发现错误

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

对于第一个字母，它在第二个条件 if( plainTex[j] > 64) and (plainTex[j] < 91 ) 中递增 j。然后随着 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 块之外并将其作为循环之前的最后一行 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

查看地图功能的工作原理。