Ну тоесть, это разве не прикладной слой? @router.post("/register") async def register(

Question

Ну тоесть, это разве не прикладной слой? @router.post("/register") async def register(

register_data: UserRegisterSchema,
pwd_context: CryptContext = Depends(),
session: Session = Depends(),
mail: FastMail = Depends(),
):
register_data_dict: dict = register_data.dict()
user_password: str = register_data_dict.get("password")

register_data_dict["password"] = User.hash_password(user_password, pwd_context)
register_data_dict["joined_at"] = datetime.datetime.utcnow()

try:
await create_user(session, register_data_dict)
except DBAPIError as exc:
return {"status": "failed", "details": str(exc)}

return {"status": "ok"}

#backend #programming #python #russian

0

26.05.2023

9 ответов

15 просмотров

Tishka17

короче, у тебя нет прикладного слоя, его логика размазана между слоем работы с БД и вьюхами

0

26.05.2023

lubaskinc0de 🎁 Автор вопроса

Tishka17
выглядит как будто create_user мог быть прикладным...

крейт юзер воть async def create_user( session: Session, user: dict, ) -> None: """ Register user """ user_obj = User( email=user.get("email"), password=user.get("password"), first_name=user.get("first_name"), last_name=user.get("last_name"), joined_at=user.get("joined_at"), is_superuser=user.get("is_superuser"), is_active=user.get("is_active"), ) session.add(user_obj) await session.commit()

0

26.05.2023

lubaskinc0de 🎁 Автор вопроса

Tishka17
короче, у тебя нет прикладного слоя, его логика ра...

и где мне тогда отправлять почту?

0

26.05.2023

Tishka17

lubaskinc0de 🎁
и где мне тогда отправлять почту?

в том слое, где у тебя планируется бизнес логика

0

26.05.2023

Alex

Почитай про слой сервисов у Фаулера. Тишка хочет сказать что вот то месиво в register это по сути и есть твоя бизнес-логика и тебе надо куда-то вынести ее и там делать в т.ч. отправку почты.

0

26.05.2023

lubaskinc0de 🎁 Автор вопроса

Alex
Почитай про слой сервисов у Фаулера. Тишка хочет с...

и куда мне ее выносить? в отдельный слой? как он должен называться?

0

26.05.2023

lubaskinc0de 🎁 Автор вопроса

мне не в тестах это надо

0

26.05.2023

Alex