Как исправить undefined method `[]' for nil:NilClass?

Question

[Роман Томченко]

Есть массив

LINE = [
    ' ', 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't',
    'u', 'v', 'w', 'x', 'y', 'z', 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O',
    'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z', '!', ',', '.', ':', ';', '-'
]

Есть хэш, который формируется так:

$square = Hash.new
def build_square
  LINE.each { |c| $square[c] = LINE.rotate(LINE.index(c)) }
end
build_square

То есть ключ хэша - символ, значение - массив.
При выводе все нормально.
Однако при получении значения хэша в переменную результат не индексируется (text, key - обычные строки равной длины).

def encode(text, key)
  res = ""
  (0..text.size).each do |i|
    line = $square[key[i]]
    n = LINE.index(text[i])
    res += line[n]
  end
  res
end

В строке res += line[n] выдаёт ошибку undefined method `[]' for nil:NilClass

Только начал разбиратся в Ruby - могу чего-то не понимать, однако при выводе line - все нормально, выводится как массив.

Answer 1

[Валентин В.]

line = $square[key[i]]
В строке key есть символы, которых нет ни в $square, ни в константе LINE, в итоге на одной из итераций переменная line равна nil

Comments

[Роман Томченко]

Большое спасибо. Достаточно было проверить выводом, какие списки генерируются
Решено путем отсечения последнего элемента строки text в функции encode:
(0..text.size-1).each do |i|

Похоже, функция захватывала символ конца строки

[GeneAYak]

Роман Томченко, лучше уж у строки вызывать метод .chomp, кторый корректно уберет все переносы

[Роман Томченко]

GeneAYak, Мысль. Проверяем...
Неа - все равно ошибку про nil-объект выдает - только с (-1) работает

[GeneAYak]

Роман Томченко, а можете привести финальный листинг программы? потому что, скорее всего, либо проблема не в переносе строки, либо вы сделали что-то не так. В любом случае просто удалять последний символ без проверки что в нем - решение очень плохое

[Роман Томченко]

GeneAYak, Прошу:

LINE = [
    " ", "a", "b", "c", "d", "e", "f", "g", "h", "i", "j", "k", "l", "m", "n", "o", "p", "q", "r", "s", "t",
    "u", "v", "w", "x", "y", "z", "A", "B", "C", "D", "E", "F", "G", "H", "I", "J", "K", "L", "M", "N", "O",
    "P", "Q", "R", "S", "T", "U", "V", "W", "X", "Y", "Z", "!", ",", ".", ":", ";", "-"
]
$square = Hash.new
LINE.each { |c| $square[c] = LINE.rotate(LINE.index(c)) }

def make_key(text, key)
  k = ""
  (0..text.size).step(key.size).each { |_| k << key if k.size < text.size }
  k[0, text.size]
end

def encode(text, key)
  res = ""
  (0..text.size-1).each do |i|
    line = $square[key[i]]
    res += line[LINE.index(text[i])]
  end
  res
end

def decode(code, key)
  res = ""
  (0..code.size-1).each do |i|
    line = $square[key[i]]
    res += LINE[line.index(code[i])]
  end
  res
end

$text = "On the other hand, we denounce with righteous indignation and dislike men who are so beguiled and demoralized by the charms of pleasure of the moment, so blinded by desire, that they cannot foresee the pain and trouble that are bound to ensue; and equal blame belongs to those who fail in their duty through weakness of will, which is the same as saying through shrinking from toil and pain. These cases are perfectly simple and easy to distinguish."
$key = make_key($text, "encodeword")

[GeneAYak]

Роман Томченко, если честно, я не очень понимаю, почему make_key работает именно так как работает, но 0..code.size-1 работает абсолютно правильно, и дело тут абсолютно не в переносе строки, а в том, что происходит выход за размеры строки

[Роман Томченко]

GeneAYak, Хм... И каким же образом .each по строке может выходить за собственно её границы?

[GeneAYak]

Роман Томченко, таким, что each итерирует не строку, а range (0..text.size), а строка перебирается по индексам

так вот, очевидно, что при длине строки в 5, взять пятый индекс не выйдет, тк 4 - максимум

ну и, соответственно, нужно либо отнимать единицу, либо итерировать range не включающий конец (1...5), либо итерировать саму строку, что предпочтительнее и избавит от подобных ошибок

[Роман Томченко]

GeneAYak, Ясно... мой косяк...
Я привык, что на Питоне функция range не включает конец диапазона.
Спасибо