mysql - 未读消息计数器

Question

<分区>

Answer 1

我认为您的域建模确实有问题。

对话的整体理念是相关各方轮流担任发送者和接收者。你模仿的是独白。

A monologue is a speech delivered by one person, or a long one-sided conversation that makes you want to pull your hair out from boredom. The Greek root word monologos translates to “speaking alone,” and that's a monologue: one person doing all the talking.

您最终在此处得到的领域模型应如下所示:

它是实际上链接到两个用户(或更多)的消息:发件人 和收件人。为了简单起见，我在这里坚持使用 1:1 消息传递(与消息可能属于许多收件人的群聊相比)。

class Message
  belongs_to :recipient, class_name: 'User'
  belongs_to :sender, class_name: 'User'
end

class User
  has_many :sent_messages, 
           class_name: 'Message', 
           foreign_key: 'sender_id' 
  has_many :messages, foreign_key: 'recipient_id'
end

请注意，当不能从关联名称派生类和外键时，我们需要告诉 Rails 类和外键。

您可能需要考虑使用 enum 而不是 bool 型 read 字段表示消息的状态。

枚举基本上是一个映射到符号列表的整数列。

class Message
  enum :status, [:unread, :read] 
  belongs_to :recipient, class_name: 'User'
  belongs_to :sender, class_name: 'User'
  belongs_to :conversation
end

枚举给你这样的范围:

Message.unread
Message.read

还有条件句，例如:

message.unread?
message.read?

如果您想添加更多状态，例如 :archieved 或 :trashed，它会变得非常简单。

有了这个，您就不需要您的 unread_messages_count 怪物了。这会消耗大量内存，因为您只是为了计算相关记录而从数据库中提取记录。

current_user.messages.unread.size

我们还应该正确定义User和Conversation之间的关系:

class Conversation
  has_many :messages
  has_and_belongs_to_many :users
end

class Users
  # ..
  has_and_belongs_to_many :conversations
end

这has_and_belongs_to_many关系会将用户和对话存储在 users_conversations 连接表中。您可以使用以下生成器来创建连接表迁移:

rails generate migration users_conversations

添加:

在您的 View 中使用 session[:user_id] 是一种非常糟糕的代码味道。您在整个应用程序中紧密耦合了身份验证逻辑。

而是创建一个助手:

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

  def user_signed_in?
     !current_user.nil?
  end 
end

应用程序的其他部分应该不知道您将当前用户存储在 session[:user_id] 中，只是因为有一个 current_user。