170 похожих чатов

Привет всем товарищи! Вопросик имеется. Вводная: Метод to_dict() класса SerializerMixin

предназначен для сериализации объекта 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: Перестать смущаться. Это нормальное поведение метода модели.

12 ответов

20 просмотров

Отдавай вместо була None И статус транзакций, типа if true: return dict, True else: return None, False

Kel0
Отдавай вместо була None И статус транзакций, тип...

Тогда поведение метода будет более явной по моему

Выглядит как нарушение single responsibility

Выноси сериализацию во вьюху

Это вообще не нормальное поведение модели. От начала и до конца

John-Does Автор вопроса
Tishka17
Выноси сериализацию во вьюху

Это апиха, вьюхи нет. Нормально ли если сериализация будет происходить в контроллере, или вынести это в отдельный класс?

John-Does Автор вопроса
Tishka17
Апиха чем не вьюха?

тем что это теплое и мягкое=)

John Does
тем что это теплое и мягкое=)

Там тот же MVC используется, это тоже самое

John-Does Автор вопроса
Tishka17
Имхо лучше в контроллере чем в модели

Согласен с тобой, тоже об этом думал

Похожие вопросы

Обсуждают сегодня

У меня это всегда вопрос вызывало.. Нафига писать код так, чтобы потом ошибки вылавливать?
Nik
44
что за асемблер кста?
Shvabrikk Nya
19
Добрый день! Пробую отловить исключение EConvertError: function _TryTextToDouble(var DoubleVar: Extended; Text: String): Boolean; begin try DoubleVar := StrToFloat...
Kirill Filippenok
19
А где @Grinyaha уважаемый пропал?
Am Ambrion
14
Сделал краткий обзор магазина из статьи и заодно там же записал инструкцию по развёртыванию. https://www.youtube.com/watch?v=8-sE1vNk95Q (чё думаете по звуку? кое-что поменял...
Andrey K
17
Привет, коллеги! Возникла задача ограничить максимальный размер вложений для определённых расширений, например, чтобы для изображений лимит был 10 МБ, а для видео — 100 МБ. Ог...
Andro
1
Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
Kraszx
24
вот не понимаю, зачем вообще лезть в осдев, если даже базы не знаешь? хотя бы про загрузку, форматы файлов железо какое-никакое. Ну реально, даже зная работу железа, с условны...
Mixail Frolov
7
Можно ли ставить FPC транк, а Лазарь нет?
А Андрей
9
Shadow Akira ты откуда инфу черпаешь?
One
9
Карта сайта