Rails4.2.8记录name字段更新时间及POST路由错误求助

我来帮你一步步搞定这两个问题——先解决眼前的路由报错，再完善姓名修改时间的逻辑。

一、先解决「No route matches [POST] "/users/udaskffa"」路由错误 #

这个报错的核心原因很明确：更新用户时表单发送了POST请求，但Rails的RESTful规则里，更新资源对应的是PATCH/PUT请求，而非POST。结合你使用friendly_id的场景，具体排查和修复步骤如下：

1. 检查路由配置是否正确 #

先打开config/routes.rb，确保你配置了标准的用户REST资源路由：

resources :users

这行代码会自动生成包括PATCH /users/:id（更新用户）在内的所有标准REST路由。如果你的路由是自定义的，一定要确保有对应的PATCH/PUT路由指向用户更新的action。

2. 强制表单使用PATCH请求 #

Rails的form_for本来会根据@user是否是新记录自动选择请求方法：新用户用POST，已存在用户用PATCH。但如果你的@user对象加载有问题（比如没正确从数据库读取，或者persisted?返回false），就会错误发送POST请求。你可以手动指定请求方法来强制使用PATCH：

<%= form_for @user, method: :patch do |f| %>
  <!-- 原来的表单内容保持不变 -->
<% end %>

3. 确认friendly_id的加载逻辑 #

你的should_generate_new_friendly_id?设置为name_changed?，意味着修改姓名会生成新的slug。要确保在渲染表单前，@user已经通过slug正确加载，比如在UsersController#edit方法中：

def edit
  @user = User.friendly.find(params[:id])
end

用friendly.find才能正确通过slug匹配到用户，避免路由匹配时找不到对应资源。

二、完善姓名修改时间的逻辑（解决单独追踪name更新时间的问题） #

你当前的代码还有几个需要补全的地方，才能实现“30天后可再次修改姓名”的需求：

1. 给数据库添加name_updated_at字段 #

你现在的代码里用到了name_updated_at，但数据库中还没有这个字段，先生成迁移文件添加它：

rails generate migration AddNameUpdatedAtToUsers name_updated_at:datetime
rake db:migrate

这样才能单独存储姓名的更新时间，不跟全局的updated_at混在一起。

2. 优化模型中的逻辑 #

你原来用validate来设置name_updated_at，其实用回调更合适，因为只有当姓名确实变更且要保存时，才需要更新这个时间戳。同时还要添加验证逻辑，确保30天内不能重复修改姓名：

class User < ActiveRecord::Base
  extend FriendlyId
  friendly_id :name, use: [:slugged, :finders]

  # 只有姓名变更时，更新name_updated_at
  before_update :update_name_updated_at, if: :name_changed?

  # 验证：30天内不能重复修改姓名
  validate :name_change_limit, if: :name_changed?

  def should_generate_new_friendly_id?
    name_changed?
  end

  private

  def update_name_updated_at
    self.name_updated_at = Time.current
  end

  def name_change_limit
    if name_updated_at.present? && (Time.current - name_updated_at) < 30.days
      errors.add(:name, "30天后可再次修改姓名")
    end
  end
end

这样拆分后逻辑更清晰：回调负责更新时间戳，验证负责检查时间限制，而且只有在姓名变更时才触发这两个操作。

3. 简化Helper和视图逻辑 #

你的edit_day Helper可以简化，还要处理用户第一次修改姓名（name_updated_at为空）的情况：

# user_helper.rb
def edit_day(user)
  return 0 unless user.name_updated_at.present?
  remaining_days = (user.name_updated_at + 30.days - Time.current).to_i / 1.day
  remaining_days > 0 ? remaining_days : 0
end

然后在视图中调用时，直接传入@user，避免依赖全局变量：

<span class="text-gray fs12">30天内只能修改一次 <span>还剩 <span id="editDays"><%= edit_day(@user) %></span> 天可以修改姓名</span></span>

4. 控制器更新逻辑要正确 #

确保UsersController#update能正确处理表单提交，验证失败时返回编辑页面：

class UsersController < ApplicationController
  def edit
    @user = User.friendly.find(params[:id])
  end

  def update
    @user = User.friendly.find(params[:id])
    if @user.update(user_params)
      redirect_to @user, notice: "信息更新成功"
    else
      render :edit
    end
  end

  private

  def user_params
    params.require(:user).permit(:name, :phone, :email, :address)
  end
end

三、最后测试验证 #

1. 启动服务器，进入用户编辑页面，尝试修改姓名：
   • 如果距离上次修改不足30天，应该看到验证错误提示
   • 修改成功后，查看数据库的name_updated_at字段，应该显示当前时间
2. 检查浏览器的网络请求，表单提交的方法应该是PATCH，不会再出现路由报错

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