From 166282ceeeae1971880a7465a6062640ad55d5bc Mon Sep 17 00:00:00 2001 From: ORENYT <106102798+ORENYT@users.noreply.github.com> Date: Wed, 11 Oct 2023 19:53:19 +0300 Subject: [PATCH] Implemented Solution,TODO fix bugs --- cinema/migrations/0001_initial.py | 53 ++++++++++++++- cinema/models.py | 17 +++++ cinema/serializers.py | 38 ++++++----- cinema/urls.py | 38 +++++++++-- cinema/views.py | 105 +++++++++++++++++++++++------- 5 files changed, 205 insertions(+), 46 deletions(-) diff --git a/cinema/migrations/0001_initial.py b/cinema/migrations/0001_initial.py index 514947a64..08101153d 100644 --- a/cinema/migrations/0001_initial.py +++ b/cinema/migrations/0001_initial.py @@ -1,15 +1,62 @@ -# Generated by Django 4.0.2 on 2022-05-03 13:41 +# Generated by Django 4.1 on 2023-10-11 16:44 from django.db import migrations, models class Migration(migrations.Migration): - initial = True dependencies = [] 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.CreateModel( name="Movie", fields=[ @@ -25,6 +72,8 @@ class Migration(migrations.Migration): ("title", models.CharField(max_length=255)), ("description", models.TextField()), ("duration", models.IntegerField()), + ("actors", models.ManyToManyField(to="cinema.actor")), + ("genres", models.ManyToManyField(to="cinema.genre")), ], ), ] diff --git a/cinema/models.py b/cinema/models.py index cc477513f..1437f88f3 100644 --- a/cinema/models.py +++ b/cinema/models.py @@ -5,6 +5,23 @@ class Movie(models.Model): title = models.CharField(max_length=255) description = models.TextField() duration = models.IntegerField() + actors = models.ManyToManyField("Actor") + genres = models.ManyToManyField("Genre") def __str__(self): return self.title + + +class Actor(models.Model): + first_name = models.CharField(max_length=255) + last_name = models.CharField(max_length=255) + + +class Genre(models.Model): + name = models.CharField(max_length=255, unique=True) + + +class CinemaHall(models.Model): + name = models.CharField(max_length=255) + rows = models.IntegerField() + seats_in_row = models.IntegerField() diff --git a/cinema/serializers.py b/cinema/serializers.py index 050db5771..51ad1e11b 100644 --- a/cinema/serializers.py +++ b/cinema/serializers.py @@ -1,24 +1,30 @@ from rest_framework import serializers -from cinema.models import Movie +from cinema.models import Movie, Actor, Genre, CinemaHall -class MovieSerializer(serializers.Serializer): - id = serializers.IntegerField(read_only=True) - title = serializers.CharField(max_length=255) - description = serializers.CharField() - duration = serializers.IntegerField() +class ActorSerializer(serializers.ModelSerializer): + class Meta: + model = Actor + fields = "__all__" - def create(self, validated_data): - return Movie.objects.create(**validated_data) - 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) +class GenreSerializer(serializers.ModelSerializer): + class Meta: + model = Genre + fields = "_all__" - instance.save() - return instance +class CinemaHallSerializer(serializers.ModelSerializer): + class Meta: + model = CinemaHall + fields = "__all__" + + +class MovieSerializer(serializers.ModelSerializer): + actors = ActorSerializer(many=True) + genres = GenreSerializer(many=True) + + class Meta: + model = Movie + fields = "__any__" diff --git a/cinema/urls.py b/cinema/urls.py index 1ae7d5cb0..28f205c53 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 import routers -from cinema.views import movie_list, movie_detail +from cinema.views import ( + CinemaHallView, + MovieView, + GenreView, + ActorListView, + ActorDetailView, +) + +halls_list = CinemaHallView.as_view( + actions={ + "get": "list", + "post": "create", + } +) + +halls_detail = CinemaHallView.as_view( + actions={ + "get": "retrieve", + "post": "update", + "patch": "partial_update", + "delete": "destroy", + } +) +router = routers.DefaultRouter() +router.register("movies", MovieView) urlpatterns = [ - path("movies/", movie_list, name="movie-list"), - path("movies//", movie_detail, name="movie-detail"), + path("", include(router.urls)), + path("genres/", GenreView.as_view(), name="genre-list"), + path("genres//", GenreView.as_view(), name="genre-detail"), + path("actors/", ActorListView.as_view(), name="actor-list"), + path("actors//", ActorDetailView.as_view(), name="actor-detail"), + path("halls/", halls_list, name="halls-list"), + path("halls//", halls_detail, name="halls-detail"), ] app_name = "cinema" diff --git a/cinema/views.py b/cinema/views.py index 78ba8a79c..f114089c2 100644 --- a/cinema/views.py +++ b/cinema/views.py @@ -1,45 +1,102 @@ -from rest_framework.decorators import api_view +from rest_framework.generics import GenericAPIView from rest_framework.response import Response -from rest_framework import status +from rest_framework import status, mixins, viewsets from django.shortcuts import get_object_or_404 +from rest_framework.views import APIView +from rest_framework.viewsets import ModelViewSet -from cinema.models import Movie -from cinema.serializers import MovieSerializer +from cinema.models import Movie, Genre, Actor, CinemaHall +from cinema.serializers import ( + MovieSerializer, + GenreSerializer, + ActorSerializer, + CinemaHallSerializer, +) -@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) +class GenreView(APIView): + def get(self, request, pk): + if pk is not None: + genres = Genre.objects.all() + serializer = GenreSerializer(genres) + else: + genre = get_object_or_404(Genre, pk) + serializer = GenreSerializer(genre) + return Response(serializer.data) - if request.method == "POST": - serializer = MovieSerializer(data=request.data) + def post(self, request): + serializer = GenreSerializer(data=request.data) if serializer.is_valid(): serializer.save() return Response(serializer.data, status=status.HTTP_201_CREATED) return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST) + def delete(self, request, pk): + genre = get_object_or_404(Genre, pk) + genre.delete() + return Response(status=status.HTTP_204_NO_CONTENT) -@api_view(["GET", "PUT", "DELETE"]) -def movie_detail(request, pk): - movie = get_object_or_404(Movie, pk=pk) + def put(self, request, pk): + genre = get_object_or_404(Genre, pk) + serializer = GenreSerializer(genre, data=request.data) + if serializer.is_valid(): + serializer.save() + return Response(serializer.data, status=status.HTTP_201_CREATED) - if request.method == "GET": - serializer = MovieSerializer(movie) - return Response(serializer.data, status=status.HTTP_200_OK) + return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST) - if request.method == "PUT": - serializer = MovieSerializer(movie, data=request.data) + def patch(self, request, pk): + genre = get_object_or_404(Genre, pk) + serializer = GenreSerializer(genre, data=request.data, partial=True) if serializer.is_valid(): serializer.save() - return Response(serializer.data, status=status.HTTP_200_OK) + return Response(serializer.data, status=status.HTTP_201_CREATED) 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 ActorListView( + mixins.CreateModelMixin, mixins.ListModelMixin, GenericAPIView +): + queryset = Actor.objects.all() + serializer_class = ActorSerializer + + def get(self, request, *args, **kwargs): + return self.list(request, *args, **kwargs) + + def post(self, request, *args, **kwargs): + return self.create(request, *args, **kwargs) + + +class ActorDetailView( + mixins.RetrieveModelMixin, + mixins.UpdateModelMixin, + mixins.DestroyModelMixin, + GenericAPIView, +): + queryset = Actor.objects.all() + serializer_class = ActorSerializer + + def get(self, request, *args, **kwargs): + return self.retrieve(request, *args, **kwargs) + + def put(self, request, *args, **kwargs): + return self.update(request, *args, **kwargs) + + def patch(self, request, *args, **kwargs): + return self.partial_update(request, *args, **kwargs) + + def delete(self, request, *args, **kwargs): + return self.destroy(request, *args, **kwargs) + + +class CinemaHallView(viewsets.GenericViewSet): + queryset = CinemaHall.objects.all() + serializer_class = CinemaHallSerializer + + +class MovieView(ModelViewSet): + queryset = Movie.objects.all() + serializer_class = MovieSerializer