Ruby on Rails记录用户邮箱方法及忘记密码POST请求代码正确性咨询

首先，咱们先看看你现有代码里的几个明显问题：

1. 未定义变量错误：你把参数存在了@password里，但后面却用了@email——这个变量根本没赋值，会直接抛出NameError。应该改成从@password里提取邮箱，比如@email = @password[:email]。
2. 业务逻辑偏差：忘记密码功能的核心是给已存在的用户发送重置邮件，而不是创建一条Email记录——这完全不符合实际业务场景。
3. 缺少响应与错误处理：你的代码没有处理邮箱不存在的情况，也没有在操作完成后返回合适的响应（比如重定向或渲染提示），用户提交表单后会得到空白页面。
4. 控制器设计不合理：用Post控制器来处理密码重置逻辑不符合Rails的RESTful设计原则，应该单独创建PasswordResets控制器来专门管理这类操作。

下面是实现忘记密码功能的标准步骤： #

1. 创建专用控制器

先删除你之前生成的Post控制器里的forgot_password动作，然后生成专门的密码重置控制器：

rails generate controller PasswordResets new create --skip-assets --skip-helper

2. 配置路由

在config/routes.rb里添加RESTful路由：

resources :password_resets, only: [:new, :create]

3. 编写控制器逻辑

在app/controllers/password_resets_controller.rb里实现new（显示表单）和create（处理提交）动作：

class PasswordResetsController < ApplicationController
  def new
    # 渲染忘记密码表单页面
  end

  def create
    @user = User.find_by(email: params[:password_reset][:email].downcase)
    
    if @user
      # 生成密码重置令牌和过期时间
      @user.create_reset_digest
      # 发送重置邮件
      @user.send_password_reset_email
      flash[:info] = "密码重置邮件已发送，请查收你的邮箱"
      redirect_to root_path
    else
      flash.now[:danger] = "该邮箱未注册，请检查输入"
      render :new
    end
  end
end

4. 在User模型中添加辅助方法

打开app/models/user.rb，添加生成重置令牌、发送邮件的方法：

class User < ApplicationRecord
  # ... 其他现有代码 ...

  # 生成密码重置令牌
  def create_reset_digest
    self.reset_token = User.new_token
    update_columns(reset_digest: User.digest(reset_token), reset_sent_at: Time.zone.now)
  end

  # 发送密码重置邮件
  def send_password_reset_email
    UserMailer.password_reset(self).deliver_now
  end

  # 类方法：生成加密令牌和摘要
  class << self
    # 生成随机令牌
    def new_token
      SecureRandom.urlsafe_base64
    end

    # 加密令牌
    def digest(string)
      cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST : BCrypt::Engine.cost
      BCrypt::Password.create(string, cost: cost)
    end
  end
end

5. 创建密码重置邮件

生成邮件处理器：

rails generate mailer UserMailer password_reset

然后编辑app/mailers/user_mailer.rb：

class UserMailer < ApplicationMailer
  default from: "your-app-email@example.com"

  def password_reset(user)
    @user = user
    mail(to: user.email, subject: "你的密码重置请求")
  end
end

最后创建邮件模板app/views/user_mailer/password_reset.html.erb（可以同时写text版本）：

<h1>密码重置</h1>
<p>你请求重置密码，请点击下面的链接：</p>
<%= link_to "重置密码", edit_password_reset_url(@user.reset_token, email: @user.email) %>
<p>如果不是你发起的请求，请忽略这封邮件，你的密码不会被修改。</p>
<p>该链接2小时内有效。</p>

6. 编写忘记密码表单视图

在app/views/password_resets/new.html.erb里添加表单：

<h1>忘记密码</h1>
<%= form_with(model: :password_reset, url: password_resets_path, local: true) do |f| %>
  <% if flash.now[:danger] %>
    <div class="alert alert-danger"><%= flash.now[:danger] %></div>
  <% end %>
  <div class="form-group">
    <%= f.label :email %>
    <%= f.email_field :email, class: "form-control", required: true %>
  </div>
  <%= f.submit "发送重置邮件", class: "btn btn-primary" %>
<% end %>

额外注意事项： #

• 别忘了给users表添加reset_digest（string类型）和reset_sent_at（datetime类型）字段，用迁移命令：

  rails generate migration AddResetDigestToUsers reset_digest:string reset_sent_at:datetime
  rails db:migrate
  

• 后续还需要实现密码重置的确认页面（edit动作）和更新密码的逻辑，这部分可以等你完成当前步骤后再扩展。

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