Relationships “backward” Python Django

Relationships “backward” Python Django
GT
  • Rejestracja:ponad 5 lat
  • Ostatnio:prawie 3 lata
  • Postów:35
0

Cześć,
tworzę prostą aplikację TODO. Ma ona polegać na tym że każdy user po zalogowaniu się ma swoja listę zadań i każda z tych list ma swoje zadania.
Przykład:
Wtorek:(to jest LISTA ZADAŃ, a poniżej są zadania)

  • posprzątaj pokój
  • umyj auto
  • przeczytaj książkę

Środa:

  • idz na uczelnie
  • idź do fryzjera

Mam problem ponieważ chce uzyskać dostęp do zadań z LISTY ZADAŃ zalogowanego usera.
Przykład:
Jestem zalogowany jako Paweł mam dwie Listy zadań "Wtorek" i "Środa" i chciałbym zobaczyć co w każdej z tej listy są za zadania.

Podsyłam modele i kod do gita gdyby ktoś chciał zajrzeć do całości. W kodzie jest dużo namącone ponieważ próbuje sobie wszystkiego po kolei.
https://gitlab.com/pawroz/my-own-todo-app

Kopiuj
class TaskList(models.Model):
    user = models.ForeignKey(User, on_delete=models.CASCADE, null=True, blank=True)
    title = models.CharField(max_length=40)
    create_date = models.DateTimeField(auto_now_add=True)

    def __str__(self):
        return self.title


class Task(models.Model):
    #user = models.ForeignKey(User, on_delete=models.CASCADE, null=True, blank=True)
    task_list = models.ForeignKey(TaskList, on_delete=models.CASCADE, null=True, blank=True)
    title = models.CharField(max_length=40)
    description = models.TextField(max_length=200, null=True, blank=True)
    complete = models.BooleanField(default=False)
    created_date = models.DateTimeField(auto_now_add=True)

    def __str__(self):
        return self.title
Sevy
abstrahując od twojego pytania bo widzę, że odpowiedź już dostałeś - czy nie uważasz, że pole user nie powinno być nullable?
GT
W sumie masz racje, bo wartość "null=True" oznacza że pole może być równe "null" ale musi być wypełnione lub zostawione bez wartości, a "blank=True" oznacza że pole nie musi zostać wypełnione i może zostać bez wartości?
RA
  • Rejestracja:prawie 8 lat
  • Ostatnio:około 2 miesiące
  • Postów:221
0
Kopiuj
TaskList= TaskList.objects.filter(user=request.user)

for task in tasklist.task_set.all()
GT
  • Rejestracja:ponad 5 lat
  • Ostatnio:prawie 3 lata
  • Postów:35
1

@Rado: Niestety nie działa. Screen poniżej
screenshot-20220111191550.png

Shalom
  • Rejestracja:około 21 lat
  • Ostatnio:około 3 lata
  • Lokalizacja:Space: the final frontier
  • Postów:26433
2

o_O to są żarty jakieś? Przecież widzisz chyba ze twoja klasa Task ma pole task_list a nie żadne task_set. Mam spore wątpliwości co do tego czy napisałeś tam chociaż jedna linijkę sam.


"Nie brookliński most, ale przemienić w jasny, nowy dzień najsmutniejszą noc - to jest dopiero coś!"
edytowany 3x, ostatnio: Shalom
RA
Xd dobry pojazd po mnie
GT
  • Rejestracja:ponad 5 lat
  • Ostatnio:prawie 3 lata
  • Postów:35
1

@Shalom: Okej, to miej spore wątpliwości. Najwazniejsze że ja wiem i z czystym sumieniem moge powiedzieć, że napisałem wszystko sam.
Co z tego ze ma pole task_list? Chce dostać wszystkie Taski, które są w konkretnych TaskListach podpiętych pod zalogowanego usera.
screenshot-20220111193728.png

Shalom
  • Rejestracja:około 21 lat
  • Ostatnio:około 3 lata
  • Lokalizacja:Space: the final frontier
  • Postów:26433
3

