Как одним запросом вытащить суммарную цену вот из этого «self.gifts.all()»?

Question

[lehubozedo]

Есть три модели, у каждой есть поле price

class Tomato(models.Model):
    price=...
class Potatoes(models.Model):
    price=...
class Eggplant(models.Model):
    price=...

И есть 4ая модель:

class Vegetable(models.Model):
    tomato = models.OneToOneField(Tomato, verbose_name='Томат', on_delete=models.CASCADE, blank=True, null=True)
    potatoes = models.OneToOneField(Potatoes, verbose_name='Potatoes', on_delete=models.CASCADE, blank=True, null=True)
    eggplant = models.OneToOneField(Eggplant, verbose_name='Eggplant', on_delete=models.CASCADE, blank=True, null=True)
    gifts = models.ManyToManyField('self', blank=True, symmetrical=False)

Как одним запросом вытащить суммарную цену вот из этого (self это Vegetable) "self.gifts.all()" ?
Проблема в том, что ЛЮБОЙ экземпляр vegetable, должен и содержит только что то одно:

print vegetable.tomato, vegetable.potatoes, vegetable.eggplant
.... "Tomato-delicious", none, none

gifts понятное дело, содержит ссылки на несколько экземпляров Vegetable, как вытащить сумму price этих экземпляров?

Comments

[Anton Kuzmichev]

Продолжаете проектировать модели по одним Вам известным принципам? :)

[sim3x]

Сделайте одну модель с полем тип_плода

[lehubozedo]

Anton Kuzmichev: да нормальные принципы) я сначала смотрел в сторону contenttypes , но все же для моих (ИМ) вышеописанное удобно.
Суммировать я бы мог и циклом, но боюсь больших накладных расходов, ведь таких объектов будет много, хотелось бы одним запросом, но как - не соображу.

[lehubozedo]

sim3x: поясните плиз, у меня же есть все данные, зачем тип то нужен?
так то тип есть, он choises у каждой из трех, но для каких целей модель создавать?

[Алексей Сергеев]

Объясните что Вы хотите сделать. Т.е. смысл этих сущностей.

[Mykola]

lehubozedo посмотрите в сторону aggregation (aggregate/annoate)

[lehubozedo]

Mykola: да я уж и так, и сяк (aggregate/annoate), не выходит что то

[sim3x]

lehubozedo:

Потому что вы не нормализировали схему БД

https://docs.djangoproject.com/en/1.11/ref/models/...

Чтоб помочь в создании модели, нужно понять, что вы вообзе делаете

[lehubozedo]

Алексей Сергеев:
3 модели - это товары, они разные, но есть одинаковые поля, name, price...
4-ая - для удобного доступа к атрибутам первых трех
например в корзине могут быть разные товары, через 4-ую удобно отображать в цикле

[lehubozedo]

sim3x: ну 5 нормальной формы тут, наверное, нет, но это ведь прикладная задача..

[sim3x]

lehubozedo: похоже на 4

Если задача прикладная, то еще не значит, что ее можно делать через одно место

[lehubozedo]

sim3x: соглаен) я уж оптимизировал как смог) так кода меньше, и быстрее пишется, только один затык остался, остальное отлично все

[Алексей Сергеев]

Зачем Вам каждому товару отдельная модель?)

[sim3x]

lehubozedo: еще раз

Модели построены не правильно

Дело не в оптимизации - тут ею и не пахнет

[lehubozedo]

Алексей Сергеев: ну сапоги и апельсины же в одну модель не очень укладываются да?

[lehubozedo]

Алексей Сергеев: или Вы меня держите за идиота и думаете что я для "Сапоги 40", "Сапоги 42" создаю разные модели?))

[Алексей Сергеев]

lehubozedo: Поясните, чем картошка отличается от Томата, как продукт?

[Anton Kuzmichev]

Вам 100500 человек говорят, что у вас что-то не то с моделями.
Создайте вопрос, чтобы вам помогли с проектированием моделей. Опишите вашу задачу в более общем виде. Типа, какими сущностями предметной области вы хотите оперировать. И каким образом оперировать - распишите кейсы. И помогут родить нормальное решение. Чтобы не надо было модифицировать БД при добавлении новых "овощей".

[lehubozedo]

Алексей Сергеев: разными характеристиками, например у картофеля будет характеристика "% содержания крахмала" и пойдут разные сорта, у одной больше, у другой - меньше

[Алексей Сергеев]

Есть же для этого всякие EAV и json поля в Postgre. Может стоит покопать в их сторону?)

[lehubozedo]

Если честно - усложнение пошло, пока забью, раз простого решения не нашлось, оставлю это:

def gifts_price(self):
        return sum(prod.price_real for prod in self.gifts.all())

пока задачу решает, в дальнейшем raw запрос напишу и все..

[Алексей Сергеев]

Отметьте тогда решением :)

А вообще - если Вам не нужно делать фильтр по % содержания крахмала и прочему, то вынесите это просто в description. :)

[lehubozedo]

Алексей Сергеев: нужно, там много таких полей

Answer 1

[Алексей Сергеев]

Вот так можно. Только погуглите в сторону select и prefetch_related для self.gifts.all()

class Vegetable(models.Model):
    tomato = models.OneToOneField(Tomato, verbose_name='Томат', on_delete=models.CASCADE, blank=True, null=True)
    potatoes = models.OneToOneField(Potato, verbose_name='Potatoes', on_delete=models.CASCADE, blank=True, null=True)
    eggplant = models.OneToOneField(Eggplant, verbose_name='Eggplant', on_delete=models.CASCADE, blank=True, null=True)
    gifts = models.ManyToManyField('self', blank=True, symmetrical=False)

    @property
    def get_price(self):
        tomato = self.tomato.price if self.tomato is not None else 0
        potatoes = self.potatoes.price if self.potatoes is not None else 0
        eggplant = self.eggplant.price if self.eggplant is not None else 0
        return sum([tomato, potatoes, eggplant])
        
    @property   
    def get_gift_price(self):
        return sum([x.get_price for x in self.gifts.all()])

Comments

[Алексей Сергеев]

Но модели нужно!!! менять.
Представьте что Вы завтра, помимо картошки с помидорами , ещё и арбузами торговать начали? Новая таблица в БД и дописывание всего кода? :)