You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Dec 19, 2024. It is now read-only.
Необходимо провести рефакторинг DBHandler, перенеся всю логику бд именно в слой бд. Что бы не было такого, что функция _check_login почему-то обитает в слое worker и строится поверх DBHandler:
def_check_login(self, login: str) ->bool:
""" Checks database for a user with the same login. Args: login: user login Returns: True if there is such a user or False if no such user exists. """try:
self._db.get_user_by_login(login)
exceptDatabaseReadError:
logger.debug("There is no user in the database")
returnFalseelse:
logger.success("User was found in the database")
returnTrue
Т.Е. в worker-е должны быть только вызовы методов DBHandler, краткие и лаконичные. А не вот это вот всё)))
Также в рамках этой задачи необходимо провести переход с SQLObjects на SQLAlchemy. Зачем? Так-то в принципе не имеет значения, какую orm юзать, можно её вообще выкинуть и писать чистые SQL запросы. Однако SQLAlchemy имеет большую распространённость и популярность - а из-за чего куда более полную доку. Также у SQLAlchemy есть инструмент миграции - Alembic, а в будущем нам без миграций прямо никак))) Так что вот.