Применить это использовать сигналы Django в том же приложении

Question

Попытка добавить уведомление по электронной почте в мое приложение самым чистым способом. При изменении определенных полей модели приложение должно отправить уведомление пользователю. Вот мое старое решение:

from django.contrib.auth import User

class MyModel(models.Model):
    user = models.ForeignKey(User)
    field_a = models.CharField()
    field_b = models.CharField()

    def save(self, *args, **kwargs):
        old = self.__class__.objects.get(pk=self.pk) if self.pk else None
        super(MyModel, self).save(*args, **kwargs)
        if old and old.field_b != self.field_b:
            self.notify("b-changed")
        # Sevelar more events here
        # ...

    def notify(self, event)
        subj, text = self._prepare_notification(event)
        send_mail(subj, body, settings.DEFAULT_FROM_EMAIL, [self.user.email], fail_silently=True)

Это работало нормально, когда у меня был один или два типа уведомлений, но после этого я чувствовал себя неправильно, когда в моем методе save() было так много кода. Итак, я изменил код на сигнал:

from django.db.models import signals

def remember_old(sender, instance, **kwargs):
    """pre_save hanlder to save clean copy of original record into `old` attribute
    """
    instance.old = None
    if instance.pk:
        try:
            instance.old = sender.objects.get(pk=instance.pk)
        except ObjectDoesNotExist:
            pass

def on_mymodel_save(sender, instance, created, **kwargs):
    old = instance.old
    if old and old.field_b != instance.field_b:
        self.notify("b-changed")
    # Sevelar more events here
    # ...

signals.pre_save.connect(remember_old, sender=MyModel, dispatch_uid="mymodel-remember-old")
signals.post_save.connect(on_mymodel_save, sender=MyModel, dispatch_uid="mymodel-on-save")

Преимущество заключается в том, что я могу разделять обработчики событий на разные модули, уменьшая размер models.py, и я могу включать / отключать их по отдельности. Недостатком является то, что это решение состоит в том, что код и обработчики сигналов отделены от самой модели, и незнающий читатель может пропустить их вообще. Итак, коллеги, вы думаете, оно того стоит?

Answer 1

Я думаю, что это хорошая идея. Обсуждение «Пользовательские сигналы для несвязанного дизайна» от самого последнего DjangoCon является отличным ресурсом того, что возможно и целесообразно для сигналов в Django.

Answer 2

Я думаю, что использование сигналов здесь - хорошее дизайнерское решение. Уведомление не является частью сохранения, оно является следствием сохранения. Работа с этими типами последствий является причиной, по которой Джанго дает сигналы.