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

22 просмотра

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

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

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

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

Господа, а что сейчас вообще с рынком труда на делфи происходит? Какова ситуация?
Rꙮman Yankꙮvsky
29
А вообще, что может смущать в самой Julia - бы сказал, что нет единого стандартного подхода по многим моментам, поэтому многое выглядит как "хаки" и произвол. Короче говоря, с...
Viktor G.
2
30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Карта сайта