使用Lock来停止并发执行不起作用?

时间:2013-07-04 14:06:29

标签: c# .net azure concurrency locking

我有一个API,我在其中进行大量处理,例如发送服务总线队列消息并接收它,向表添加条目,然后最终向socket.io服务器发送事件。我希望所有这些都受到并发执行的保护。我正在使用Lock,但它似乎没有做到这一点。我错过了什么吗?下面是我的代码

 public class BroadcastController : ApiController
{
private readonly Object _Lock = new Object();

    [HttpPost]
    [ActionName("UploadRecording")]
    public async Task<HttpResponseMessage> UploadRecording()
    {
        // Check if the request contains multipart/form-data.
        if (!Request.Content.IsMimeMultipartContent())
        {
            throw new HttpResponseException(HttpStatusCode.UnsupportedMediaType);
        }

        string path = Path.GetTempPath();            

        var provider = new MultipartFormDataStreamProvider(path);                       

        // Read the form data and return an async task.
        var response = await Request.Content.ReadAsMultipartAsync(provider);

// processing the mime content

lock (_Lock)
            {

                // sending and receiving service bus messages
                // adding records to table
               // sending an event to socket.io

return Request.CreateResponse(HttpStatusCode.OK, new ResponseMessage<PersistedAudioRecord> { SuccessCode = 1, Message = "Broadcast Uploaded", Data = updatedRecord });
}   }    }

1 个答案:

答案 0 :(得分:5)

使_Lock对象保持静态。否则,您为控制器的每个实例使用不同的锁。由于框架创建了一个新的控制器实例来处理每个请求,因此每个请求都锁定在不同的_Lock对象上,因此不会提供并发安全性。

但请注意,即使是静态锁定对象,只有拥有一台服务器才能生效。如果您有多个服务器处理请求,则需要通过其他方式管理并发。