Общая карта идентичности в C #. Не хочу публичного конструктора

Question

Я пытаюсь реализовать карту идентификации с использованием обобщений. У меня есть абстрактный класс Entity и ограничение деривации на карте для Entity. Поскольку моя карта должна иметь возможность создавать экземпляры сущностей, моя карта также имеет ограничение конструктора.

Однако, чтобы карта была полезной, подклассы Entity не должны иметь возможности создания экземпляров из клиентского кода, а это значит, что мне нужен внутренний конструктор, а не публичные конструкторы. Это противоречит ограничению конструктора.

Есть ли что-то, что я пропускаю? Есть ли способ рефакторинга, чтобы получить желаемый результат?

Следующий код компилируется как есть, но в идеале конструкторы подклассов Entity должны быть внутренними:

public abstract class Entity
{
    public int Id { get; protected internal set; }
}

public sealed class Widget : Entity
{
    // Client code should not be allowed to instantiate entities.
    // But the constraints on EntityMap require that entities have
    // a public constructor.
    public Widget() { }
}

public sealed class Gadget : Entity
{
    public Gadget() { }
}

// The new() constraint is required so that Get() can instantiate Ts.
public class EntityMap<T> where T : Entity, new()
{
    private Dictionary<int, T> _entities = new Dictionary<int, T>();
    private object _getLock = new object();

    public T Get(int id)
    {
        lock (_getLock)
        {
            if (!_entities.ContainsKey(id))
                _entities.Add(id, new T() { Id = id });
        }

        return _entities[id];
    }

    // Client code should not be allowed to instantiate maps.
    internal EntityMap() { }
}

// Ideally, the client would only be able to obtain Entity
// references through EntityMaps, which are only accessible
// through the ApplicationMap.
public static class ApplicationMap
{
    public static EntityMap<Widget> Widgets = new EntityMap<Widget>();
    public static EntityMap<Gadget> Gadgets = new EntityMap<Gadget>();
}

Answer 1

Вместо того, чтобы требовать ограничения конструктора, передайте Func<T> в конструктор карты. Таким образом, конструктор может быть внутренним, но карта все равно может эффективно вызывать его:

public class EntityMap<T> where T : Entity
{
    private readonly Dictionary<int, T> _entities = new Dictionary<int, T>();
    private readonly object _getLock = new object();
    private readonly Func<T> _entityGenerator;

    public T Get(int id)
    {
        lock (_getLock)
        {
            T ret;
            if (!_entities.TryGetValue(id, ret))
            {
                ret = entityGenerator();
                newEntity[id] = ret;
                ret.Id = id;
            }

            return ret;
        }
    }

    internal EntityMap(Func<T> entityGenerator)
    {
        _entityGenerator = entityGenerator;
    }
}

Затем инициализируйте его с помощью:

EntityMap<Widget> widgetMap = new EntityMap(() => new Widget());

Вместо этого вы могли бы сделать его Func<int, T> и назначить делегата ответственным за создание объекта с правильным идентификатором. Таким образом, вы можете сделать свой идентификатор доступным только для чтения, передав его в качестве параметра конструктору Entity.

(я позволил себе сделать ваш метод Get более эффективным, кстати.)

Answer 2

Спасибо Джону, вот рабочий код:

public abstract class Entity
{
    private readonly int _id;

    public int Id
    {
        get { return _id; }
    }

    internal Entity(int id)
    {
        _id = id;
    }
}

public sealed class Widget : Entity
{
    internal Widget(int id) : base(id) { }
}

public sealed class Gadget : Entity
{
    internal Gadget(int id) : base(id) { }
}

public class EntityMap<T> where T : Entity
{
    private readonly Dictionary<int, T> _entities = new Dictionary<int, T>();
    private readonly object _getLock = new object();
    private readonly Func<int, T> _entityGenerator;

    public T Get(int id)
    {
        lock (_getLock)
        {
            T entity;

            if (!_entities.TryGetValue(id, out entity))
                _entities[id] = entity = _entityGenerator(id);

            return entity;
        }
    }

    internal EntityMap(Func<int, T> entityGenerator)
    {
        _entityGenerator = entityGenerator;
    }
}

public static class ApplicationMap
{
    public static readonly EntityMap<Widget> Widgets = new EntityMap<Widget>(id => new Widget(id));
    public static readonly EntityMap<Gadget> Gadgets = new EntityMap<Gadget>(id => new Gadget(id));
}