Rails API自定义方法无法获取路由动态ID，如何正确传递参数？

核心问题 #

1. 类型不匹配：路由传递的params[:id]是字符串类型，而Order模型的user_id是整数类型，直接用==对比会因类型不同返回false，导致筛选出空数组。
2. 低效的内存筛选：你用了Ruby数组的select方法，会先把所有Order记录加载到内存再筛选，数据量大时性能极差，且完全没必要。

推荐解决方案（数据库层面筛选） #

改用ActiveRecord的where方法，它会在数据库端完成筛选，性能更优，且ActiveRecord会自动处理类型转换：

def user_orders
  orders = Order.where(user_id: params[:id])
  if orders.exists?
    render json: orders, status: :ok
  else
    render json: {error: "No orders available"}, status: :not_found
  end
end

• where(user_id: params[:id])：直接生成SQL查询条件，数据库会自动将字符串params[:id]转换为整数匹配user_id。
• orders.exists?：替代原有的if orders，因为ActiveRecord::Relation对象即使为空也是真值，必须用exists?判断是否存在匹配数据。

临时修复（若坚持用内存筛选） #

如果一定要用Ruby数组的select，需先将params[:id]转换为整数，同时规范属性访问方式：

def user_orders
  target_user_id = params[:id].to_i
  orders = Order.select { |item| item.user_id == target_user_id }
  if orders.present?
    render json: orders, status: :ok
  else
    render json: {error: "No orders available"}, status: :not_found
  end
end

• params[:id].to_i：将字符串ID转为整数，确保和user_id类型一致。
• item.user_id：用模型属性访问器替代item[:user_id]，更符合Rails规范。
• orders.present?：判断数组是否非空，避免空数组被误判为真值。

路由说明 #

你的路由配置get "/orders/user/:id", to: "orders#user_orders"是正确的，params[:id]确实能获取到路由中的动态ID，问题出在后续的类型处理和筛选逻辑上。

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