Код ревью by samsonovroma2011-cmd · Pull Request #1 · samsonovroma2011-cmd/code-review-task

samsonovroma2011-cmd · 2026-08-06T07:05:01Z

Я проведу код ревью проекта https://github.com/theshohidul/FastAPI-Boilerplate

samsonovroma2011-cmd · 2026-08-06T07:35:59Z

+from sqlalchemy.orm import as_declarative
+
+
+@as_declarative()


не знакомый декоратор

Советую изучить, что он делает и для чего

samsonovroma2011-cmd · 2026-08-06T07:40:40Z

+def init_handlers(app_: FastAPI) -> None:
+
+    @app_.exception_handler(CustomException)
+    async def custom_exception_handler(request: Request, exc: CustomException):
+        return JSONResponse(
+            status_code=exc.status_code,
+            content={
+                "status": exc.status,
+                "status_type": exc.status_type,
+                "message": exc.message
+            },
+        )
+
+    @app_.exception_handler(RequestValidationError)
+    async def validation_exception_handler(request: Request, exc: RequestValidationError):
+        reformatted_message = defaultdict(list)
+        field_string = ''
+        msg = ''
+
+        for pydantic_error in exc.errors():
+            loc, msg, error_type = pydantic_error["loc"], pydantic_error["msg"], pydantic_error["type"]
+
+            if error_type == "value_error.jsondecode":
+                return await custom_exception_handler(request=request, exc=BadRequestException())
+
+            filtered_loc = loc[1:] if loc[0] in ("body", "query", "path") and loc[1:] else loc
+            field_string = ".".join(filtered_loc)
+            reformatted_message[field_string].append(msg)
+
+        return JSONResponse(
+            status_code=UnprocessableEntity().status_code,
+            content={
+                "status": UnprocessableEntity.status,
+                "status_type": UnprocessableEntity.status_type,
+                "message": field_string.capitalize() + ' ' + msg,
+                "errors": reformatted_message
+            }
+        )


Слишком нагруженная функция

Как бы сделал по другому. Интересно решение

samsonovroma2011-cmd · 2026-08-06T07:43:14Z

+    def __getattr__(self, name):
+        return getattr(self._session, name)
+
+    async def init(self):


Хорошо что используется асинхронность

samsonovroma2011-cmd · 2026-08-06T07:44:47Z

+loguru = "^0.6.0"
+cryptography = "^38.0.1"
+bcrypt = "^4.0.0"
+sqlalchemy = "^1.4.46"


Используется старая версия sqlalchemy

samsonovroma2011-cmd

Мне показалось что в целом у проекта хорошая архитектура. Могу выделить в ней несколько плюсов: используется паттерн репозиторий, есть сервисный слой и виден очень сильный ООП подход.

Я понял этот проект на 50-60%.

samsonovroma2011-cmd · 2026-08-06T08:02:18Z

+class Settings(BaseSettings):
+    ENV: str = "dev"
+    PROJECT_NAME: str = "Boilerplate"
+    PROJECT_TITLE: str = "FastAPI Boilerplate"
+    PROJECT_DESCRIPTION: str = "FastAPI Boilerplate"
+    PROJECT_VERSION: str = "1.0.0"
+
+    ENCODING: str = 'utf-8'
+
+    API_V1_PREFIX: str = "/api/v1"
+    JWT_ALGORITHM: str = "HS256"
+    JWT_ACCESS_SECRET_KEY: str = "MySuperSecret"
+    JWT_REFRESH_SECRET_KEY: str = "MySuperSecretRefresh"


Секретные данные должны быть в в файле .env и от туда их должен брать класс с настройками

Да. Тут и нету секретных данных. Это шаблоны заглушки

samsonovroma2011-cmd

Мне показалось что в целом у проекта хорошая архитектура. Могу выделить в ней несколько плюсов: используется паттерн репозиторий, есть сервисный слой и виден очень сильный ООП подход.

Я понял этот проект на 50-60%.

kykyrzik · 2026-08-07T13:38:11Z

