diff --git a/cinema/models.py b/cinema/models.py index cc477513f..cf4118218 100644 --- a/cinema/models.py +++ b/cinema/models.py @@ -1,9 +1,35 @@ from django.db import models +class Actor(models.Model): + first_name = models.CharField(max_length=63) + last_name = models.CharField(max_length=63) + + def __str__(self) -> str: + return f"{self.first_name} {self.last_name}" + + +class Genre(models.Model): + name = models.CharField(max_length=63, unique=True) + + def __str__(self) -> str: + return self.name + + +class CinemaHall(models.Model): + name = models.CharField(max_length=63) + rows = models.IntegerField() + seats_in_row = models.IntegerField() + + def __str__(self) -> str: + return f"{self.name} ({self.rows * self.seats_in_row} places)" + + class Movie(models.Model): title = models.CharField(max_length=255) description = models.TextField() + actors = models.ManyToManyField(Actor, related_name="movies") + genres = models.ManyToManyField(Genre, related_name="movies") duration = models.IntegerField() def __str__(self): diff --git a/cinema/serializers.py b/cinema/serializers.py index 050db5771..18b6e9c83 100644 --- a/cinema/serializers.py +++ b/cinema/serializers.py @@ -1,6 +1,6 @@ from rest_framework import serializers -from cinema.models import Movie +from cinema.models import Movie, Actor, Genre, CinemaHall class MovieSerializer(serializers.Serializer): @@ -22,3 +22,58 @@ def update(self, instance, validated_data): instance.save() return instance + + +class ActorSerializer(serializers.Serializer): + id = serializers.IntegerField(read_only=True) + first_name = serializers.CharField(max_length=63) + last_name = serializers.CharField(max_length=63) + + def create(self, validated_data): + return Actor.objects.create(**validated_data) + + def update(self, instance, validated_data): + instance.first_name = validated_data.get( + "first_name", instance.first_name + ) + instance.last_name = validated_data.get( + "last_name", instance.last_name + ) + + instance.save() + return instance + + +class GenreSerializer(serializers.Serializer): + id = serializers.IntegerField(read_only=True) + name = serializers.CharField(max_length=63) + + def create(self, validated_data): + return Genre.objects.create(**validated_data) + + def update(self, instance, validated_data): + instance.name = validated_data.get("name", instance.name) + + instance.save() + return instance + + +class CinemaHallSerializer(serializers.Serializer): + id = serializers.IntegerField(read_only=True) + name = serializers.CharField(max_length=63) + rows = serializers.IntegerField() + seats_in_row = serializers.IntegerField() + + def create(self, validated_data): + return CinemaHall.objects.create(**validated_data) + + def update(self, instance, validated_data): + instance.name = validated_data.get("name", instance.name) + instance.rows = validated_data.get("rows", instance.rows) + instance.seats_in_row = validated_data.get( + "seats_in_row", instance.seats_in_row + ) + + instance.save() + return instance + diff --git a/cinema/urls.py b/cinema/urls.py index 1ae7d5cb0..bbf258ffc 100644 --- a/cinema/urls.py +++ b/cinema/urls.py @@ -1,10 +1,32 @@ -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 (GenreList, + GenreDetail, + ActorList, + ActorDetail, + CinemaHallViewSet, + MovieModelViewSet) + + +cinema_hall_list = CinemaHallViewSet.as_view(actions={"get": "list", + "post": "create"}) +cinema_hall_detail = CinemaHallViewSet.as_view(actions={"get": "retrieve", + "put": "update", + "patch": "partial_update", + "delete": "destroy"}) + +router = routers.DefaultRouter() +router.register("movies", MovieModelViewSet) urlpatterns = [ - path("movies/", movie_list, name="movie-list"), - path("movies//", movie_detail, name="movie-detail"), + path("genres/", GenreList.as_view(), name="genre-list"), + path("genres//", GenreDetail.as_view(), name="genre-detail"), + path("actors/", ActorList.as_view(), name="actor-list"), + path("actors//", ActorDetail.as_view(), name="actor-detail"), + path("cinemahalls/", cinema_hall_list, name="cinemahall-list"), + path("cinemahalls//", cinema_hall_detail, name="cinemahall-detail"), + path("", include(router.urls)) ] app_name = "cinema" diff --git a/cinema/views.py b/cinema/views.py index 78ba8a79c..35508b053 100644 --- a/cinema/views.py +++ b/cinema/views.py @@ -1,22 +1,29 @@ -from rest_framework.decorators import api_view -from rest_framework.response import Response from rest_framework import status -from django.shortcuts import get_object_or_404 - -from cinema.models import Movie -from cinema.serializers import MovieSerializer +from rest_framework.mixins import (ListModelMixin, + CreateModelMixin, + RetrieveModelMixin, + UpdateModelMixin, + DestroyModelMixin) +from rest_framework.response import Response +from rest_framework.viewsets import GenericViewSet, ModelViewSet +from rest_framework.generics import GenericAPIView +from rest_framework.views import APIView +from cinema.serializers import (MovieSerializer, + GenreSerializer, + ActorSerializer, + CinemaHallSerializer) +from cinema.models import Movie, Genre, Actor, CinemaHall -@api_view(["GET", "POST"]) -def movie_list(request): - if request.method == "GET": - movies = Movie.objects.all() - serializer = MovieSerializer(movies, many=True) +class GenreList(APIView): + def get(self, request): + movies = Genre.objects.all() + serializer = GenreSerializer(movies, many=True) return Response(serializer.data, status=status.HTTP_200_OK) - 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) @@ -24,22 +31,84 @@ def movie_list(request): return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST) -@api_view(["GET", "PUT", "DELETE"]) -def movie_detail(request, pk): - movie = get_object_or_404(Movie, pk=pk) +class GenreDetail(APIView): + def get_genre(self, pk): + try: + return Genre.objects.get(pk=pk) + except Genre.DoesNotExist: + return Response(status=status.HTTP_404_NOT_FOUND) - if request.method == "GET": - serializer = MovieSerializer(movie) + def get(self, request, pk: int): + serializer = GenreSerializer(self.get_genre(pk)) return Response(serializer.data, status=status.HTTP_200_OK) - if request.method == "PUT": - serializer = MovieSerializer(movie, data=request.data) + def put(self, request, pk: int): + serializer = GenreSerializer(self.get_genre(pk), data=request.data) + if serializer.is_valid(): + serializer.save() + return Response(serializer.data, status=status.HTTP_200_OK) + + return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST) + + def patch(self, request, pk: int): + serializer = GenreSerializer(self.get_genre(pk), data=request.data, + partial=True) if serializer.is_valid(): serializer.save() return Response(serializer.data, status=status.HTTP_200_OK) return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST) - if request.method == "DELETE": - movie.delete() + def delete(self, request, pk: int): + self.get_genre(pk).delete() return Response(status=status.HTTP_204_NO_CONTENT) + + +class ActorList(ListModelMixin, CreateModelMixin, 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 ActorDetail( + RetrieveModelMixin, + UpdateModelMixin, + 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 CinemaHallViewSet( + ListModelMixin, + CreateModelMixin, + RetrieveModelMixin, + UpdateModelMixin, + DestroyModelMixin, + GenericViewSet +): + queryset = CinemaHall.objects.all() + serializer_class = CinemaHallSerializer + + +class MovieModelViewSet(ModelViewSet): + queryset = Movie.objects.all() + serializer_class = MovieSerializer diff --git a/requirements.txt b/requirements.txt index 99938e15f..84d6438bd 100644 --- a/requirements.txt +++ b/requirements.txt @@ -1,7 +1,7 @@ django==4.1 -flake8==5.0.4 -flake8-quotes==3.3.1 -flake8-variables-names==0.0.5 -pep8-naming==0.13.2 -django-debug-toolbar==3.2.4 -djangorestframework==3.13.1 +flake8 +flake8-quotes +flake8-variables-names +pep8-naming +django-debug-toolbar +djangorestframework