调用模块：

Question

寻找有关如何修复此错误的建议，并重构此代码以改进它。

require 'mechanize'
require 'pry'
require 'pp'




module Mymodule
  class WebBot 

    agent = Mechanize.new { |agent| 
        agent.user_agent_alias = 'Windows Chrome'
    }

    def form(response)
      require "addressable/uri"
      require "addressable/template"
      template = Addressable::Template.new("http://www.domain.com/{?query*}")
      url = template.expand({"query" => response}).to_s
      page = agent.get(url)  
    end 

    def get_products
      products = []
      page.search("datatable").search('tr').each do |row|
        begin
          product =  row.search('td')[1].text
        rescue => e
            p e.message
        end
        products << product
      end  
      products
    end  
  end 
end

调用模块：

response = {size: "SM", color: "BLUE"}

t = Mymodule::WebBot.new
t.form(response)
t.get_products

错误：

NameError: undefined local variable or method `agent'

Answer 1

Ruby有一个命名约定。 agent是类范围中的局部变量。要使其对其他方法可见，您应该通过命名@@agent使其成为类变量，并且它将在WebBot的所有对象之间共享。但首选的方法是通过命名它@agent使其成为实例变量。 WebBot的每个对象都有自己的@agent。但是你应该把它放在initialize中，当initialize

创建一个新对象时，会调用new

class WebBot
  def initialize
    @agent = Mechanize.new do |a|
      a.user_agent_alias = 'Windows Chrome'
    end
  end
.....

page也会出现同样的错误。您在form中将其定义为本地变量。当form完成执行时，它将被删除。您应该将其设为实例变量。幸运的是，您不必将其放在initialize中。您可以在form中定义它。在调用@page后，对象将拥有自己的form。在form中执行此操作：

def form(response)
  require "addressable/uri"
  require "addressable/template"
  template = Addressable::Template.new("http://www.domain.com/{?query*}")
  url = template.expand({"query" => response}).to_s
  @page = agent.get(url)  
end

请务必将page和agent的每次出现更改为@page和@agent。在您的get_products例如：

def get_products
  products = []
  @page.search("datatable").search('tr').each do |row|
  .....

这些更改将解决名称错误。重构是另一个故事。

机械化模块，名称错误＆＃39;剂＆＃39;

调用模块：

错误：

1 个答案: