Ruby on Rails: If Else 基于 params[:example]

Question

我有一个控制器订单。该控制器根据 POST 请求执行不同的操作。 当用户购买产品时，它会进入他的库存。所以只能销售库存产品。 Stock 表有一个 product_id 列。简单来说，当用户在 products/product_id 页面上时，订单必须是买入，而在股票/stock_id 页面上时，订单必须是卖出。

Routes.rb:

resources :products, only: [:index, :show] do
  resources :orders, only: [:create]
end

resources :stocks, only: [:index, :show] do
  resources :orders, only: [:create]
end

耙子路线：

products_orders POST       /products/:product_id/orders(.:format) orders#create
products_index GET        /products(.:format) products#index
product GET        /products/:id(.:format) products#show
stocks_orders POST       /stocks/:stock_id/orders(.:format) orders#create
stocks_index GET        /stocks(.:format) stocks#index
stock GET        /stock/:id(.:format) stocks#show

我的模特：

class Order < ApplicationRecord
    belongs_to :product
end

class Product < ApplicationRecord
    has_many :orders
    has_many :stocks
end

class Stock < ApplicationRecord
    belongs_to :product
end

所以我这样做了：

class OrdersController < ApplicationController
def create
    if params[:product_id].present?
        order.type = 'buy'

    elsif params[:stock_id].present?
        order.type = 'sell'
    end
end
end

这段代码安全吗？有没有办法做得更好？ 根据 params[:product_id] 和 params[:stock_id] 使用此代码是否正确？

if params[:product_id].present?
    order.type = 'buy'

elsif params[:stock_id].present?
    order.type = 'sell'
end

是否可以通过某种方式在请求中注入参数？ 例如，将 params[:product_id] 注入到 stoks/stock_id/orders 会造成一些损害吗？例如卷曲。 非常感谢。

Answer 1

是否可以通过某种方式在请求中注入参数？例如，将 params[:product_id] 注入到 stoks/stock_id/orders 会造成一些损害吗？例如 curl

是的。没有什么能阻止您向/stocks/123/orders?product_id=123 发送请求。因此，params[:product_id] 将出现，订单将收到错误的类型。

为每种类型的订单创建一个单独的控制器怎么样？

resources :products, only: [:index, :show] do
  resources :product_orders, only: [:create]
end

resources :stocks, only: [:index, :show] do
  resources :stock_orders, only: [:create]
end

那么ProductOrdersController 就这么简单：

class ProductOrdersController < ApplicationController
  def create
    order.type = 'buy'
  end
end

那么StockOrdersController 会变成这样：

class StockOrdersController < ApplicationController
  def create
    order.type = 'sell'
  end
end

这将帮助您停止依赖传递的params 并摆脱if 语句。