From 32f0f60d51186791d8e9db66606d1d3e57ad6d21 Mon Sep 17 00:00:00 2001 From: Sofiia Zholud Date: Fri, 1 Nov 2024 12:50:08 +0200 Subject: [PATCH] Solution --- .gitignore | 12 ++- ...emahall_genre_movie_actors_movie_genres.py | 47 +++++++++ cinema/models.py | 41 ++++++++ cinema/serializers.py | 50 ++++++---- cinema/urls.py | 42 ++++++-- cinema/views.py | 95 ++++++++++++------- 6 files changed, 232 insertions(+), 55 deletions(-) create mode 100644 cinema/migrations/0002_actor_cinemahall_genre_movie_actors_movie_genres.py diff --git a/.gitignore b/.gitignore index 9c18d6ac5..307dd2c3b 100644 --- a/.gitignore +++ b/.gitignore @@ -7,4 +7,14 @@ venv/ .pytest_cache/ **__pycache__/ *.pyc -db.sqlite3 \ No newline at end of file +db.sqlite3 +*.log +env/ +*.pyo +*.pyd +.env +celerybeat-schedule +htmlcov/ +.tox/ +.cache/ +staticfiles/ diff --git a/cinema/migrations/0002_actor_cinemahall_genre_movie_actors_movie_genres.py b/cinema/migrations/0002_actor_cinemahall_genre_movie_actors_movie_genres.py new file mode 100644 index 000000000..edb868b60 --- /dev/null +++ b/cinema/migrations/0002_actor_cinemahall_genre_movie_actors_movie_genres.py @@ -0,0 +1,47 @@ +# Generated by Django 4.1 on 2024-11-01 09:33 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('cinema', '0001_initial'), + ] + + operations = [ + migrations.CreateModel( + name='Actor', + fields=[ + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('first_name', models.CharField(max_length=255)), + ('last_name', models.CharField(max_length=255)), + ], + ), + migrations.CreateModel( + name='CinemaHall', + fields=[ + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('name', models.CharField(max_length=255)), + ('rows', models.IntegerField()), + ('seats_in_row', models.IntegerField()), + ], + ), + migrations.CreateModel( + name='Genre', + fields=[ + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('name', models.CharField(max_length=255, unique=True)), + ], + ), + migrations.AddField( + model_name='movie', + name='actors', + field=models.ManyToManyField(related_name='movies', to='cinema.actor'), + ), + migrations.AddField( + model_name='movie', + name='genres', + field=models.ManyToManyField(related_name='movies', to='cinema.genre'), + ), + ] diff --git a/cinema/models.py b/cinema/models.py index cc477513f..066850b36 100644 --- a/cinema/models.py +++ b/cinema/models.py @@ -1,10 +1,51 @@ from django.db import models +class Genre(models.Model): + name = models.CharField( + max_length=255, + unique=True + ) + + def __str__(self): + return self.name + + +class Actor(models.Model): + first_name = models.CharField( + max_length=255 + ) + last_name = models.CharField(max_length=255) + + def __str__(self): + return f"{self.first_name} {self.last_name}" + + class Movie(models.Model): title = models.CharField(max_length=255) description = models.TextField() + actors = models.ManyToManyField( + to=Actor, + related_name="movies" + ) + genres = models.ManyToManyField( + to=Genre, + related_name="movies" + ) duration = models.IntegerField() def __str__(self): return self.title + + +class CinemaHall(models.Model): + name = models.CharField(max_length=255) + rows = models.IntegerField() + seats_in_row = models.IntegerField() + + @property + def capacity(self) -> int: + return self.rows * self.seats_in_row + + def __str__(self) -> str: + return self.name diff --git a/cinema/serializers.py b/cinema/serializers.py index 050db5771..a11598242 100644 --- a/cinema/serializers.py +++ b/cinema/serializers.py @@ -1,24 +1,42 @@ from rest_framework import serializers +from cinema.models import Genre, Actor, CinemaHall, Movie -from cinema.models import Movie +class GenreSerializer(serializers.ModelSerializer): + class Meta: + model = Genre + fields = ["id", "name"] -class MovieSerializer(serializers.Serializer): - id = serializers.IntegerField(read_only=True) - title = serializers.CharField(max_length=255) - description = serializers.CharField() - duration = serializers.IntegerField() - def create(self, validated_data): - return Movie.objects.create(**validated_data) +class ActorSerializer(serializers.ModelSerializer): + class Meta: + model = Actor + fields = ["id", "first_name", "last_name"] - def update(self, instance, validated_data): - instance.title = validated_data.get("title", instance.title) - instance.description = validated_data.get( - "description", instance.description - ) - instance.duration = validated_data.get("duration", instance.duration) - instance.save() +class CinemaHallSerializer(serializers.ModelSerializer): + class Meta: + model = CinemaHall + fields = ["id", "name", "rows", "seats_in_row"] - return instance + +class MovieSerializer(serializers.ModelSerializer): + actors = serializers.PrimaryKeyRelatedField( + queryset=Actor.objects.all(), + many=True + ) + genres = serializers.PrimaryKeyRelatedField( + queryset=Genre.objects.all(), + many=True + ) + + class Meta: + model = Movie + fields = [ + "id", + "title", + "description", + "duration", + "actors", + "genres" + ] diff --git a/cinema/urls.py b/cinema/urls.py index 1ae7d5cb0..70113e9be 100644 --- a/cinema/urls.py +++ b/cinema/urls.py @@ -1,10 +1,40 @@ -from django.urls import path +from django.urls import path, include +from rest_framework.routers import DefaultRouter +from cinema.views import ( + GenreList, + GenreDetail, + ActorList, + ActorDetail, + CinemaHallViewSet, + MovieViewSet +) -from cinema.views import movie_list, movie_detail +app_name = "cinema" + +router = DefaultRouter() +router.register("cinema_halls", CinemaHallViewSet) +router.register("movies", MovieViewSet) urlpatterns = [ - path("movies/", movie_list, name="movie-list"), - path("movies//", movie_detail, name="movie-detail"), + path("", include(router.urls)), + path( + "genres/", + GenreList.as_view(), + name="genre-list-create" + ), + path( + "genres//", + GenreDetail.as_view(), + name="genre-detail" + ), + path( + "actors/", + ActorList.as_view(), + name="actor-list-create" + ), + path( + "actors//", + ActorDetail.as_view(), + name="actor-detail" + ), ] - -app_name = "cinema" diff --git a/cinema/views.py b/cinema/views.py index 78ba8a79c..ce7e14fa4 100644 --- a/cinema/views.py +++ b/cinema/views.py @@ -1,45 +1,76 @@ -from rest_framework.decorators import api_view +from rest_framework import viewsets, mixins, status, generics +from rest_framework.views import APIView from rest_framework.response import Response -from rest_framework import status +from cinema.models import Genre, Actor, CinemaHall, Movie +from cinema.serializers import ( + GenreSerializer, + ActorSerializer, + CinemaHallSerializer, + MovieSerializer +) -from django.shortcuts import get_object_or_404 -from cinema.models import Movie -from cinema.serializers import MovieSerializer +class GenreList(APIView): + def get(self, request): + genres = Genre.objects.all() + serializer = GenreSerializer(genres, many=True) + return Response(serializer.data) + def post(self, request): + serializer = GenreSerializer(data=request.data) + serializer.is_valid(raise_exception=True) + serializer.save() + return Response(serializer.data, status=status.HTTP_201_CREATED) -@api_view(["GET", "POST"]) -def movie_list(request): - if request.method == "GET": - movies = Movie.objects.all() - serializer = MovieSerializer(movies, many=True) - return Response(serializer.data, status=status.HTTP_200_OK) - if request.method == "POST": - serializer = MovieSerializer(data=request.data) - if serializer.is_valid(): - serializer.save() - return Response(serializer.data, status=status.HTTP_201_CREATED) +class GenreDetail(APIView): + def get(self, request, pk): + genre = generics.get_object_or_404(Genre, pk=pk) + serializer = GenreSerializer(genre) + return Response(serializer.data) - return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST) + def put(self, request, pk): + genre = generics.get_object_or_404(Genre, pk=pk) + serializer = GenreSerializer(genre, data=request.data) + serializer.is_valid(raise_exception=True) + serializer.save() + return Response(serializer.data) + def patch(self, request, pk): + genre = generics.get_object_or_404(Genre, pk=pk) + serializer = GenreSerializer(genre, data=request.data, partial=True) + serializer.is_valid(raise_exception=True) + serializer.save() + return Response(serializer.data) -@api_view(["GET", "PUT", "DELETE"]) -def movie_detail(request, pk): - movie = get_object_or_404(Movie, pk=pk) + def delete(self, request, pk): + genre = generics.get_object_or_404(Genre, pk=pk) + genre.delete() + return Response(status=status.HTTP_204_NO_CONTENT) - if request.method == "GET": - serializer = MovieSerializer(movie) - return Response(serializer.data, status=status.HTTP_200_OK) - if request.method == "PUT": - serializer = MovieSerializer(movie, data=request.data) - if serializer.is_valid(): - serializer.save() - return Response(serializer.data, status=status.HTTP_200_OK) +class ActorList(generics.ListCreateAPIView): + queryset = Actor.objects.all() + serializer_class = ActorSerializer - return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST) - if request.method == "DELETE": - movie.delete() - return Response(status=status.HTTP_204_NO_CONTENT) +class ActorDetail(generics.RetrieveUpdateDestroyAPIView): + queryset = Actor.objects.all() + serializer_class = ActorSerializer + + +class CinemaHallViewSet( + mixins.ListModelMixin, + mixins.CreateModelMixin, + mixins.RetrieveModelMixin, + mixins.UpdateModelMixin, + mixins.DestroyModelMixin, + viewsets.GenericViewSet +): + queryset = CinemaHall.objects.all() + serializer_class = CinemaHallSerializer + + +class MovieViewSet(viewsets.ModelViewSet): + queryset = Movie.objects.all() + serializer_class = MovieSerializer