Ошибка в коде Python?

Question

[Govnokod12]

Доброго времени суток!
1. По непонятной для меня причине при вводе рандомных символов при первом вопросе в коде вылазит ошибка. Ругается на 20 строчку.
2. Как сделать в скрипте так, чтоб он не закрывался, а повторялся? (гуглил - не понял)
3. Необходимо в двух закомментированых местах проигрывание аудиофайлов. Как лучше всего реализовать? Я попытался через Pyglet воспроизвести mp3, но выдаёт ошибку.

import time

what = input ("Начать? [Y/N/]:")

if what == "Y":
    print(time.asctime())
    print("Отсчитываю 1 час...")
    time.sleep(3600)
    print("время вышло")
    #тут место для аудиосигнала
elif what == "N":
    kdks = input ("Хотите завести другой таймер? [Y/N]:")
else:
    print("Ошибка! Вы будете возвращены в начальное меню")
    
    
if kdks == "Y":
    clockers = input ("На сколько секунд поставить таймер?:")
    print("Таймер заведён на " + str(clockers) + " секунд ")
    time.sleep(int(clockers))
    print("Время вышло!")
    #место для аудиосигнала
elif kdks == "N":
    print ("Вы будете возвращены в начальное меню")
else:
    print ("Ошибка! Вы будете возвращены в начальное меню")

Comments

[Александр]

1- код можно вставить сюда и скрыть спойлером.
2- ошибку надо приложить
3- начертите ветвление и поймете что при Y попадаешь в 1ю ветку, пропуска дальше elif и else,
следовательно в след блоке питон не знает что за переменная kdks

[When you dont want to litter on stackoverflow]

Для начала выучи основы Python, и только потом учи его библиотеки! Для того чтобы скрипт повторялся, нужно использовать циклы, а для того чтобы не было таких ошибок, нужно уметь писать исключения.

[Сергей Горностаев]

Александр Шабельский, там 26 строчек, даже под спойлер пихать нечего.

[Govnokod12]

longclaps, говнокод-то мой, а не чей-то другой из присутствующих здесь :)

Answer 1

[Ivan Yakushenko]

pip install playsound

import time
from playsound import playsound


while True:
    start = input("Начать? [Y/N/]:")
    clockers = 3600
    timer = None
    if start.lower() == "y":
        print(time.asctime())
        print("Отсчитываю 1 час...")
    elif start.lower() == "n":
        timer = input("Хотите завести другой таймер? [Y/N]:")
        if timer.lower() == "y":
            while True:
                clockers = input("На сколько секунд поставить таймер?:")
                try:
                    isinstance(int(clockers), int)
                    break
                except ValueError:
                    print("Повторите ввод. Необходимо ввести число.")
            print("Таймер заведён на " + str(clockers) + " секунд ")
    if start.lower() not in ["y", "n"] and timer != "y":
        print("Ошибка! Вы будете возвращены в начальное меню")
        continue
    time.sleep(int(clockers))
    print("время вышло")
    playsound("D:/test.mp3")
    break

Comments

[Govnokod12]

Спасибо!

[Govnokod12]

А за что отвечает это строчка в коде?: if start.lower() not in ["y", "n"] and timer != "y":
И для чего .lower в 9 12 14 23 строчках?

[Ivan Yakushenko]

Govnokod12, .lower() переводит все буквы в строке в нижний регистр и происходит проверка на 'y' таким образом вне зависимости от того введете вы Y или y оба варианта будут работать, что избавляет от возможных мелких проблем.

if start.lower() not in ["y", "n"] and timer != "y":

нужна для проверки, что на начальном этапе был выбран правильный вариант и при установке таймера так-же, иначе - возврат к началу с помощью continue

Answer 2

[Александр]

import time
 
what = input ("Начать? [Y/N/]:")
while True:
    
    if what == "Y":
        print(time.asctime())
        print("Отсчитываю 1 час...")
        time.sleep(5)
        print("время вышло")
        #тут место для аудиосигнала
        break
    elif what == "N":
        kdks = input ("Хотите завести другой таймер? [Y/N]:")
        if kdks == "Y":
            clockers = input ("На сколько секунд поставить таймер?:")
            print("Таймер заведён на " + str(clockers) + " секунд ")
            time.sleep(int(clockers))
            print("Время вышло!")
            #место для аудиосигнала
            break
        elif kdks == "N":
            print ("Вы будете возвращены в начальное меню")
        else:
            print ("Ошибка! Вы будете возвращены в начальное меню")
    else:
        print("Ошибка! Вы будете возвращены в начальное меню")

Comments

[Ivan Yakushenko]

time.sleep(int(clockers)) тут ошибка будет, если ввести не число.

[Александр]

Ivan Yakushenko, всё правильно, я не отлавливал ValueError. Так как у ТС ошибка в том, что переменная не определена, если идти по ветки N в начале

[Ivan Yakushenko]

Александр Шабельский, и что? это ошибка логики, ТС же в свою очередь говорил об ошибке на 20 строке, где как раз таки time.sleep(int(clockers)) и просил помощи зациклить ввод. Мало того, что вы не исправили ошибку о которой говорил ТС, так еще и предложили вместо неё мешанину с еще более убогим кодом. Молодец.

[Ivan Yakushenko]

Не позорьтесь, удалите ответ. Из 3х вопросов вы умудрились дать ответ на 0 из них и сделать код еще более неработоспособным.

[javedimka]

Ivan Yakushenko, А твой код не мешанина? :)

[Ivan Yakushenko]

javedimka, мешанина, но он близкий к оригинальному коду и он полностью работает так, как ТС того хочет, в отличии от предложенного Александр Шабельский вариантом, который мало того что не решает проблем упомянутых ТС, так еще и зацикливается на 2 вложенности.