Is this static method threadsafe?

[Diffident]

Hello All,

Following is a static method. Can you please tell me if this is
threadsafe...wh y? If it is not threadsafe...wh y?

Thank you.

public static string GetCachedApplic ationWideObject (string HashtableKey)
{
//Cache["DefaultSampleQ Page"] holds the hashtable
if(Cache["DefaultSampleQ Page"]==null)
{
CacheApplicatio nWideObjects();
_sHTable = ((Hashtable)Cac he["DefaultSampleQ Page"]);
return ((string)HTable[HashtableKey]);
}
else
{
_sHTable = ((Hashtable)Cac he["DefaultSampleQ Page"]);
return ((string)HTable[HashtableKey]);
}
}

[Diffident]

I forgot to add this information:

_sHTable is a static variable which is like a const variable for the entire
application; so no problem if two threads access that information at the same
time.

But my only question is if the parameter 1 is passed by thread 1 and
parameter 2 is passed by thread 2 is there any possibility that thread 1 will
be returned thread 2's result and vice versa?

Thanks

"Diffident" wrote:

Hello All,

Following is a static method. Can you please tell me if this is
threadsafe...wh y? If it is not threadsafe...wh y?

Thank you.

public static string GetCachedApplic ationWideObject (string HashtableKey)
{
//Cache["DefaultSampleQ Page"] holds the hashtable
if(Cache["DefaultSampleQ Page"]==null)
{
CacheApplicatio nWideObjects();
_sHTable = ((Hashtable)Cac he["DefaultSampleQ Page"]);
return ((string)HTable[HashtableKey]);
}
else
{
_sHTable = ((Hashtable)Cac he["DefaultSampleQ Page"]);
return ((string)HTable[HashtableKey]);
}
}

[Peter Bromberg [C# MVP]]

If all you are ever doing is reading from the Cache, i fail to see where the
problem lies, because the data will always be the same.
It's when you might have another thread that attempts to WRITE to the same
Cache item that you could run into issues.

Take a look at the lock keyword. Better yet, read MVP Jon Skeet's articles
on the subject:

http://www.yoda.arachsys.com/csharp/threads/

Peter
--
Co-founder, Eggheadcafe.com developer portal:
http://www.eggheadcafe.com
UnBlog:
http://petesbloggerama.blogspot.com


"Diffident" wrote:

Hello All,

Following is a static method. Can you please tell me if this is
threadsafe...wh y? If it is not threadsafe...wh y?

Thank you.

public static string GetCachedApplic ationWideObject (string HashtableKey)
{
//Cache["DefaultSampleQ Page"] holds the hashtable
if(Cache["DefaultSampleQ Page"]==null)
{
CacheApplicatio nWideObjects();
_sHTable = ((Hashtable)Cac he["DefaultSampleQ Page"]);
return ((string)HTable[HashtableKey]);
}
else
{
_sHTable = ((Hashtable)Cac he["DefaultSampleQ Page"]);
return ((string)HTable[HashtableKey]);
}
}

[Bruce Barker]

no its not threadsafe. you do not serialize access to the hashtable, so race
condiditons can occur. also two threads can run CacheApplicatio nWideObjects
at the same time (is it threadsafe?)

the treading issue you worried about should not be a problem if
CacheApplicatio nWideObjects is threadsafe.

-- bruce (sqlwork.com)


"Diffident" <Di*******@disc ussions.microso ft.com> wrote in message
news:26******** *************** ***********@mic rosoft.com...

I forgot to add this information:

_sHTable is a static variable which is like a const variable for the
entire
application; so no problem if two threads access that information at the
same
time.

But my only question is if the parameter 1 is passed by thread 1 and
parameter 2 is passed by thread 2 is there any possibility that thread 1
will
be returned thread 2's result and vice versa?

Thanks

"Diffident" wrote:

Hello All,

Following is a static method. Can you please tell me if this is
threadsafe...wh y? If it is not threadsafe...wh y?

Thank you.

public static string GetCachedApplic ationWideObject (string HashtableKey)
{
//Cache["DefaultSampleQ Page"] holds the hashtable
if(Cache["DefaultSampleQ Page"]==null)
{
CacheApplicatio nWideObjects();
_sHTable = ((Hashtable)Cac he["DefaultSampleQ Page"]);
return ((string)HTable[HashtableKey]);
}
else
{
_sHTable = ((Hashtable)Cac he["DefaultSampleQ Page"]);
return ((string)HTable[HashtableKey]);
}
}