Что не так с моим кодом(Будильник)?

Question

[Keste ...]

import time # Для работы с временем
import re # Для работы с регулярными выражениями
import threading # Модуль для работы с потоками

def thread(my_func):
	def wrapper(*args, **kwargs):
		my_thread = threading.Thread(target=my_func, args=args, kwargs=kwargs)
		my_thread.start()
	return wrapper

z = input("> ")

if 'поставь' in z and 'будильник' in z or 'установи' in z and 'будильник' in z or 'будильник' in z and 'на' in z:
	@thread
	def Alarm(t1, t2):
		while t1 != t2:
			time.sleep(1)
		print("Проснись!!!")
	def time_to(text):
		datex = re.findall(r'\d{2}[.-:]\d{2}',text)
		if datex:
			date = datex[0].replace('-','').replace('.','').replace(':','')
			whattime = date
		return f"{whattime}"
	def what_time():
		time_list = time.localtime()
		hours = time_list[3]
		minut = time_list[4]
		return f"{hours}{minut}"
	print("Будильник установлен!")
	Alarm(time_to(z), what_time())

Comments

[Павел Дунаев]

А что вы от него хотите и что у вас работает не так?

[Keste ...]

Он не выводит "Проснись!!!"

Answer 1

[Павел Дунаев]

Он выведет "Проснись!!!", если установить будильник на текущее время (т.е. чтобы он сработал сразу). А все потому, что надо в каждой итерации заново узнавать текущее время через what_time(), вместо сравнения с переменной t2

def Alarm(t1, t2):
    while t1 != what_time():
      time.sleep(1)  
    print("Проснись!!!")

И еще: хранить время в строках и все время сравнивать их не самое лучшее решение. Гораздо быстрее и удобнее хранить и сравнивать время как число.

Comments

[Keste ...]

Эх, как я сам не додумался про узнавать текущее время!?
Спасибо!
А про строки, я просто не помну как складывать числа не прибовляя т.е. 2155 а не 76.
если знаешь как это сделать то буду признателен!

[Павел Дунаев]

Использовать unix-время, например так.

import time  # Для работы с временем
import re  # Для работы с регулярными выражениями
import threading  # Модуль для работы с потоками


def thread(my_func):
    def wrapper(*args, **kwargs):
        my_thread = threading.Thread(target=my_func, args=args, kwargs=kwargs)
        my_thread.start()

    return wrapper


@thread
def Alarm(t):
    time.sleep(t)
    print("Проснись!!!")


def time_to(text):
    datex = re.match(r'(\d{2})[.-:](\d{2})', text)
    if datex:
        hour = int(datex[1])
        min = int(datex[2])
            
        # Чтобы получить текущий год, месяц, день...
        t = time.localtime()
        return time.mktime((t[0], t[1], t[2], hour, min, 0, t[6], t[7], t[8]))

z = input("> ")

if 'поставь' in z and 'будильник' in z or 'установи' in z and 'будильник' in z or 'будильник' in z and 'на' in z:
    alarm_time = time_to(z)

    # Текущее время в секундах с полуночи 1 января 1970 года (unix время)
    current_time = time.time()

    if alarm_time > current_time:
        # Вычислить время до будильника, в секундах
        delta = alarm_time - current_time
        Alarm(delta)
        print("Будильник установлен!")

[Keste ...]

Павел Дунаев, Круто, хотя вылезает ошибка!

if alarm_time > current_time:
TypeError: '>' not supported between instances of 'NoneType' and 'float'

P.S. И имя min вообще-то зарезервированае.

[Павел Дунаев]

Keste ..., точно, не заметил, должно быть так:

def time_to(text):
    datex = re.match(r'.*(\d{2})[.-:](\d{2})', text)

имя min вообще-то зарезервированае.

Если чуть-чуть, то можно (c)

[Keste ...]

Павел Дунаев,
Спасибо!!!