Как быть с запросами, которые повторяются в нескольких роутах?

Question

[pcdesign]

Например, вот есть запрос:

News.query.filter_by(n_active=1).order_by(News.n_id.desc()).all()

Он используется в 3-х роутах.
/
/news
/admin/editnews/

Как правильно с повторяющимися запросами поступать?
Вынести в отдельную функцию? Или может в отдельный файл и обернуть в класс?
Посоветуйте, плз, как правильно?

Answer 1

[Александр]

class News(db.Model):
  ...
  @classmethod
  get_active_news(cls):
    return cls.query.filter_by(n_active=1).order_by(News.n_id.desc()).all()

Затем используете:
active_news = News.get_active_news()

Comments

[pcdesign]

Спасибо. А подскажите, плз, зачем декоратор @classmethod в данном случае?

[Александр]

pcdesign: это чтобы можно было использовать метод вне рамок конкретных экземпляров, а в рамках класса. Что-то вроде статик-метода, но со своими плюшками.

[pcdesign]

Александр: А если запрос такого вида:
News.query.filter_by(n_id=self.n_id).first()
Тогда как быть?

[pcdesign]

И вот с такими запросами до кучи:
db.session.query(User).join(UserItem, Lang).filter(User.id == u_id)

[Александр]

self - это ссылка на экземпляр класса News, то есть, на уже существующую(возможно, пока не в базе) сущность. Иными словами, запись - News.query.filter_by(n_id=self.n_id).first() - тавтология.

Чтобы сделать то, что вы хотите, передавайте в метод дополнительные параметры:
@classmethod
def get_news_by_name(cls, name):
return cls.query.filter_by(name=name).order_by(cls.n_id.desc()).all()

[pcdesign]

Александр: спасибо. Так и сделал. А как быть с запросами, где несколько таблиц? Где используются join-ы?

Куда их всабачить?

Answer 2

[nagibator8000]

Можно вынести в класс модели в виде отдельной функции

Comments

[pcdesign]

То есть в файл models.py
В нем что-то вроде этого:
class News(db.Model):
__tablename__ = 'news'
n_id = db.Column(db.SmallInteger, primary_key=True)
n_date = db.Column(db.Date, nullable=False)
n_text = db.Column(db.Text, nullable=False)
n_active = db.Column(db.Integer, nullable=False)

И прямо сюда впихнуть функцию:
def get_active(self):
return News.query.filter_by(n_active=1).order_by(News.n_id.desc()).all()

Как-то так?

[Roman K]

pcdesign: Да. И в конце all уберите. Глаза режет.

[pcdesign]

Roman Kitaev: А у меня без .all() не работает вывод в print(get_active())