Where do I put my code in a Singleton?

Question

I am linking to the following question --> https://stackoverflow.com/a/2550935/46724 specifically Jon Skeet's code:

public sealed class Singleton
 {
     private static readonly Singleton instance = new Singleton();
     public static Singleton Instance { get { return instance; } }

     static Singleton() {}
     private Singleton() {}
 }

What I would like to know is where would I put any logic I need on Initialization? The private or static constructor? By logic, I mean my Singleton is my DapperLite connection so I need to Initialize the mapping like so:

Database = new SqliteDatabase<int>(myConn);
Database.Init();

---

EDIT: I am stuck on Compact Framework 3.5 and VS 2008 for reference.

Answer 1

If you have any static field needed in your class it should initialized in this constructor static Singleton() {}for any other instance fields or property it should goes private Singleton() {} for more take a look at the following code

 public class Bus
    {
        // Static variable used by all Bus instances. 
        // Represents the time the first bus of the day starts its route. 
        protected static readonly DateTime globalStartTime;

        // Property for the number of each bus. 
        protected int RouteNumber { get; set; }

        // Static constructor to initialize the static variable. 
        // It is invoked before the first instance constructor is run. 
        static Bus()
        {
            globalStartTime = DateTime.Now;

            // The following statement produces the first line of output,  
            // and the line occurs only once.
            Console.WriteLine("Static constructor sets global start time to {0}",
                globalStartTime.ToLongTimeString());
        }

        // Instance constructor. 
        public Bus(int routeNum)
        {
            RouteNumber = routeNum;
            Console.WriteLine("Bus #{0} is created.", RouteNumber);
        }

        // Instance method. 
        public void Drive()
        {
            TimeSpan elapsedTime = DateTime.Now - globalStartTime;

            // For demonstration purposes we treat milliseconds as minutes to simulate 
            // actual bus times. Do not do this in your actual bus schedule program!
            Console.WriteLine("{0} is starting its route {1:N2} minutes after global start time {2}.",
                                    this.RouteNumber,
                                    elapsedTime.TotalMilliseconds,
                                    globalStartTime.ToShortTimeString());
        }
    }

    class TestBus
    {
        static void Main()
        {
            // The creation of this instance activates the static constructor.
            Bus bus1 = new Bus(71);

            // Create a second bus.
            Bus bus2 = new Bus(72);

            // Send bus1 on its way.
            bus1.Drive();

            // Wait for bus2 to warm up.
            System.Threading.Thread.Sleep(25);

            // Send bus2 on its way.
            bus2.Drive();

            // Keep the console window open in debug mode.
            System.Console.WriteLine("Press any key to exit.");
            System.Console.ReadKey();
        }
    }

so in your case is an instance initialization

 private Singleton() 
{
Database = new SqliteDatabase<int>(myConn);
Database.Init();
}

Answer 2

Here's a link from Jon Skeet's book C# in Depth

If you're using .NET 4.0:

public sealed class Singleton
{
    private static readonly Lazy<Singleton> lazy =
        new Lazy<Singleton>(() => new Singleton());

    public static Singleton Instance { get { return lazy.Value; } }

    private Singleton()
    {
        try 
        {
            Database = new SqliteDatabase<int>(myConn);
            Database.Init();
        }
        catch(Exception ex)
        {
           // Handle errors
        }
    }

}

EDIT: Pre-4.0, solution #4 from the provided link. Fully thread safe without locks, his preferred solution.

public sealed class Singleton
{
    private static readonly Singleton instance = new Singleton();

    // Explicit static constructor to tell C# compiler
    // not to mark type as beforefieldinit
    static Singleton()
    {
    }

    private Singleton()
    {
        try
        {
            Database = new SqliteDatabase<int>(myConn);
            Database.Init();
        }
        catch (Exception ex)
        {
            // Handle errors
        }
    }

    public static Singleton Instance
    {
        get
        {
            return instance;
        }
    }
}