@GoT98wppl kajam się, nie doczytałem dokładnie co robisz :( Teraz rozumiem co chcesz zrobić. O ile dobrze rozumiem to nie możesz zrobić tak jak próbujesz, bo masz w ręku QuerySet (bo zrobiłeś filtracje ale nadal masz potencjalnie X obiektów wybranych) a do tego odwołania przez _set potrzebujesz jeden obiekt. Bardzo możliwe że musisz przeiterować po tym co trzymasz jako tasklist (ale ze to jakiś QuerySet to moze trzeba coś na tym jeszcze zawołać)


"Nie brookliński most, ale przemienić w jasny, nowy dzień najsmutniejszą noc - to jest dopiero coś!"
edytowany 1x, ostatnio: Shalom
Sevy
  • Rejestracja:prawie 6 lat
  • Ostatnio:ponad 2 lata
  • Lokalizacja:Katowice
  • Postów:20
0

tutaj z usera wyciagasz liste zadan

Kopiuj
task_list = request.user.tasklist_set.filter(id=TUTAJ_ID).first()

tutaj z listy zadan wyciagasz zadania

Kopiuj
tasks = task_list.task_set.all()

zamiast all mozesz sobie filtrowac na rozne sposoby

edytowany 3x, ostatnio: Sevy
GT
tylko to task_list = request.user.tasklist_set.filter(id=TUTAJ_ID).first() zwróci mi po prostu jeden przypisany TaskList do usera, a nie wszystkie
RA
  • Rejestracja:prawie 8 lat
  • Ostatnio:około 2 miesiące
  • Postów:221
2

Zamiast filter daj tam get jeśli chcesz tylko jeden po np ID i powinno być ok daj jeszcze makemigrations i migrate Shaklom dobrze pisze tylko trzeba by machając tak jak wcześniej ale dodać jeszcze jednego for

Kopiuj
For x  in TaskList:
   For task in x.task_set.all()
edytowany 4x, ostatnio: Rado
GT
  • Rejestracja:ponad 5 lat
  • Ostatnio:prawie 3 lata
  • Postów:35
0

Dzięki wszystkim za komentarze i zaangażowanie!

Kopiuj
tasklist = TaskList.objects.filter(user=request.user)
for i in tasklist:
    print(i.task_set.all())
veneficus
Zmień sobie to i na inną zmienną.
RA
Zaakceptuj moja ostatnia odpowiedz albo @Shalom pierwsza nie jest poprawna
GT
okej zaakceptowałem. Zmieniłem, używam w takich pętlach z reguły innych zmiennych, tutaj podałem przykład na szybko, ale dzięki ze uwagę!
masochista
  • Rejestracja:około 6 lat
  • Ostatnio:4 dni
  • Lokalizacja:Warszawa
  • Postów:77
1

@Rado: Pobieranie tasków w pętli jest skrajnie nieoptymalne pod względem liczby uderzeń do bazy.
@GoT98wppl coś takiego na ten moment nie zrobi Ci to krzywdy, ale na przyszłość doczytaj o prefetch_related i select_related

edytowany 1x, ostatnio: masochista
RA
Ta ale ja tylko z telefonu pisze i to iphona se miałem dopisać selextrd ale mi się nie chciało już na tym małym dziadostwie dobrze ze czuwasz lepi choć trochę coś podpowiedzieć niż nic chociaż i tak złe ale moderator podpowiedział
RA
Django toolbar jest fajne zeby chyba to zobaczyć
masochista
Tak, django debug toolbar jest kozackie, tylko chyba nie działa jak się robi resta i np. wypluwa w swagger ui, a szkoda bo trzeba wtedy samemu dbać o takie rzeczy
GT
  • Rejestracja:ponad 5 lat
  • Ostatnio:prawie 3 lata
  • Postów:35
0

@masochista: Cześć, wiem że długo się nie odzywałem, ale czy mógłbyś pokazać na moim przykładzie jak użyć zagadnień, o których napisałeś, aby uzyskać efekt którego oczekuję?

GT
Teraz chce jeszcze raz tego uzyc i jak tak patrze to rzeczywiście wychodzi duzo kodu, i myślę że django na pewno na to coś zaradziło. A ja tworzę słownik w którym kluczem jest moja "TaskList" a wartościami są "Taski" które mają ten sam klucz obcy, wszystko to jest adekwatne dla aktualnie zalogowanego usera i dopiero to przenoszę do mojej templatki. BEZSENSU :(
masochista
Wrzuć aktualną wersję kodu, to będę mógł dokładniej podpowiedzieć, co należałoby zrobić. Przyda się zarówno kod pythonowy, jak i to, co siedzi w templatce.
GT
Super, dziękuję za pomoc @masochista Templatka ma "extends base", którego nie podsyłam Views: https://pastebin.com/dLTiJ6G8 Template: https://pastebin.com/msZhN41J

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.