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 ответов

15 просмотров

Отдавай вместо була 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
Имхо лучше в контроллере чем в модели

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

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

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

Типа вызывать GetParent и проверять на соответствие GetModuleHandle?
The Bird of Hermes
67
Do any of you guys have interesting projects one could join? I'm a Middle Full-Stack developer (JS/TS, React & Node)
Lev Shapiro
40
$res = json_decode($наша строка из респонса); $res1 = array_map(fn($o) => $o->name, $res->breadcrumbs[0]->entities); Как такое будет на Хаскеле?.. В начале весь джейсон, в ко...
Хаскель Моисеевич Гопник
27
В чем сила брат, в NASM или FASM?
Isaac Kleiner
18
Вопрос по диагностике ошибок (я знаю в чем, в данном конкретном примере, я знаю, как исправить, пример модельный, понятно, что в реальности бывает намного запутаннее). module...
ⰄⰎⰋⰐⰐⰑⰛⰤⰧⰧⰩⰄ ⰊⰑⰁⰓⰡⰛⰦⰕⰫ
11
А чем вам питонисты не угодили?😂
.
79
Есть какой-нибудь для Delphi/FPC T*Compression(Decompression)Stream на базе LZ4/Zstd/любой другой быстрый(и хорошо сжимающий) алгоритм А ещё лучше в pure pascal А ещё лучше од...
notme
52
Хтось використовував Vapor на Windows?
Jaroshevskii
15
Какое виндузовое сообщение приходит TTabSheet, что риэлайняться контролы на нем, даже у которых парент другой? Ситуация: открываю форму - кнопок нет, перелистываю на другой т...
Катерина Свиридова
7
Тут кста кто-нибудь NeoVim использует?
Simple Sorcerer
13
Карта сайта