Состояние гонки ASP.NET/статического класса?

Если приложение ASP.NET обрабатывает только один запрос за раз, все сайты ASP.NET столкнутся с серьезными проблемами. ASP.NET может обрабатывать десятки одновременно (обычно 25 на ядро ​​ЦП).

Вы должны использовать кэш ASP.NET вместо использования собственного словаря для хранения вашего объекта. Операции с кешем потокобезопасны.

Обратите внимание, что вы должны быть уверены, что операция чтения объекта, который вы храните в кеше, является потокобезопасной, к сожалению, большинство классов .NET просто заявляют, что члены экземпляра не являются потокобезопасными, не пытаясь указать что-либо, что может быть.

Изменить:

В комментарии к этому ответу говорится: -

Только атомарные операции с кешем являются потокобезопасными. Если вы делаете что-то вроде проверки
, существует ли ключ, а затем добавляете его, это НЕ является потокобезопасным и может привести к перезаписи элемента
.

Стоит отметить, что если мы чувствуем, что нам нужно сделать такую ​​операцию атомарной, то кеш, вероятно, не является подходящим местом для ресурса.

У меня есть довольно много кода, который работает точно так, как описано в комментарии. Однако сохраняемый ресурс будет одинаковым в обоих местах. Следовательно, если существующий элемент в редких случаях перезаписывается, единственной ценой является то, что один поток без необходимости генерирует ресурс. Стоимость этого редкого события намного меньше, чем стоимость попытки сделать операцию атомарной каждый раз, когда делается попытка доступа к ней.

Это очень легко исправить:

private _clientsLock = new Object();

public static ClientData GetClientData(Guid fk_client)
{
  if (_clients == null)
    lock (_clientsLock)
      // Check again because another thread could have created a new 
      // dictionary in-between the lock and this check
      if (_clients == null) 
        _clients = new Dictionary<Guid, ClientData>();

  if (_clients.ContainsKey(fk_client))
    // Don't need a lock here UNLESS there are also deletes. If there are
    // deletes, then a lock like the one below (in the else) is necessary
    return _clients[fk_client];
  else
  {
    ClientData client = new ClientData(fk_client);

    lock (_clientsLock)
      // Again, check again because another thread could have added this
      // this ClientData between the last ContainsKey check and this add
      if (!clients.ContainsKey(fk_client))
       _clients.Add(fk_client, client);

    return client;
  }
}

Имейте в виду, что всякий раз, когда вы возитесь со статическими классами, у вас могут возникнуть проблемы с синхронизацией потоков. Если есть какой-то статический список на уровне класса (в данном случае _clients, объект Dictionary), ОПРЕДЕЛЕННО возникнут проблемы с синхронизацией потоков.

Ваш код действительно предполагает, что в каждый момент времени в функции находится только один поток.

Этого просто не будет в ASP.NET.

Если вы настаиваете на этом, используйте статический семафор, чтобы заблокировать область вокруг этого класса.

вам нужна многопоточность и минимальная блокировка.
см. блокировку с двойной проверкой (http://en.wikipedia.org/wiki/Double-checked_locking)

напишите просто с TryGetValue.

public static object lockClientsSingleton = new object();

public static ClientData GetClientData(Guid fk_client)
{
    if (_clients == null) {
        lock( lockClientsSingleton ) {
            if( _clients==null ) {
                _clients = new Dictionary``();
            }
        }
    }
    ClientData client;
    if( !_clients.TryGetValue( fk_client, out client ) )
    {
        lock(_clients) 
        {
            if( !_clients.TryGetValue( fk_client, out client ) ) 
            {
                client = new ClientData(fk_client)
                _clients.Add( fk_client, client );
            }
        }
    }
    return client;
}