Skipping locked section already in use
-
Quick Q - I have a critical section which is to be executed only once but is invoked by many scenarios. How can I execute this thread proc and skip all the rest of the calls? Thanks in Advance -- Varun
-
Quick Q - I have a critical section which is to be executed only once but is invoked by many scenarios. How can I execute this thread proc and skip all the rest of the calls? Thanks in Advance -- Varun
Do you mean something like what's used in a Singleton to enable lazy instantiation? Have an object to lock on and something to indicate that the code has already been called.
private object tolock = new object() ;
private bool initialized = false ;...
if ( !initialized )
{
lock ( tolock )
{
if ( !initialized )
{
/* do whatever */initialized = true ; }}
}And if you are writing a Singleton, please don't, or at least read http://csharpindepth.com/Articles/General/Singleton.aspx[^]
-
Do you mean something like what's used in a Singleton to enable lazy instantiation? Have an object to lock on and something to indicate that the code has already been called.
private object tolock = new object() ;
private bool initialized = false ;...
if ( !initialized )
{
lock ( tolock )
{
if ( !initialized )
{
/* do whatever */initialized = true ; }}
}And if you are writing a Singleton, please don't, or at least read http://csharpindepth.com/Articles/General/Singleton.aspx[^]
Thanx a lot... Still, you recommended not to use Singleton. Why? -- Varun
-
Thanx a lot... Still, you recommended not to use Singleton. Why? -- Varun
They're silly. I've never yet found a reason to use one. They seem to solve a problem that doesn't exist. They limit your future options. They were devised prior to C# -- in C# (since V2) a static class may be a better fit, otherwise just use a regular class. At any rate, if you do write a Singleton (and I can't stop you), follow Skeet's advice.
-
They're silly. I've never yet found a reason to use one. They seem to solve a problem that doesn't exist. They limit your future options. They were devised prior to C# -- in C# (since V2) a static class may be a better fit, otherwise just use a regular class. At any rate, if you do write a Singleton (and I can't stop you), follow Skeet's advice.
i would like to hear more about why singleton are silly... I've a case where a server application would manage a list of users, the server could add a user,
getLoggedUsers, log a user in, log out a user, get a list of all users and contact one user to send a chat request (the client contacts the server about the chat request, the server alerts the target user and then the users would chat directly), to implement the functionality of the contacts list (not a simple List, sadly) i've made a static class who controls the underling List, synchronizing it with a XML file (used to serialize the list), this way, all the users contacting the server would see the same list of users. i was thinking in replacing the static class with a singleton, you think it's a bad idea? i would appreciate your opinion.I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
-
i would like to hear more about why singleton are silly... I've a case where a server application would manage a list of users, the server could add a user,
getLoggedUsers, log a user in, log out a user, get a list of all users and contact one user to send a chat request (the client contacts the server about the chat request, the server alerts the target user and then the users would chat directly), to implement the functionality of the contacts list (not a simple List, sadly) i've made a static class who controls the underling List, synchronizing it with a XML file (used to serialize the list), this way, all the users contacting the server would see the same list of users. i was thinking in replacing the static class with a singleton, you think it's a bad idea? i would appreciate your opinion.I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
Sentenryu wrote:
i was thinking in replacing the static class with a singleton, you think it's a bad idea?
Mainly I just don't see the point. Does it work as a static class? What do you think a Singleton would provide that a static class doesn't? If the users are running in different systems then they wouldn't share the Singleton anyway -- and if it's behind a Service then it's just a black box so what difference does it make to the clients? If you've read up on the Singleton Pattern, then you should know that it provides a shared instance -- if it isn't going to be shared, then it probably isn't the right tool for the job. Nor am I sure that a static class is either; I'd likely just instantiate one instance of a regular class. When I wrote a chat system, it had a database and each client connected to the database server to log in/out, get the list of users, send/get messages, etc. -- I haven't gotten around to writing a Web Service for it yet, and haven't experimented with Web Services at all for two years. But I wouldn't automatically run off and write a Singleton to hide behind the Service.
-
Sentenryu wrote:
i was thinking in replacing the static class with a singleton, you think it's a bad idea?
Mainly I just don't see the point. Does it work as a static class? What do you think a Singleton would provide that a static class doesn't? If the users are running in different systems then they wouldn't share the Singleton anyway -- and if it's behind a Service then it's just a black box so what difference does it make to the clients? If you've read up on the Singleton Pattern, then you should know that it provides a shared instance -- if it isn't going to be shared, then it probably isn't the right tool for the job. Nor am I sure that a static class is either; I'd likely just instantiate one instance of a regular class. When I wrote a chat system, it had a database and each client connected to the database server to log in/out, get the list of users, send/get messages, etc. -- I haven't gotten around to writing a Web Service for it yet, and haven't experimented with Web Services at all for two years. But I wouldn't automatically run off and write a Singleton to hide behind the Service.
this "chat" is a submission to my university, i can't use a database and need to user a server application that communicates witch tcp/ip (if i can use a database, this wouldn't be a problem :) ) in theory it works with the static class, the problem is that i'm afraid it's not safe, what happens if two users connected (so, two threads) request the server to add a new user, then the server call the Add method 2 times simultaneously, the Add method writes a XML file and then reads it again (yes, poor implementation, hopefully not mine). the point in making it a static class is that in this way all the threads would have access to the same list of users, but i've seen very strange comportment of this class, in the static constructor, the class instantiate and fills the list of contacts, i've put a breakpoint here and tested, for some reason, when i called the add method, the static constructor was not executed, with scared the shit out of me, everything was null, then, when the method returned (the method executed with success, even with the List null, they were able to add to it, i don't know how) then the execution point moved to the static constructor, in my breakpoint. after much test, i've found that when 2 threads call a method of this class, the secund call execute before the static constructor, i think it's a bug with my installation of visual studio, i prefer not to consider that this can occur i in a production environment, but just in case...
I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
-
i would like to hear more about why singleton are silly... I've a case where a server application would manage a list of users, the server could add a user,
getLoggedUsers, log a user in, log out a user, get a list of all users and contact one user to send a chat request (the client contacts the server about the chat request, the server alerts the target user and then the users would chat directly), to implement the functionality of the contacts list (not a simple List, sadly) i've made a static class who controls the underling List, synchronizing it with a XML file (used to serialize the list), this way, all the users contacting the server would see the same list of users. i was thinking in replacing the static class with a singleton, you think it's a bad idea? i would appreciate your opinion.I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
The biggest problem with singletons lies in the fact that it reduces parallelism. As you only have a single instance present, any threaded operations must be serialized in and out of the singleton, reducing the efficiency of the threading. They can also make it harder to unit test code because it can introduce global state.
*pre-emptive celebratory nipple tassle jiggle* - Sean Ewington
"Mind bleach! Send me mind bleach!" - Nagy Vilmos
CodeStash - Online Snippet Management | My blog | MoXAML PowerToys | Mole 2010 - debugging made easier
-
The biggest problem with singletons lies in the fact that it reduces parallelism. As you only have a single instance present, any threaded operations must be serialized in and out of the singleton, reducing the efficiency of the threading. They can also make it harder to unit test code because it can introduce global state.
*pre-emptive celebratory nipple tassle jiggle* - Sean Ewington
"Mind bleach! Send me mind bleach!" - Nagy Vilmos
CodeStash - Online Snippet Management | My blog | MoXAML PowerToys | Mole 2010 - debugging made easier
i see, this really helps, in my case I've global state (the list of users), and i've to synchronize the access, so this wont help... i think i'll just rewrite this to use normal instances and a single shared user list, apparently, is the best solution at the moment, i'm working on this with a plenty of beginners and the only thing that will be sent to the appraiser, thanks :)
I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
-
this "chat" is a submission to my university, i can't use a database and need to user a server application that communicates witch tcp/ip (if i can use a database, this wouldn't be a problem :) ) in theory it works with the static class, the problem is that i'm afraid it's not safe, what happens if two users connected (so, two threads) request the server to add a new user, then the server call the Add method 2 times simultaneously, the Add method writes a XML file and then reads it again (yes, poor implementation, hopefully not mine). the point in making it a static class is that in this way all the threads would have access to the same list of users, but i've seen very strange comportment of this class, in the static constructor, the class instantiate and fills the list of contacts, i've put a breakpoint here and tested, for some reason, when i called the add method, the static constructor was not executed, with scared the shit out of me, everything was null, then, when the method returned (the method executed with success, even with the List null, they were able to add to it, i don't know how) then the execution point moved to the static constructor, in my breakpoint. after much test, i've found that when 2 threads call a method of this class, the secund call execute before the static constructor, i think it's a bug with my installation of visual studio, i prefer not to consider that this can occur i in a production environment, but just in case...
I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
Sentenryu wrote:
in theory it works with the static class, the problem is that i'm afraid it's not safe, what happens if two users connected (so, two threads) request the server to add a new user, then the server call the Add method 2 times simultaneously, the Add method writes a XML file and then reads it again (yes, poor implementation, hopefully not mine).
That question has nothing to do with whether it is a singleton or not.
Sentenryu wrote:
but i've seen very strange comportment of this class
That has nothing to do with the correct behavior of the class. Could be something odd with how you were using the debugger, could be mismatch in classes, could bug in your code or something even more exotic like a bug in the debugger. I wouldn't get to wrapped up in the implementation of the idea of a singleton. Conceptually a singleton is a representation of a single instance of a class. Nothing magical about that. You can use a static class to manage the access to a single instance (a different class) without strictly implementing the singleton pattern and yet still conceptually implement it.
-
The biggest problem with singletons lies in the fact that it reduces parallelism. As you only have a single instance present, any threaded operations must be serialized in and out of the singleton, reducing the efficiency of the threading. They can also make it harder to unit test code because it can introduce global state.
*pre-emptive celebratory nipple tassle jiggle* - Sean Ewington
"Mind bleach! Send me mind bleach!" - Nagy Vilmos
CodeStash - Online Snippet Management | My blog | MoXAML PowerToys | Mole 2010 - debugging made easier
Pete O'Hanlon wrote:
The biggest problem with singletons lies in the fact that it reduces parallelism. As you only have a single instance present, any threaded operations must be serialized in and out of the singleton, reducing the efficiency of the threading. They can also make it harder to unit test code because it can introduce global state.
I don't agree. The biggest problem with singletons is incorrect usage and from that overusage. For your points... For the first problem sometime the very nature of the need dictates serialization anyways. And in other cases there are ways that can allow multiple access. As for the second problem there are trivial solutions for most problems of that nature, for example just implementing a Reset() method.
-
this "chat" is a submission to my university, i can't use a database and need to user a server application that communicates witch tcp/ip (if i can use a database, this wouldn't be a problem :) ) in theory it works with the static class, the problem is that i'm afraid it's not safe, what happens if two users connected (so, two threads) request the server to add a new user, then the server call the Add method 2 times simultaneously, the Add method writes a XML file and then reads it again (yes, poor implementation, hopefully not mine). the point in making it a static class is that in this way all the threads would have access to the same list of users, but i've seen very strange comportment of this class, in the static constructor, the class instantiate and fills the list of contacts, i've put a breakpoint here and tested, for some reason, when i called the add method, the static constructor was not executed, with scared the shit out of me, everything was null, then, when the method returned (the method executed with success, even with the List null, they were able to add to it, i don't know how) then the execution point moved to the static constructor, in my breakpoint. after much test, i've found that when 2 threads call a method of this class, the secund call execute before the static constructor, i think it's a bug with my installation of visual studio, i prefer not to consider that this can occur i in a production environment, but just in case...
I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
Sentenryu wrote:
submission to my university
If it's classwork, then considering and trying out different ways of implementing it is a good thing. :thumbsup:
Sentenryu wrote:
if two users connected (so, two threads)
My understanding is that if they're in separate App Domains, then a Singleton won't help anyway -- each would have its own instance. And that would be true of a static class as well. You may need to look into a Mutex, which is sort of like the locking object, but system-wide.
-
Sentenryu wrote:
submission to my university
If it's classwork, then considering and trying out different ways of implementing it is a good thing. :thumbsup:
Sentenryu wrote:
if two users connected (so, two threads)
My understanding is that if they're in separate App Domains, then a Singleton won't help anyway -- each would have its own instance. And that would be true of a static class as well. You may need to look into a Mutex, which is sort of like the locking object, but system-wide.
now you have confused me, the console application that acts as the server is the only one who will use this class, what will be sent to the client is a xml response of their request. the threads are created in the server with a ThreadPool (subject to change, i think i'm using the wrong class, maybe i'm confusing it with java's ThreadPool...) in my understanding, all of those threads run in the App Domain of the console application, is this wrong?
I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
-
now you have confused me, the console application that acts as the server is the only one who will use this class, what will be sent to the client is a xml response of their request. the threads are created in the server with a ThreadPool (subject to change, i think i'm using the wrong class, maybe i'm confusing it with java's ThreadPool...) in my understanding, all of those threads run in the App Domain of the console application, is this wrong?
I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
Sentenryu wrote:
all of those threads run in the App Domain of the console application
Oh, yes, but then why do you have threads?
Sentenryu wrote:
the console application
The main method of which is static already, so I'd likely stick with static. Hmmm... so, if I understand what you're saying, each client makes a connection to the Server and gets its own thread, which I suppose is a session, and it continues to interact with the server that way until it disconnects? I'm not sure that's a good architecture*, but not being an expert on that sort of thing, I'd better keep quiet and let others provide guidance. * I'm fairly sure that a "connectionless" technique is more robust.
-
Sentenryu wrote:
i was thinking in replacing the static class with a singleton, you think it's a bad idea?
Mainly I just don't see the point. Does it work as a static class? What do you think a Singleton would provide that a static class doesn't? If the users are running in different systems then they wouldn't share the Singleton anyway -- and if it's behind a Service then it's just a black box so what difference does it make to the clients? If you've read up on the Singleton Pattern, then you should know that it provides a shared instance -- if it isn't going to be shared, then it probably isn't the right tool for the job. Nor am I sure that a static class is either; I'd likely just instantiate one instance of a regular class. When I wrote a chat system, it had a database and each client connected to the database server to log in/out, get the list of users, send/get messages, etc. -- I haven't gotten around to writing a Web Service for it yet, and haven't experimented with Web Services at all for two years. But I wouldn't automatically run off and write a Singleton to hide behind the Service.
PIEBALDconsult wrote:
What do you think a Singleton would provide that a static class doesn't?
The ability for the singleton to be passed as a parameter. /ravi
My new year resolution: 2048 x 1536 Home | Articles | My .NET bits | Freeware ravib(at)ravib(dot)com
-
PIEBALDconsult wrote:
What do you think a Singleton would provide that a static class doesn't?
The ability for the singleton to be passed as a parameter. /ravi
My new year resolution: 2048 x 1536 Home | Articles | My .NET bits | Freeware ravib(at)ravib(dot)com
Yes, but you're not supposed to do that with a Singleton -- being a Singleton means you don't have to pass it, all parts of the system know where to get it: "Provide a global point of access to the object."
-
Yes, but you're not supposed to do that with a Singleton -- being a Singleton means you don't have to pass it, all parts of the system know where to get it: "Provide a global point of access to the object."
There are cases where we can have different singletons, all of which implement an interface
IFoo(for example) and a method that has accepts anIFooparameter. Due to separation of concerns, the method may have no idea about the existence of the concrete singletons. For example:static class PrettyPrinter
{
public static string PrettyPrint
(string textToBePrettyPrinted,
IPrettyPrinterRuleProvider rulesToUse)
{
...
return prettyPrintedString;
}
}...
string text = "...";
string result1 = PrettyPrinter.PrettyPrint (text, KAndRTypeFormattingRules.Instance);
string result2 = PrettyPrinter.PrettyPrint (text, MicrosoftFormattingRules.Instance);/ravi
My new year resolution: 2048 x 1536 Home | Articles | My .NET bits | Freeware ravib(at)ravib(dot)com
-
There are cases where we can have different singletons, all of which implement an interface
IFoo(for example) and a method that has accepts anIFooparameter. Due to separation of concerns, the method may have no idea about the existence of the concrete singletons. For example:static class PrettyPrinter
{
public static string PrettyPrint
(string textToBePrettyPrinted,
IPrettyPrinterRuleProvider rulesToUse)
{
...
return prettyPrintedString;
}
}...
string text = "...";
string result1 = PrettyPrinter.PrettyPrint (text, KAndRTypeFormattingRules.Instance);
string result2 = PrettyPrinter.PrettyPrint (text, MicrosoftFormattingRules.Instance);/ravi
My new year resolution: 2048 x 1536 Home | Articles | My .NET bits | Freeware ravib(at)ravib(dot)com
Well sure, but would you design a class to do that with and make it a Singleton? Why? I see no reason to do so. Look at, perhaps, System.StringComparer and its fields.
-
Well sure, but would you design a class to do that with and make it a Singleton? Why? I see no reason to do so. Look at, perhaps, System.StringComparer and its fields.
PIEBALDconsult wrote:
make it a Singleton? Why?
For performance reasons. I would prefer to not have to
newup a concreteIPrettyPrinterRuleProvidereach time I callPrettyPrinter.PrettyPrint(). /raviMy new year resolution: 2048 x 1536 Home | Articles | My .NET bits | Freeware ravib(at)ravib(dot)com
-
PIEBALDconsult wrote:
make it a Singleton? Why?
For performance reasons. I would prefer to not have to
newup a concreteIPrettyPrinterRuleProvidereach time I callPrettyPrinter.PrettyPrint(). /raviMy new year resolution: 2048 x 1536 Home | Articles | My .NET bits | Freeware ravib(at)ravib(dot)com
And you don't; nor does it need to be a Singleton. It can be a static field of the class, for instance.
