Почему функция не видит переменную?

Question

[nano_e_t_4]

def actions():
    global flash
    node_name = request.args.get('node_name')
    if node_name:
        form = AddForm(obj = get_node_values(node_name))
        check = True
        return render_template('actions.html', form = form)
    form = AddForm()
    if request.method == 'POST' and form.validate_on_submit():
        operator = form.operator_name.data.lower()
       db_values = models.node(operator_name = operator, node_name = node,\
        node_id = nod_id)
        current_page = request.form['current_page']
        if check:
            flash(change_value_in_db(db_value))
            return redirect(url_for('actions'))

При попытке выполнить блок условия if check возникает exeption:

local variable 'check' referenced before assignment

. Не понимаю, отчего оно может возникать, ведь переменная check объявляется и отрабатывается в одной функции (есть подозрение, что из-за ветвления такая ситуация, поэтому гугл в помощь, но если здесь кто знает, подскажите пожалуйста)
Спасибо

Comments

[nirvimel]

Заверните свой код в тег <code></code> а то читать невозможно.

[nano_e_t_4]

упс. Сорри, нада было завернуть. Поспешил, исправляюсь

Answer 1

[nirvimel]

check объявляется только при if node_name: В противном случае остается не объявленной.

Answer 2

[Dimonchik]

global - плохая практика

а отсутствие CODE - "еще более хужее"

def actions():
    global flash
    node_name = request.args.get('node_name')
    if node_name:
        form = AddForm(obj = get_node_values(node_name))
        check = True
        return render_template('actions.html', form = form)

    form = AddForm()
    if request.method == 'POST' and form.validate_on_submit():
        operator = form.operator_name.data.lower()
        db_values = models.node(operator_name = operator, node_name = node,\
                                                            node_id = nod_id)
        current_page = request.form['current_page']
        if check:
            flash(change_value_in_db(db_value))
    return redirect(url_for('actions'))

Comments

[nano_e_t_4]

однако функцию flash по другому пока что встроить не получилось. А что такое CODE?

[nano_e_t_4]

вопрос CODE закрыт, исправляюьс

Answer 3

[nano_e_t_4]

следую хорошим советам:

def actions():
    global flash
    node_name = request.args.get('node_name')
    if node_name:
        form = AddForm(obj = get_node_values(node_name))
        check = True
        return render_template('actions.html', form = form)
    form = AddForm()
    if request.method == 'POST' and form.validate_on_submit():
        operator = form.operator_name.data.lower()
        node = form.node_name.data.lower()
        nod_id = form.node_id.data.lower()
        db_values = models.node(operator_name = operator, node_name = node,\
        node_id = nod_id)
        current_page = request.form['current_page']
        if check:
            flash(change_value_in_db(db_value))
            return redirect(url_for('actions'))
        flash(add_value_in_db(db_values))
        return redirect(url_for('actions'))

Comments

[Dimonchik]

stackoverflow.com/questions/10851906/python-3-unbo... оно?

[Dimonchik]

ну конечно оно, check в одной ветке будет отсутствовать

[Dimonchik]

объяви ее выше как None да и все

[nano_e_t_4]

нет, не совсем. Вопрос - почему переменная check объявленная в одном блоке функции не видна в другом блоке этой же функции. Ответ - в книге Бизли о питоне:
"То есть такие переменные определены только в пределах тела функции, и они уничтожаются, когда функция возвращает управление вызывающей программе"

[Dimonchik]

дык она не объявлена )
впиши ее после node_name = request.args.get('node_name')
тогда будет объявлена, а так в ветке условного оператора как она может быть объявлена?

как мигалка у ментов - то работает, то не работает? )

[nano_e_t_4]

неее. при обработке программы в первом блоке if значение переменной check печатается (в дебаггере имею ввиду). Но т.к. этот блоке заканчивается return-ом то питон эту переменную уничтожает

[nano_e_t_4]

не совсем по делу написал. спешу, сорри. хочу уже закончить наконец :)

[angru]

nano_e_t_4: nirvimel и dimonchik2013 вам правильно ответили, а вы спорите. перечитайте ответ nirvimel'а еще раз. А вообще вам эта переменная совсем не нужна, в том месте, где она обьявляется делается возврат из функции. Про global для функций вообще никогда не слышал, он обычно нужен только для того, чтобы изменить какое-нибудь глобальное значение, у вас вообще ничего не меняется, так что и смысла - ноль, не говоря уже о том, что моветон.

[nano_e_t_4]

а я и не спорю, скорее дополняю. В частности, мне просто было непонятно, что если переменная check создается, почему в следующий проход функции у нее значение false. Немного почитав понял, да и здешние советы были не лишними
Про global, полностью согласен, сам пока не могу разобраться почему при вызове flash без global питон ругается, что flash не объявлен