Singleton class, use another class?

[=?Utf-8?B?ZGF2ZWJ5dGhlc2Vh?=]

Hi,

I have created a Singleton class to provide some database functionality in
my mobile application. I have a public class called Utility which performs
various operations on data. Is it ok to use the utility class in my Singleton
class? Here is some code to hopefully make it clearer what i want to do -

namespace MyApp
{
class SingletonConnec tion
{
private volatile static SingletonConnec tion instance;

private Utility utility;

private SingletonConnec tion()
{
}

public static SingletonConnec tion GetInstance()
{
if (instance == null)
{
instance = new SingletonConnec tion();
}
return instance;
}

public void DoStuff()
{
utility = new Utility();

utility.DoStuff ();
}
}

class Utility
{
public Utility()
{
}

public void DoStuff()
{

}
}
}

Thanks for any tips,
Dav

Sure. Why wouldn't it be ok?
--

Chris Tacke, eMVP
Join the Embedded Developer Community
http://community.opennetcf.com
"davebythes ea" <da**********@d iscussions.micr osoft.comwrote in message
news:24******** *************** ***********@mic rosoft.com...

Hi,

I have created a Singleton class to provide some database functionality in
my mobile application. I have a public class called Utility which performs
various operations on data. Is it ok to use the utility class in my
Singleton
class? Here is some code to hopefully make it clearer what i want to do -

namespace MyApp
{
class SingletonConnec tion
{
private volatile static SingletonConnec tion instance;

private Utility utility;

private SingletonConnec tion()
{
}

public static SingletonConnec tion GetInstance()
{
if (instance == null)
{
instance = new SingletonConnec tion();
}
return instance;
}

public void DoStuff()
{
utility = new Utility();

utility.DoStuff ();
}
}

class Utility
{
public Utility()
{
}

public void DoStuff()
{

}
}
}

Thanks for any tips,
Dav

[Brian Gideon]

On Nov 19, 7:49 am, davebythesea
<davebythe...@d iscussions.micr osoft.comwrote:

Hi,

I have created a Singleton class to provide some database functionality in
my mobile application. I have a public class called Utility which performs
various operations on data. Is it ok to use the utility class in my Singleton
class? Here is some code to hopefully make it clearer what i want to do -

namespace MyApp
{
class SingletonConnec tion
{
private volatile static SingletonConnec tion instance;

private Utility utility;

private SingletonConnec tion()
{
}

public static SingletonConnec tion GetInstance()
{
if (instance == null)
{
instance = new SingletonConnec tion();
}
return instance;
}

public void DoStuff()
{
utility = new Utility();

utility.DoStuff ();
}
}

class Utility
{
public Utility()
{
}

public void DoStuff()
{

}
}

}

Thanks for any tips,
Dav

Yes, it is okay to use the Utility class (or any other class for that
matter) if it helps your implementation of the singleton.

As written your singleton is not thread-safe. I would remove the
volatile modifier and wrap the contents of the GetInstance method with
a lock construct. Better yet, create a new SingletonConnec tion
instance inline with the declaration and have the GetInstance method
just return the reference. Or if you don't require any fancy
initialization or polymorphism then just create a static class.

[Marc Gravell]

Since DoStuff() creates a new instance each time, why not just hold
this as a local variable? Unless I missed something in the usage, this
could also be implemented just as a static method. A lot here depends
on whether utility is itself thread-safe...

Note that your SingletonConnec tion is not guaranteed to be singleton.
First, you might want to seal it, and second there is a race-condition
on the first access; if two threads get past the null check you could
get 2. Unless there is a huge cost associated with initializing a
SingletonConnec tion (which the ctor doesn't indicate), then perhaps
just use:

private static readonly SingletonConnec tion instance = new
SingletonConnec tion();
public static SingletonConnec tion Instance {get {return instance;}}

Of course, unless the singleton is implementing a specific base /
interface, you might be able to get away with just a static class with
static methods (etc).
Did I miss the point?

Marc

[=?Utf-8?B?ZGF2ZWJ5dGhlc2Vh?=]

"Brian Gideon" wrote:

On Nov 19, 7:49 am, davebythesea

snip

Yes, it is okay to use the Utility class (or any other class for that
matter) if it helps your implementation of the singleton.

As written your singleton is not thread-safe. I would remove the
volatile modifier and wrap the contents of the GetInstance method with
a lock construct. Better yet, create a new SingletonConnec tion
instance inline with the declaration and have the GetInstance method
just return the reference. Or if you don't require any fancy
initialization or polymorphism then just create a static class.

Thanks for the tips. I think i saw some code which used the lock construct
so I will probably hunt for it again and add it in there.

Cheers

[=?Utf-8?B?ZGF2ZWJ5dGhlc2Vh?=]

Hi,

Thanks for your reply.

"Marc Gravell" wrote:

Note that your SingletonConnec tion is not guaranteed to be singleton.
First, you might want to seal it, and second there is a race-condition
on the first access; if two threads get past the null check you could
get 2. Unless there is a huge cost associated with initializing a
SingletonConnec tion (which the ctor doesn't indicate), then perhaps
just use:

Would the following be enough to seal the class correctly -

namespace MyApp
{
sealed class SingletonConnec tion
{

}
}

Cheers,
Dav

[Brian Gideon]

On Nov 19, 8:48 am, davebythesea
<davebythe...@d iscussions.micr osoft.comwrote:

Thanks for the tips. I think i saw some code which used the lock construct
so I will probably hunt for it again and add it in there.

Cheers

Unless you have a specific reason for doing manual synchronization I
would stick with what Marc posted.

private static readonly SingletonConnec tion instance = new
SingletonConnec tion();
public static SingletonConnec tion Instance {get {return instance;}}

[Marc Gravell]

There is a full discussion of various approaches (pros, cons, etc)
here:

http://www.pobox.com/~skeet/csharp/singleton.html

Marc

[Marc Gravell]

yep; actually, this only becomes necessary if there is a non-private
constructor (since you can't inherit from something with only private
constructors), but it is a good idea anyway

Marc

[=?Utf-8?B?ZGF2ZWJ5dGhlc2Vh?=]

Thanks for this link. Its very useful for me.

David

"Marc Gravell" wrote:

There is a full discussion of various approaches (pros, cons, etc)
here:

http://www.pobox.com/~skeet/csharp/singleton.html

Marc