предназначен для сериализации объекта SqlAlchemy.
Дано: метод модели Users
def get_user_by_id(id: int) -> Union[Dict, bool]:
user = Users.query.filter_by(id=id).first()
if user is not None:
return user.to_dict()
else:
return False
Проблема: меня смущает, что метод возвращает либо словарь либо bool, есть ощущение что метод модели не должен обладать таким поведением, а должен только заниматься возвратом инфы из БД.
Вопрос: Как сделать лучше? Или выбрать вариант решения из предложенных.
Предложения по решению:
Вариант 1. Вместо False возвращать пустой словарь. Но тогда будет ли это приемлемо учитывая что тогда либо в контроллере прийдётся всё равно проверять лишний раз пришёл ли не пустой словарь, либо тогда уже фронт должен будет делать эти проверки.
Вариант2. Делегировать это на контроллер, то есть перенести сериализацию в него. Тогда и делать проверку пришел ли None, а если не None то и делать сериализацию. Но тогда контроллер будет толще.
Вариант3: Перестать смущаться. Это нормальное поведение метода модели.
Отдавай вместо була None И статус транзакций, типа if true: return dict, True else: return None, False
Тогда поведение метода будет более явной по моему
Выглядит как нарушение single responsibility
Выноси сериализацию во вьюху
Это вообще не нормальное поведение модели. От начала и до конца
Это апиха, вьюхи нет. Нормально ли если сериализация будет происходить в контроллере, или вынести это в отдельный класс?
Апиха чем не вьюха?
тем что это теплое и мягкое=)
Там тот же MVC используется, это тоже самое
Имхо лучше в контроллере чем в модели
Согласен с тобой, тоже об этом думал
Обсуждают сегодня