如何仅允许一个请求执行诉讼并拒绝所有其他并发请求？

Question

当多个并发请求尝试调用控制器拒绝操作时。在竞赛条件下，它多次生成相同的注释，尽管request> request和order被拒绝。但是在种族条件下，请求无法识别订单已经被其中一个请求之一拒绝，因为所有这些过程都在很少的时间内执行订单更改为拒绝。为了避免此竞赛条件问题，我已锁定请求和订单在DB行级别。但这不起作用。这是我的示例代码：

我使用Ruby 2.5.8，Rails 5.0.7。

我的模型

• 订单有许多请求
• 请求有一个批准者

class Request < ApplicationRecord
  def self.reject_request(key)
    request = Request.where(:reject_key => key).first
    order = request&.order
    raise ActiveRecord::Rollback if request.blank? || order.blank?
    raise ActiveRecord::Rollback if request.rejected? || order.rejected?

    order.lock!
    request.lock!

    request.status = 'rejected'
    request.save!
    order.update(status: 'rejected')
    true
  end
end

我的控制器

class OrdersController < ApplicationController
  def reject
    @request ||= Request.find_by(reject_key: params[:reject_key])
    @order = @request&.purchase_order

    ActiveRecord::Base.transaction do
      rejected = Request.reject_request(params[:reject_key])
      @order.comments.create(comment: "Order rejected by approver: #{@request.approver.name}") if rejected
      if rejected
        redirect_to root_path, notice: 'Rejected'
      else
        redirect_to root_path, notice: 'Try again!'
      end
    end
  end
end

问题

应该只有一个正确的评论如下所示。

批准者拒绝的订单：乔（正确）

批准者拒绝的订单：杰克（不正确）

批准者拒绝的订单：乔（不正确）

批准者拒绝的订单：杰克（不正确）

当拒绝行动执行拒绝时，它在执行拒绝时工作通过应用程序链接。但是，当多个请求通过多线程流程运行时，它就无法正常工作。如何仅允许一个请求执行该动作并拒绝所有其他行动？任何反馈都将不胜感激，谢谢！

Answer 1

在检查状态之前，您需要锁定订单。

如果2个并发请求执行检查请求。 ||订单。拒绝？与您的解决方案同时获得false，并在锁定下进行。

更改您的模型

class Request < ApplicationRecord
  def self.reject_request(key)
    request = Request.where(:reject_key => key).first
    order = request&.order

    # make sure both request and order objects are not empty
    raise ActiveRecord::Rollback if request.blank? || order.blank?

    # lock your object
    order.lock!
    request.lock!

    # put your conditions here after the `lock`
    raise ActiveRecord::Rollback if request.rejected? || order.rejected?


    request.status = 'rejected'
    request.save!
    order.update(status: 'rejected')
    true
  end
end