UnboundLocalError: local variable 'metawallet' referenced before assignment. Что делать?

Question

[Kemper5454]

Вот фрагмент кода:

def get_metawallet(self,  user_id):
    with self.connection:
        result = self.cursor.execute("SELECT `wallet` FROM `users` WHERE `user_id` = ?", (user_id,)).fetchall()
        for row in result:
            metawallet = str(row[0])
        return metawallet

Перегуглил все что можно было не нашел. Надеюсь кто-то сможет мне помочь. Я новичок

Comments

[Kemper5454]

Когда исправил на .fetchone
пишет:
for row in result:
TypeError: 'NoneType' object is not iterable

[Александр Нестеров]

Kemper5454, а по чему он должен итерировать?

[Дмитрий]

Kemper5454, запрос возвращает больше одной записи?

[Kemper5454]

Дмитрий, нет, одну, текст из полученного сообщения в тг

[Дмитрий]

Kemper5454, тогда и цикл там не нужен

[Kemper5454]

Дмитрий, а как это написать в коде?

[Kemper5454]

def get_metawallet(self, user_id):
    with self.connection:
        result = self.cursor.execute("SELECT `wallet` FROM `users` WHERE `user_id` = ?", (user_id,)).fetchone()
        return result

Так?

Answer 1

[Vindicar]

Если в result нет ни одной строки, какое значение будет у metawallet?

Comments

[Kemper5454]

Дефолтное, то есть 0, в таблице db это указано

[Сергей Горностаев]

Kemper5454, неправильно. Дефолтное значение будет у строк, для которых не определено значение колонки. А если строк вообще нет?

[Kemper5454]

Сергей Горностаев, пусто...?

[Сергей Горностаев]

Kemper5454, да, result буде пустым кортежем, цикл ни разу не выполнится и переменная metawallet не будет определена. О чём в ошибке и написано.

[Kemper5454]

Сергей Горностаев, и как это исправить в коде?

[Сергей Горностаев]

Kemper5454, разными можно способами. Например определять metawallet не в цикле или проверять result на пустоту.

[Kemper5454]

Сергей Горностаев, допустим если определять metawallet не в цикле, то...что?

[Vindicar]

Kemper5454, включи уже голову.
Если определять не в цикле, то есть только два варианта:
1. определять после цикла, но до вывода
2. определять до цикла, после цикла проверять, изменилось ли значение.
Если определять после цикла, то тогда мы перезапишем значение, которое получили в цикле. Плохо.
Если определять до цикла, то если цикл произойдёт, то начальное значение перезапишется на значение из БД. Если цикл не произойдёт, начальное значение останется. Таким образом, можно будет проверить, произошел ли цикл.

Это же не специфически питонья штука, это просто логика. Есть причина, есть следствие.

[Kemper5454]

Vindicar, я то понял как это работает, но как это написать в коде?

[Влад Григорьев]

Kemper5454, а для этого надо взять и прочитать хоть пару глав учебника, что было мочь написать такие элементарные вещи

[Kemper5454]

Влад Григорьев, можешь посоветовать книги?(◕‿◕)и главы

Answer 2

[Александр Нестеров]

Перегуглил все что можно было не нашел.

Очень сомневаюсь.
Дело в том, что вы объявляете metawallet в цикле, который проходит по кортежу из значений, полученных из бд.
Далеко не факт, что в базе есть подходящие данные. К тому-же с каждой итерацией данные будут перезаписываться.

Comments

[Kemper5454]

В том то и дело что есть, а он выдает ошибку

[Сергей Горностаев]

Kemper5454, вы демонстрируете схему таблицы. Покажите её содержимое.

[Александр Нестеров]

Kemper5454, то, что в таблице есть столбец 'wallet', ничего не значит, т.к.
1. Я говорил о конкретных значениях (например вы пытаетесь получить юзера с id 228, но в базе такой записи нет)
2.Это не отменяет того, что переменную нужно объявлять до цикла.