Rails中params.nil?为true却执行else分支的问题排查

这是一个典型的Ruby变量提升特性导致的坑，再加上对Rails控制器中params的误解，才让你陷入了困惑。我们一步步拆解：

核心问题：局部变量覆盖了控制器的params方法 #

你在else分支里写了params = params.merge(dealer: {})——这行代码会在整个方法范围内创建一个名为params的局部变量，哪怕它是在else分支里定义的。Ruby的规则是：只要方法内任何位置有变量赋值操作，该变量在整个方法中都会被当作局部变量，优先于同名的方法调用。

所以在你执行binding.pry时，这个局部变量还没被赋值，处于未初始化状态，此时params.nil?返回true——但这个params是局部变量，不是Rails控制器提供的请求参数对象！控制器的实际请求参数需要用self.params来调用，它在Rails中几乎永远不会是nil（哪怕没有任何请求参数，它也是一个空的ActionController::Parameters对象）。

为什么代码会走到else分支？ #

你说明明params.nil?是true却执行了else分支，大概率是因为你在Pry会话中不小心修改了局部变量params的值，比如执行了params = self.params（把控制器的实际参数赋值给了局部变量），此时局部变量params不再是nil，继续执行代码就会进入else分支。而如果后续操作中这个局部变量又变成了nil，就会触发undefined method 'merge' for nil:NilClass的异常。

修复方案 #

1. 避免覆盖控制器的params方法：把else分支里的变量名改掉，不要用params，这是最稳妥的做法：

def build_resource
  binding.pry
  if self.params.empty? # Rails中params不会是nil，判断空更合理
    model_class.new
  else
    merged_params = self.params.merge(dealer: {})
    model_class.new(dealer_params)
  end
end

2. 明确调用控制器的params：如果一定要用同名变量，记得用self.params来区分局部变量和控制器方法，但更推荐第一种方案，避免混淆。

另外补充一句：在Rails控制器中，永远不要判断params.nil?，因为框架保证params始终是一个ActionController::Parameters对象，哪怕没有任何参数，用params.empty?或者判断特定参数是否存在才是正确的做法。

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