Словарь/Хашмап ......... что происходит на земле?

Question

else if (!registryData.ContainsKey(keyName)) 
{ 
    keyInvolved = new RegistryKy(keyName); 
    lock (registryDataLock) 
    { 
     registryData.Add(keyName, keyInvolved); 
    } 
    processInvolved = new Proces(procInvolved); 
    keyInvolved.addProcessToDict(processInvolved); 
} 

keyName - это строка, представляющая раздел реестра. keyInvolved - это фактический объект ключа реестра.

Мне говорят, что я добавляю ключ, который уже существует, но я уже проверил, есть ли он там или нет ???

Answer 1

Это удар в темноте, но тот факт, что у вас есть lock на registryDataLock, говорит мне, что это многопоточность. Возможно ли, что другой поток добавляет ключ в словарь после вызова ContainsKey, но перед вызовом Add?

Кроме того, аналогичным именованные переменные делают этот код довольно трудно читать ...

Answer 2

Это должно быть так:

else if (!registryData.ContainsKey(kyInvolved)) 
{ 
    //keyInvolved = new RegistryKy(kyInvolved); 
    lock (registryDataLock) 
    { 
     //registryData.Add(kyInvolved, keyInvolved); 
     registryData.Add(kyInvolved, new RegistryKy(kyInvolved)); 
    } 
    processInvolved = new Proces(procInvolved); 
    keyInvolved.addProcessToDict(processInvolved); 
} 

Answer 3

Вы должны вызвать ContainsKey и Add в пределах одного блока lock , иначе другой поток может добавить ключ между моментом, когда вы вызываете ContainsKey, и вы получаете блокировку. Это один из способов сделать это:

// check to see if the key exists 
else if (!registryData.ContainsKey(keyName)) 
{ 
    bool foundKey; 
    // lock the dictionary 
    lock (registryDataLock) 
    { 
     // make sure another thread didn't add the key while waiting on the lock 
     if (!(foundKey = registryData.ContainsKey(keyName))) 
     { 
      keyInvolved = new RegistryKy(keyName); 
      registryData.Add(keyName, keyInvolved); 
     } 
     // release the lock as soon as we're done with registryData 
    } 
    // now perform operations that need to be done when we've added 
    // a key but without holding the registry lock 
    if (!foundKey) 
    { 
     processInvolved = new Proces(procInvolved); 
     keyInvolved.addProcessToDict(processInvolved); 
    } 
}