+
+from sqlalchemy import Column, Integer, String, ForeignKey, Table, PrimaryKeyConstraint
+from sqlalchemy.orm import declarative_base, relationship
+
+from core.db.base import Base
+
+
+user_roles_table = Table(
+    "user_roles",
+    Base.metadata,
+    Column("user_id", ForeignKey("users.id")),
+    Column("role_id", ForeignKey("roles.id")),
+    PrimaryKeyConstraint('user_id', 'role_id'),
+)
+
+
+class RoleModel(Base):
+    def __init__(self):
+        pass
+
+    __tablename__ = "roles"
+
+    id: int = Column(Integer, primary_key=True, autoincrement=True)
+    role: str = Column(String(255), nullable=False)
+    __mapper_args__ = {"eager_defaults": True}
+
+    users = relationship(
+        "UserModel", secondary=user_roles_table, back_populates="roles"
+    )
+
+
+class UserModel(Base):
+    def __init__(self):
+        pass
+
+    __tablename__ = "users"
+
+    id: int = Column(Integer, primary_key=True, autoincrement=True)
+    name: str = Column(String(255), nullable=False)
+    email: str = Column(String(255), nullable=False, unique=True)
+    password: str = Column(String(255), nullable=False)
+    phone: str = Column(String(255), nullable=False)
+    __mapper_args__ = {"eager_defaults": True}
+
+    roles: List[RoleModel] = relationship(
+        "RoleModel",
+        secondary=user_roles_table,
+        back_populates="users",
+    )


Смешивание подходов создание моделей. Декларативный и императивный подходы смешаны

kykyrzik · 2026-08-07T13:39:17Z

+        await self.db.rollback()
+
+        user_dict = user.dict()
+        role_name = user.role
+        del user_dict["role"]


Для чего rollback до выполнение операции? Почему del? Советую изучить

kykyrzik · 2026-08-07T13:39:58Z

+def init_handlers(app_: FastAPI) -> None:
+
+    @app_.exception_handler(CustomException)
+    async def custom_exception_handler(request: Request, exc: CustomException):
+        return JSONResponse(
+            status_code=exc.status_code,
+            content={
+                "status": exc.status,
+                "status_type": exc.status_type,
+                "message": exc.message
+            },
+        )
+
+    @app_.exception_handler(RequestValidationError)
+    async def validation_exception_handler(request: Request, exc: RequestValidationError):
+        reformatted_message = defaultdict(list)
+        field_string = ''
+        msg = ''
+
+        for pydantic_error in exc.errors():
+            loc, msg, error_type = pydantic_error["loc"], pydantic_error["msg"], pydantic_error["type"]
+
+            if error_type == "value_error.jsondecode":
+                return await custom_exception_handler(request=request, exc=BadRequestException())
+
+            filtered_loc = loc[1:] if loc[0] in ("body", "query", "path") and loc[1:] else loc
+            field_string = ".".join(filtered_loc)
+            reformatted_message[field_string].append(msg)
+
+        return JSONResponse(
+            status_code=UnprocessableEntity().status_code,
+            content={
+                "status": UnprocessableEntity.status,
+                "status_type": UnprocessableEntity.status_type,
+                "message": field_string.capitalize() + ' ' + msg,
+                "errors": reformatted_message
+            }
+        )


Как бы сделал по другому. Интересно решение

kykyrzik · 2026-08-07T13:40:39Z

+class Settings(BaseSettings):
+    ENV: str = "dev"
+    PROJECT_NAME: str = "Boilerplate"
+    PROJECT_TITLE: str = "FastAPI Boilerplate"
+    PROJECT_DESCRIPTION: str = "FastAPI Boilerplate"
+    PROJECT_VERSION: str = "1.0.0"
+
+    ENCODING: str = 'utf-8'
+
+    API_V1_PREFIX: str = "/api/v1"
+    JWT_ALGORITHM: str = "HS256"
+    JWT_ACCESS_SECRET_KEY: str = "MySuperSecret"
+    JWT_REFRESH_SECRET_KEY: str = "MySuperSecretRefresh"


Да. Тут и нету секретных данных. Это шаблоны заглушки

kykyrzik · 2026-08-07T13:40:57Z

+from sqlalchemy.orm import as_declarative
+
+
+@as_declarative()


Советую изучить, что он делает и для чего

add code for review

b261644

samsonovroma2011-cmd commented Aug 6, 2026

View reviewed changes

samsonovroma2011-cmd changed the title ~~add code for review~~ Код ревью Aug 6, 2026

kykyrzik reviewed Aug 7, 2026

View reviewed changes

Conversation

samsonovroma2011-cmd commented Aug 6, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

samsonovroma2011-cmd left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

samsonovroma2011-cmd left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

samsonovroma2011-cmd left a comment •

edited

Loading