在从抽象基类派生的一组类中添加功能的正确方法是什么？

Question

设置

我的朋友告诉我，在OOP中，您通常不想修改现有代码库中的任何抽象基类，因为这意味着您必须对每个派生的类实施新的更改。我对使用Pythonic的方式更喜欢对代码库进行的修改感兴趣。重点是更改现有代码库。

示例场景

我有一个抽象基类，称为Animal，因为使用此库的代码必须与Animal对象进行交互。 我有多个儿童类实现：dog和cat每个都有不同的字段，它们每个字段都需要自己的内部功能。因此，在这一点上，代码库看起来像：

from abc import ABC, abstractmethod


class Animal(ABC):
    @abstractmethod
    def feed(self, foo: str) -> None:
        raise NotImplementedError


class Dog(Animal):
    def __init__(self):
        self.woof = "woof"

    def feed(self, food: str):
        print(f"Dog is being fed {food}. It says {self.woof}")


class Cat(Animal):
    def __init__(self):
        self.meow = "meow"
        self.purr = "purr"

    def feed(self, food: str):
        print(f"Cat is being fed {food}. It says {self.meow}")

实现此操作后的修改

，开发人员意识到他们想从Animal对象登录相关字段（或状态），以及记录的数据是什么因素而变化。儿童课程的儿童课。

选项A

最初，我的想法是实现另一个AbstractMethod并以这种方式添加功能。这迫使每个动物>以任何方式实现新的get_fields（）。

class Animal(ABC):
    @abstractmethod
    def feed(self, foo: str) -> None:
        raise NotImplementedError

    @abstractmethod
    def get_fields(self) -> list:
        raise NotImplementedError

class Dog(Animal):
    def __init__(self):
        self.woof = "woof"

    def feed(self, food: str):
        print(f"Dog is being fed {food}. It says {self.woof}")

    def get_fields(self) -> list:
        return [self.woof]


class Cat(Animal):
    def __init__(self):
        self.meow = "meow"
        self.purr = "purr"

    def feed(self, food: str):
        print(f"Cat is being fed {food}. It says {self.meow}")

    def get_fields(self) -> list:
        return [self.meow, self.purr]

选项B

我的朋友说我们不应该修改抽象类，但是，我们提出的唯一其他选择是要做以下操作：

def get_field(animal: Animal) -> list:
    if isinstance(animal, Dog):
        return [animal.woof]
    elif isinstance(animal, Cat):
        return [animal.meow, animal.purr]
    else:
        raise TypeError

您会选择哪一个？还有另一种更好的方法吗？哪一个是更多的Pythonic？

Answer 1

在ABC上实现一种通用机制，作为具体方法，但将配置转移到子类，并且不使用硬编码名称。

我在此处使用了meta，因为这是您在Django模型中看到的东西的类型，在嵌入式meta 中，name-stacs name-stacs name-stacs in name-stacs in name-spacs in name-space /代码>类。 django专门使用非常相似的系统哪些字段显示用于数据输入的自动生成管理面板中的位置。

from abc import ABC, abstractmethod

from typing import Optional, List

class Animal(ABC):

    class Meta:
        #could also set to [] as default...
        fields_of_interest : Optional[List[str]] = None

    @abstractmethod
    def feed(self, foo: str) -> None:
        raise NotImplementedError

    def get_fields(self) -> List:

        if self.Meta.fields_of_interest is None:
            # could also raise NotImplementedError("need to set `fields_of_interest` on class Meta in class {self.__class__.__name__}")
            return []

        res = [getattr(self, field) for field in self.Meta.fields_of_interest]
        return res

class Dog(Animal):

    class Meta:
        fields_of_interest = ["woof"]

    def __init__(self):
        self.woof = "woof"

    def feed(self, food: str):
        print(f"Dog is being fed {food}. It says {self.woof}")


class Cat(Animal):

    class Meta:
        fields_of_interest = ["purr", "meow"]

    def __init__(self):
        self.meow = "meow"
        self.purr = "purr"

    def feed(self, food: str):
        print(f"Cat is being fed {food}. It says {self.meow}")

class Mouse(Animal):

    def feed(self, foo: str) -> None:
        print(f"{self} feed")


for cls in [Cat, Dog, Mouse]:
    animal = cls()
    print(f"{animal} {animal.get_fields()}")

输出：

<__main__.Cat object at 0x1079f67d0> ['purr', 'meow']
<__main__.Dog object at 0x1079f6320> ['woof']
<__main__.Mouse object at 0x1079f67d0> []

此外，就抽象与混凝土而言，它有助于创造性地思考以保持方法行为统一（并因此是通用），而不是过于挑剔。例如，要么原始设计模式书或一本带有一本书是在谈论复合图案，该图案涉及“树”。好吧，他们所说的是，当您在叶子上（没有孩子）并试图迭代其不存在的孩子时，他们可以返回一个空名单，而不是抛出例外。