ConcurrentDictionaryを使用するために、Dictionaryを使用していくつかのコードを書き直そうとしています。いくつかの例を確認しましたが、AddOrUpdate関数の実装にまだ問題があります。これは元のコードです:
dynamic a = HttpContext;
Dictionary<int, string> userDic = this.HttpContext.Application["UserSessionList"] as Dictionary<int, String>;
if (userDic != null)
{
if (useDic.ContainsKey(authUser.UserId))
{
userDic.Remove(authUser.UserId);
}
}
else
{
userDic = new Dictionary<int,string>();
}
userDic.Add(authUser.UserId, a.Session.SessionID.ToString());
this.HttpContext.Application["UserDic"] = userDic;
更新部分に追加するものがわかりません:
userDic.AddOrUpdate(authUser.UserId,
a.Session.SessionID.ToString(),
/*** what to add here? ***/);
任意のポインタをいただければ幸いです。
更新の場合に辞書に保存される値を返すFunc
を渡す必要があります。あなたの場合(追加と更新を区別しないため)、これは次のようになります:
var sessionId = a.Session.SessionID.ToString();
userDic.AddOrUpdate(
authUser.UserId,
sessionId,
(key, oldValue) => sessionId);
つまりFunc
は常にsessionIdを返すため、追加と更新の両方で同じ値が設定されます。
ところで: MSDNページ にサンプルがあります。
私はあなたの質問に何も見逃していないことを願っていますが、なぜこれが好きではないのですか?簡単で、アトミックで、スレッドセーフです(以下を参照)。
userDic[authUser.UserId] = sessionId;
キー/値ペアを無条件に辞書に保存し、キーが既に存在する場合はそのキーの値を上書きします:インデクサーのセッターを使用します
(参照: http://blogs.msdn.com/b/pfxteam/archive/2010/01/08/9945809.aspx )
インデクサーもアトミックです。代わりに関数を渡すと、次のようになりません。
これらの操作はすべてアトミックであり、ConcurrentDictionaryの他のすべての操作に関してスレッドセーフです。各操作の原子性に対する唯一の注意点は、デリゲート、つまりAddOrUpdateとGetOrAddを受け入れる操作です。 [...]これらのデリゲートはロックの外側で呼び出されます
参照: http://blogs.msdn.com/b/pfxteam/archive/2010/01/08/9945809.aspx
私は拡張メソッドを実装することになりました:
static class ExtensionMethods
{
// Either Add or overwrite
public static void AddOrUpdate<K, V>(this ConcurrentDictionary<K, V> dictionary, K key, V value)
{
dictionary.AddOrUpdate(key, value, (oldkey, oldvalue) => value);
}
}
興味がある人のために、私は現在、新しい値を強制するのではなく、既存の値である「oldValue」を使用するための素晴らしい例であるケースを実装しています(個人的には「oldValue」という用語は好きではありません並列スレッド内からわずか数プロセッサティック前に作成されたとき).
dictionaryCacheQueues.AddOrUpdate(
uid,
new ConcurrentQueue<T>(),
(existingUid, existingValue) => existingValue
);