Python beginners

09 October 2024 19:02

Потому что where(id==123) эквивалентно where(False)

Python beginners

09 October 2024 19:01

оно и просто с id работает. но все равно подчеркивает

Python beginners

09 October 2024 19:00

Чекер в пайчарме это тот ещё бред

Python beginners

08 October 2024 19:38

а если чел ник сменил, то что?

Python beginners

08 October 2024 19:33

потому что у него коммит именно тут,а ты его вырезал

Python beginners

08 October 2024 19:32

я и не сказал - СЮДА

Python beginners

08 October 2024 19:32

Непонятно где выставляется этот user_id.. Такое ощущение что create не обновляет его

Python beginners

08 October 2024 19:31

    async def get_user(self) -> UserModel | None:
        query = select(User).filter_by(id=self.user_id)
        result = await self.session.execute(query)
        result = result.first()

        return UserModel.from_list(result)

Python beginners

08 October 2024 19:30

Как выглядит get_user? У тебя реально всегда одна запись в БД, или как он определяет какую взять?

Python beginners

08 October 2024 19:29

так как чат называется python begineers я бы для начала посоветовал разбить эту огромную функцию на 2-3

Python beginners

08 October 2024 19:28

глобалы тредлокалы контекст-вары 🌚

Python beginners

08 October 2024 19:27

кринжанистан какой-то

Python beginners

08 October 2024 19:27

скорее всего там используется self.user_id чтобы идентифицировать пользователя 😭

Python beginners

08 October 2024 19:25

У тебя только один юзер?

Python beginners

08 October 2024 19:23

if user.username != username:
    await self.update_table('username', username)
...

if not user:
    return

Python beginners

09 October 2024 19:02

filter_by(id=123) работает, where(id==123) нет

Python beginners

09 October 2024 19:00

с фильтр бай он не ругался

Python beginners

08 October 2024 19:39

использовать функцию update_user_name?

Python beginners

08 October 2024 19:34

Так всё что я вырезал нужно, а не только коммит, но в другом месте.

Python beginners

08 October 2024 19:33

я сказал, что коммит нужен если не автоматический

Python beginners

08 October 2024 19:32

Не надо коммит сюда засовывать

Python beginners

08 October 2024 19:31

ну и коммит если не автоматический

Python beginners

08 October 2024 19:30

пришли в чат get_user плиз

Python beginners

08 October 2024 19:29

Это больше всего смущает

Ещё то что ты используешь create как update. По идее они должны вызываться в разных местах

Python beginners

08 October 2024 19:28

ну как бы одна сущность. да

Python beginners

08 October 2024 19:28

У тебя сама функция слишком много делает

Python beginners

08 October 2024 19:27

может у него там в классе он хранится ,что обрабатывается 😂😂

Python beginners

08 October 2024 19:26

Судя по get_user() без параметров только один, или чего-то не понимаю?

Python beginners

08 October 2024 19:24

    async def create_user(self, username: str | None, name: str) -> UserModel:
        user = await self.get_user()

        name = slugify(
            name,
            allow_unicode=True,
            lowercase=False,
            separator=' '
        )

        if name.isspace() or len(name) < 1:
            name = username or str(self.user_id)

        if not user:
            query = insert(User).values(id=self.user_id, name=name, username=username)
            await self.session.execute(query)
            await self.session.commit()
            return await self.get_user()
            
        if not user.username == username:
            await self.update_table('username', username)

        if not user.name == name:
            await self.update_table('name', name)
                
        return await self.get_user()

Python beginners

08 October 2024 19:23

а в чем разница username и name?

kek