Можете покритиковать мой код прототипа инвентаря и крафта на Python(терминал\консоль)?

Question

[longset4]

Занимаюсь изучением программирования совсем недавно
Прототип инвентаря для "выживалки" в терминале\консоли на Python.
Покритикуйте, пожалуйста, архитектуру и логику. Буду рад любым советам по улучшению.

import random

items = ["Дерево","Камень","Кремень","Палка","Волокно","Ягоды","Грибы","Уголь","Железная руда","Тряпка"]
inventory = []

print("1 - Подобрать ресурс\n2 - Скрафтить железный меч\n"
      "3 - Очистить инвентарь(выбростить все)\n 4- Посмотреть инвентарь\n"
      "5 - Да\n6 - Нет\n7 - выход из инвентаря")

while True:
    answer = int(input())

    if answer < 1 or answer > 7:
        print('Вводите корректное число!')
        print("Попробуйте еще раз")
        continue
        
    elif answer == 1:
        random_index = random.randint(0, len(items) - 1)
        print("Вы подобрали:", items[random_index])
        inventory.append(items[random_index])

    elif answer == 4:
        print("\n\n\nВ вашем инвентаре:")
        print(inventory)

    elif answer == 2:
        print("*** СОЗДАНИЕ ПРЕДМЕТОВ ***")
        print("Для крафта требуется:\n Палка, Железная руда, Тряпка")
        answer_1 = int(input("Хотите продолжить?\n"))

        if answer_1 == 5:

            if 'Палка' in inventory and 'Железная руда'in inventory and 'Тряпка' in inventory:
                inventory.remove('Палка')
                inventory.remove('Железная руда')
                inventory.remove('Тряпка')
                inventory.append('Железный меч')
                print("Предмет успешно создан!")

            else:
                print("Недостаточно предметов для крафта!")

        elif answer_1 == 6:
            print("Приходите как будете уверены!")

    elif answer == 3:
        inventory.clear()
        print("Ваш инвентарь успешно очищен!")
        print(f"В вашем инвентаре сейчас: {inventory}")

    else:
        print("Вы успешно вышли из инвентаря!")
        exit()

Comments

[Алексей Уколов]

У вас варианты 5 и 6 выкидывают из программы, потому что не обрабатываются явно в основном цикле.

[longset4]

Алексей Уколов, это же вроде вложенное условие в другое условие что бы понять крафт true or false, или я что то не так понял в этом всем?

[Invoidy]

Я конечно не спец в архитектуре. Но мне кажется было бы читабельнее написать

if answer == 1:
elif answer == 2:
elif answer == 3:
elif answer == 4:
elif answer == 5:
elif answer == 6:
elif answer == 7:
print("Вы успешно вышли из инвентаря!")
exit()
else:
print('Вводите корректное число!')
print("Попробуйте еще раз")

[longset4]

Invoidy, я приму любую критику к сведению, я сам толком не разбираюсь

[Михаил Р.]

answer = int(input())

Попробуйте указать букву или просто нажать enter.

random.randint(0, len(items) - 1)

Может проще random.choice(items)?

exit()

В данном случае лучше break.

В данном случае, вместо if/elif попробуйте свитч.

[longset4]

Михаил Р., не знаю что такое свитч, объясните пожалуйста :(

[Михаил Р.]

longset4, в python эта конструкция называется match-case, но чаще называют именно свитч/switch (как в большинстве ЯП) - python switch.

Answer 1

[Пума Тайланд]

int(input()) без try/except уронит всю программу на любом нечисловом вводе или на пустой строке — а не только через ответы 5/6, как уже разобрали в комментах. Заодно инвентарь копит дубликаты строк без счётчика, рано или поздно захочется Counter или словарь {предмет: количество} вместо списка.