Как правильно структурировать функции датакласса?

Question

[Ninzalo]

У меня есть датакласс, похожий на этот:

from dataclasses import dataclass
from typing import List

@dataclass()
class My_class:
    value: str
    main_key: str = "type"
    keys: List[str] = field(default_factory=list)

    def change_main_key(self, main_key: str):
        self.main_key = main_key
        print(f"Main key was changed to '{self.main_key}'")

    def add_key(self, key: str | List[str]) -> None:
        if isinstance(key, list):
            for k in key:
                self.add_key(key=k)
            return
        if key in self.keys:
            raise ValueError("Key value already exists in keys")
        self.keys.apppend(key)
        print(f"Keys list was appended by '{key}'")

Как можно переделать класс, чтобы структурировать функции?
Мое видение результата:

c = My_class(value="my_value")
c.main_keys.change_main_key(main_key='subtype')
c.keys.add_key(key=["first_key", "second_key"])
print(c)

Ответ:

> Main key was changed to 'subtype'
> Keys list was appended by 'first_key'
> Keys list was appended by 'second_key'
> My_class(main_key='subtype', keys=["first_key", "second_key"])

Можно ли так сделать? Или я выдумываю нереальные вещи?

Answer 1

[Ninzalo]

В итоге, своими силами получилось нечто такое:

class Main_key:
    def __init__(self, outer) -> None:
        self.outer = outer

    def change_main_key(self, main_key: str):
        if not self.outer.main_key:
            error_str = f"Outer class don't have .main_key param"
            raise Exception(error_str)
        old_main_key = self.outer.main_key
        self.outer.main_key = main_key
        print(
            f"Main key '{old_main_key}' was changed to '{self.outer.main_key}'"
        )


class Keys:
    def __init__(self, outer) -> None:
        self.outer = outer

    def add_key(self, key: str | List[str]):
        if isinstance(key, list):
            for k in key:
                self.append_keys(key=k)
            return
        if key in self.outer.keys:
            error_str = f"Key value '{key}' already exists: {self.outer.keys}"
            raise ValueError(error_str)
        self.outer.keys.append(key)
        print(f"Keys list was appended by {key}")


@dataclass()
class My_class:
    main_key: str = "type"
    keys: List[str] = field(default_factory=list)
    main_key_funcs: Main_key | None = None
    keys_funcs: Keys | None = None

    def get(self):
        self.main_key_funcs = Main_key(outer=self)
        self.keys_funcs = Keys(outer=self)


c = My_class()
c.get()
c.main_key_funcs.change_main_key(main_key="subtype")
c.keys_funcs.add_key('first_key')
c.keys_funcs.add_key('second_key')
print(c)

Работает превосходно, так, как и планировалось

Main key 'type' was changed to 'subtype'
Keys list was appended by first_key
Keys list was appended by second_key
My_class(main_key='subtype', keys=['first_key', 'second_key'], main_key_funcs=<__main__.Main_key object at 0x102acd1d0>, keys_funcs=<__main__.Keys object at 0x102acd250>)

Можно ли как-то улучшить? Открыт к предложениям

Comments

[Ninzalo]

Dr. Bacon, Для чего я это сделал? -Все просто! Потому что мне это было нужно.
А от тебя внятного ответа с примерами я так и не увидел

По понятиям ООП, ты как раз должен прятать подобную реализацию c.keys_funcs.add_key в c.add_key или т.п

А если таких функций сотня, а может и больше? Код становится нечитабельным. Мне необходима была структуризация и классификация, я ее и сделал

[Сергей П]

Ninzalo, вы не нервничайте. Dr. Bacon прав. что вы нагородили дохрена лишнего кода.
В вашем условии не читалось, что таких функций будет "сотня", но даже если и так, ставлю на то, что проблема не здесь. Ваш код нарушает принципы S.O.L.I.D Почитайте про них.

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

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

Попробуйте оисать, возможно даже в отдельном вопросе, вашу реальную задачу, лучше даже не на асбтрактном уровне, а вот как есть. Мне интересно что там за сотни функций. И мы подумаем все вместе как можно сделать лучше.
На текущий момент я бы снял, на самом деле, галочку с вашего ответа. Это не ответ, а антипаттерн.
Но для более обстоятельного обсуждения нужно больше сведений чего вы хотите добиться в итоге и в чем проблема.