Django Quiz - Code Review

Question

Cześć, uczę się frameworka Django od jakiegoś czasu i postanowiłem stworzyć jakąś własną aplikacje. Jest to strona internetowa z quizami, oferuje użytkownikom między innymi możliwość tworzenia własnych quizów, rozwiązywanie ich, komentowanie, dodałem również możliwość oceny przy pomocy łapek w górę lub w dół. Prosiłbym o ocenę kodu tzn. co mógłbym napisać lepiej, czy nie zastosowałem jakiś złych praktyk itd..

Link do repozytorium:

https://github.com/CodingForFuture/django_quiz

Answer 1

Pomijając brak testów i mniejsze lub większe pierdoły (np osobiście bym zamienił te widoki z funkcji na widoki klasowe, powinno być trochę mniej kodu), to najbardziej w oczy się rzuca słabe używanie ORMa.

    answer_2 = question.answers.filter(is_correct=False).all()[0]
    answer_3 = question.answers.filter(is_correct=False).all()[1]
    answer_4 = question.answers.filter(is_correct=False).all()[2]

            for j in range(4):
                choices.append([
                    question.answers.all()[j], question.answers.all()[j].is_correct,
                    int(question.answers.all()[j].id) == int(data.get(f'answer{question.id}'))])

    if request.user in quiz.likes.all():
        quiz.likes.remove(request.user)
    else:
        if request.user in quiz.dislikes.all():

    question = get_object_or_404(Question, pk=pk)
    if request.user != question.quiz.author:

Robisz tutaj kilka razy więcej zapytań SQLowych niż jest potrzeba. Polecam poczytać o funkcjonalności querysetów, w szczególności select_related, prefetch_related.

Na przykład w tym ostatnim samo `question.quiz` wygeneruje nowe zapytanie bazowanowe wyciągające quiz tego pytania. To jest podręcznikowe miejsce na użycie `select_related`.

Comments

Dzięki za odpowiedź. Poczytałem trochę na temat select_related i prefetch_related i spróbowałem wprowadzić zmiany do mojego kodu, są one już dostępne na githubie, więc byłbym wdzięczny jeśli w wolnej chwili sprawdziłbyś, czy wprowadzone zmiany są poprawne