Code Review DJANGO REST

Question

Witam, jestem dość świeży, jeśli chodzi o pisanie backendu natomiast prosiłbym was o ocenę mojego kodu napisanego w Djnago i o jakieś ogólne uwagi. Aplikacja to proste API "portalu społecznościowego", możliwość tworzenia usera/logowanie/wstawiania filmów/zdjęć/możliwość ich likowania.

Kod: https://github.com/PuszkinN/django-rest-api

Answer 1

Kilka rzeczy na oko:

                    follow_request = FollowRequest.objects.create(
                        sender=request.user,
                        receiver=user,
                        is_active=True
                    )
                    follow_request.save()

Z tego co wiem, `create()` to skrót na stworzenie obiektu + save, więc ręczne `save()` nie powinno być potrzebne. To samo w kilku innych miejscach.

if not FollowRequest.objects.filter(receiver=user).filter(sender=request.user).first():
->
if not FollowRequest.objects.filter(receiver=user, sender=request.user).exists():

Tutaj:

def has_paid(self, current_date=datetime.date.today()):

To `current_date=datetime.date.today()` zrobi raczej co innego, niż myślisz :) Przeczytaj jak działają domyślne argumenty w Pythonie.

A, no i dodaj .gitignore do repo - takie rzeczy jak db.sqlite3, __pycache__ czy videos/ raczej nie powinny być w repo.

Comments

A jeśli chodzi o samą komunikację z bazą danych niema jakichś większych zastrzeże? Bo niekiedy wydaje mi się, że robię nadmierne zapytania lub są one nieco nieoptymalne. Tak czy siak wielkie dzięki za review.

Answer 2

Code review się nie podejmę (DRF nie znam wcale), ale kod wygląda na sensownie napisany. Jakbyś zrobił front i wystawił demo, to łatwiej byłoby przetestować tą appkę.