Rails中新增投票通知：需新建控制器还是复用现有通知控制器？

• 优先复用现有逻辑，不需要新建控制器。Rails控制器的核心职责是处理HTTP请求，而通知发送属于业务逻辑范畴，应该和控制器解耦，避免冗余代码。
• 先梳理现有GroupNotificationController的职责：
  • 如果它只是调用通知发送逻辑（比如触发模型推送、调用通知模板），建议把通知发送的核心代码抽离到独立的服务类（比如NotificationService），在里面新增send_new_vote_notification方法，之后在投票创建的流程中直接调用这个方法即可。
  • 如果它是负责通知的CRUD管理（比如用户查看、标记已读通知），那发送投票通知的逻辑本来就不该放在这里，而是在Vote模型的after_create回调里，或者在VotesController#create动作执行完成后调用通知服务。
• 具体实现示例：
  1. 新建服务类app/services/notification_service.rb：

     class NotificationService
       def self.send_group_member_join_notification(group, member)
         # 原有的成员加入通知逻辑
       end
     
       def self.send_new_vote_notification(group, vote)
         # 新增投票创建的通知逻辑，比如给群内所有成员发送通知
         group.members.each do |member|
           Notification.create(
             user: member,
             group: group,
             content: "群内新增投票：#{vote.title}"
           )
         end
       end
     end
     

  2. 在投票创建的控制器动作里调用服务：

     # app/controllers/votes_controller.rb
     def create
       @vote = current_group.votes.build(vote_params)
       if @vote.save
         NotificationService.send_new_vote_notification(current_group, @vote)
         redirect_to group_votes_path(current_group), notice: '投票创建成功'
       else
         render :new
       end
     end
     

• 这种做法的优势：代码复用性高，逻辑分层清晰，符合Rails的"瘦控制器、胖模型"或"服务对象"最佳实践，后续新增其他类型通知（比如活动创建）时，只需在服务类中扩展方法即可。

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