Rails 6中编写私信功能是否有更优实现方式？

你的代码存在重复渲染逻辑和嵌套判断问题，可以从两个层面优化：

1. 视图层直接简化（无需修改模型） #

将嵌套条件合并，消除重复的DOM渲染代码：

<% if !@message.private? || @message.user_id == current_user.id || @service_request.user_id == current_user.id %>
  <div class="jumbotron">
    <%= render 'service_request_messages', service_request_message: @message %>
  </div>
<% end %>

优化点说明： #

• 合并判断逻辑：用单个条件直接判断「消息非私密」或「当前用户有权查看私密消息」，去掉嵌套if-else
• 消除重复代码：重复的<div>和render调用只保留一次，避免冗余
• 改用Ruby风格布尔方法private?替代private == true，更符合Rails约定

2. 模型层抽离业务逻辑（更符合DRY与Rails最佳实践） #

把权限判断逻辑移到ServiceRequestMessage模型中，让视图只负责渲染，业务逻辑统一维护：

首先在模型中添加方法：

# app/models/service_request_message.rb
def visible_to?(user)
  !private? || user_id == user.id || service_request.user_id == user.id
end

然后视图代码会极度简洁：

<% if @message.visible_to?(current_user) %>
  <div class="jumbotron">
    <%= render 'service_request_messages', service_request_message: @message %>
  </div>
<% end %>

优势： #

• 符合DRY原则：权限规则集中在模型，后续调整只需修改模型方法，无需改动多个视图
• 视图更简洁：只关注渲染逻辑，不掺杂业务判断
• 可测试性提升：模型方法可单独编写单元测试，确保权限逻辑正确

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