2

Есть абстрактный тип Data, от него наследуются конкретные типы DataA, DataB, DataC и т.д. (по хорошему, в моей задаче DataA, DataB, ... - это структуры, а не классы, поэтому их не может объединять абстрактный тип, но для простоты пусть это будут классами). У каждого типа Data есть свой id [0, 255]. Необходимо реализовать метод, который будет возвращать Data в зависимости от id:

public Data CreateDataInstance(int id);

Если решать задачу в лоб, то получится что-то такое:

public Data CreateDataInstance(int id)
{
    switch (id)
    {
        case 0:
            return new DataA();
        case 1:
            return new DataB();
        //...
        case 255:
            return new DataXXX();
        default:
            return null;
    }
}

Мне этот большой switch не очень нравится, потому что потом в случае изменения кода придется менять этот switch/case. Мне это все напоминает фабричный метод, но вот уже долго мучаюсь и никак не могу допереть, как его здесь применить, чтобы избавиться от switch/case.

В момент написания вопроса появилась идея создать все типы фабрик при запуске программы и запихать их в массив или список, и по id их доставать и создавать нужный Data. Но, может быть, есть более элегантный метод?

UPD. Решил вопрос с помощью фабрики. Теперь возник следующий вопрос: вот у меня есть список List<Data>. Как теперь, зная Data.id, совершить восходящее преобразование от общего Data к конкретному DataX? Неужели switch/case не избежать?

UPD2. Есть подозрение, что у меня не все в порядке с архитектурой приложения

14
  • А вам точно нужны именно разные структуры DataA..Z?
    – srvr4vr
    18 июл 2019 в 12:01
  • @Сергей не обязательно. Просто у меня из массива байт данные запоминаются в структуры через Marshal.PtrToStructure и я бы хотел работать именно с этими структурами, а так придется создавать классы помимо структур. Все структуры имеют различный набор полей. И этот набор полей зависит от id 18 июл 2019 в 12:55
  • связанный вопрос: Способы создания объектов в C#
    – Grundy
    18 июл 2019 в 17:41
  • Как теперь, зная Data.id, совершить восходящее преобразование от общего Data к конкретному DataX? — ??? Вы уверены, что это вам нужно? Вы это даже написать в принципе не сможете, все типы переменных должны быть определены на этапе компиляции 19 июл 2019 в 8:40

2 ответа 2

1

Например, так (в предположении, что все наследники в одной сборке, и что наплевать на производительность):

using System;
using System.Linq;
using System.Collections.Generic;
using System.Text;
using System.Reflection;

namespace ConsoleApp1
{
    abstract class Data
    {
        public static Data CreateDataInstance(int id)
        {
            var derived = typeof(Data).Assembly.GetTypes().Where(t => t.BaseType == typeof(Data));
            var types = derived.Where(
                    t => (t.GetCustomAttribute(typeof(DataIdAttribute)) as DataIdAttribute).ID == id
                );
            if (types.Count() == 0) throw new MissingMemberException("No such type with ID "+id.ToString());            

            Type type = types.First();
            return (Data)Activator.CreateInstance(type);
        }
    }

    class DataIdAttribute : Attribute
    {
        int _id;

        public int ID { get { return _id; } }

        public DataIdAttribute(int id)
        {
            _id = id;
        }
    }

    [DataId(1)]
    class DataA : Data
    {
        public override string ToString()
        {
            return "DataA";
        }
    }

    [DataId(2)]
    class DataB : Data
    {
        public override string ToString()
        {
            return "DataB";
        }
    }
}

Если же производительность важна, но допустимы некоторые накладные расходы при первом обращении, можно предложить более громоздкое оптимизированное решение:

using System;
using System.Linq;
using System.Linq.Expressions;
using System.Collections.Generic;
using System.Text;
using System.Reflection;

namespace ConsoleApp1
{
    abstract class Data
    {
        static Dictionary<int, Delegate> delegcache = new Dictionary<int, Delegate>();

        public static Data CreateDataInstance(int id)
        {
            Delegate deleg;
            if (delegcache.ContainsKey(id))
            {
                deleg = delegcache[id];
            }
            else
            {
                var derived = typeof(Data).Assembly.GetTypes().Where(t => t.BaseType == typeof(Data));
                var types = derived.Where(
                        t => (t.GetCustomAttribute(typeof(DataIdAttribute)) as DataIdAttribute).ID == id
                    );
                if (types.Count() == 0) throw new MissingMemberException("No such type with ID " + id.ToString());

                Type type = types.First();
                NewExpression expr = Expression.New(type);
                LambdaExpression lambda = Expression.Lambda(expr);
                deleg = lambda.Compile();
                delegcache[id] = deleg;
            }

            return (Data)deleg.DynamicInvoke();
        }
    }

    class DataIdAttribute : Attribute
    {
        int _id;

        public int ID { get { return _id; } }

        public DataIdAttribute(int id)
        {
            _id = id;
        }
    }

    [DataId(1)]
    class DataA : Data
    {
        public override string ToString()
        {
            return "DataA";
        }
    }

    [DataId(2)]
    class DataB : Data
    {
        public override string ToString()
        {
            return "DataB";
        }
    }  
}
1

Вот так как-то

interface ICommonData
{
    int Id { get; set; }
    string Name { get; set; }
}
struct DataA : ICommonData
{
    string DataA1 { get; set; }
    public int Id { get; set; }
    public string Name { get; set; }
}

и т.д....затем

     private Dictionary<byte, Type> types;
     private List<ICommonData> data = new List<ICommonData>();

        types = new Dictionary<byte, Type>();
        types[1] = typeof(DataA);
        types[2] = typeof(DataB);
        types[3] = typeof(DataC);
        types[4] = typeof(DataD);
        types[5] = typeof(DataE);

и уже далее

        for (byte i = 0; i < byte.MaxValue; i++)
        {
            if (types.ContainsKey(i))
            {
                data.Add((ICommonData)Activator.CreateInstance(types[i]));
            }
        }
8
  • Про Type и не догадался, неплохо! Может быть, вы знаете, насколько по производительности такой подход отличается по производительности от того же фабричного метода? Больше всего меня интересует эта строка по производительности: (ICommonData)Activator.CreateInstance(types[i])); 18 июл 2019 в 13:53
  • Отличается в худшую сторону. Это же рефлексия...(ну все так говорят). А вообще нужно мерить и если это разовая акция, то я думаю не критично...Ежели надо тысячами создавать структуры постоянно, то будет заметно 18 июл 2019 в 13:59
  • Тогда не очень устроит, потому как нужно будет вызывать эту штуку около 10000-20000 раз 18 июл 2019 в 14:01
  • Это не много. Будет быстро работать 18 июл 2019 в 14:03
  • @AndreiKhotko, можно взять CustomActivator здесь: ru.stackoverflow.com/a/608238/218063 18 июл 2019 в 18:45

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.