Ruby: Undefined method 'length' for nil: NilClass (NoMethodError)

Question

Я пытаюсь написать программу, которая берет строку и выводит самое длинное слово в этой строке. Теперь я знаю, что мой код выглядит довольно волосатым, но я довольно новичок в языке Ruby, поэтому, пожалуйста, просто несите меня. Я не понимаю каких-либо других объяснений, касающихся этой проблемы. Я не ищу ответа. Все, что я хочу, - это доброжелательный человек, чтобы объяснить мне, почему моя программа останавливается в строке 16 с проблемой, указанной в названии этого вопроса. Пожалуйста и спасибо!

# longest_word.rb 
# A method that takes in a string and returns the longest word 
# in the string. Assume that the string contains only 
# letters and spaces. I have used the String 'split' method to 
# aid me in my quest. Difficulty: easy. 

def longest_word(sentence) 
    array = sentence.split(" ") 

    idx = 0 
    ordered_array = [] 

    puts(array.length) 

    while idx <= array.length 
    if (array[idx].length) < (array[idx + 1].length) 
     short_word = array[idx] 
     ordered_array.push(short_word) 
     idx += 1 
    elsif array[idx].length > array[idx + 1].length 
     long_word = array[idx] 
     ordered_array.unshift(long_word) 
     idx += 1 
    else l_w = ordered_array[0] 
     return l_w 
    end 
    end 
end 

puts("\nTests for #longest_word") 
puts(longest_word("hi hello goodbye")) 

Answer 1

В какой-то момент вашего цикла while вы попадаете в состояние, когда idx указывает на последний элемент массива. В этот момент запрос array[idx+1] возвращает нуль, а у NilClass нет метода 'length'

Простым решением было бы изменить условие цикла while так, чтобы idx+1 всегда находилось внутри массива.

Answer 2

Могу я рекомендовать более короткое решение.

words1= "This is a sentence." # The sentence 

words2 = words1.split(/\W+/) # splits the words by via the space character 

words3 = words2.sort_by {|x| x.length} #sort the array 

words3[-1] #gets the last word 

или

def longest_word(sentence) 
sentence.split(/\W+/).sort_by {|x| x.length}[-1] 
end