andisaev / async_api_sprint_1 Goto Github PK

Хорошая работа! Ваш код структурирован и разбит на адекватные по размеру методы. Ниже несколько замечаний:

1. Стандартная практика делить импорты на три группы: "системные", внешние пакеты, модули проекта. Чтобы с этим проще жить предлагаю вам использовать isort. Если вы вдруг ее уже используете, то правильно настройте параметр https://pycqa.github.io/isort/docs/configuration/options.html#known-first-party
2. По поводу логирования в docker-контейнере. Максимально не рекомендую писать данные в файл. Мотивация: если вы не пробросили volume, то вы будете писать во временный слой, который потеряете при удалении контейнера и более важная причина - вы будете вынуждены самостоятельно заниматься ротацией логов (иначе контейнер потенциально может скушать весь ваш диск). А докер уже из коробки по-умолчанию предоставляет вам инструмент, который забирает весь stdout и stderr и позволяет с ним работать, подробнее тут: https://docs.docker.com/config/containers/logging/. А в docker-compose есть дополнительная возможность управлять размером и ротацией логов: https://docs.docker.com/compose/compose-file/compose-file-v3/#logging
3. Под main желательно оставить только вызов save_elastic иначе вы теряете гибкость в плане тестирования, а также возможность импорта этих функций
4. Пустый файлы из вашего шаблона лучше удалить, чтобы они не захламляли проект
5. Вы на версии python 3.9. А с этой версии для стандартных коллекций больше не нужен typing: https://docs.python.org/3.9/whatsnew/3.9.html#type-hinting-generics-in-standard-collections. Поправьте во всем проекте
6. Тут вы немного перемудрили с сортировкой..Представьте, что вам нужно сортировать по всем полям модели, в итоге вы получите очень сложную структуру. Это можно максимально упростить, если сделать строковое поле sort. Направление сортировки обозначить через минус. И парсить - эту сортировку как -field - по убыванию, field - по возрастанию. А само поле получать как sort.removeprefix('-') (новый стороковый метод в 3.9)
7. У вас тут query является обязательным параметром. А вам нужно реализовать две вещи: поиск и просто отдачу списка элементов без фильтрации. Поэтому query надо тоже сделать опциональным
8. Можно лучше: main.py обычно все-таки размещают в корне проекта, чтобы избежать различные проблемы с циклическим импортом в будущем
9. Тут опечатка в имени файла видимо
10. Кеширование в redis необходимо так же реализовать для списка объектов. Так же посмотрите на свои сервисы, некоторую часть кода можно вынести в базовый класс и использовать во всех сервисах
11. Не очень хорошая практика оборачивать такую большую часть кода в try/except. Ошибка, которую вы обрабатываете может произойти только в self.elastic.search