能否用其他类的方法覆写实例方法？该做法是否属于不良实践？

示例代码 #

class test:
  def func(self):
    print(5)

class test1:
  def testing(self, t):
    t.func = self.another_func
  def another_func(self):
    print(6)

t = test()
t1 = test1()
t1.testing(t)
# now t.func() uses test1's another_func and prints 6
t.func()

问题描述 #

这是针对工作场景编写的简化示例。此前从未见过这种方法覆写方式，虽然运行能得到预期结果，但感觉不够规范，不确定是否属于不良编程实践。

另外这种做法会严重降低代码可读性：查看func的原始定义时，完全无法得知它已经被外部覆写，后续维护时很容易出现逻辑混淆或bug。

分析与结论 #

这种**动态修改实例属性（方法）**的写法确实属于不推荐的不良编程实践，核心问题包括：

• 可读性极差：test类中func的定义和实际运行逻辑完全脱节，其他开发者（甚至一段时间后的你自己）查看test类时，根本想不到这个方法会被外部对象篡改，排查问题时会浪费大量时间。
• 破坏封装性：面向对象编程的核心原则之一是封装，类的行为应该由自身定义和控制，外部直接修改实例的方法会打破这种封装，导致代码逻辑混乱、难以维护。
• 隐藏上下文依赖：t.func被替换成test1实例的another_func后，another_func的运行依赖test1实例的状态，如果后续test1实例被销毁或状态改变，调用t.func()可能会引发难以预料的错误。

更规范的替代方案 #

如果需要实现类似逻辑，推荐使用以下几种符合Python面向对象规范的方式：

1. 继承与重写 #

如果test类的行为需要被扩展，直接通过继承重写方法：

class test:
    def func(self):
        print(5)

class test_extended(test):
    def func(self):
        print(6)

t = test_extended()
t.func()  # 输出6

2. 组合模式 #

如果test和test1是不同职责的类，通过组合复用逻辑，而非篡改实例方法：

class test:
    def __init__(self, func_impl=None):
        # 允许传入自定义实现逻辑
        self.func_impl = func_impl or self.default_func
    
    def default_func(self):
        print(5)
    
    def func(self):
        self.func_impl()

class test1:
    def another_func(self):
        print(6)

t1 = test1()
t = test(func_impl=t1.another_func)
t.func()  # 输出6

3. 装饰器（适合动态增强逻辑） #

如果需要动态给方法添加功能，装饰器是更清晰的方式：

def replace_func_with_another(func):
    def wrapper(self):
        print(6)
    return wrapper

class test:
    def func(self):
        print(5)

t = test()
t.func = replace_func_with_another(t.func)
t.func()  # 输出6

这种方式能明确看到func被修改的逻辑，比直接替换更直观。

内容的提问来源于stack exchange，提问作者h8n2