Как лучше спроектировать в данном примере классы?

Question

[vilix]

Есть устройства контроллеры - Controllers, у них одинаковые наборы атрибутов. Но они могут быть разных производителей и в зависимости от производителей разные возможности и функции. Пока у меня класс

Controller:

• Id
  
• Name
  
• Address
  
• DeviceId - по этому ID можно будет достать техн характеристики данного типа устройства, напр производителя.
  

Этот класс я планирую посредством orm хранить в таблице Controllers.

Дальше я хочу поместить функции для работы с контроллерами, они должны быть сгруппированы по производителям.

В голове у меня создать по классу на производителя наследующих Controller. Например :

ControllerVendor1 : Controller

• setData()
  

ControllerVendor2 : Controller

• setData()
  

Хоть название функций одно и тоже, код будет отличатся.

Далее мне нужно, при получении из базы, получать контроллер нужного типа. Для этого я думаю в методе Controller.get() - проверять, по deviceId производителя и в зависимости от него создавать уже нужный контроллер и return его. return new ControllerVendor1(controller)

Насколько это нормальное решение, и как правильнее это сделать?

Язык C#;

в Device отображает возможности модели устройства.

Device:

• Id
  
• ModelName
  
• Vendor
  
• WorkModes
  

Пример из жизни: Допустим у нас есть два производителя контроллеров Vendor1 и Vendor2, мы купили 2 штуки первого и 1 штуку второго, назвали их ctr1, ctr2, ctr3.
Тогда

Devices: { {id: 1, vendor: "Vendor1"},  {id: 2, vendor: "Vendor2"} }
Controllers { {id: 1, name: "ctr1", deviceId: 1}, { id: 2, name: "ctr2", deviceId: 1}, { id:3, name: "ctr3", deviceId: 2} }

P.S Device собираюсь хранить не в базе, а где то зашито в коде будет.

Comments

[Александр Евгеньевич]

укажите язык

[Александр Евгеньевич]

Дальше я хочу поместить функции для работы с контроллерами,

Что за функции ?

DeviceId - по этому ID можно будет достать техн характеристики данного типа устройства, напр производителя.

Device - это базовая сущность, Controller - наследуемая, как я понял. Почему тогда в Controller существует поле - name ? Или только у контроллеров будет name ?

[vilix]

Александр Евгеньевич: обновил вопрос. Device это грубо говоря перечень моделей контроллеров с некими характеристиками, name у контроллеров это его пользовательское наименование.

[Александр Евгеньевич]

vilix: т.е. Device - это по сути Manufacturer ?

[vilix]

Александр Евгеньевич: да manufacturer, а так же тип модели, и режимы работы данной модели

[Александр Евгеньевич]

vilix: т.е. по сути вырисовывается такая структура:
DeviceModel
- id
- original_name
- manufacturer (я бы вынес в отдельную таблицу) : IManufacturer::getName()
- work_modes (не понял, что это)

Controller extends DeviceModel
- id
- name
- address

IManufacturer
- id
- name

Далее мне нужно, при получении из базы, получать контроллер нужного типа.

Самое главное, почему вам нужно получать определенный класс контроллера для разных Manufacturer , что вы хотите в дальнейшем с этим делать ? По идее ,если пользуетесь ORM, там можно настроить маппинг в зависимости от типа (содержимого поля, которое вы укажите как тип).

[vilix]

Александр Евгеньевич: я хотел разделить методы для разных мануфакторов в разные классы, чтобы например manufactirer1instance давал список методов, которые только для него. Ну и чтобы методы с одинаковыми названиями, выполняли разную логику. Но пока я отказался от этой идеи, так как мне нужно использовать эти данные и для отображения на UI, а условный метод сервиса getAll должен отдавать List<конкретного типа>, а у меня бы после orm было бы много типов. Вообщем сам себя запутал)

А вам спасибо за помощь) скопируйте свой коммент в ответ, я помечу как решение.

[Александр Евгеньевич]

vilix: а так всегда бывает, когда пытаешься решить задачу в уме, в итоге достаточно пообщаться с кем-нибудь, просто пытаясь обьяснить задачу, как все становится на свои места.
Думаю не стоит мой комментарий делать ответом, так как он не решает вашей задачи, тут лучши вы сделайте свой комментарий как ответ, грамотно обьяснив, по какой причине вам не подходит вариант, который вы изначально выбрали. Так будет полезней.

[vilix]

Александр Евгеньевич: вообще в идеале хотел провернуть конструкцию, что то вроде:
Controller controllers = Controller.getAll();
foreach (var ctr in controllers)
{
ctr.callMethod() -> где в классе контроллера в этом методе, из атрибута deviceType автоматически понимается класс и для этого класса вызывается метод callMethod(), без всяких switch и if.
}

пока вариант в голове, создать helper который будет принимать ctr на вход, в таблице device хранить атрибут ClassName, и в функции хелпера, через reflection вызывать метод класса например Macdonals.callMethod(ctr)

[Александр Евгеньевич]

vilix:
Ну можно таким вариантом обойтись:

IController
- createFromController(IController controller) : IController (но возвращаете конкретный тип, который вам нужен)

Т.е. если у вас, например ControllerManufacturer1 и у него свой конструктор, свои доп. методы, то можете его создавать через свой же метод createFromController(IController controller (который пришел из базы))

А когда вы принимаете данные, то можете просто вызывать этот класс, но вам, возможно (не знаю способности ОРМ в с#) создавать из него стандартный IController , чтобы поместить в базу.

Но для такого финта ушами, нужен хендлер\менеджер\etc. который будет это все дело разруливать. Т.е. внутри него вы должны будете зарегистрировать типы контроллеров. Т.е. когда из базу получаете контроллер, далее у вас есть список Manufacture\Type\Че-то там любой ключ => ClassName (класс конкретного контроллера), при получении вызываете у него метод createFromController и получаете конкретный экземпляр с методами и т.д.

[vilix]

Александр Евгеньевич: спасибо за вариант, буду думать над этим, если получится реализовать, добавлю в ответ.

Answer 1

[alexchin]

Все вроде ок. Есть подозрения?