在django中使用观察者模式的问题

Question

我正在一个销售产品的网站上工作（一类销售，一类产品）。每当我销售产品时，我都想将该操作保存在历史表中，并且我决定使用观察者模式来执行此操作。

也就是说：我的 Sales 类是主题，History 类是观察者，每当我调用 Sales 类的 save_sale() 方法时，我都会通知观察者。 （我决定使用这种模式，因为稍后我还会发送电子邮件，通知管理员等）

这是我的主题类（销售类由此扩展）

class Subject:
    _observers = []

    def attach(self, observer):
        if not observer in self._observers:
            self._observers.append(observer)

    def detach(self, observer):
        try:
            self._observers.remove(observer)
        except ValueError:
            pass

    def notify(self,**kargs):
        for observer in self._observers:
            observer.update(self,**kargs)

在视图上我做了这样的事情

sale = Sale()
sale.user = request.user
sale.product = product
h = History() #here I create the observer
sale.attach(h) #here I add the observer to the subject class
sale.save_sale() #inside this class I will call the notify() method

这是历史记录的更新方法

def update(self,subject,**kargs):
    self.action = "sale"
    self.username = subject.user.username
    self.total = subject.product.total
    self.save(force_insert=True)

第一次工作正常，但是当我尝试进行另一次销售时，我收到一条错误消息，说由于主键约束，我无法插入历史记录。

我的猜测是，当我第二次调用视图时，第一个观察者仍在主题类中，现在我有两个历史观察者正在监听销售，但我不确定这是否是问题所在（天哪，我错过了来自 php 的 print_r）。

我做错了什么？我什么时候必须“附加”观察者？或者有更好的方法吗？

顺便说一句：我使用的是 Django 1.1，我无权安装任何插件。

Answer 1

这可能不是一个可接受的答案，因为它与架构更多相关，但是您是否考虑过使用信号来通知系统更改？看来您正在尝试完全按照信号的设计目的进行操作。 Django 信号具有与观察者模式相同的最终结果功能。

http://docs.djangoproject.com/en/1.1/topics/signals/

Answer 2

我认为这是因为 _observers = [] 的作用类似于静态共享字段。因此 Subject 的每个实例都会更改 _observers 实例，并且会产生不需要的副作用。

在构造函数中初始化该变量：

class Subject:

    def __init__(self):
        self._observers = []

Answer 3

@Andrew Sledge 的 answer 指出了解决此问题的好方法。我想建议一种替代方法。

我遇到了类似的问题并开始使用信号。它们工作得很好，但我发现我的单元测试变得更慢，因为每次我使用固定装置加载关联类的实例时都会调用信号。这使测试运行增加了数十秒。有一个解决办法，但我发现它很笨拙。我定义了一个自定义测试运行程序，并在加载夹具之前断开了我的函数与信号的连接。后来我重新连接了它们。

最后，我决定完全放弃信号并覆盖模型的适当 save() 方法。就我而言，每当 Order 发生更改时，都会在 OrderHistory 表中自动创建一行。为此，我添加了一个函数来创建 OrderHistory 实例，并从 Order.save() 方法中调用它。这也使得单独测试 save() 和函数成为可能。

看看这个SO问题。它讨论了何时重写 save() 以及何时使用信号。

Answer 4

谢谢大家的回答，阅读有关信号的内容给了我另一个视角，但由于学习目的我不想使用它们（我想在网络开发中使用观察者模式：P）最后，我解决了做这样的事情：

class Sales(models.Model,Subject):
    ...
    def __init__(self):
        self._observers = []  #reset observers
        self.attach(History()) #attach a History Observer
    ...
    def save(self):
        super(Sales,self).save()
        self.notify() # notify all observers

现在每次我调用 save() 时，观察者都会收到通知，如果我需要它，我可以添加或删除观察者，

你觉得怎么样？这是解决问题的好方法吗？