Как передать значение переменной в другую функцию?

Question

[Corn385]

Есть такая машина состояния. В нее приходит значение, которое хранится в переменной "message".

class FSMAdmin(StatesGroup):
    names = State()
    id_tg = State()
    all_game = State()
    winners = State()

async def cm_start(message : types.Message):
    await FSMAdmin.all_game.set()
    await message.answer('Введите количество игр, которое вы сыграли:')
    moon = message.text

async def load_win(message : types.Message, state: FSMContext, moon):
    async with state.proxy() as data: 
        data['names'] = moon
        data['id_tg'] = message.from_user.id
        data['all_game'] = message.text
    await FSMAdmin.next()
    await message.answer('Введите количество побед:')

async def load_lose(message : types.Message, state: FSMContext):
    async with state.proxy() as data:
        data['winners'] = message.text
    async with state.proxy() as data:
        await message.answer(str(data))
    await message.answer("Данные успешно внесены!")
    await state.finish()

т.е. я перенес значение переменной message в переменную moon. И это значение moon, мне нужно перенести в другую функцию, в которой это значение должно перейти во временную бд машины состояния. Но на выходе мне выдает:

TypeError: load_win() missing 1 required positional argument: 'moon'

Как правильно перенести значение moon в следующую функцию?

Comments

[Василий]

в месте вызова функции не вставлен позиционный аргумент.
Кинь кусок кода, где вызывается функция load_win()
и ещё не понятно, как moon выходит наружу из функции cm_start, нужно её или глобалом объявлять, или возвращать из функции

[Corn385]

Василий, dp.register_message_handler(load_win, state=FSMAdmin.all_game)

[Corn385]

Если я правильно понял, то так должно выглядит?

async def cm_start(message : types.Message):
    await FSMAdmin.all_game.set()
    await message.answer('Введите количество игр, которое вы сыграли:')
    moon = message.text
    return moon

async def load_win(moon, message : types.Message, state: FSMContext):
    async with state.proxy() as data: 
        data['names'] = moon
        data['id_tg'] = message.from_user.id
        data['all_game'] = message.text
    await FSMAdmin.next()
    await message.answer('Введите количество побед:')

[Алан Гибизов]

Corn385, так, но надо при вызове cm_start возвращаемое значение в переменную сохранить, а при вызове load_win в параметр передать.
Примерно так:

def first_func():
    result = 2 + 2
    return result


def second_func(moon):
    print(moon)


my_var = first_func()
second_func(moon=my_var)

[Corn385]

Алан Гибизов, сделал переменную moon глобальной, и получилось именно так, как я хотел. Это нормально?

[Алан Гибизов]

Corn385, это очень плохой вариант. Потому что с глобальной переменной вы никогда не уверены, что в ней в данный момент.

Глобальные лучше вовсе не использовать. Опять же, положим, в глобальной переменной есть какие-то данные. К кому они относятся? К какому пользователю? К какому циклу действий какого пользователя?

Если уж вы делаете глобальную переменную, то в неё надо передавать не только собственно данные, но и их принадлежность, порядковый номер цикла вашей машины состояний (который надо организовать тоже), также желательно контролировать время установки глобальной переменной, тайм-ауты, в общем - если делать по уму, проще использовать базу данных.

[Алан Гибизов]

Corn385, тут каждый день практически приходят с вопросом «я сделал глобальную переменную и у меня теперь данные все перепутались».

Answer 1

[Алан Гибизов]

Вам надо одно из двух: либо вернуть значение из первой функции через return и затем использовать его, передав в другую функцию как параметр той функции, либо внутри первой функции поменять объект вне функции (файл, БД, глобальную переменную).

Answer 2

[Иван Фурсов]

Почему ты в функции cm_start не запишешь в словарь FSM?, ты же там установил состояние, так и запиши там же!)

async def cm_start(message : types.Message, state: FSMContext):
    await FSMAdmin.all_game.set()

    await message.answer('Введите количество игр, которое вы сыграли:')

    async with state.proxy() as data: 
        data['names'] = message.text

Comments

[Corn385]

пробовал так, выдает

cm_start() missing 1 required positional argument: 'state'