Rails 5：用户试用期过期时的重定向逻辑优化方案

• 重复代码太多：两个前置动作都在判断current_user.trial_expired?，没有把这段核心判断逻辑复用起来
• 没匹配核心需求：现有代码跳回index的时候根本没带用户能看到的提示消息，用户跳转完完全不知道为什么被引导回首页
• 逻辑是割裂的：check_if_trial_expired只给视图塞了个@trial_expired变量，完全没承接过期场景的统一处理，和重定向的逻辑分开编写，后续维护很容易漏改

核心思路是把两个动作的校验逻辑合并为同一个前置钩子，分场景做对应处理，既避免重定向循环，又能完整跑通需求流程：

class HomeController < ApplicationController
  # 对所有需要校验试用期的动作统一挂载前置方法
  before_action :handle_trial_expiration, only: [:index, :show]

  protected

  def handle_trial_expiration
    # 试用期未过期，直接放行执行后续动作
    return unless current_user.trial_expired?

    # 如果当前已经在index动作，仅给视图赋值过期标记，不触发重定向避免死循环
    if action_name == 'index'
      @trial_expired = true
      return
    end

    # 其他动作遇到过期场景，携带提示消息重定向回index
    flash[:warning] = '您的账户试用期已到期，请完成续费操作'
    redirect_to home_path
  end
end

• 代码量更少：整个校验流程只需要写一次判断，不用拆成两个功能重叠的前置方法
• 完整覆盖需求链路：
  • 用户访问show触发过期条件时，会自动带上提示消息跳回index
  • 不管是直接访问index、还是从别的动作被重定向回来，视图都能拿到@trial_expired标记渲染对应的提示样式，同时不会出现反复重定向的死循环
• 后续改起来方便：新加需要校验的动作时，只要把动作名往before_action的only列表里加就行，不用重复写判断逻辑
• 复用成本低：要是别的控制器也要加同款试用期校验，直接把handle_trial_expiration方法挪到ApplicationController里，其他控制器直接引用即可

内容的提问来源于stack exchange，提问作者opensource-developer