Не могу понять, почему код работает не так?

Question

[Anastasia_fox]

Условие задачи: Я работаю секретарем и мне постоянно приходят различные документы. Я должен быть очень внимателен чтобы не потерять ни один документ. Каталог документов хранится в следующем виде: Перечень полок, на которых находятся документы хранится в следующем виде: Необходимо реализовать пользовательские команды, которые будут выполнять следующие функции: p – people – команда, которая спросит номер документа и выведет имя человека, которому он принадлежит

documents = [
        {"type": "passport", "number": "2207 876234", "name": "Василий Гупкин"},
        {"type": "invoice", "number": "11-2", "name": "Геннадий Покемонов"},
        {"type": "insurance", "number": "10006", "name": "Аристарх Павлов"}
      ]

directories = {
        '1': ['2207 876234', '11-2'],
        '2': ['10006'],
        '3': []
      }

def documents_number(people):
  user_input = input('Введите номер документа')
  for documents in people:
    if documents_number == (documents['number']):
      print(documents['name'])
  print(documents['name'])
  return user_input
print(documents_number(documents))

Comments

[Андрей Шубин]

Код бы конечно в тэг обернуть и отформатировать по-человечески.

Ну и в коде дичь, конечно. Вы же спрашиваете у пользователя номер документа, а по нему потом не сверяетесь.

Да и с переменными беда аргумент people, а передаете туда documents. Что вообще происходит?

У вас есть документы и директории (полки?). Полки вроде даже и не используются в задаче. Соответсвтенно, нужно перебрать список документов, сверить номера с введенным пользователем и вывести имя.

[Алан Гибизов]

Это задача? Какое отношение к задаче имеет приведённый код - он входит в условие задачи или это попытка решения? Если это попытка решения, то чего ожидали и что не получается? Как «не так» работает код? Было бы неплохо прокомментировать код, если вы его автор.
Или вы хотите, чтобы вам дали решение?

[Алан Гибизов]

Оберните код в вопросе тегом code, удобнее при помощи кнопки < / >

[Александр Прокопенко]

Думаю код лучше переписать с нуля.

Answer 1

[PRoGRamm_InG]

Ты всегда выводишь последний документ, так как documents которая прошлась в цикле оставила у себя значение последнего повторения и user_input никак не используется. Правильный вариант намного легче

Правильный вариант:

def documents_number(people):
  user_input = int(input('Введите номер документа'))-1
  return people[user_input]["name"]

print(documents_number(documents))