Реализация вложенных атрибутов класса?

Question

[mkone112]

Периодически возникает желание использовать вложенные атрибуты класса. Обычно я использую внутренний класс:

class Context:
    _event_loop = None

    class states:  # noqa
        INITIAL = 0
        CONNECTING = 1
        CONNECTED = 2
        CLOSED = 3
    ...

class async_socket(Context):  # noqa
    def __init__(self, *args):
        ...
        self._state = self.states.INITIAL

Но мне это решение кажется довольно грязным. Есть ли более чистый вариант?
P.S.

1. Я знаю про возможность использования Enum и SimpleNamespace, но ищу более элегантное решение
   
2. Приведенный код это просто пример иллюстрирующий вложенность атрибутов, ничего более.
   
3. Я знаю что класс в классе это не гуд, но если ._states нужен только для использования в тех классах где они объявлены - это приведет с созданию дополнительного класса для каждого такого класса - две тысячи классов это не лучше чем тысяча классов с инкапсулированной реализацией.
   
4. Я знаю про PEP8 и PascalCase, но сохраняю консистентность именования(как кстати рекомендует сам pep), а еще потому что мне так захотелось
   

Comments

[HCEL]

Мда и этот чел мне что то про solid с ооп пытался внушить до сих пор кста свои слова не аргументировал. Надо было сразу чекнуть твой профиль и кинуть в игнор.

Answer 1

[LXSTVAYNE]

Написать пакет, в котором всё разбить на модули types.py, context.py, async_socket.py. В types.py вынеси State, унаследованный от Enum.enum, в context.py свой Context, а в async_socket.py выноси AsyncSocket. Соблюдай PEP, придерживайся правил в названии классов.

Comments

[mkone112]

В types.py вынеси State

С чего вдруг State в types? Эти состояния относятся только к async_socket они не должны быть переиспользуемыми, и уж тем более это не какой-то тип - делать в файле types.py им нечего.

Соблюдай PEP, придерживайся правил в названии классов.

В самом pep написано о том что единообразие кода имеет приоритет над pep. Класс socket в lowercase, его наследник тоже будет в lowercase.

[LXSTVAYNE]

mkone112, не знаю какая структура у твоего кода, но я бы вынес всё по пакетам. Твой пакет должен относиться только к твоему сокету, тогда путаницы никакой не будет. Если types.py не нравится, назови state.py, но обычно перечисления и типы я выношу в types.py. Был у меня опыт, когда я инкапсулировал классы в класс, но я от этого отошёл, потому что были проблемы в будущем с тайпхинтингом и наследованием этого чуда.
Что имеется ввиду под единообразием кода? Я понимаю это как единообразие кода в твоём проекте, так что я бы придерживался PascalCase в названии классов, но тут кто как понимает.

Можешь написать всё в один файл как это сделано в peewee, но потом будет очень сложно редактировать код.

[mkone112]

проблемы в будущем с тайпхинтингом

Тайпхинтинг это сатанизм.
Ты попер куда-то не туда. Все это вообще никак не относится к вопросу. Перечитай заголовок.

Answer 2

[Павел Дунаев]

class _ContextState(enum.Enum):
   ...

class AsyncSocket(Context):
    def __init__(self, *args):
        self._state = _ContextState.INITIAL

Если короче, то:

1. вложенные классы - это плохо
   
2. названия классов всегда в PascalCase
   
3. как и методы, классы могут быть приватными, если их название начинается с символа подчеркивания
   
4. если все объекты класса предназначены только для внутреннего использования, то этот класс можно сделать приватным
   

Хотя иногда все же можно использовать вложенные классы, но только если они приватные и нужны только в одном классе:

class Context:
    class _State(enum.Enum):
        ...

    def __init__(self):
        self._state = Context._State.INITIAL

Comments

[Михаил]

Стоит отметить, что в Python вообще не существует приватности. Одно подчеркивание, это декларативная приватность, для других разработчиков. Два подчеркивания, это мутация имени переменной. Напрямую к ней обратиться будет невозможно, однако есть способы и на этот случай, это же Python)))

[mkone112]

вложенные классы - это плохо

Замечательно.

названия классов всегда в PascalCase

Глянь исходники socket

как и методы, классы могут быть приватными,

Не могут.
Короче твой вариант это по классу на каждый чих.

[Павел Дунаев]

Михаил, в python приватность - это соглашение. Разработчики посовещались и согласились не использовать поля, методы и классы, начинающиеся с подчеркивания нигде, кроме объекта self.

mkone112, socket - это стандартная библиотека (как и str, int, list и т. д.). В PEP 8 говорится, что в стандартной библиотеке это соглашение почти не используется.

Не могут.

Могут.

Кстати в той же java приватность тоже обходится через рефлексию, но никто не говорит, что так надо делать.

Короче твой вариант это по классу на каждый чих.

Я говорю только про именование и расположение классов, а не про их количество.

[mkone112]

Павел Дунаев,

Я говорю только про именование и расположение классов, а не про их количество.

Твое решение это (число классов) * 2.

[Павел Дунаев]

mkone112, ещё раз. Я не говорю ничего про количество классов. Я говорю как лучше разместить класс, если он нужен.

При этом если класс расположен внутри другого, то классом быть он не перестаёт. И не надо считать, что больше классов == хуже. Enum достаточно легковесный, чтобы его использовать.

[mkone112]

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

[Павел Дунаев]

mkone112, я понял о чем вы и да, вы правы. Если есть переменная (поле), которой присваивается одно из нескольких значений, то в современном коде на python используется Enum. Собственно, модуль enum был придуман именно для того, чтобы не делать константы. Но это все рекомендации. Если enum протит производительность, то использовать его не нужно. Хотя я считаю, что python и производительность не очень сочетаются.

Если ответить на ваш изначальный вопрос: да, более чистый вариант есть - enum, но никто не заставляет вас всегда писать чистый код.

[mkone112]

Павел Дунаев, вообще по сравнению с Enum мне гораздо больше нравится types.SimpleNamespace

from types import SimpleNamespace

class Context:
    _event_loop = None

    states = SimpleNamespace(
        INITIAL=0,
        CONNECTING=1,
        CONNECTED=2,
        CLOSED=3
    )

class async_socket(Context):  # noqa
    def __init__(self, *args):
        ...
        self._state = self.states.INITIAL

Хотя я надеялся что есть какое-то еще более чистое решение.