在图形模拟开发中如何正确实现渲染与逻辑分离？

你的当前实现已经走在了正确的路上——用ShapePrinter统一处理渲染逻辑，让形状类只负责存储自身数据，完全符合关注点分离原则。不过现有实现还有几个可以优化的地方，主要集中在扩展性和代码严谨性上：

当前实现的小问题 #

• 依赖ShapeType枚举做类型判断，新增形状时需要同时修改枚举、形状类和ShapePrinter的注册表，违反开闭原则
• render_triangle方法里的打印文本写错了（把triangle写成circle了）
• 抛出的异常类型太宽泛，应该用更具体的异常（比如ValueError）

优化方案：使用访问者模式 #

访问者模式是处理这种"数据类+多操作"场景的经典方案，它能在不修改形状类的前提下，新增不同的渲染（或其他操作）逻辑，完美适配开闭原则。

优化后的代码示例 #

from abc import ABC, abstractmethod

# 定义访问者接口，规范对不同形状的操作方法
class ShapeVisitor(ABC):
    @abstractmethod
    def visit_rectangle(self, rectangle: 'Rectangle'):
        pass

    @abstractmethod
    def visit_circle(self, circle: 'Circle'):
        pass

    @abstractmethod
    def visit_triangle(self, triangle: 'Triangle'):
        pass

# 形状基类，定义接受访问者的统一入口
class Shape(ABC):
    @abstractmethod
    def accept(self, visitor: ShapeVisitor):
        pass

# 具体形状类，实现accept方法，将自身传递给访问者对应方法
class Rectangle(Shape):
    def __init__(self, x, y):
        self.x = x
        self.y = y

    def accept(self, visitor: ShapeVisitor):
        visitor.visit_rectangle(self)

class Circle(Shape):
    def __init__(self, r):
        self.r = r

    def accept(self, visitor: ShapeVisitor):
        visitor.visit_circle(self)

class Triangle(Shape):
    def __init__(self, a, b, c):
        self.a = a
        self.b = b
        self.c = c

    def accept(self, visitor: ShapeVisitor):
        visitor.visit_triangle(self)

# 控制台渲染器，实现访问者接口，专注处理控制台打印逻辑
class ConsoleShapeRenderer(ShapeVisitor):
    def __init__(self, surface: str):
        self.surface = surface

    def visit_rectangle(self, rectangle: Rectangle):
        print(f"Rendering a rectangle ({rectangle.x}, {rectangle.y}) on the {self.surface}")

    def visit_circle(self, circle: Circle):
        print(f"Rendering a circle ({circle.r}) on the {self.surface}")

    def visit_triangle(self, triangle: Triangle):
        print(f"Rendering a triangle ({triangle.a}, {triangle.b}, {triangle.c}) on the {self.surface}")

# 使用示例
renderer1 = ConsoleShapeRenderer('surface1')
renderer2 = ConsoleShapeRenderer('surface2')

Rectangle(10,5).accept(renderer1)
Circle(7).accept(renderer2)
Triangle(3,4,5).accept(renderer1)

其他优化建议 #

• 拆分渲染器接口：如果后续需要支持不同的渲染目标（比如OpenGL窗口、SVG文件），可以定义Renderer基类，再实现OpenGLRenderer、SVGRenderer等子类，每个子类专注于一种渲染方式
• 移除冗余枚举：访问者模式不需要依赖ShapeType枚举，形状类自身的accept方法已经能明确类型，减少代码冗余
• 使用更具体的异常：如果遇到未实现的形状操作，抛出NotImplementedError比通用Exception更清晰
• 保持单一职责：如果形状类需要处理计算面积、碰撞检测等逻辑，把这些逻辑放到独立的工具类里，让形状类只负责存储数据

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