Вложенный словарь для цикла

Question

Я новичок в программировании. Я пытаюсь выяснить, как вычесть «бюджет» из «фактического», а затем обновить значение до «дисперсии», используя вложенный для l oop. Однако я читал, что менять словарь во время итерации - не лучшая практика. До сих пор я не понимал, как действовать дальше.

for i in properties:
    for j in properties[i]:
        if j == "actual":
            sum = properties[i][j]
            print('\nActual:' , sum)
        if j == "budgeted":
            sum_two = properties[i][j]
            print('Budgeted:' , sum_two)
            diff = sum_two - sum
            print('Variance:', diff)    

default_value = 0

properties = {587: {'prop_name': 'Collington'}, 'rental_income': {'apartment_rent': '5120-0000', 'resident_assistance': '5121-0000', 'gain_loss': '5120-0000'}, 51200000: {'actual': 29620, 'budgeted': 30509, 'variance': default_value}, 51210000: {'actual': 25620, 'budgeted': 40509, 'variance': default_value}, ............

Answer 1

просто переберите словарь и проверьте, есть ли во внутреннем словаре, существуют ли actual, variance и budgeted или нет, если да, то измените значение variance

for k, v in properties.items():
    if (('actual' in v.keys()) and ('variance' in v.keys()) and ('budgeted' in v.keys())):
            properties[k]['variance'] = properties[k]['actual']-properties[k]['budgeted']

Answer 2

Ваши данные имеют странный формат, я всегда стараюсь сгруппировать похожие объекты в словарях, а не иметь метаданные и «списки» элементов на одном уровне словаря. Это сработает для вас:

for prop in properties:
    p = properties[prop]
    if 'actual' or 'budgeted' in p.keys():
        # get() wont error if not found, also default to 0 if not found
        p['variance'] = p.get('budgeted', 0) - p.get('actual', 0)

import json
print(json.dumps(properties, indent=4))

Вывод:

{
    "587": {
        "prop_name": "Collington"
    },
    "rental_income": {
        "apartment_rent": "5120-0000",
        "resident_assistance": "5121-0000",
        "gain_loss": "5120-0000"
    },
    "51200000": {
        "actual": 29620,
        "budgeted": 30509,
        "variance": 889
    },
    "51210000": {
        "actual": 25620,
        "budgeted": 40509,
        "variance": 14889
    }
}

Answer 3

Попробуйте что-то вроде:

for i in properties:
    properties[i]['variance'] = properties[i]['budgeted'] - properties[i]['actual']

Если вы не уверены, что bugeted и actual существуют в словарях, вы должны поймать KeyError и обработать его соответствующим образом:

for i in properties:
    try:
        properties[i]['variance'] = properties[i]['budgeted'] - properties[i]['actual']
    except KeyError:
        properties[i]['variance'] = -1 # Set to some special value or just pass

Answer 4

Нет ничего плохого в изменении значений внутри словаря во время итерации. Единственное, что не рекомендуется - это модифицировать сам словарь , то есть добавлять / удалять элементы.

Answer 5

sum = None
sum_two = None
for i in properties:
        for j in i:
            if j=="actual":
                sum = properties [i]["actual"] 
                print('\nActual:' , sum)
            if j == "budgeted":
                sum_two = properties[i]["budgeted"]
                print('Budgeted:' , sum_two)
                diff = sum_two - sum
                print('Variance:', diff)

Я не понял, что именно означает, но это должно сработать.