非nil对象触发nil:NilClass错误？Ruby代码问题求助

代码场景 #

puts document.nil?  # 输出False

if document.nil?  # 按道理document非nil，不该进入该分支
  begin
    document = conect_documents.find("123")
  rescue ActiveRecord::RecordNotFound => e
    return false
  end
end

attrs = document.attributes.key?('import')  # 抛出错误：undefined method `attributes' for nil:NilClass

错误调用栈 #

NoMethodError:
   undefined method `attributes' for nil:NilClass
# ./path/permission.rb:217:in `edit?'
# ./path/documents_controller.rb:67:in `update'
# ./spec/login_spec_helper.rb:11:in `block in process'
# ./spec/login_spec_helper.rb:32:in `catch'
# ./spec/login_spec_helper.rb:32:in `_catch_warden'
# ./spec/login_spec_helper.rb:11:in `process'
# ./path/documents_controller_spec.rb:468:in `block (5 levels) in <top (required)>'

核心原因分析 #

1. 并发场景下的对象被修改

如果这段代码运行在多线程环境中，puts document.nil?执行后、if document.nil?判断前，其他线程可能把document改成nil，导致进入if分支。如果conect_documents.find("123")找不到记录且不抛ActiveRecord::RecordNotFound异常（比如自定义的find方法），就会把document赋值为nil，后续执行document.attributes时直接报错。

2. conect_documents.find的行为不符合预期

你默认find找不到记录会抛ActiveRecord::RecordNotFound，但如果conect_documents不是标准ActiveRecord模型，或者find方法被重写过，找不到记录时直接返回nil而非抛异常，那么进入分支后document会被设为nil，跳过rescue块，继续执行到后续代码就会触发错误。
另外检查拼写：conect_documents是不是应该是connect_documents？笔误可能导致调用不存在的方法，但从调用栈看代码走到了edit?方法，这个可能性较低，但仍需确认。

3. 延迟加载导致对象失效

如果document是ActiveRecord的延迟加载对象（比如未加载的关联对象），document.nil?会返回false（因为对象本身存在），但后续访问attributes时，对应的数据库记录可能已被删除或事务回滚，导致实际加载时变成nil。

验证与修复方案 #

加日志确认执行流程

在关键位置添加日志，确认是否进入了if分支，以及document的变化：

puts document.nil?  # 输出False
puts "Before if: document = #{document.inspect}"
if document.nil?
  puts "Entered if branch!"
  begin
    document = conect_documents.find("123")
    puts "After find: document = #{document.inspect}"
  rescue ActiveRecord::RecordNotFound => e
    puts "Rescued RecordNotFound, returning false"
    return false
  rescue => e
    puts "Unexpected error: #{e.message}"
    return false
  end
end
puts "Before attributes: document = #{document.inspect}"
attrs = document.attributes.key?('import')

处理并发问题

如果是多线程场景，对document的访问加锁，或者使用线程局部变量避免并发修改。

修正find逻辑

确认conect_documents.find的实现，必要时改用find_by（返回nil而非抛异常）并显式判断：

if document.nil?
  document = conect_documents.find_by(id: "123")
  return false unless document
end

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