KeyError при добавлений элементов в словарь,как исправить эту ошибку?

Question

[The_Beginer]

По заданию надо выяснить быстрый заказ.
В моём понимании - нужно взять 2 даты(Дата Заказа,Дата Выдачи Заказа).
Подсчитать разницу дней между ними и уже с помощью условий найти самый короткий промежуток дней между датами и следовательно вывести его с фамилией заказчика.

Для этого создал отдельно словарь,где в качестве ключа хранится количество дней,а значение это фамилия заказчика.В рабочем цикле я считаю каждую дату заказа и дату выдачи заказа(dateorder,datetakeorder). После начинаю заносить полученное в словарь вместе с фамилией.То есть ищем разницу дней первой фамилии,нашли и заносим и дни и фамилию в словарь.

def fastorders(self):
            dateformat='%d.%m.%Y'
            print("Вывод тех записей,где был самый маленький срок выполнения заказа:")
            storagedatedict={}# Словарь для хранения полученных дней
            for bookorder in bookorders:
                  dat1=datetime.strptime(bookorder.dateorder,dateformat)
                  dat2=datetime.strptime(bookorder.datetakeorder,dateformat) 
                  difdayz=dat2-dat1
                  dayzstr=str(difdayz)                  
                  storagedatedict[(dayzstr.split()[0]),bookorder.surnm]+=1
            for daiz,srn in storagedatedict.items():
                  print(daiz,srn)
                  if daiz=='1' or '2':
                        print(daiz, ' - ',srn)

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

Answer 1

[soremix]

storagedatedict[(dayzstr.split()[0]),bookorder.surnm]+=1

Вы не создаете элемент словаря, вы пытаетесь изменить уже существующий (который у вас не существует на данный момент)

Comments

[The_Beginer]

Я делал другими способами.То есть по умолчанию,какие-то значения,строка и None и всё равно ошибка не исчезла.

storagedatedict={'key',None}

[soremix]

storagedatedict={'key',None} - не словарь

[o5a]

The_Beginer, чтобы можно было менять значения пустых ключей словаря, стоит использовать defaultdict вместо обычного словаря.

from collections import defaultdict

storagedatedict = defaultdict(int)

# тогда можно будет использовать увеличение значения любого ключа
storagedatedict[(dayzstr.split()[0]),bookorder.surnm]+=1

И в целом не особо понятно, зачем вычислять разницу дат, затем конвертировать ее в строку и делать split(). И в конце увеличивать значение ключа.

Если нужно просто найти минимальный срок заказа, можно напрямую отсортировать список, используя параметр key. Примерно так

result = sorted(bookorder, key=lambda x: datetime.strptime(x.datetakeorder,dateformat) 
 - datetime.strptime(x.dateorder,dateformat))

[The_Beginer]

o5a, я изменил код,добавил defaultdict и всё равно показывает вот такое исключение


Отвечу на вопрос про конвертацию и split.
Просто дату-время ввожу в строку,затем вычисляю и обратно возвращаю в строку.
На скриншоте,как вы видите, указан полученный день и рядом фамилия человека,которому оформили заказ за конкретное количество дней.
Количество полученных дней хочу использовать как ключ,а рядом фамилия.То есть за 1 день оформили заказ,например,Иванову.

[o5a]

The_Beginer,

я изменил код,добавил defaultdict и всё равно показывает вот такое исключение

В данном случае немного по-другому надо инициализировать, вот так:
storagedatedict = defaultdict(int)

Просто дату-время ввожу в строку,затем вычисляю и обратно возвращаю в строку.

Так а зачем цифровое значение разницы дней (с которым можно производить арифметические действия) переводить назад в строку (по которой нельзя будет что-то считать напрямую).
Зачем вместо (3, "Иванов"): 1 делать ("3", "Иванов"): 1

[The_Beginer]

o5a, увы возникает проблема с отображением,поэтому я и использовал строку,чтобы отсечь лишнее

def fastorders(self):
            dateformat='%d.%m.%Y'
            print("Вывод тех записей,где был самый маленький срок выполнения заказа:")
            storagedatedict=defaultdict(int)# Словарь для хранения полученных дней
            for bookorder in bookorders:
                  dat1=datetime.strptime(bookorder.dateorder,dateformat)
                  dat2=datetime.strptime(bookorder.datetakeorder,dateformat) 
                  difdayz=dat2-dat1
                  #dayzstr=str(difdayz)                  
                  storagedatedict[difdayz,bookorder.surnm]+=1
            for daiz,srn in storagedatedict.items():
                  if daiz==0 or daiz==1 in storagedatedict.items():
                        print(daiz,srn)

Вот пример работы программы
И дата отображается вот так,если я в строку не преобразовываю,а использую изначально int


То есть дни считает,всё хорошо и к фамилии сопостовляет,но с выводом проблема(поэтому я и в строке split вставлял).
А из-за неправильного вывода уже трудность по условию вывести короткие дни между заказами(в моём случае 0 или 1 дней)

[o5a]

The_Beginer, для объекта timedelta (который получается от разницы дат) можно брать только то, что нужно, в данном случае для отображения только дней достаточно добавить

.days
difdayz.days

И это будет цифрой, а не строкой.

Насчет проблем с короткими днями я не понял.

[The_Beginer]

o5a, Короткие дни,это я не так выразился,извините,это маленький срок выполнения заказа.Если между заказами будет 0 дней(в этот же день) или 1 день,тогда это маленький срок заказа.

.

По скриншоту видно, введены 2 фамилии,у одного между выполнениями заказа 1 день,а у 2-ого 4 дня.
По задумке моего кода,который ниже, программа должна вывести только одну запись где 1 день,в примере это (1,Иванов).
А он выводит все введённые записи

for daiz,srn in storagedatedict.items():
                  if daiz==0 or daiz==1 in storagedatedict.items():
                        print(daiz,srn)

Как я лично представляю этот код:
Инициализируются 2 переменные для перебора в словаре.
С помощью цикла начинается перебор всех элементов словаря.
Дальше условие если daiz(переменная отвечающая за отображение дней в этом цикле) равна 0 или 1 в элементах словаря,то надо вывести ту запись словаря, где минимальное количество дней и фамилия заказчика.

[o5a]

The_Beginer, потому что это условие неправильное
if daiz==0 or daiz==1 in storagedatedict.items():
Не похоже, что приведенный последний код работает так, как описали, что-то путаете.

Во-первых, раз учитываются только дни без времени, лучше изменить
storagedatedict[difdayz,bookorder.surnm]+=1
и записывать сразу дни
storagedatedict[difdayz.days,bookorder.surnm]+=1

Во-вторых, storagedatedict.items() возвращает кортеж (ключ, значение) при этом сам ключ тоже является кортежем, так что его надо отдельно разбирать (назначить переменными или по индексу)
Например так:

for key, value in storagedatedict.items():
    # key[0] соответственно кол-во дней заказа, key[1] - фамилия
    if key[0] in [0,1]:
        print(key, value)

Если считать количество дней заказа не нужно, то можно всю систему было сделать проще: ключом сделать фамилию, а время исполнения заказов заносить в значение списком, например
{"Иванов": [1,4,3,5,0], "Петров": [4]}

Еще как вариант - всю систему данных перевести на pandas DataFrame, там как раз такой поиск/сортировка/агрегирование выглядели бы уместно.

[The_Beginer]

o5a, Теперь правильно считает,спасибо вам!Очень всё объяснили и помогли!