remove duplicate objects from queryset

[AndrzejMorawski00]

Problem wynikał z dodania do funkcji ThesesQuerySet.visible warunku Q(students__user=user). Ten warunek jest potrzebny, aby wyświetlić pracę dyplomową osobom, które powinny ją zobaczyć, gdy jest ona weryfikowana przez komisję lub została zwrócona do poprawek. Ponieważ pole students w modelu Thesis jest relacją m2m to mamy do czynienia ze złączeniem wielu tabel. W tym momencie otrzymujemy tyle wyników danej pracy dyplomowej ilu jest przypisanych do niej studentów. Aby rozwiązać ten problem usuwamy duplikaty z zapytania dopisując .distinct() do zapytania.

[AndrzejMorawski00]

Z tego co zrozumiałem to w tej funkcji mamy taką konstrukcję: * Jeżeli użytkownik jest członkiem rady lub ma status 'is_staff' to zwracamy wszystkie prace coś na zasadzie Thesis.objects.all() * W przeciwnym wypadku sprawdzamy następujące rzeczy: 1. Filtrujemy tylko te prace, które nie są ewaluowane lub nie zostały zwrócone do poprawek 2. Do tego zbioru dodajemy te prace, które spełniają kolejne warunki. Na samym końcu warunku sprawdzamy, czy użytkownik może zobaczyć tę pracę a ponieważ to może być relacja wiele-wiele to będziemy mieli kilka par dla tej samej pracy np: Praca -> student1 Praca -> student2 Praca -> student3 I to nam zwraca powtórzenia, które pojawiają się tylko w momencie, gdy do jednej pracy jest przypisany więcej niż 1 student. Więc wydaje mi się, że to wynika po prostu ze sposobu w jaki django tworzy zapytania. wt., 29 paź 2024 o 14:54 Michal Augustyn ***@***.***> napisał(a):

…

Ja chyba dalej nie czaje czemu te zwielokrotnienia prac pokazywały się jednemu typowi użytkowników, a drugim nie — Reply to this email directly, view it on GitHub <#1755 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AROHVUT7CBVLEQBUHLJ44CTZ56HQ5AVCNFSM6AAAAABQZOPTGGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBUGMZTGNZUGA> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[lgpawel]

Ad dlaczego to czasami działało: w instrukcji warunkowej ThesesQuerySet.visible wchodziliśmy do gałęzi, gdzie nie było problematycznego zapytania do bazy, tylko zwykłe return self. Przy okazji przekonałem się, że są jeszcze podstrony, na których efekt (w bieżącym stanie) jest jeszcze gorszy – wyjątek MultipleObjectsReturned, oczywiście niezłapany...

Rozwiązanie wydaje się działać, ale z punktu widzenia pisania zapytań do bazy danych to jest trochę hotfix, zaciemniający (moim zdaniem) naturę zapytania. Jasne, że ostatni obiekt Q dopisany w filtrze jest źle napisany, ale lepiej byłoby napisać go od razu dobrze, żeby sprawdzał, czy bieżący użytkownik jest wśród przypisanych do tematu (a nie równy któremuś z nich). Analogicznie do SQLowego operatora IN [podzapytanie] mamy w Django takie coś https://docs.djangoproject.com/en/3.2/ref/models/querysets/#in choć efekt może nie być jednolinijkowcem...

[AndrzejMorawski00]

Poprawiłem zapytanie tak, żeby generowało queryset bez duplikatów w bardziej czytelny sposób.

[lgpawel]

Chyba działa i faktycznie jest czytelne. Może jeszcze fajniej byłoby, gdyby pomocniczy obiekt (wtedy lepiej o nazwie np. my_theses) wybierał wszystkie tematy, z którymi związany jest użytkownik (w dowolnej z trzech wiadomych ról) i potem sumowałoby się to (mnogościowo) z tematami "ogólnodostępnymi"? To nie jest pytanie retoryczne.