Question

我经常发现自己覆盖了父类的方法，并且永远无法决定是否应该明确列出给定的参数，或者只是使用毯子*args, **kwargs构造。一个版本比另一个好吗？有最好的做法吗？我缺少什么（dis-）优势？

class Parent(object):

    def save(self, commit=True):
        # ...

class Explicit(Parent):

    def save(self, commit=True):
        super(Explicit, self).save(commit=commit)
        # more logic

class Blanket(Parent):

    def save(self, *args, **kwargs):
        super(Blanket, self).save(*args, **kwargs)
        # more logic

明确变体的好处

更明确（Python的禅宗）
更容易掌握
轻松访问的功能参数

一揽子变体的好处

更多DRY
父类很容易互换
在不触及其他代码的情况下传播父方法中的默认值更改

Answer 1

Liskov替代原则

通常，您不希望方法签名在派生类型中有所不同。如果要交换派生类型的使用，这可能会导致问题。这通常被称为Liskov Substitution Principle。

明确签名的好处

与此同时，我认为所有方法都没有*args，**kwargs的签名是正确的。明确签名：

通过良好的参数名称帮助记录方法
通过指定哪些args是必需的以及哪些具有默认值
提供隐式验证（缺少必需的args抛出明显的异常）

可变长度参数和耦合

不要将可变长度参数误认为是良好的耦合实践。父类和派生类之间应该有一定的内聚力，否则它们就不会彼此相关。相关代码导致耦合反映内聚水平是正常的。

使用可变长度参数的位置

使用可变长度参数不应该是您的第一选择。当你有充分的理由时应该使用它：

定义函数包装器（即装饰器）。
定义参数化多态函数。
当您可以采用的参数确实是完全可变的时（例如，通用的DB连接函数）。数据库连接函数通常采用许多不同形式的connection string，以单个arg形式和多arg形式。不同的数据库也有不同的选项集。
...

你做错了吗？

如果您发现自己经常创建带有许多参数的方法或带有不同签名的派生方法，那么您在组织代码方面可能会遇到更大的问题。

Answer 2

我的选择是：

class Child(Parent):

    def save(self, commit=True, **kwargs):
        super(Child, self).save(commit, **kwargs)
        # more logic

它避免了从*args和**kwargs访问提交参数，并且如果Parent:save的签名发生更改（例如添加新的默认参数），它会保证安全。

更新：在这种情况下，如果将新的位置参数添加到父级，则使用* args会导致麻烦。我只保留**kwargs并仅使用默认值管理新参数。它可以避免传播错误。

Answer 3

如果您确定Child会保留签名，那么明确的方法肯定是可取的，但是当Child改变签名时我个人更喜欢使用这两种方法：

class Parent(object):
    def do_stuff(self, a, b):
        # some logic

class Child(Parent):
    def do_stuff(self, c, *args, **kwargs):
        super(Child, self).do_stuff(*args, **kwargs)
        # some logic with c

这样，签名中的更改在Child中非常易读，而原始签名在Parent中非常易读。

在我看来，当你有多重继承时，这也是更好的方法，因为当你没有args和kwargs时，调用super几次是非常恶心的。

对于它的价值，这也是很多Python库和框架中的首选方式（Django，Tornado，Requests，Markdown，仅举几例）。虽然不应该根据这些事情做出选择，但我只是暗示这种做法非常普遍。

Answer 4

不是一个真正的答案，而是更多的旁注：如果你真的，真的想确保父类的默认值传播到子类，你可以做类似的事情：

class Parent(object):

    default_save_commit=True
    def save(self, commit=default_save_commit):
        # ...

class Derived(Parent):

    def save(self, commit=Parent.default_save_commit):
        super(Derived, self).save(commit=commit)

但是我必须承认这看起来很丑陋，如果我觉得我真的需要它，我只会使用它。

Answer 5

我更喜欢显式参数，因为自动完成允许您在进行函数调用时查看函数的方法签名。

Answer 6

除了其他答案：

具有可变参数可以使父级与子级“分离”，但是会在创建的对象与父级之间建立耦合，我认为这更糟，因为现在您创建了“长距离”对（更难发现，维护起来比较困难，因为您可能会在应用程序中创建多个对象）

如果您要进行去耦，请查看composition over inheritance

继承最佳实践：* args，** kwargs或显式指定参数

6 个答案: