Ruby on Rails方法内调用其他方法报NameError的原因与代码优化方案

Ruby的局部变量有严格的方法级作用域边界：在某个方法内定义的局部变量，仅能在当前方法内部访问，方法执行结束后局部变量就会被销毁，其他方法无法直接读取。
你最初的代码里，active_campaign_id、contact都是定义在create_token方法里的局部变量，当代码执行到update_contact方法时，这些变量已经不存在了。Ruby会先尝试找同名局部变量，找不到就会认为你要调用同名方法，找不到对应方法就抛出NameError异常。
把所有逻辑合并到update_contact方法后，所有变量都处于同一个方法作用域内，可以互相访问，因此代码可以正常运行。

你可以根据场景选下面几种拆分方式，避免作用域问题同时保持代码结构清晰：

• 用返回值传递数据：让拆分出的子方法返回后续逻辑需要的对象，调用方接收返回值后使用，这是最推荐的低耦合写法。
  示例改造：

  def create
    update_contact
  end
  
  def update_contact
    # 接收create_token返回的contact实例
    contact = create_token
    url = URI("https://acme.api-us1.com/api/3/contacts/#{contact.active_campaign_id}")
    http = Net::HTTP.new(url.host, url.port)
    http.use_ssl = true
  
    request = Net::HTTP::Put.new(url)
    request['Accept'] = 'application/json'
    request['Content-Type'] = 'application/json'
    request['api-token'] = API_KEY
    data = { contact: { fieldValues: [{ field: '1', value: contact.email_token }] } }
    request.body = JSON.dump(data)
    response = http.request(request)
  end
  
  def create_token
    active_campaign_id = params[:contact][:id].to_i
    generate_token = SecureRandom.urlsafe_base64(12)
    contact = Contact.find_or_initialize_by(active_campaign_id: active_campaign_id, email_token: generate_token)
    contact.save!
    # 方法最后返回contact实例，供调用方使用
    contact
  end
  

• 抽离业务逻辑到Service层：控制器只保留参数校验、响应返回的薄逻辑，把生成token、调用第三方接口这类业务逻辑封装到单独的Service类中，避免控制器里堆砌太多业务方法，适合逻辑复杂的场景。
• 合理使用实例变量：如果多个动作方法都需要用到同一个变量，可以用带@前缀的实例变量（比如@contact），实例变量的作用域是整个控制器实例生命周期，跨方法也能访问。注意不要滥用实例变量，否则会提升代码维护成本，很难追踪变量定义位置。

• 先系统学习Ruby基础语法中变量作用域、方法定义与调用的相关规则，Rails是基于Ruby构建的框架，搞懂Ruby本身的作用域规则是解决这类问题的核心。
• 阅读Rails官方指南中Action Controller相关章节，了解控制器的执行流程、方法可见性规则。
• 《Ruby元编程》一书中对Ruby的作用域门、方法查找路径有非常清晰的讲解，读完可以从底层理解这类问题的成因。
• 日常调试可以使用byebug、pry这类断点调试工具，运行时直接查看当前作用域下的可用变量、方法，比死记规则更直观。

内容的提问来源于stack exchange，提问作者theKid