Как выбрать классы в ООП?

Question

[Руслан]

Здравствуйте.

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

На практике у меня есть запрос, который содержит список устройств и список команд, которые надо выполнить на устройствах.
[
{'device_name': 'device1', 'commands': ['sh run']},
{'device_name': 'device2', 'commands': ['sh run', 'sh ip int brie']}
]
В качестве класса я выбрал device и добавил ему свойство commands_to_execute куда поместил список команд.
Но в реальном мире устройства и команды это разные сущности и тогда нужно создать классы device и command, но я не понимаю как тогда красиво связать их, ведь мне нужно выполнить определенные команды на определенных устройствах.

Как быть в подобной ситуации?

Answer 1

[montray]

Вообще, здесь все зависит от того, что конкретно делает ваша система и будет ли у нее развитие.

Но, если мыслить объектно, то устройство - это то, чему на вход подают некую команду, он ее выполняет и отдает ответ. Соответственно, вам нужна реализация Device, которая через метод execute(Executable command) принимает объект, который реализует интерфейс Executable. А список команд уже никакого отношения к устройству не имеет, поэтому и хранится должен отдельно.

Comments

[Алексей П]

Про ЯП никакой информации нет, поэтому вести рассуждения про интерфейсы, возможно, не является корректным решением.

[montray]

Вы меня, конечно, извините, но, неужели в ваших учебниках не рассказывают про один из главных принципов ООП - абстракцию?

[Антон]

montray, поддерживаю.

[Руслан]

Это микросервис который принимает запрос из очереди (в котором есть список приведеный в вопросе и дополнительные уточнения), собирает необходимую информацию(ip устройства, креды, ОС устройства, поддерживаемые протоколы и т.д.) из других сервисов, выполняет команды на устройствах, парсит вывод если нужно, и отправляет ответ. Устройства - сетевые устройства.

Если я вас правильно понял, то нужно сделать примерно так:
Request имеет методы parse_message (принимает json), get_command_list (возвращает список объектов CommandList)
CommandList содержит список объектов Command и объект Device
Device имеет метод execute_command, аттрибуты os, login, pass, protocol и set методы дл них.
Command содержит в себе собственно команду и параметр parse: True/False (одно из требований запроса)
Creds (делает запрос логина и пароля в сервисе Creds) имеет метод get_credentials принимает объект Device, с помощью set методов добавляет login и pass.
DeviceDB (делает запрос в сервис DeviceDB) имеет методы get_os, get_protocol оба принимают объект Device.

Прежде всего я хочу разобраться в ООП.
ЯП проекта - Python, имею базовые знания Java, но ведь это не имеет определяющего значения в данном случае?
Сильно не пинайте, вообще я сетевик:)

[montray]

Если методы выполнения и парсинга вывода команд зависят от протокола и ос, то ООП тут в самый раз - вы динамически будете определять как выполнять команду в зависимости от условий.

Ещё момент - не увлекайтесь сеттерами, старайтесь соблюдать целостность объекта. Т.е. креды вы должны получить ещё до создания объекта устройства(можно опять же ввести абстракцию подключения).

По выполнению команд можно пойти тем же путем, что я предлагал в ответе, только отдавать в методе execute() вывод этой команды. Над реализацией парсинга предлагаю подумать самому)

[montray]

А вообще научитесь мыслить объектно, если вы действительно хотите познать ООП.

Бездумное клепание классов != ООП

Удачи.

[Руслан]

Да, научиться мыслить объектно я пытаюсь.
Примерно понятно куда двигаться.

Большое спасибо за консультацию!

Answer 2

[Алексей П]

Всё упирается в Вашу фантазию, как и в учебниках по программированию.
Здесь можно выделить множество классов:
1. класс Array, который содержит список девайсов или команд
2. класс Device, который отображает сущность девайса и содержит список команд
3. класс Command, который отображает сущность команды

примерная имплементация:

class Device {
  name: String
  commands: Array<Command>
}
class Command {
  shell: String
}

Сюда можно зафигачить accessor methods для доступа к данным или другую муть, которую захотите

Comments

[montray]

Судя по всему, дальше учебников вы и не ушли.

[Алексей П]

Не буду спорить.

[Руслан]

Спасибоа за ответ.

Answer 3

[Антон Р.]

Почитай что такое Композиция и Агрегация в ООП, многое поймёшь.