Как обновлять модель в django автоматически?

Question

[mezigar]

Есть 2 модели:
1 - это средний рейтинг игры и сама игра соответственно.
Хочется, чтобы при каждой новой оценке пользователя, средний рейтинг обновлялся
Для этого условно написана функция new_avg. Как мне отслеживать, что пользователь ставит новую оценку и автоматом вызывать функцию new_avg?
В качестве пользователя использую стандартного джангового юзера

class AvgRating(models.Model):
    avg = models.FloatField(default=0)
    all = models.IntegerField(default=0)


    def __init__(self, rate = 0):
        self.avg = rate
        self.all = 1

    def new_avg(self, prev_avg, prev_all, rate):
        self.all = prev_all + 1
        self.avg = (prev_avg * prev_all + rate) / self.all
        return self.avg

class Game(models.Model):
    Rate = (
        ('1 star', 1),
        ('2 stars', 2),
        ('3 stars', 3),
        ('4 stars', 4),
        ('5 stars', 5),
    )

    rating = models.CharField(max_length=7, choices=Rate)
    title = models.CharField(max_length=100, default='Mario')
    publisher = models.CharField(max_length=100, null=True)
    developer = models.CharField(max_length=100, null=True)
    avg = models.ForeignKey(AvgRating, on_delete=models.CASCADE, default=0)

    def __str__(self):
        return self.title + " " + str(self.rating)

Comments

[Модератор]

В вопросе по django не должно быть тэга Python, см.п.3.1 правил.

Answer 1

[Александр Нестеров]

1.Сигналы
2.Переопределите метод save.
Но прежде чем делать это учтите что сейчас это, уж извините, это ужас а не структура данных.

В качестве пользователя использую стандартного джангового юзера

Да? Ну и где это отображено в моделях? Сейчас я вижу, что для каждой игры может быть выставлена только одна оценка и средняя всегда будет равна ей. При этом с юзером эта оценка никак не связана. При этом - достаточно удалить это оценку и запись об игре тоже удалиться.
Далее - документация не рекомендует переопределять конструктор.
Допустим, вы это знаете, но решили, что так будет правильнее, ибо это именно что рекомендация. Но далее вопрос: вам не приходило в голову, что у модели уже есть какая-то важная логика в конструкторе?
Следующий вопрос: почему вы уверены что данные, передаваемые через параметры функции будут актуальнее, чем полученные путем выборки из базы.

Comments

[mezigar]

Уф, жестко, но по факту. Спасибо за ваш комментарий, почитаю про сигналы, уберу конструктор, модели допилю, спасибо ещё раз за развернутый фидбек!

[mezigar]

Ещё раз спасибо, а данного кода не будет достаточно для моих нужд?

class AvgRating(models.Model):
    _avg = models.FloatField(default=0)
    all = models.IntegerField(default=0)

def set_avg(self, rate = 0):
    self.is_updated = True
    self.all += 1
    self._avg = (self._avg * self.all + rate) / self.all
    self.is_set = True

def get_avg(self):
    return self._avg

avg = property(get_avg, set_avg)

[Александр Нестеров]

1.Перенести среднюю оценку в модель игры.
2.Во второй - указать 3 поля: связь с юзером/игрой и оценка
3.Высчитывать среднее c помощью агрегации (можно как постоянно сохранять это значение в поле, для уменьшения количества запросов, так и считать каждый раз по новой_

[mezigar]

Спасибо огромное за ваш фидбек!Постарался сделать то, что вы написали.К сожалению у меня не загружается сайт доки джанго(пробовал разные сайты и разные впн), но сделал что-то погуглив агрегацию:

class Game(models.Model):
    Rate = (
        ('1 star', 1),
        ('2 stars', 2),
        ('3 stars', 3),
        ('4 stars', 4),
        ('5 stars', 5),
    )

    rating = models.CharField(max_length=7, choices=Rate)
    title = models.CharField(max_length=100, default='Mario')
    publisher = models.CharField(max_length=100, null=True)
    developer = models.CharField(max_length=100, null=True)
    avg = models.FloatField(default=0)

    def update_avg(self, title):
        self.avg = Game.objects.filter(title=title).aggregate(Avg('rating'))

    def __str__(self):
        return self.title + " " + str(self.rating)

    def save(self, *args, **kwargs):
        self.avg = self.update_avg(self.title)
        super(Game, self).save(*args, **kwargs)


class Profile(models.Model):
    user = models.OneToOneField(User, on_delete=models.SET_DEFAULT, default='user')
    # games = models.ManyToManyField(Game)
    def __str__(self):
        return str(self.user.username)

Проблема теперь в том, что при сохранении объекта game всё крашится с ошибкой "NOT NULL constraint failed Ругается на строчку super(Game, self).save(*args, **kwargs)
Не пойму, что у меня не так
*Update:
Скорее всего что-то не так с тем, как я вызываю агреацию внутри функции update_avg, пока что остановился на агрегации

[Александр Нестеров]

mezigar,
Обновление стоит делать со стороны модели рейтинга:

связь с юзером/игрой и оценка

(ее я кстати, не вижу у вас)
И собственно, считать не по игра, с соответствующим названием, а по оценкам, которые относятся к указанной в поле игре.
Т.е. условно:

self.avg = [Модель рейтинга].objects.filter(game_id=game_id).aggregate(Avg('rating'))

[mezigar]

Александр Нестеров, спасибо за ваш фидбек, но я плохо понимаю, как правильно связать модели. Начну по порядку "Game" - есть название, издатель, и т.д. , но важно автоматически созданное id. Имеется рейтинг, у каждый игры есть свой средний рейтинг, тут вроде бы связь один к одному, если я не ошибаюсь. У профиля- наследника юзера есть пул игр, в которых он голосовал, причем у каждой игры много голосующих, получается многие ко многим, но в каком порядке и где прописывать связи, я пока не понимаю. Очень рассчитываю на вашу помощь!

[Александр Нестеров]

игры есть свой средний рейтинг, тут вроде бы связь один к одному

Не плодите сущности. Что будет плохого в том, что средний балл будет храниться в модели игры? Ничего. Что плохого в том, чтобы выносить этот балл в другую таблицу? Как минимум - не очевидность структуры.

У профиля- наследника юзера есть пул игр, в которых он голосовал, причем у каждой игры много голосующих, получается многие ко многим, но в каком порядке и где прописывать связи, я пока не понимаю.

Юзер (стандартная модель) (Либо ваша модель профиля, но в таком случае могут понадобиться лишние запросы)

Игра (ваша модель)

Оценки (новая модель):
--Юзер (FK)
--Игра (FK)
--Оценка (int)

Собственно, когда пользователь ставит оценку, данные записываются в таблицу "Оценки".

[mezigar]

Александр Нестеров, Огромное спасибо за помощь, модели работают так, как я и хотел! Однако я не понимаю, как агрегатные функции прикрутить к полям модели? Нужно где-то отдельно считать, вне класса модел
Update: узнал про аннотирование

[Александр Нестеров]

mezigar,

как агрегатные функции прикрутить к полям модели?

Напрямую - никак, они не предназначены для этого. Дальше - все зависит от того, что именно вам нужно.
Подозреваю, что вы хотите, чтобы среднее значение, полученное из агрегатной функции записывалось в соответствующее поле.
Тут вариантов несколько:
Django-style: Менеджеры моделей
Python-style: Декоратор property
Чуть менее ресурсозатратный (в теории): Считать среднее значение при сохранении каждой оценки и записывать вручную.