Почему выдает ошибку?

Question

[andrey13879]

До добавления функции 'filter_bad_items' все работало изумительно

class ItemService:
    def enrich_with_data(self, session: requests.Session, items: list[Item]) -> list[Item]:
        for item in items:
            response = self.repository.get_additional_data(session, item)
            item_info = response.json()['iteminfo']
            item.full_item_name = item_info['full_item_name']
            #print(item.full_item_name)

        return items, item.full_item_name

    def filter_bad_items(self,full_item_name, items: list[Item]) -> list[Item]:
        print(full_item_name + '1234123')
        if full_item_name in ['XM1014 | Seasons (Factory New)']:
            return [item for item in items if (item.floatvalue in config.GOOD_FLOAT_VALUES2)]
        elif full_item_name in ['Glock-18 | High Beam (Factory New)']:
            return [item for item in items if (item.floatvalue in config.GOOD_FLOAT_VALUES1)]

items, full_item_name = item_service.enrich_with_data(session, items)
    print(full_item_name)
    items = item_service.filter_bad_items(full_item_name,items)

Код ошибки:

!!   File "C:\Users\User\Desktop\PATERN\app\main.py", line 42, in perform_item_instances_check
    items, full_item_name = item_service.enrich_with_data(session, items)
!!   File "C:\Users\User\Desktop\PATERN\app\services.py", line 85, in enrich_with_data
    return items, item.full_item_name
!! UnboundLocalError: local variable 'item' referenced before assignment

Comments

[Vindicar]

На какой строке ошибка? Дай полный стектрейс.

[Wispik]

В полном трейсбеке ошибки есть номер строки, в которой переменная item используется до объявления, там соответственно и ошибка

[andrey13879]

Wispik, Поменял на нормальный трейсбек

[Dmitrii]

andrey13879, обновил свой ответ на вопрос, проверяй

Answer 1

[Dmitrii]

Так, ты поправил вопрос, а я поправлю ответ.

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

def enrich_with_data(self, session: requests.Session, items: list[Item]) -> list[Item]:
        for item in items:
            response = self.repository.get_additional_data(session, item)
            item_info = response.json()['iteminfo']
            item.full_item_name = item_info['full_item_name']
            #print(item.full_item_name)

        return items, item.full_item_name

Answer 2

[Wispik]

class ItemService:
    def enrich_with_data(self, session: requests.Session, items: list[Item]) -> list[Item]:
        for item in items:
            response = self.repository.get_additional_data(session, item)
            item_info = response.json()['iteminfo']
            item.full_item_name = item_info['full_item_name']
            #print(item.full_item_name)

        return items, item.full_item_name

в return item.full_item_name, что за переменная item? она доступна только в цикле, поэтому и ошибка

Comments

[Dmitrii]

Прям копия моего ответа)

Answer 3

[Маленький Скрипт]

items: list[Item]) -> list[Item]:

Попробуйте без -> list[Item]

Comments

[andrey13879]

нет, не сработало(

[Dmitrii]

andrey13879, а что на счет того, чтобы убрать полностью type hint?
def filter_bad_items(self,full_item_name, items):

Или
def foo(it: list) -> list:

[andrey13879]

Dmitrii, не сработало

Answer 4

[andrey13879]

Перепробовал все решения из предложенных, но ни одно не сработало, тогда я просто объединил эти две функции в одну и все заработало.