tc_origins_stats_microservice's People
tc_origins_stats_microservice's Issues
Review
https://github.com/ivangermes/testovoe_mongo/blob/main/source/service.py#L9
PEP8 - комментарий в конце должен быть отделен двумя символами пробела перед символом #
и одним после:
# from datetime import datetime # python3.11 isoparse
https://github.com/ivangermes/testovoe_mongo/blob/main/source/service.py#L16
LOG_LEVEL
определен, но реально не используется. Это вводит в заблуждение, нарушение общепринятых правил в программировании.
https://github.com/ivangermes/testovoe_mongo/blob/main/source/service.py#L47
Это не информативный способ ответа - http клиент не получит никакой внятной информации о произошедшей ошибке, в логах тоже ничего не будет. Правильным будет либо явное указание ошибки (а в этом месте она точно известная) либо передача хотя бы служебной информации (если такое позволительно в проекте):
try:
time_from = isoparse(time_from_str)
except ValueError:
raise HTTPBadRequest(f"Parameter 'time_from' has incorrect format: '{time_from_str}'")
try:
time_from = isoparse(time_from_str)
except ValueError as exc:
raise HTTPBadRequest(exc)
https://github.com/ivangermes/testovoe_mongo/blob/main/source/service.py#L57
нету проверки на тот случай если заданы оба параметра и time_from > time_to
. В таких случаях запрос вернет пустой результат, однако на деле это должно сигнализировать об ошибке входных данных.
https://github.com/ivangermes/testovoe_mongo/blob/main/source/service.py#L59
мне не ясно что такое SON. Если я предположу что это BSON, то данный TODO не имеет, на мой взгляд, практической ценности, поскольку итак будет происходить неявное преобразование этого словаря в BSON и без этого код будет компактнее и практичнее.
https://github.com/ivangermes/testovoe_mongo/blob/main/source/service.py#L93
Общепринятое использовать в таких случаях null/None, наличие любого другого статического значения предполагается быть осмысленным (необходимым для бизнес-логики). Таким образом повышается читаемость кода, так как фокус на этом не теряется.
https://github.com/ivangermes/testovoe_mongo/blob/main/source/service.py#L132
В отличии от других БД в монга работает со своими единицами хранилищ (коллекциями в данном случае) таким образом что нету необходимости в проверке или явном создании их. То есть если коллекции не будет. то данный проект все равно будет работать, а результат будет аналогичному если коллекция будет существовать пустой. Тоже самое и с добавлением документов - если коллекции в этот момент не будет, то она будет создана.
Поэтому данный кусок кода в практическом смысле скорее не нужен. Исключения могут быть, например если есть жесткой условие что коллекций должна быть, а ее отсутствие сигнал о проблеме и не нужно запускать сервис.
В целом хорошо.
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.