Ruby-Rails-用户模型上的NoMethodError

Question

我正在为current_user.user_id收到此NoMethodError 。 下面是我的用户模型表。 我不知道该错误意味着什么，因为current_user.user_id存在。 是什么导致此问题？ 我在这里迷路了。 据我所记得，我没有收到这个错误，我一直在向项目中添加新的东西，以及我弄乱的地方。 尽管我所做的更改与用户模式无关。 为什么如果current_user.user_id存在，则为什么出现NoMethodError？ 笨拙而困惑。

错误-

Started GET "/item/list" for 10.10.10.10 at 2012-05-01 23:19:19 -0400
Processing by Item#list as */*
Completed 500 Internal Server Error in 0ms

NoMethodError (undefined method `user_id' for nil:NilClass):
  app/controllers/Item_controller.rb:52:in `list'

用户模型表-

mysql> explain users;
+------------+--------------+------+-----+---------+----------------+
| Field      | Type         | Null | Key | Default | Extra          |
+------------+--------------+------+-----+---------+----------------+
| id         | int(11)      | NO   | PRI | NULL    | auto_increment |
| user_id    | varchar(255) | YES  |     | NULL    |                |
| name       | varchar(255) | YES  |     | NULL    |                |
| created_at | datetime     | NO   |     | NULL    |                |
| updated_at | datetime     | NO   |     | NULL    |                |
+------------+--------------+------+-----+---------+----------------+

使用current_user.user_id的项目控制器-

    def list
    @mylist = Item.find(:all, :select => 'item_num', :conditions => { :id => current_user.user_id, :item_status => ["New"]} )
    respond_to do |format|
    format.html { render :partial => 'item_list'}
    format.js
    end
    return @mylist
    end


    def current_user
    @current_user ||= User.find(session[:user_id]) if session[:user_id]
    end

    def debug
    render :text => current_user.user_id #prints correct user_id
    end

Answer 1

错误消息告诉您，当您尝试在其上调用user_id时， current_user设置为nil 。 我无法根据所包含的代码告诉您current_user为何nil ，但是如果您可以跟踪current_user分配用户的方式/位置，则可以在其中找到问题。 它与您的数据库没有任何关系。

Answer 2

问题似乎不在于您的用户模型没有响应方法user_id，而是根据错误消息，方法current_user返回nil且nil不响应user_id。 因此，原因可能是即使没有任何用户登录也调用此操作。

解决该问题的一种方法是，在调用user_id之前先检查current_user是否返回任何内容。 也许是这样的：

@mylist = Item.find(:all, :select => 'item_num', :conditions => { :id => (current_user && current_user.user_id), :item_status => ["New"]} )

如果curret_user为nil，它将计算为false，然后不会调用user_id，而是查询id为NULL的Items。 如果那是适合您的应用的行为，那我就由您决定。

另外，您的语法实际上不是Rails3。我将其更改为：

@mylist = Item.select(:item_num).where(:id => (current_user && current_user.user_id), :item_status => ["New"])

Answer 3

之所以会这样，是因为current_user值为nil。 您当前的用户功能并没有您认为的那样，让我们​​看一下：

def current_user
    @current_user ||= User.find(session[:user_id]) if session[:user_id]
end

因此，这将按以下顺序执行：

1. 如果有session[:user_id] ，请继续| 否则返回nil

2. 然后，如果@current_user设置，返回@current_user | 否则继续

3. 然后，将@current_user设置为User.find(session[:user_id])

注意： User.find()将在找不到用户的情况下引发异常。 User.find_by_user_id()只会返回nil而不引发异常。 确定并选择所需的行为。

您的方法正常，这意味着每当未设置session[:user_id]时，您将得到nil 。 您只需要在代码中捕获它即可。 因为nil是虚假的，所以这很容易。 在任何红宝石代码中：

any_method if current_user

在您的控制器中，您需要执行以下操作：

def list
  if current_user
    @mylist = Item.find(:all, :select => 'item_num', :conditions => { :id => current_user.user_id, :item_status => ["New"]} )
    respond_to do |format|
      format.html { render :partial => 'item_list'}
      format.js
    end
  else
    redirect_to login_path, :notice => "You must be logged in to continue."
  end
end

请注意，（1）在Ruby中您不需要说return ，每个方法的最后一行都会自动返回，并且（2）Rails控制器方法在您重定向或渲染时结束，没有返回值。

为了使这个更清洁，我可能会说您需要一个before_filter。 控制器中的类似内容应该可以解决问题：

before_filter :require_login

def require_login
  redirect_to login_path, :message => 'You must be logged in to view this page.' unless current_user
end

然后，如果尚未登录，则您的所有方法都将重定向该人以登录。 您也可以使用：only或：except选项仅指定某些方法来使用此过滤器。

因为您可能会在整个应用程序中使用它，所以可以将current_user和require_login方法移至应用程序控制器。

然而，最后一件事，您的代码和问题反映出您对Rails和REST的一些基本概念并不熟悉。 如果您学习做事的“轨道方法”，那么您将真正加快您的开发速度。 迈克·哈特尔（Michael Hartl）的Rails 3 Tutorial是使您步入正轨的绝佳免费资源。 您应该努力完成它，您会感到非常高兴，当您完成之后，您将遥遥领先于现在。