[Romantyczny_widelec]

Robię aplikację w Django, która często komunikuje się z bazą. Dla uproszczenia przyjmijmy, że za każdym razem gdy ktoś wchodzi na konkretną stronę moja aplikacja zwiększa licznik odwiedzin tej strony o 1.

Powiedzmy, że mam takie 7 linijek kodu: https://pastebin.com/eqh6tDYu

Jak łatwo się domyślić przy większej liczbie wywołań, a nawet już przy dwóch jednoczesnych wyświetleniach strony coś może pójść nie tak:

wywołanie1 pobiera licznik z bazy: views=4

wywołanie1 zwiększa licznik o 1
wywołanie2 pobiera licznik z bazy: views=4 | już tu jest źle, aktualny licznik wynosi 5, ale nie został zapisany do bazy

wywołanie1 zapisuje licznik do bazy: views=5

wywołanie2 zwiększa licznik o 1
wywołanie2 chce zapisać do bazy: views=5 | a powinno być 6

Chyba wiadomo o co mi chodzi. Jak tym się ładnie zająć, jak zapewnić integralność bazy? Jakie są do tego mechanizmy w Django?
Jeżeli to ma jakieś znaczenie to jako bazy używam Postrgresa.

#programowanie #python #django

[biczek]

@Romantyczny_widelec: zrob kolejkowanie ;P

[Romantyczny_widelec]

@biczek: A oprócz tego? Nie powinienem jakichś dodatkowych mechanizmów wprowadzić?

[Ercefet]

@Romantyczny_widelec: Nie znam się zbytnio, ale czy Celery (moduł do robienia asynchronicznych tasków) nie eliminuje tego problemu?

[m.....o]

@Romantyczny_widelec: jeśli to nie jest uproszczony przykład, to pobierasz views, ale w bazie zwiększając licznik robisz coś typu update pages set views = views +1

[maykel]

@Romantyczny_widelec:
Poczytaj ( ͡° ʖ̯ ͡°)
https://docs.djangoproject.com/en/1.11/ref/models/querysets/#select-for-update
https://docs.djangoproject.com/en/1.11/ref/models/expressions/#f-expressions

[maledorak]

@Romantyczny_widelec: przenieś logike dodawania +1 do modelu (łatwiej obtestować)

object.save() zrób na object.save(update_fields=['views']) bo jak przerzucisz to do workerów celerowych to ci się będzie instancje obiektu nadpisywać.

[AlphaGoldfish]

Jezusie drogi. Celery? Workery? maykel dobrze podlinkował F-expressions. Zamiast pobierać dane z bazy, dodawać +1 a następnie zapisywać, "powiedz" bazie danych "dodaj +1 do licznika". Problem rozwiązany.

I dodaj może do FunnyWebsiteView metodę increment_views(self, delta=1) czy coś.

[krzysiek_pl]

@Romantyczny_widelec: j.w. @maykel

from django.db.models import F

counter = Counter.objects.get(website_name='Funny')
counter.views = F('views') + 1
counter.save()

[dog_meat]

@Romantyczny_widelec: transakcje i poziomy izolacji

[alex-fortune]

@Romantyczny_widelec:
Jeśli Twój silnik bazy danych wspiera atomiczność, możesz po prostu wykonać query na zasadzie SET counter = counter + 1
Jeśli wspiera transakcje - możesz zakładać transakcję ( czyli de facto lock ), ale nie polecam tego rozwiązania.

[alex-fortune]

@AlphaGoldfish: swoją drogą, fajnie na tym widać, jak ludzie próbują narzędziami rozwiązywać problemy ( ͡° ͜ʖ ͡°) to chyba efekt tego, że prawie każdy problem novadays rozwiąże za nas jakiś tool/jakaś paczka