Rails胖模型瘦控制器：密码恢复控制器重构疑问

我来帮你把这段控制器逻辑完美迁移到User模型，彻底贯彻Rails的胖模型瘦控制器原则！咱们一步步来：

第一步：把业务逻辑迁移到User模型 #

首先在app/models/user.rb里添加一个类方法，把原来控制器里的登录凭证判断、用户存在性检查都封装进来：

class User < ApplicationRecord
  # 你的其他模型代码...

  # 处理密码恢复的登录凭证识别
  def self.password_recovery_status(login)
    return :missing_login unless login.present?

    if exists?(id: login)
      :recover_by_id
    elsif valid_email_format?(login)
      # 注意：原代码未检查邮箱对应的用户是否存在，若需修复此逻辑，可添加 && exists?(email: login)
      :recover_by_email
    else
      :invalid_credentials
    end
  end

  private

  # 把邮箱格式验证封装到模型内部，消除对外部Utils模块的依赖
  def self.valid_email_format?(string)
    string.match?(/\A[\w+\-.]+@[a-z\d\-]+(\.[a-z\d\-]+)*\.[a-z]+\z/i)
  end
end

这里做了几个关键优化：

• 用语义化符号（比如:recover_by_id）代替直接返回字符串，逻辑更清晰
• 将邮箱格式验证移到模型内部，让模型更内聚，不再依赖外部工具模块
• 给每种场景定义明确的状态标识，方便控制器后续处理

第二步：简化控制器代码 #

现在你的控制器可以变得非常“瘦”，只需要处理请求参数和返回HTTP响应：

def recover_password
  recovery_status = User.password_recovery_status(params[:login])

  case recovery_status
  when :recover_by_id
    render plain: "Recover by id", status: :ok
  when :recover_by_email
    render plain: "Recover by email", status: :ok
  when :missing_login
    render plain: "Login not found", status: :unauthorized
  when :invalid_credentials
    render plain: "Email not found", status: :unauthorized
  end
end

为什么这么重构？ #

• 职责分离：模型负责处理业务逻辑（识别登录凭证、判断用户状态），控制器只负责HTTP层面的请求响应，完全符合Rails的设计原则
• 可维护性：以后如果要修改登录凭证规则（比如支持用户名），只需要修改User模型的方法，不需要改动控制器
• 可测试性：模型里的方法可以单独写单元测试，比测试控制器逻辑更简单高效

可选优化：修复原逻辑的潜在问题 #

原代码存在一个小漏洞：如果输入合法邮箱但无对应用户，依然会返回"Recover by email"。若需修正此逻辑，只需修改模型方法里的邮箱判断部分：

elsif valid_email_format?(login) && exists?(email: login)
  :recover_by_email

这样只有当邮箱存在对应用户时，才会进入邮箱恢复流程，否则返回错误。

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