如何在Rails 5控制器中更新另一个模型的属性？

首先，先梳理下你现有代码里的几个小问题，再一步步实现更新其他模型的需求：

1. 优化现有代码的基础逻辑 #

你的update动作里有几个可以改进的地方，同时能让后续更新其他模型的逻辑更清晰：

• 用Rails关联直接获取@game，不用手动find_by_id，更高效且避免N+1查询
• 用update替代update_attributes（后者在Rails 5+已被弃用），它会直接返回布尔值，判断保存是否成功
• 只有在棋子保存成功后，再执行更新其他模型的操作，避免无效操作
• 把@move = @piece.moves.all移到else分支里，只有渲染编辑页面时才加载，优化性能

修正后的基础update动作：

def update
  @piece = ChessPiece.find(params[:id])
  @game = @piece.game # 直接通过关联获取游戏，无需手动查询

  if @piece.update(pieces_params)
    # 在这里调用方法更新其他模型
    update_related_models
    redirect_to game_path(@game), notice: "Chess piece updated successfully."
  else
    @move = @piece.moves.all # 仅渲染编辑页时加载移动记录
    render :edit, status: :unprocessable_entity
  end
end

2. 实现更新其他模型的具体逻辑 #

接下来，你可以创建一个专门的方法（比如update_related_models）来处理其他模型的更新，根据你的需求不同，这里提供几个常见场景的示例：

场景1：更新关联的Game模型属性

比如你想记录游戏的最后移动时间、最后移动的棋子ID：

def update_related_models
  # 更新Game模型的指定属性
  update_success = @game.update(
    last_move_at: Time.current,
    last_moved_piece_id: @piece.id
  )

  # 可选：处理更新失败的情况（比如Game有验证规则）
  unless update_success
    Rails.logger.error "Failed to update game ##{@game.id}: #{@game.errors.full_messages.join(', ')}"
    # 也可以添加flash警告给用户
    # flash[:alert] = "Chess piece updated, but game metadata failed to save."
  end

  # 如果你还要保留原来的update_moves方法，可以在这里调用它
  update_moves
end

# 修正你未完成的update_moves方法
def update_moves
  puts "The piece has been updated!"
  # 这里可以添加更多移动相关的模型更新逻辑，比如标记最新移动记录
  @piece.moves.last&.update(is_latest: true)
end

场景2：更新无关模型（比如当前用户的属性）

如果需要更新当前登录用户的某个字段（比如最后操作时间）：

def update_related_models
  # 假设你用Devise之类的认证系统，current_user是当前登录用户
  current_user.update(last_chess_action_at: Time.current)
end

场景3：快速更新时间戳字段

如果只是需要更新模型的时间戳（比如Game的updated_at或者自定义的last_activity_at），可以用Rails的touch方法，更简洁：

def update_related_models
  # 仅更新Game的updated_at字段
  @game.touch
  # 或者更新自定义的时间戳字段
  @game.touch(:last_move_at)
end

关键注意事项 #

• 只在保存成功后执行：确保只有当棋子的更新验证通过、保存成功时，才更新其他模型，避免数据不一致
• 利用关联查询：尽量通过模型间的关联（比如@piece.game）获取对象，不要手动用find_by_id，减少数据库查询次数
• 处理错误：如果其他模型的更新有验证规则，记得处理更新失败的情况，避免静默失败

内容的提问来源于stack exchange，提问作者Hans Martin Hanken