Click here to Skip to main content
15,507,526 members
Home / Discussions / C#
   

C#

 
AnswerRe: C# Performance Issue Pin
Richard MacCutchan17-Aug-14 21:41
mveRichard MacCutchan17-Aug-14 21:41 
GeneralRe: C# Performance Issue Pin
Mycroft Holmes17-Aug-14 21:59
professionalMycroft Holmes17-Aug-14 21:59 
GeneralRe: C# Performance Issue Pin
Richard MacCutchan17-Aug-14 22:34
mveRichard MacCutchan17-Aug-14 22:34 
GeneralRe: C# Performance Issue Pin
SledgeHammer0118-Aug-14 5:46
SledgeHammer0118-Aug-14 5:46 
AnswerRe: C# Performance Issue Pin
Rob Philpott18-Aug-14 0:45
Rob Philpott18-Aug-14 0:45 
AnswerRe: C# Performance Issue Pin
Richard Deeming18-Aug-14 3:30
mveRichard Deeming18-Aug-14 3:30 
GeneralRe: C# Performance Issue Pin
SledgeHammer0118-Aug-14 5:51
SledgeHammer0118-Aug-14 5:51 
GeneralRe: C# Performance Issue Pin
Richard Deeming18-Aug-14 8:31
mveRichard Deeming18-Aug-14 8:31 
Your second example should look something like:
C#
theValue = concurrentDict.GetOrAdd(theKey, key =>
{
    DoSomeOtherStuff();
    DoAFewMoreThings();
    
    var theNewObject = NewUpAnObject(key);
    return theNewObject;
});

The entire delegate would be executed within the scope of a lock, as with your non-generic dictionary code. However, the lock would be more granular, based on the number of buckets and the concurrency level of the collection.

If you wanted to execute the "other stuff" outside of the lock, your code would look like:
C#
if (!concurrentDict.TryGetValue(theKey, out theValue))
{
    DoSomeOtherStuff();
    DoAFewMoreThings();
    
    var theNewObject = NewUpAnObject(theKey);
    theValue = concurrentDict.GetOrAdd(theKey, theNewObject);
    
    // Or:
    // theValue = concurrentDict.GetOrAdd(theKey, key => NewUpAnObject(key));
}

In this case, DoSomeOtherStuff, DoAFewMoreThings and NewUpAnObject would run outside of any lock. However, the insert to the dictionary would not suffer from a race condition, as the class is specifically written to cope with this type of code.



"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer


GeneralRe: C# Performance Issue Pin
SledgeHammer0118-Aug-14 9:31
SledgeHammer0118-Aug-14 9:31 
AnswerRe: C# Performance Issue Pin
Pete O'Hanlon18-Aug-14 4:09
mvaPete O'Hanlon18-Aug-14 4:09 
GeneralRe: C# Performance Issue Pin
SledgeHammer0118-Aug-14 5:52
SledgeHammer0118-Aug-14 5:52 
GeneralRe: C# Performance Issue Pin
Pete O'Hanlon18-Aug-14 5:56
mvaPete O'Hanlon18-Aug-14 5:56 
AnswerRe: C# Performance Issue Pin
jschell18-Aug-14 12:51
jschell18-Aug-14 12:51 
QuestionC#/JS problematic iteration with table... Pin
OmegaExtern16-Aug-14 18:23
OmegaExtern16-Aug-14 18:23 
AnswerRe: C#/JS problematic iteration with table... Pin
Richard MacCutchan16-Aug-14 22:11
mveRichard MacCutchan16-Aug-14 22:11 
QuestionC# Keybind for Autoclicker Pin
Aaudiio16-Aug-14 17:39
Aaudiio16-Aug-14 17:39 
AnswerRe: C# Keybind for Autoclicker Pin
Zain Ul Abidin16-Aug-14 20:49
Zain Ul Abidin16-Aug-14 20:49 
AnswerRe: C# Keybind for Autoclicker Pin
OmegaExtern16-Aug-14 21:44
OmegaExtern16-Aug-14 21:44 
GeneralRe: C# Keybind for Autoclicker Pin
Aaudiio16-Aug-14 21:47
Aaudiio16-Aug-14 21:47 
GeneralRe: C# Keybind for Autoclicker Pin
OmegaExtern16-Aug-14 21:50
OmegaExtern16-Aug-14 21:50 
Questioncheck array value exists Pin
scottichrosaviakosmos16-Aug-14 4:11
scottichrosaviakosmos16-Aug-14 4:11 
AnswerRe: check array value exists Pin
onelopez16-Aug-14 4:34
onelopez16-Aug-14 4:34 
GeneralRe: check array value exists Pin
scottichrosaviakosmos16-Aug-14 5:03
scottichrosaviakosmos16-Aug-14 5:03 
AnswerRe: check array value exists Pin
OriginalGriff16-Aug-14 4:39
mveOriginalGriff16-Aug-14 4:39 
QuestionCatching exceptions on backgroundworker Pin
pjank4215-Aug-14 20:44
pjank4215-Aug-14 20:44 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.