Почему цикл не добавляет переменную в словарь(список) Python?

Question

[Ya172602]

Столкнулся с проблемой - беру данные с одного файла ищу по соответствию в другом и обновляю значение цены(в соседней колонке).
Все здорово. НО:
При проходе цикла переменная не хочет добавлять в словарь(список) - пробовал и так и так. И через append, setdefault, update на выходе перезаписывает последним значением!

import numpy as np
import pandas as pd


def new():
#  перебираем товары с помощью цикла с домашнего файла
    exel = pd.read_excel("homefile.xlsx")
    full_rea_xl = pd.DataFrame(exel)
    tovar_list = full_rea_xl["name"]
    for tovar in tovar_list:
# находим товар у поставщика и выводим цену и количество
        df = pd.read_excel("прайсвечер1.xlsx")
        rows, cols = np.where(df == tovar)
        try:
            row, col = rows[0], cols[0]
        except IndexError:
            print("kjlkhj")
            break
        if col + 1 > df.shape[1]:

            print("Ошибка2")
        price_product = int(df.iat[row, col+1])
        list = []
        list.append(tovar)
        # total_dict = {tovar: price_product}
        # total_dict.setdefault(tovar,price_product)
        print(list)
print(new())

Comments

[0xD34F]

3.4, 3.8

[Hemul GM]

Учите "мат часть". Тут нет никакой проблемы, только ваша некомпетентность. При чем в самых базовых вещах.

[Модератор]

Фрагменты кода надо размещать в виде текста и оборачивать тэгом code для корректного отображения. Удобно делать кнопкой </>
Это обязательно, см.п.3.8 Регламента.
Сюда же относится traceback, ввод и вывод в консоли и другая структурированная текстовая инфа.

[Bright144]

табуляция или 4 х пробел в питоне имеет большую значение. Табуляция не работает по этому тебе придется поставить 4 пробела при открытие блока.

[Ya172602]

Bright144, Да, было совсем не понятно, сейчас добавил в теги

[Ya172602]

Hemul GM, Скажите в чем именно? Если как написали ниже проблема в области видимости, то пробовал перемещать и внутрь for и за пределами.

[Ya172602]

0xD34F, благодарю исправил

[Ya172602]

def new():                                             
# # перебираем товары с помощью цикла с домашнего файла
    exel = pd.read_excel("homefile.xlsx")              
    full_rea_xl = pd.DataFrame(exel)                   
    tovar_list = full_rea_xl["name"]                   
    list = []                                          
    list_2 = []                                        
                                                       
    for tovar in tovar_list:                           
        df = pd.read_excel("прайсвечер1.xlsx")         
        rows, cols = np.where(df == tovar)             
        row, col = rows[0], cols[0]                    
                                                       
        price_product = int(df.iat[row, col+1])        
                                                       
        list.append(tovar)                             
        list_2.append(price_product)                   
        #                                              
        # total_dict = {tovar: price_product}          
        # total_2.update(total_dict)                   
    print(list)                                        
    print(list_2)                                      
                                                       
print(new())

Решение:
объявление list вывел из цикла for.
Убрал проверку Try (не понял как связано) - но заработало
По поводу setdefault - другой совсем принцип действия
Всем спасибо

Answer 1

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

Потому что написавший его разработчик ничего не знает об областях видимости.

Comments

[Bright144]

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

[Ya172602]

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

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

Bright144, области видимости - это не только про пространства имён, но и про то, где и когда переменные определяются, а на это влияет уже каждый блок. Впрочем, это не единственная проблема автора, очевидно например ещё и не понимание основных структур данных.