Как можно отрефакторить данный код?

Question

[anton_zaboev]

Всем здравствуйте! Я начинающий питон программист, недавно начал писать код, но очень волнуюсь что пишу не красиво. Кое как решил вот такую задачу: Задача вроде тренажера по английскому. Задаются вопросы и на них надо отвечать. Если правильный ответ то и с первой попытке 3 бала, со 2 попытке 2 база и с 3 соответственно 1 балл, а если не отгадал с 3 попытке то 0 балов и переходим к следующему вопросу. Если кто может, посмотрите пожалуйста какие можете дать рекомендации как можно упростить код, что лишнее ,а что вообще не стоит писать
https://colab.research.google.com/drive/100OPOi5b4...

Comments

[Vindicar]

Ну для тестовой задачи норм. Я бы выделил в отдельную функцию сам процесс задавания вопроса (попытку), разве что... но тут это не критично.

[anton_zaboev]

Vindicar, спасибо большое что откликнулись

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

С телефона очень неудобно смотреть код на том ресурсе. Почему бы не показать код в вопросе?

[Влад Григорьев]

anton_zaboev,
1. вместо цикла while лучше использовать for attempt in range()
2. while > 0, означает что elif attempt == 0 0 никогда не выполнится, изменение attempt происходит в конце
3. весь блок if attempt == 3: scores += 3 ... можно выкинуть оставив scores += attempt
4. ну и в целом логику задавания и проверки ответа неплохо было бы выкинуть в отдельную функцию

[anton_zaboev]

Алан Гибизов, Да конечно

questions = ["My name ___ Vova ", "I ___ a coder ", "I live ___ Moscow "]
answers = ["is", "am", "in"]

questions.append("What is the best programming language ")
answers.append("Python")

user_message = input(f"Привет! Предлагаю проверить свои знания английского! Наберите 'ready', чтобы начать ")


def faq():
    """
    Функция для вывода опроса
    """

    counter = 0
    scores = 0

    for question, correct_answer in zip(questions, answers):
        attempt = 3
        while attempt > 0:
            answer = input(question)

            if answer.lower() == correct_answer.lower():
                counter += 1
                if attempt == 3:
                    scores += 3
                elif attempt == 2:
                    scores += 2
                elif attempt == 1:
                    scores += 1
                else:
                    scores += 0
                print("Ответ верный!")
                print(f"Баллы за правильный ответ {scores}")
                print(f"Попытка  {attempt}")
                break
            elif attempt == 0:
                print(f"Увы, но нет. Верный ответ {correct_answer}")
                print(f"Баллы за правильный ответ {scores}")
                break
            else:
                attempt -= 1

                print(f"Неправильно. Осталось {attempt} попыток")
                print(f"Баллы за правильный ответ {scores}")

    sum_questions = len(questions)
    percent = round((100 / sum_questions) * counter)
    print(f"Вот и все, "
          f"\nВы ответили на "
          f"{counter} вопросов из {sum_questions} верно,\nэто "
          f"{percent} процентов и {scores} баллов")


def count_attempt(attempt, scores):
    if attempt == 3:
        scores += 3
    elif attempt == 2:
        scores += 2
    elif attempt == 1:
        scores += 1
    else:
        scores += 0
    return scores


if user_message == "ready":
    faq()
else:
    print("Кажется вы не хотите играть. Очень жаль")

[mkone112]

anton_zaboev, боже какой трешак. Давай ты сначала Лутца до конца дочитаешь, а потом придешь с такими вопросам.

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

Влад Григорьев, вы дали на мой взгляд достаточно полный и внятный ответ, рекомендую перенести его из комментариев в ответы.