Search code examples
c#nested-classaccess-levels

How to restrict access to nested class member to enclosing class?


Is it possible to specify that members of a nested class can be accessed by the enclosing class, but not other classes ?

Here's an illustration of the problem (of course my actual code is a bit more complex...) :

public class Journal
{
    public class JournalEntry
    {
        public JournalEntry(object value)
        {
            this.Timestamp = DateTime.Now;
            this.Value = value;
        }

        public DateTime Timestamp { get; private set; }
        public object Value { get; private set; }
    }

    // ...
}

I would like to prevent client code from creating instances of JournalEntry, but Journal must be able to create them. If I make the constructor public, anyone can create instances... but if I make it private, Journal won't be able to !

Note that the JournalEntry class must be public, because I want to be able to expose existing entries to client code.

Any suggestion would be appreciated !


UPDATE: Thanks everyone for your input, I eventually went for the public IJournalEntry interface, implemented by a private JournalEntry class (despite the last requirement in my question...)


Solution

  • If your class is not too complex, you could either use an interface which is publicly visible and make the actual implementing class private, or you could make a protected constructor for the JornalEntry class and have a private class JornalEntryInstance derived from JornalEntry with a public constructor which is actually instantiated by your Journal.

    public class Journal
    {
        public class JournalEntry
        {
            protected JournalEntry(object value)
            {
                this.Timestamp = DateTime.Now;
                this.Value = value;
            }
    
            public DateTime Timestamp { get; private set; }
            public object Value { get; private set; }
        }
    
        private class JournalEntryInstance: JournalEntry
        { 
            public JournalEntryInstance(object value): base(value)
            { }
        }
        JournalEntry CreateEntry(object value)
        {
            return new JournalEntryInstance(value);
        }
    }
    

    If your actual class is too complex to do either of that and you can get away with the constructor being not completely invisible, you can make the constructor internal so it is only visible in the assembly.

    If that too is infeasible, you can always make the constructor private and use reflection to call it from your journal class:

    typeof(object).GetConstructor(new Type[] { }).Invoke(new Object[] { value });
    

    Now that I think about it, another possibility would use a private delegate in the containing class which is set from the inner class

    public class Journal
    {
        private static Func<object, JournalEntry> EntryFactory;
        public class JournalEntry
        {
            internal static void Initialize()
            {
                Journal.EntryFactory = CreateEntry;
            }
            private static JournalEntry CreateEntry(object value)
            {
                return new JournalEntry(value);
            }
            private JournalEntry(object value)
            {
                this.Timestamp = DateTime.Now;
                this.Value = value;
            }
    
            public DateTime Timestamp { get; private set; }
            public object Value { get; private set; }
        }
    
        static Journal()
        {
            JournalEntry.Initialize();
        }
            
        static JournalEntry CreateEntry(object value)
        {
            return EntryFactory(value);
        }
    }
    

    This should give you your desired visibility levels without needing to resort on slow reflection or introducing additional classes / interfaces