未登录时将实例变量设为nil是否可行？Rails控制器报错求助

首先，你可以通过条件赋值或者安全导航操作符来避免未登录时的无方法错误，同时让代码更简洁易维护。下面是几种实用的优化方案：

方案1：分支明确处理登录/未登录状态 #

直接在条件分支里给未登录场景的变量统一赋值为nil，逻辑清晰直观，后续视图也能基于变量是否为nil做对应渲染：

class ItemsController < ApplicationController
  def show
    if user_signed_in?
      @fruit = current_user.fruits.build
      @car = current_user.cars.build
      @building = current_user.buildings.find_by(item_id: params[:id]) # 推测你这里的@i是请求参数params[:id]
    else
      @fruit = @car = @building = nil
    end
    # 后续业务逻辑或视图渲染代码
  end
end

方案2：用安全导航操作符简化代码 #

Ruby的安全导航操作符&.可以帮你省去else分支——它会先检查调用对象是否为nil，若为nil则直接返回nil，不会触发NoMethodError，代码会紧凑很多：

class ItemsController < ApplicationController
  def show
    @fruit = current_user&.fruits&.build
    @car = current_user&.cars&.build
    @building = current_user&.buildings&.find_by(item_id: params[:id])
  end
end

这种写法下，用户未登录时current_user为nil，三个变量会自动被赋值为nil，完美规避报错问题。

方案3：提取公共方法（复用场景适用） #

如果多个控制器动作都需要类似的变量赋值逻辑，可以把这段代码提取到私有方法里，提升代码复用性：

class ItemsController < ApplicationController
  def show
    assign_user_related_items
  end

  private

  def assign_user_related_items
    if user_signed_in?
      @fruit = current_user.fruits.build
      @car = current_user.cars.build
      @building = current_user.buildings.find_by(item_id: params[:id])
    else
      @fruit = @car = @building = nil
    end
  end
end

另外提醒下，你代码里的@i看起来应该是请求参数（比如params[:id]），记得确认变量名是否正确，避免出现未定义变量的额外错误哦。

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