如何在 Python 类中正确调用类方法并避免重复计数问题-Python教程-PHP中文网

本文讲解为何在 __init__ 中直接调用类方法会导致计数逻辑错误，并通过职责分离（furniture 实体类 + furniturecollection 管理类）重构代码，实现单次、精准的材质统计。

本文讲解为何在 __init__ 中直接调用类方法会导致计数逻辑错误，并通过职责分离（furniture 实体类 + furniturecollection 管理类）重构代码，实现单次、精准的材质统计。

在原始代码中，Furniture.count_material() 被设计为遍历整个类变量 materials 并重新累加所有材质频次，但它被每次实例化时重复调用——这导致了严重的问题：第一次创建 "Chair" 后，materials = ["Wood"]，count_material() 统计得 {"Wood": 1}；第二次创建 "Cushion" 时，materials = ["Wood", "Cotton"]，而 count_material() 再次全量扫描并累加，结果 "Wood" 被重算为 2，"Cotton" 为 1；第三次后 "Wood" 变成 3，依此类推。最终输出 {'Chair': 3, 'Cushion': 2, 'Table': 1} 实际是误将 self.name 当作键名（因 materials 被错误地追加了 name？但原代码实为追加 material），更关键的是：逻辑混淆了「增量更新」与「全量重算」。

根本症结在于：__init__ 是实例生命周期起点，而 count_material() 承担了本应由外部聚合器完成的「状态维护」职责。正确的做法是分离关注点：

Furniture 应仅表示不可变的事实数据（如名称、材质）；
计数、集合管理、查询等行为应交由独立的容器类（如 FurnitureCollection）负责，确保状态单一、可预测。

以下是重构后的专业实践方案：

from dataclasses import dataclass
from collections import Counter

@dataclass
class Furniture:
    name: str
    material: str

class FurnitureCollection:
    def __init__(self):
        self.pieces: list[Furniture] = []
        self.material_count: Counter = Counter()

    def add(self, furniture: Furniture) -> None:
        self.pieces.append(furniture)
        self.material_count[furniture.material] += 1  # 增量更新，O(1)

    @property
    def materials(self) -> list[str]:
        return list(self.material_count.keys())

# 使用示例
fc = FurnitureCollection()
fc.add(Furniture("Chair", "Wood"))
fc.add(Furniture("Cushion", "Cotton"))
fc.add(Furniture("Table", "Glass"))

print(dict(fc.material_count))  # 输出: {'Wood': 1, 'Cotton': 1, 'Glass': 1}

✅ 优势说明：