неопределенный метод каждый для нулевого класса в использовании нокигири

Question

Я пытаюсь получить все ссылки по данной ссылке, но это дает мне ошибку неопределенный метод `each 'для nil: NilClass

require 'nokogiri'
def find_links(link)
page = Nokogiri::HTML(open(link))
link_size = page.css('li')
(0..link_size.length).each do |index|
    b = link_size[index]['href']
    return b
end
end
find_links('http://code.tutsplus.com/tutorials/you-dont-know-anything-about-regular-expressions-a-complete-guide--net-7869').each do |url|
puts url
end

Answer 1

require 'nokogiri'
require 'open-uri'
def find_links(link)
page = Nokogiri::HTML(open(link))
link_array = page.css('li')
(1..link_array.length).each do |f|
    a=Array.new.push(page.css('li a')[f]['href'])
    puts a
end
end
find_links('http://code.tutsplus.com/tutorials/you-dont-know-anything-about-regular-expressions-a-complete-guide--net-7869')

Answer 2

В вашем коде есть несколько проблем.Найти объяснение в строке ниже:

def find_links(link)
  page = Nokogiri::HTML(open(link))
  link_size = page.css('li') 
  (0..link_size.length).each do |index|
     b = link_size[index]['href'] # You are expecting to get 'href' on list item which is wrong.
     return b # You shouldn't return from this block if you are expecting to get all the links. return from here will return from this method itself after first iteration. 
     # That's why you are getting nil error since link_size[index]['href'] doesn't have href attribute in first list item
  end
end

Измените свой код на: (найти объяснения в строке)

require 'nokogiri'
require 'open-uri'
def find_links(link)
  page = Nokogiri::HTML(open(link))
  # You want to iterate on anchor tags rather than list. 
  # See the use of `map`, it will return the array and since this is the last statement it will return from the method, giving all the links.
  # .css('li a') will give all the anchor tags which have list item in it's parent chain.
  page.css('li a').map { |x| x['href'] } 
end
find_links('http://code.tutsplus.com/tutorials/you-dont-know-anything-about-regular-expressions-a-complete-guide--net-7869').each do |url|
  puts url
end