如何显示在线用户

Question

我正在编写一个简单的聊天，我需要列出在线用户。我不使用devise 进行身份验证，有一个自定义的user 模型通过omniauth 进行身份验证。

user.rb

class User < ActiveRecord::Base
    has_many :messages, dependent: :delete_all
    class << self
        def from_omniauth(auth)
            provider = auth.provider
            uid = auth.uid
            info = auth.info.symbolize_keys!
            user = User.find_or_initialize_by(uid: uid, provider: provider)
            user.name = info.name
            user.avatar_url = info.image
            user.profile_url = info.urls.send(provider.capitalize.to_sym)
            user.save!
            user
        end
    end
end

application_controller.rb

def current_user
    @current_user ||= User.find_by(id: cookies[:user_id]) if cookies[:user_id]
end
helper_method :current_user

我试图这样做：添加到application_controller.rb show_online 方法：

def show_online
    @users = User.where(status: online)
end

helper_method :online_users

然后添加到视图中：

<%= online_users.each do |user| %>
<ul>
    <li><%= user.name %></li>
</ul>
<%end%>

但它会抛出异常ActionView::Template::Error (undefined method 'online_users' for #<MessagesController:0x007f52d7f82740>)

source code这里

编辑

我找到了here 对我来说最好的解决方案，但我完全不知道如何正确实现它:( 但这绝对是我需要的

Answer 1

应该是<% %> 而不是<%= %>

<% @users.each do |user| %>
    <ul>
    <li><%= user.name %></li>
</ul>
<% end%>

其次

但您还需要检查 @users 是否为 nil，因此 nil.each 每个都会​​抛出该错误 ActionView::Template::Error (undefined method 'each' for nil:NilClass)

它看起来像

<% if @users %>
 <% @users.each do |user| %>
  <ul>
    <li><%= user.name %></li>
  </ul>
 <% end%>
<% end %>

或在控制器中

def show_online
  @users = User.where(status: 'Online')
end

和

<% @users.each do |user| %>
<ul>
    <li><%= user.try(:name) %></li>
</ul>
<%end%>

我为什么选择where not find all

Answer 2

您的应用程序控制器代码错误：

class ApplicationController < ActionController::Base
  def show_online
    @users = User.where(status: 'online')
  end
  helper_method :online_users
end

应该是：

class ApplicationController < ActionController::Base
  def online_users
    @users ||= User.where(status: 'online')
  end
  helper_method :online_users
end

Answer 3

编辑：

您遇到的错误可以通过以下两种方式之一解决。
您可以使用辅助方法并从您的视图中调用它们，就像您想要做的那样。
或者，您可以完全避免使用它们，而只在加载视图时从当前正在使用的任何方法调用 show_online 方法。如果你要去show，它将是 show 方法，依此类推。

您自己的答案确实使用第一种方法正确修复了错误，但我推荐这种方式。

实施这些修复需要做些什么：

• 在加载 new 时调用 show_online，以便视图可以访问 @users 变量。我们可以通过 before_action

做到这一点

• 在视图中，您有一个循环遍历online_users，但它应该遍历@users

• 在视图中的同一个循环中，您有一个简单的语法错误。第一行以<%= 开头，但应以<% 开头，不带=。无论您以何种方式编写代码，都应该更改此设置。

所以所有的代码是：

application_controller.rb

#put this line at the top of the controller, just below the line ApplicationController:: .....
before_action :show_online, only: [:new]

def show_online
    @users = User.where(online: true)

视图文件

<% @users.each do |user| %>
  <ul>
      <li><%= user.name %></li>
  </ul>
<% end %>

为什么要用这种方法？

• 一种获取在线用户的方法意味着逻辑只有一个地方
• 将您的逻辑/代码放在一个地方意味着您永远不会重复自己，并且您知道在出现问题时该去哪里查看
• 使用 before_action 意味着除非需要，否则不会进行调用
• 如果您稍后添加需要获取在线用户列表的页面，您只需将这些页面添加到括号中的方法列表中：only: [:new]
• 在将逻辑放置在视图中或控制器中进行选择时，正确的答案几乎总是控制器

Answer 4

从错误消息看来，您的 @users 不是数组或 ActiveRecord::Relation。

我会打印出视图上的@users 以进行调试。此外，find(:all, :conditions => ["status = ?", "online"]) 也不是最好的查询方式。

使用User.where(:status => "online")。参考——http://guides.rubyonrails.org/active_record_querying.html#conditions